Re: [PATCH 1/1] of: unittest: rename overlay source files from .dts to .dtso

Geert Uytterhoeven
2022-04-28T06:26:23Z

Hi Rob,

On Wed, Apr 27, 2022 at 11:14 PM Rob Herring <robh@kernel.org> wrote:
> On Wed, Jan 26, 2022 at 1:31 PM Rob Herring <robh@kernel.org> wrote:
> > On Fri, Jan 14, 2022 at 3:25 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > > On Fri, Jan 14, 2022 at 3:10 AM Rob Herring <robh@kernel.org> wrote:
> > > > On Thu, Jan 6, 2022 at 11:23 AM Frank Rowand <frowand.list@gmail.com> wrote:
> > > > > Patient Geert has pinged again.
> > > >
> > > > If it's not a patch to be reviewed, then I'm not going to see it most
> > > > likely. I don't read the DT list regularly...
> > >
> > > Fair enough...
> > >
> > > > > If I remember correctly you guys were not thrilled with this idea, but
> > > > > also did not seem strongly against it. Are you willing to go along
> > > > > with .dtso for overlay source files? If so, I will revive this patch
> > > > > series.
> > > > >
> > > > > David, if you are against supporting .dtso in the dtc compiler then
> > > > > the kernel can still support it through make rules.
> > > >
> > > > I'm not really interested in diverging from dtc. I'd suggest moving
> > > > the discussion to dtc list and/or devicetree-spec if you want to get
> > > > more attention on this.
> > >
> > > What needs to be supported in the dtc compiler?
> > > The fallback passed to guess_input_format() is "dts".
> > > So this has been working out-of-the-box since forever?
> >
> > Ah, okay.
> >
> > > > Also, keep in mind that extensions also affect MIME types which
> > > > someone was also asking about recently.
> > >
> > > You mean "MIME type of Devicetree Blobs and Sources"[1]?
> > > According to [2](2022-01-13), none of that has happened.
> >
> > This is what I was thinking of:
> >
> > https://github.com/devicetree-org/devicetree-specification/issues/46
> >
> > In any case, given everyone is ambivalent, send me an updated patch
> > and I'll apply it.
>
> Ping! Anyone still want this?
>
> What I don't want to see is a mixture of .dts and .dtso. And now I'm
> reviewing RPi overlay patches[1] with .dts.

I still prefer .dtso over .dts, as it allows tools to detect the file
type without having to read the file's contents.
Without this, e.g. make needs to have all overlays listed explicitly
in a Makefile.

We do have .c, .h, .s (even .S), .dtsi. So why not .dtso?

> [1] https://lore.kernel.org/r/20220427185243.173594-3-detlev.casanova@collabora.com

Thanks!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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