[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230816-asleep-move-ab8746612698@spud>
Date: Wed, 16 Aug 2023 15:38:07 +0100
From: Conor Dooley <conor@...nel.org>
To: Mihai.Sain@...rochip.com
Cc: claudiu.beznea@...on.dev, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
Nicolas.Ferre@...rochip.com, Cristian.Birsan@...rochip.com,
alexandre.belloni@...tlin.com, andre.przywara@....com,
Jerry.Ray@...rochip.com, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
Andrei.Simion@...rochip.com
Subject: Re: [PATCH] ARM: dts: at91: sama5d29_curiosity: Add device tree for
sama5d29_curiosity board
On Wed, Aug 16, 2023 at 09:05:19AM +0000, Mihai.Sain@...rochip.com wrote:
> Hi Claudiu,
>
> ----------------------
>
> Hi, Mihai,
>
> On 8/10/23 09:47, Mihai.Sain@...rochip.com wrote:
> >> +&i2s0 {
> >> + pinctrl-names = "default";
> >> + pinctrl-0 = <&pinctrl_i2s0_default>;
> >> + status = "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 and 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.
> >
>
> 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.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists