linux-kernel - RE: [PATCH] ARM: dts: at91: sama5d29_curiosity: Add device tree for sama5d29

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <PH8PR11MB6804B1C6466C7A450A180D1D8215A@PH8PR11MB6804.namprd11.prod.outlook.com>
Date:   Wed, 16 Aug 2023 09:05:19 +0000
From:   <Mihai.Sain@...rochip.com>
To:     <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>
CC:     <Andrei.Simion@...rochip.com>
Subject: RE: [PATCH] ARM: dts: at91: sama5d29_curiosity: Add device tree for
 sama5d29_curiosity board

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.