[PATCH] ARM: dts: at91/trivial: fix typo in crypto DT naming

nicolas.ferre@microchip.com posted 1 patch 2 years, 3 months ago
arch/arm/boot/dts/microchip/sama5d4.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] ARM: dts: at91/trivial: fix typo in crypto DT naming
Posted by nicolas.ferre@microchip.com 2 years, 3 months ago
From: Nicolas Ferre <nicolas.ferre@microchip.com>

Fix typo in DT name for TDES node.

Fixes: 4b6140b96dfe ("ARM: dts: at91: Use the generic "crypto" node name for the crypto IPs")
Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>
---
 arch/arm/boot/dts/microchip/sama5d4.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/microchip/sama5d4.dtsi b/arch/arm/boot/dts/microchip/sama5d4.dtsi
index 41284e013f53..cbfc60fed37c 100644
--- a/arch/arm/boot/dts/microchip/sama5d4.dtsi
+++ b/arch/arm/boot/dts/microchip/sama5d4.dtsi
@@ -694,7 +694,7 @@ aes: crypto@fc044000 {
 				clock-names = "aes_clk";
 			};
 
-			tdes: crpyto@fc04c000 {
+			tdes: crypto@fc04c000 {
 				compatible = "atmel,at91sam9g46-tdes";
 				reg = <0xfc04c000 0x100>;
 				interrupts = <14 IRQ_TYPE_LEVEL_HIGH 0>;
-- 
2.39.2
Re: [PATCH] ARM: dts: at91/trivial: fix typo in crypto DT naming
Posted by claudiu beznea 2 years, 3 months ago

On 06.09.2023 16:58, nicolas.ferre@microchip.com wrote:
> From: Nicolas Ferre <nicolas.ferre@microchip.com>
> 
> Fix typo in DT name for TDES node.
> 
> Fixes: 4b6140b96dfe ("ARM: dts: at91: Use the generic "crypto" node name for the crypto IPs")
> Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Applied, thanks!

Hi, Nicolas,

Also, I've removed the fixes tag as the node is not referenced anywhere by
its name thus no need for fixes in this case. Let me know if you consider
differently.

Thank you,
Claudiu Beznea
> ---
>  arch/arm/boot/dts/microchip/sama5d4.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/microchip/sama5d4.dtsi b/arch/arm/boot/dts/microchip/sama5d4.dtsi
> index 41284e013f53..cbfc60fed37c 100644
> --- a/arch/arm/boot/dts/microchip/sama5d4.dtsi
> +++ b/arch/arm/boot/dts/microchip/sama5d4.dtsi
> @@ -694,7 +694,7 @@ aes: crypto@fc044000 {
>  				clock-names = "aes_clk";
>  			};
>  
> -			tdes: crpyto@fc04c000 {
> +			tdes: crypto@fc04c000 {
>  				compatible = "atmel,at91sam9g46-tdes";
>  				reg = <0xfc04c000 0x100>;
>  				interrupts = <14 IRQ_TYPE_LEVEL_HIGH 0>;
Re: [PATCH] ARM: dts: at91/trivial: fix typo in crypto DT naming
Posted by Nicolas Ferre 2 years, 3 months ago
On 18/09/2023 at 06:08, claudiu beznea wrote:
> On 06.09.2023 16:58, nicolas.ferre@microchip.com wrote:
>> From: Nicolas Ferre <nicolas.ferre@microchip.com>
>>
>> Fix typo in DT name for TDES node.
>>
>> Fixes: 4b6140b96dfe ("ARM: dts: at91: Use the generic "crypto" node name for the crypto IPs")
>> Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> 
> Applied, thanks!
> 
> Hi, Nicolas,
> 
> Also, I've removed the fixes tag as the node is not referenced anywhere by
> its name thus no need for fixes in this case. Let me know if you consider
> differently.

Agreed, sure Claudiu, let's not add more work to the "stable" people.

Thanks, best regards,
   Nicolas

>> ---
>>   arch/arm/boot/dts/microchip/sama5d4.dtsi | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/microchip/sama5d4.dtsi b/arch/arm/boot/dts/microchip/sama5d4.dtsi
>> index 41284e013f53..cbfc60fed37c 100644
>> --- a/arch/arm/boot/dts/microchip/sama5d4.dtsi
>> +++ b/arch/arm/boot/dts/microchip/sama5d4.dtsi
>> @@ -694,7 +694,7 @@ aes: crypto@fc044000 {
>>                                clock-names = "aes_clk";
>>                        };
>>
>> -                     tdes: crpyto@fc04c000 {
>> +                     tdes: crypto@fc04c000 {
>>                                compatible = "atmel,at91sam9g46-tdes";
>>                                reg = <0xfc04c000 0x100>;
>>                                interrupts = <14 IRQ_TYPE_LEVEL_HIGH 0>;
Re: [PATCH] ARM: dts: at91/trivial: fix typo in crypto DT naming
Posted by Tudor Ambarus 2 years, 3 months ago

On 9/6/23 14:58, nicolas.ferre@microchip.com wrote:
> From: Nicolas Ferre <nicolas.ferre@microchip.com>
> 
> Fix typo in DT name for TDES node.
> 
> Fixes: 4b6140b96dfe ("ARM: dts: at91: Use the generic "crypto" node name for the crypto IPs")
> Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Reviewed-by: Tudor Ambarus <tudor.ambarus@linaro.org>

> ---
>  arch/arm/boot/dts/microchip/sama5d4.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/microchip/sama5d4.dtsi b/arch/arm/boot/dts/microchip/sama5d4.dtsi
> index 41284e013f53..cbfc60fed37c 100644
> --- a/arch/arm/boot/dts/microchip/sama5d4.dtsi
> +++ b/arch/arm/boot/dts/microchip/sama5d4.dtsi
> @@ -694,7 +694,7 @@ aes: crypto@fc044000 {
>  				clock-names = "aes_clk";
>  			};
>  
> -			tdes: crpyto@fc04c000 {
> +			tdes: crypto@fc04c000 {
>  				compatible = "atmel,at91sam9g46-tdes";
>  				reg = <0xfc04c000 0x100>;
>  				interrupts = <14 IRQ_TYPE_LEVEL_HIGH 0>;