Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <a5f5c64d-672d-2d1b-5243-af0e2a7186f5@foss.st.com>
Date:   Wed, 16 Aug 2023 09:16:27 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH 2/3] ARM: dts: st: move leds out of soc in
 stih418-b2264.dts
Content-Language: en-US
To:     Alain Volmat <avolmat@me.com>, Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>
CC:     <linux-arm-kernel@lists.infradead.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20230702161936.134429-1-avolmat@me.com>
 <20230702161936.134429-3-avolmat@me.com>
From:   Patrice CHOTARD <patrice.chotard@foss.st.com>
In-Reply-To: <20230702161936.134429-3-avolmat@me.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk


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