Re: [PATCH v2] fdtoverlay: Remove bogus type info from help text

[Rob Herring <robh@kernel.org>]

On Fri, May 31, 2024 at 9:07 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Rob,
>
> On Fri, May 31, 2024 at 3:43 PM Rob Herring <robh@kernel.org> wrote:
> > On Thu, May 30, 2024 at 8:32 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > > On Thu, May 30, 2024 at 2:31 PM Rob Herring <robh@kernel.org> wrote:
> > > > On Wed, May 29, 2024 at 5:11 AM Geert Uytterhoeven
> > > > <geert+renesas@glider.be> wrote:
> > > > >
> > > > > "fdtoverlay -h" shows a.o.:
> > > > >
> > > > >     <type>      s=string, i=int, u=unsigned, x=hex
> > > > >             Optional modifier prefix:
> > > > >                     hh or b=byte, h=2 byte, l=4 byte (default)
> > > > >
> > > > > However, unlike fdtget and fdtput, fdtoverlay does not support the
> > > > > "-t"/"--type" option.
> > > > >
> > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > > > ---
> > > > > Upstream dtc has Uwe's commit 2cdf93a6d402a161 ("fdtoverlay: Fix usage
> > > > > string to not mention "<type>"").
> > > >
> > > > So you want me to sync upstream? That's how upstream commits get into
> > > > the kernel. I don't take patches (unless there's some urgent
> > > > breakage).
> > >
> > > If that's the policy, then yes please ;-)
> >
> > With current dtc main branch, I get these errors:
> >
> > Failed to apply
> > 'arch/arm64/boot/dts/renesas/salvator-panel-aa104xd12.dtbo':
> > FDT_ERR_NOTFOUND
>
> Interesting...
>
> Do you know what's the problem?

No. Since it is conveniently platforms you care about, I thought I'd
leave it to you. ;)

Well, it seems that if I use the upstream built dtc and fdtoverlay it
works fine. Using the one in scripts/dtc/ doesn't. Looks like that
issue is on me...

> Are these the only ones failing?

Yes. And all of arm32 was fine.

Rob