Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 2 Aug 2023 16:39:07 +0100
From:   Conor Dooley <conor@kernel.org>
To:     Mihai.Sain@microchip.com
Cc:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        conor+dt@kernel.org, Nicolas.Ferre@microchip.com,
        alexandre.belloni@bootlin.com, Claudiu.Beznea@microchip.com,
        andre.przywara@arm.com, Andrei.Simion@microchip.com,
        Jerry.Ray@microchip.com, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Cristian.Birsan@microchip.com
Subject: Re: [PATCH 1/2] dt-bindings: ARM: at91: Document Microchip SAMA5D29
 Curiosity
Message-ID: <20230802-scoring-pony-0cb3ad094018@spud>
References: <20230801111151.6546-1-mihai.sain@microchip.com>
 <20230801-unbalance-baguette-cd0d4e7e0107@spud>
 <PH8PR11MB6804073D5A1F26D8699756F8820BA@PH8PR11MB6804.namprd11.prod.outlook.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="99q6Lptlzcoa5Bia"
Content-Disposition: inline
In-Reply-To: <PH8PR11MB6804073D5A1F26D8699756F8820BA@PH8PR11MB6804.namprd11.prod.outlook.com>
Precedence: bulk


--99q6Lptlzcoa5Bia
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Aug 02, 2023 at 12:42:52PM +0000, Mihai.Sain@microchip.com wrote:
> On Tue, Aug 01, 2023 at 02:11:50PM +0300, Mihai Sain wrote:

> > ---
> >  Documentation/devicetree/bindings/arm/atmel-at91.yaml | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >=20
> > diff --git a/Documentation/devicetree/bindings/arm/atmel-at91.yaml=20
> > b/Documentation/devicetree/bindings/arm/atmel-at91.yaml
> > index dfb8fd089197..89d75fbb1de4 100644
> > --- a/Documentation/devicetree/bindings/arm/atmel-at91.yaml
> > +++ b/Documentation/devicetree/bindings/arm/atmel-at91.yaml
> > @@ -79,6 +79,13 @@ properties:
> >            - const: atmel,sama5d2
> >            - const: atmel,sama5
> > =20
> > +      - description: Microchip SAMA5D29 Curiosity
> > +        items:
> > +          - const: microchip,sama5d29-curiosity
> > +          - const: atmel,sama5d29
> > +          - const: atmel,sama5d2
> > +          - const: atmel,sama5
>=20
> What is the benefit of adding the two familial compatibles? Is there soft=
ware that actually uses these?
>=20
> Thanks,
> Conor.
>=20
> Currently there is no software.
> In this file we've just documented the previous boards from sam9, sama5, =
sama7 MPUs.

Please fix your quoting. If no software cares about this, what is the
benefit of adding familial compatibles?

--99q6Lptlzcoa5Bia
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZMp4mwAKCRB4tDGHoIJi
0piVAP9G3ga7bbRbo8QSR420Fhqag5yUVcino8bfFrLs7ssDaQEA/X+Kkb3OBTGT
ET5ozafhFE9pZqdveoRIppIc0UnOtwM=
=TrOH
-----END PGP SIGNATURE-----

--99q6Lptlzcoa5Bia--