ARM: dts: st: dts fixes in stih418-b2264.dts

[PATCH 2/3] ARM: dts: st: move leds out of soc in stih418-b2264.dts

Posted by Alain Volmat 2 years, 7 months ago

Move the leds node out of the soc section
and correct the following warning:
Warning (simple_bus_reg): /soc/leds: missing or empty reg/ranges property

Signed-off-by: Alain Volmat <avolmat@me.com>
---
 arch/arm/boot/dts/st/stih418-b2264.dts | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/st/stih418-b2264.dts b/arch/arm/boot/dts/st/stih418-b2264.dts
index f496ca28850d..d4874282cfba 100644
--- a/arch/arm/boot/dts/st/stih418-b2264.dts
+++ b/arch/arm/boot/dts/st/stih418-b2264.dts
@@ -73,15 +73,15 @@ aliases {
 		ethernet0 = &ethernet0;
 	};
 
-	soc {
-		leds {
-			compatible = "gpio-leds";
-			led-green {
-				gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
-				default-state = "off";
-			};
+	leds {
+		compatible = "gpio-leds";
+		led-green {
+			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
+			default-state = "off";
 		};
+	};
 
+	soc {
 		pin-controller-sbc@961f080 {
 			gmac1 {
 				rgmii1-0 {
-- 
2.34.1

Re: [PATCH 2/3] ARM: dts: st: move leds out of soc in stih418-b2264.dts

Posted by Patrice CHOTARD 2 years, 5 months ago


On 7/2/23 18:19, Alain Volmat wrote:
> Move the leds node out of the soc section
> and correct the following warning:
> Warning (simple_bus_reg): /soc/leds: missing or empty reg/ranges property
> 
> Signed-off-by: Alain Volmat <avolmat@me.com>
> ---
>  arch/arm/boot/dts/st/stih418-b2264.dts | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/st/stih418-b2264.dts b/arch/arm/boot/dts/st/stih418-b2264.dts
> index f496ca28850d..d4874282cfba 100644
> --- a/arch/arm/boot/dts/st/stih418-b2264.dts
> +++ b/arch/arm/boot/dts/st/stih418-b2264.dts
> @@ -73,15 +73,15 @@ aliases {
>  		ethernet0 = &ethernet0;
>  	};
>  
> -	soc {
> -		leds {
> -			compatible = "gpio-leds";
> -			led-green {
> -				gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
> -				default-state = "off";
> -			};
> +	leds {
> +		compatible = "gpio-leds";
> +		led-green {
> +			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
> +			default-state = "off";
>  		};
> +	};
>  
> +	soc {
>  		pin-controller-sbc@961f080 {
>  			gmac1 {
>  				rgmii1-0 {
Applied on sti-next

Thanks
Patrice

Re: [PATCH 2/3] ARM: dts: st: move leds out of soc in stih418-b2264.dts

Posted by Patrice CHOTARD 2 years, 5 months ago


On 8/16/23 09:16, Patrice CHOTARD wrote:
> 
> 
> On 7/2/23 18:19, Alain Volmat wrote:
>> Move the leds node out of the soc section
>> and correct the following warning:
>> Warning (simple_bus_reg): /soc/leds: missing or empty reg/ranges property
>>
>> Signed-off-by: Alain Volmat <avolmat@me.com>
>> ---
>>  arch/arm/boot/dts/st/stih418-b2264.dts | 14 +++++++-------
>>  1 file changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/st/stih418-b2264.dts b/arch/arm/boot/dts/st/stih418-b2264.dts
>> index f496ca28850d..d4874282cfba 100644
>> --- a/arch/arm/boot/dts/st/stih418-b2264.dts
>> +++ b/arch/arm/boot/dts/st/stih418-b2264.dts
>> @@ -73,15 +73,15 @@ aliases {
>>  		ethernet0 = &ethernet0;
>>  	};
>>  
>> -	soc {
>> -		leds {
>> -			compatible = "gpio-leds";
>> -			led-green {
>> -				gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
>> -				default-state = "off";
>> -			};
>> +	leds {
>> +		compatible = "gpio-leds";
>> +		led-green {
>> +			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
>> +			default-state = "off";
>>  		};
>> +	};
>>  
>> +	soc {
>>  		pin-controller-sbc@961f080 {
>>  			gmac1 {
>>  				rgmii1-0 {
> Applied on sti-next
> 
> Thanks
> Patrice
Applied on sti-next

Thanks
Patrice

Re: [PATCH 2/3] ARM: dts: st: move leds out of soc in stih418-b2264.dts

Posted by Patrice CHOTARD 2 years, 5 months ago


On 7/2/23 18:19, Alain Volmat wrote:
> Move the leds node out of the soc section
> and correct the following warning:
> Warning (simple_bus_reg): /soc/leds: missing or empty reg/ranges property
> 
> Signed-off-by: Alain Volmat <avolmat@me.com>
> ---
>  arch/arm/boot/dts/st/stih418-b2264.dts | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/st/stih418-b2264.dts b/arch/arm/boot/dts/st/stih418-b2264.dts
> index f496ca28850d..d4874282cfba 100644
> --- a/arch/arm/boot/dts/st/stih418-b2264.dts
> +++ b/arch/arm/boot/dts/st/stih418-b2264.dts
> @@ -73,15 +73,15 @@ aliases {
>  		ethernet0 = &ethernet0;
>  	};
>  
> -	soc {
> -		leds {
> -			compatible = "gpio-leds";
> -			led-green {
> -				gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
> -				default-state = "off";
> -			};
> +	leds {
> +		compatible = "gpio-leds";
> +		led-green {
> +			gpios = <&pio1 3 GPIO_ACTIVE_HIGH>;
> +			default-state = "off";
>  		};
> +	};
>  
> +	soc {
>  		pin-controller-sbc@961f080 {
>  			gmac1 {
>  				rgmii1-0 {

Acked-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks
PAtrice