Date:   Mon, 13 Mar 2023 16:28:34 +0800
From:   Shawn Guo <shawnguo@kernel.org>
To:     Michal =?utf-8?B?Vm9rw6HEjQ==?= <michal.vokac@ysoft.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/6] ARM: dts: imx6dl-yapp4: Change LED channel names
 according to dt-binding
Message-ID: <20230313082834.GO143566@dragon>
References: <20230210154855.3086900-1-michal.vokac@ysoft.com>
 <20230210154855.3086900-3-michal.vokac@ysoft.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20230210154855.3086900-3-michal.vokac@ysoft.com>
Precedence: bulk

On Fri, Feb 10, 2023 at 04:48:51PM +0100, Michal Vokáč wrote:
> The lp55xx LED controller binding was converted to schema and dtbs_check
> now complains:
> 
>   led-controller@30: 'chan@0', 'chan@1', 'chan@2' do not match any of the regexes
> 
> Use correct names to fix that.
> 
> Signed-off-by: Michal Vokáč <michal.vokac@ysoft.com>

It's been address as part of this commit:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4b0d1f2738899dbcc7a026d826373530019aa31b

Shawn

> ---
>  arch/arm/boot/dts/imx6dl-yapp4-common.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx6dl-yapp4-common.dtsi b/arch/arm/boot/dts/imx6dl-yapp4-common.dtsi
> index 52162e8c7274..aacbf317feea 100644
> --- a/arch/arm/boot/dts/imx6dl-yapp4-common.dtsi
> +++ b/arch/arm/boot/dts/imx6dl-yapp4-common.dtsi
> @@ -274,7 +274,7 @@ leds: led-controller@30 {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
>  
> -		chan@0 {
> +		led@0 {
>  			chan-name = "R";
>  			led-cur = /bits/ 8 <0x20>;
>  			max-cur = /bits/ 8 <0x60>;
> @@ -282,7 +282,7 @@ chan@0 {
>  			color = <LED_COLOR_ID_RED>;
>  		};
>  
> -		chan@1 {
> +		led@1 {
>  			chan-name = "G";
>  			led-cur = /bits/ 8 <0x20>;
>  			max-cur = /bits/ 8 <0x60>;
> @@ -290,7 +290,7 @@ chan@1 {
>  			color = <LED_COLOR_ID_GREEN>;
>  		};
>  
> -		chan@2 {
> +		led@2 {
>  			chan-name = "B";
>  			led-cur = /bits/ 8 <0x20>;
>  			max-cur = /bits/ 8 <0x60>;
> -- 
> 2.25.1
>