[PATCH] pinctrl: Check pinconfig nodes pre-reloc status recursively

[Simon Glass]

Hi Jonas,

On Sat, 5 Aug 2023 at 08:29, Jonas Karlman <jonas at kwiboo.se> wrote:
>
> Hi Massimo,
>
> On 2023-08-05 16:06, Massimo Pegorer wrote:
> > Hi Jonas,
> >
> > Il giorno sab 5 ago 2023 alle ore 13:11 Jonas Karlman <jonas at kwiboo.se> ha
> > scritto:
> >
> >> Pinconfig nodes normally bind recursively with PINCTRL_FULL and
> >> PINCONF_RECURSIVE enabled. However, during U-Boot proper pre-relocation
> >> any node marked with e.g. bootph-all will not bind unless its parent is
> >> also marked for pre-reloc.
> >>
> >>   group1 {
> >>       pinconf1 {
> >>           bootph-all;
> >>       };
> >>   };
> >>
> >> This cause the following warning message to be shown during U-Boot
> >> proper pre-reloc stage on Rockchip RK3568 devices.
> >>
> >>   ns16550_serial serial at fe660000: pinctrl_select_state_full:
> >> uclass_get_device_by_phandle_id: err=-19
> >>
> >> Check pinconfig nodes pre-reloc status recursively to fix this and to
> >> make pinconfig_post_bind work same at both U-Boot proper pre-reloc and
> >> at TPL/SPL stage.
> >>
> >> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
> >> ---
> >>  drivers/pinctrl/pinctrl-uclass.c | 18 +++++++++++++++++-
> >>  1 file changed, 17 insertions(+), 1 deletion(-)

This is actually something we want to change about fdtgrep. I believe
we should try to adjust that tool rather than loading this ontoU-Boot,
not least because it controls what properties are visible in SPL.

Regards,
Simon