On Mon, Sep 2, 2024 at 4:03 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
> On Tue, Feb 13, 2024 at 11:59 AM Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> > In order to use #address-cells = <1> and start making use of the
> > standard reg property, let's prepare things to ignore the possible
> > address in the clock node name.
> >
> > Unless the clock-output-names property is used, the legacy clocks still
> > fall back to matching the clock data based on the node name.
> >
> > We use cleanup.h to simplify the return path for freeing tmp.
> >
> > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
>
> Thanks for your patch, which is now commit 3516338543cafb65 ("clk: ti:
> Handle possible address in the node name") in v6.9-rc1.
> This causes an early boot crash on BeagleBone Black:
>
> ti_dt_clocks_register: failed to lookup clock node
> clk-24mhz-clkctrl:0000:0, ret=-517
I found the culprit: after the move of .dts files to vendor
sub-directories, I had updated my boot script to:
DTB=arch/arm/boot/dts/ti/am335x-boneblack.dtb
if [ ! -e $DTB ]; then
DTB=arch/arm/boot/dts/am335x-boneblack.dtb
fi
I.e. I missed the "/omap" part, causing the install to fall back to
an old DTB file that no longer works with modern kernels.
Sorry for the noise.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
