Re: [PATCH] dt-bindings: mfd: syscon: Add more simple compatibles

Miquel Raynal
2024-05-13T08:05:36Z

Hi Conor,

conor@kernel.org wrote on Mon, 13 May 2024 08:59:34 +0100:

> On Mon, May 13, 2024 at 09:55:04AM +0200, Miquel Raynal wrote:
> > Hi Rob,
> >
> > robh@kernel.org wrote on Fri, 10 May 2024 07:30:14 -0500:
> >
> > > Add another batch of various "simple" syscon compatibles which were
> > > undocumented or still documented with old text bindings. Remove the old
> > > text binding docs for the ones which were documented.
> >
> > ...
> >
> > > .../devicetree/bindings/mtd/atmel-nand.txt | 9 ------
> >
> > I don't know how useful are these compatibles, but if I take the case
> > of atmel,sama5d3-nfc-io, it is only described in this file, while
> > several DTSI reference it. After this change they will no longer be
> > referenced at all but still in use. Is this intended? Shall these
> > compatibles be totally dropped from the device trees as well?
> >
> > $ git grep -c atmel,sama5d3-nfc-io arch/
> > arch/arm/boot/dts/microchip/sama5d2.dtsi:1
> > arch/arm/boot/dts/microchip/sama5d3.dtsi:1
> > arch/arm/boot/dts/microchip/sama5d4.dtsi:1
> > arch/arm/boot/dts/microchip/sama7g5.dtsi:1
>
> atmel,sama5d3-nfc-io is added to the syscon yaml binding in this patch,
> so it's a move not a removal, no?

Ah sorry, I missed the additions in this file, fine then, sorry for the
misunderstanding.

Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com> # mtd

Thanks,
Miquèl