Re: [PATCH 0/6] Rename DTB overlay source files

[Andrew Davis]

On 10/24/22 7:28 AM, Geert Uytterhoeven wrote:
> Hi Andrew,
>
> On Sun, Oct 23, 2022 at 8:24 PM Andrew Davis <afd@xxxxxx> wrote:
> > This is a series based on my patch here[0]. As suggested by Rob
> > I've resurrected Frank's patch and appended it to mine as a series.
> >
> > First patch here is my original patch, 3rd is Frank's patch but with
> > the unittest changes pulled out into the 2nd patch. That was re-worked
> > moving the source building macro into scripts/Makefile.lib.
> >
> > Patches 4, 5, and 6 are an attempt at renaming all the existing DTB
> > overlays. Split out by platform so they could be taken by platform
> > maintainers or if easier ACK'd here and taken all together.
> >
> > This should cover all the DTB overlays so we can remove the old .dts
> > rule for overlays and make .dtso the only supported way, let me know
> > if we want that this cycle and I can post that too.
>
> Thanks a lot for picking this up!
>
> Everything builds still fine, and the OF unit tests still run fine, so
> Tested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
>
> BTW, you missed the Smarthome-Wolf Pi433 overlay[1] and its
> documentation[2] under drivers/staging/, but perhaps that was
> intentional, as it is not tied into the build system?

Wasn't really sure about that one, but it seems simple enough to rename,
will add a patch for that in v2. Will also fix the wording in the last
3 patches as suggested.

Thanks,
Andrew

> [1] drivers/staging/pi433/Documentation/devicetree/pi433-overlay.dts
> [2] drivers/staging/pi433/Documentation/devicetree/pi433.txt
>
> 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