linux-kernel - Re: [PATCH 1/2] ARM: dts: at91: align SPI NOR node name with dtschema

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: <0c0b19ac-9047-509b-1112-cb25fa00ae22@microchip.com>
Date:   Tue, 12 Apr 2022 10:41:04 +0000
From:   <Tudor.Ambarus@...rochip.com>
To:     <krzysztof.kozlowski@...aro.org>, <robh+dt@...nel.org>,
        <krzysztof.kozlowski+dt@...aro.org>, <Nicolas.Ferre@...rochip.com>,
        <alexandre.belloni@...tlin.com>, <Claudiu.Beznea@...rochip.com>,
        <Lars.Povlsen@...rochip.com>, <Steen.Hegelund@...rochip.com>,
        <UNGLinuxDriver@...rochip.com>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/2] ARM: dts: at91: align SPI NOR node name with dtschema

On 4/12/22 13:32, Tudor Ambarus wrote:
> On 4/7/22 17:32, Krzysztof Kozlowski wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> The node names should be generic and SPI NOR dtschema expects "flash".
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> 
> Reviewed-by: Tudor Ambarus <tudor.ambarus@...rochip.com>
> 
>> ---
>>  arch/arm/boot/dts/at91-dvk_su60_somc.dtsi     | 2 +-
>>  arch/arm/boot/dts/at91-q5xr5.dts              | 2 +-
>>  arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi   | 2 +-
>>  arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dts | 2 +-
>>  arch/arm/boot/dts/at91-sama5d2_xplained.dts   | 2 +-
>>  arch/arm/boot/dts/at91-sama5d4_ma5d4.dtsi     | 2 +-
>>  arch/arm/boot/dts/at91-sama5d4ek.dts          | 2 +-
>>  arch/arm/boot/dts/at91-vinco.dts              | 2 +-
>>  arch/arm/boot/dts/at91sam9n12ek.dts           | 2 +-
>>  arch/arm/boot/dts/at91sam9x5ek.dtsi           | 2 +-
>>  arch/arm/boot/dts/sama5d3xmb.dtsi             | 2 +-
>>  arch/arm/boot/dts/sama5d3xmb_cmp.dtsi         | 2 +-
>>  12 files changed, 12 insertions(+), 12 deletions(-)
>>

I've just noticed that there are other node names that can be fixed,
if you feel like resubmitting:
arch/arm/boot/dts/at91rm9200ek.dts:				mtd_dataflash@0 {
arch/arm/boot/dts/at91rm9200ek.dts:		nor_flash@...00000 {
arch/arm/boot/dts/at91sam9260ek.dts:				mtd_dataflash@1 {
arch/arm/boot/dts/at91sam9261ek.dts:				mtd_dataflash@0 {
arch/arm/boot/dts/at91sam9263ek.dts:				mtd_dataflash@0 {
arch/arm/boot/dts/at91sam9g20ek_common.dtsi:				mtd_dataflash@1 {
arch/arm/boot/dts/at91sam9m10g45ek.dts:				mtd_dataflash@0 {
arch/arm/boot/dts/at91sam9rlek.dts:				mtd_dataflash@0 {