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-next>] [day] [month] [year] [list]
Message-ID: <20220208111225.234685-1-tudor.ambarus@microchip.com>
Date:   Tue, 8 Feb 2022 13:12:25 +0200
From:   Tudor Ambarus <tudor.ambarus@...rochip.com>
To:     <nicolas.ferre@...rochip.com>, <claudiu.beznea@...rochip.com>,
        <alexandre.belloni@...tlin.com>
CC:     <robh+dt@...nel.org>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <kavyasree.kotagiri@...rochip.com>,
        <krzysztof.kozlowski@...onical.com>,
        Tudor Ambarus <tudor.ambarus@...rochip.com>
Subject: [PATCH] ARM: dts: at91: Use the generic "crypto" node name for the crypto IPs

The DT specification recommeds that:
"The name of a node should be somewhat generic, reflecting the function of
the device and not its precise programming model. If appropriate, the name
should be one of the following choices:"
"crypto" being the recommendation for the crypto nodes. Follow the DT
recommendation and use the generic "crypto" node name for the at91 crypto
IPs. While at this, add labels to the crypto nodes where they missed, for
easier reference purposes.

Signed-off-by: Tudor Ambarus <tudor.ambarus@...rochip.com>
---
 arch/arm/boot/dts/sam9x60.dtsi | 6 +++---
 arch/arm/boot/dts/sama5d2.dtsi | 6 +++---
 arch/arm/boot/dts/sama5d3.dtsi | 6 +++---
 arch/arm/boot/dts/sama5d4.dtsi | 6 +++---
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi
index ec45ced3cde6..ec686f617ec7 100644
--- a/arch/arm/boot/dts/sam9x60.dtsi
+++ b/arch/arm/boot/dts/sam9x60.dtsi
@@ -270,7 +270,7 @@ pit64b: timer@...28000 {
 				clock-names = "pclk", "gclk";
 			};
 
-			sha: sha@...2c000 {
+			sha: crypto@...2c000 {
 				compatible = "atmel,at91sam9g46-sha";
 				reg = <0xf002c000 0x100>;
 				interrupts = <41 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -291,7 +291,7 @@ trng: trng@...30000 {
 				status = "okay";
 			};
 
-			aes: aes@...34000 {
+			aes: crypto@...34000 {
 				compatible = "atmel,at91sam9g46-aes";
 				reg = <0xf0034000 0x100>;
 				interrupts = <39 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -307,7 +307,7 @@ AT91_XDMAC_DT_PERID(32))>,
 				status = "okay";
 			};
 
-			tdes: tdes@...38000 {
+			tdes: crypto@...38000 {
 				compatible = "atmel,at91sam9g46-tdes";
 				reg = <0xf0038000 0x100>;
 				interrupts = <40 IRQ_TYPE_LEVEL_HIGH 0>;
diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
index c700c3b19e4c..51e80611aa02 100644
--- a/arch/arm/boot/dts/sama5d2.dtsi
+++ b/arch/arm/boot/dts/sama5d2.dtsi
@@ -306,7 +306,7 @@ qspi1: spi@...24000 {
 				status = "disabled";
 			};
 
-			sha@...28000 {
+			sha: crypto@...28000 {
 				compatible = "atmel,at91sam9g46-sha";
 				reg = <0xf0028000 0x100>;
 				interrupts = <12 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -319,7 +319,7 @@ sha@...28000 {
 				status = "okay";
 			};
 
-			aes@...2c000 {
+			aes: crypto@...2c000 {
 				compatible = "atmel,at91sam9g46-aes";
 				reg = <0xf002c000 0x100>;
 				interrupts = <9 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -1084,7 +1084,7 @@ pioBU: secumod@...40000 {
 				#gpio-cells = <2>;
 			};
 
-			tdes@...44000 {
+			tdes: crypto@...44000 {
 				compatible = "atmel,at91sam9g46-tdes";
 				reg = <0xfc044000 0x100>;
 				interrupts = <11 IRQ_TYPE_LEVEL_HIGH 0>;
diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
index d1841bffe3c5..8fa423c52592 100644
--- a/arch/arm/boot/dts/sama5d3.dtsi
+++ b/arch/arm/boot/dts/sama5d3.dtsi
@@ -381,7 +381,7 @@ usart3: serial@...24000 {
 				status = "disabled";
 			};
 
-			sha@...34000 {
+			sha: crypto@...34000 {
 				compatible = "atmel,at91sam9g46-sha";
 				reg = <0xf8034000 0x100>;
 				interrupts = <42 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -391,7 +391,7 @@ sha@...34000 {
 				clock-names = "sha_clk";
 			};
 
-			aes@...38000 {
+			aes: crypto@...38000 {
 				compatible = "atmel,at91sam9g46-aes";
 				reg = <0xf8038000 0x100>;
 				interrupts = <43 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -402,7 +402,7 @@ aes@...38000 {
 				clock-names = "aes_clk";
 			};
 
-			tdes@...3c000 {
+			tdes: crypto@...3c000 {
 				compatible = "atmel,at91sam9g46-tdes";
 				reg = <0xf803c000 0x100>;
 				interrupts = <44 IRQ_TYPE_LEVEL_HIGH 0>;
diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi
index f6e3e6f57252..1dff79a29012 100644
--- a/arch/arm/boot/dts/sama5d4.dtsi
+++ b/arch/arm/boot/dts/sama5d4.dtsi
@@ -673,7 +673,7 @@ adc0: adc@...34000 {
 				status = "disabled";
 			};
 
-			aes@...44000 {
+			aes: crypto@...44000 {
 				compatible = "atmel,at91sam9g46-aes";
 				reg = <0xfc044000 0x100>;
 				interrupts = <12 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -687,7 +687,7 @@ aes@...44000 {
 				status = "okay";
 			};
 
-			tdes@...4c000 {
+			tdes: crypto@...4c000 {
 				compatible = "atmel,at91sam9g46-tdes";
 				reg = <0xfc04c000 0x100>;
 				interrupts = <14 IRQ_TYPE_LEVEL_HIGH 0>;
@@ -701,7 +701,7 @@ tdes@...4c000 {
 				status = "okay";
 			};
 
-			sha@...50000 {
+			sha: crypto@...50000 {
 				compatible = "atmel,at91sam9g46-sha";
 				reg = <0xfc050000 0x100>;
 				interrupts = <15 IRQ_TYPE_LEVEL_HIGH 0>;
-- 
2.25.1

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ