[PATCH] pinctrl: single: Fix memleak in pcs_dt_node_to_map
Tony Lindgren
tony at atomide.com
Mon Jul 17 22:29:20 PDT 2023
Hi,
* guomengqi (A) <guomengqi3 at huawei.com> [230712 10:00]:
> 在 2023/7/6 12:07, Tony Lindgren 写道:
> > Thanks for looking into it. I wonder if we can rely on naming for
> > pinmux_func_name_to_selector() though. Can things change in a way where
> > we need to release everything and reparse? Mostly wondering what happens
> > with DT overlays?
>
> Let me confirm, you mean when the pin controller dtsi changed at runtime,
> some functions and groups can change silently while the dt-node name remains
> same, so the old data needs to be released and reparsed, right?
>
> I don't know much about DT overlays. I can look deeper into revelant codes,
> maybe do some experiments too.
>
> My guess now is DT overlay will first remove the old parsed nodes, then
> create new ones. If so, the modification to pcs_dt_node_to_map() in this
> patch is not affected.
OK yeah good to check it to confirm.
Regards,
Tony
More information about the linux-arm-kernel
mailing list