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, 16 Aug 2023 15:38:07 +0100
From:   Conor Dooley <conor@kernel.org>
To:     Mihai.Sain@microchip.com
Cc:     claudiu.beznea@tuxon.dev, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        Nicolas.Ferre@microchip.com, Cristian.Birsan@microchip.com,
        alexandre.belloni@bootlin.com, andre.przywara@arm.com,
        Jerry.Ray@microchip.com, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Andrei.Simion@microchip.com
Subject: Re: [PATCH] ARM: dts: at91: sama5d29_curiosity: Add device tree for
 sama5d29_curiosity board
Message-ID: <20230816-asleep-move-ab8746612698@spud>
References: <20230804101043.4063-1-mihai.sain@microchip.com>
 <127fc712-d924-f27f-5449-33385e89d6c3@tuxon.dev>
 <PH8PR11MB6804E49B5946F4A9D60837E18213A@PH8PR11MB6804.namprd11.prod.outlook.com>
 <98d01db3-3eba-0731-0dd9-4310ed293bd6@tuxon.dev>
 <PH8PR11MB6804B1C6466C7A450A180D1D8215A@PH8PR11MB6804.namprd11.prod.outlook.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="TC0LRDS6Pz+cbGoi"
Content-Disposition: inline
In-Reply-To: <PH8PR11MB6804B1C6466C7A450A180D1D8215A@PH8PR11MB6804.namprd11.prod.outlook.com>
Precedence: bulk


--TC0LRDS6Pz+cbGoi
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Aug 16, 2023 at 09:05:19AM +0000, Mihai.Sain@microchip.com wrote:
> Hi Claudiu,
>=20
> ----------------------
>=20
> Hi, Mihai,
>=20
> On 8/10/23 09:47, Mihai.Sain@microchip.com wrote:
> >> +&i2s0 {
> >> +     pinctrl-names =3D "default";
> >> +     pinctrl-0 =3D <&pinctrl_i2s0_default>;
> >> +     status =3D "okay";
> > I see no sound bindings on DT. Is there any reason for having this here?
> > # i2s0 bus is wired to RPi 40-pin connector. I want to have this node a=
nd its pinctrl here.
> > # i2c1, flx4-spi, pwm0, uart1 are also wired to Rpi connector, and are =
defined here.
> > # The same logic we apply also for mikroBUS sockets: i2c, pwm, spi and =
uart nodes are defined here.
> >
>=20
> Can you use it (either with user space tools or other kernel consumers) w=
/o additional device tree bindings?
> # Yes. We have added in dt-overlay the codec which uses the i2s0 and i2c1=
 nodes and pinctrls.

Can you please fix your mail client so that your replies are properly
quoted? I don't understand who is saying what here.

--TC0LRDS6Pz+cbGoi
Content-Type: application/pgp-signature; name="signature.asc"

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

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZNzfTgAKCRB4tDGHoIJi
0mskAQDqB7yvGCNZL5HLPqbFd4miuOsz8BFLD9vxbjY5k+x9XAD+NKe0JvyQzcBk
MW+H0TBbpXXo7VMYE0ttbiSKddZE1QI=
=X1ZD
-----END PGP SIGNATURE-----

--TC0LRDS6Pz+cbGoi--