Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Luca Weiss <luca@z3ntu.xyz>
To:     Lee Jones <lee@kernel.org>, Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        ~postmarketos/upstreaming@lists.sr.ht
Cc:     Daniel Thompson <daniel.thompson@linaro.org>,
        Jingoo Han <jingoohan1@gmail.com>, Pavel Machek <pavel@ucw.cz>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Helge Deller <deller@gmx.de>,
        Uwe =?ISO-8859-1?Q?Kleine=2DK=F6nig?= 
        <u.kleine-koenig@pengutronix.de>,
        Artur Weber <aweber.kernel@gmail.com>,
        dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-msm@vger.kernel.org, linux-tegra@vger.kernel.org,
        linux-fbdev@vger.kernel.org, linux-pwm@vger.kernel.org,
        ~postmarketos/upstreaming@lists.sr.ht,
        Artur Weber <aweber.kernel@gmail.com>
Subject: Re: [PATCH 3/4] ARM: dts: adapt to LP855X bindings changes
Date:   Sat, 29 Apr 2023 13:14:12 +0200
Message-ID: <3414865.QJadu78ljV@z3ntu.xyz>
In-Reply-To: <20230429104534.28943-4-aweber.kernel@gmail.com>
References: <20230429104534.28943-1-aweber.kernel@gmail.com>
 <20230429104534.28943-4-aweber.kernel@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Precedence: bulk

On Samstag, 29. April 2023 12:45:33 CEST Artur Weber wrote:
> Change underscores in ROM node names to dashes, and remove deprecated
> pwm-period property.
> 
> Signed-off-by: Artur Weber <aweber.kernel@gmail.com>

Reviewed-by: Luca Weiss <luca@z3ntu.xyz>

> ---
>  .../dts/qcom-apq8026-samsung-matisse-wifi.dts |  1 -
>  ...-msm8974pro-sony-xperia-shinano-castor.dts | 23 ++++++++++---------
>  2 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8026-samsung-matisse-wifi.dts
> b/arch/arm/boot/dts/qcom-apq8026-samsung-matisse-wifi.dts index
> 91b860e24681..884d99297d4c 100644
> --- a/arch/arm/boot/dts/qcom-apq8026-samsung-matisse-wifi.dts
> +++ b/arch/arm/boot/dts/qcom-apq8026-samsung-matisse-wifi.dts
> @@ -99,7 +99,6 @@ backlight@2c {
> 
>  			dev-ctrl = /bits/ 8 <0x80>;
>  			init-brt = /bits/ 8 <0x3f>;
> -			pwm-period = <100000>;
> 
>  			pwms = <&backlight_pwm 0 100000>;
>  			pwm-names = "lp8556";
> diff --git
> a/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts
> b/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts index
> 04bc58d87abf..2396253f953a 100644
> --- a/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts
> +++ b/arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dts
> @@ -150,47 +150,48 @@ lp8566_wled: backlight@2c {
>  		bl-name = "backlight";
>  		dev-ctrl = /bits/ 8 <0x05>;
>  		init-brt = /bits/ 8 <0x3f>;
> -		rom_a0h {
> +
> +		rom-a0h {
>  			rom-addr = /bits/ 8 <0xa0>;
>  			rom-val = /bits/ 8 <0xff>;
>  		};
> -		rom_a1h {
> +		rom-a1h {
>  			rom-addr = /bits/ 8 <0xa1>;
>  			rom-val = /bits/ 8 <0x3f>;
>  		};
> -		rom_a2h {
> +		rom-a2h {
>  			rom-addr = /bits/ 8 <0xa2>;
>  			rom-val = /bits/ 8 <0x20>;
>  		};
> -		rom_a3h {
> +		rom-a3h {
>  			rom-addr = /bits/ 8 <0xa3>;
>  			rom-val = /bits/ 8 <0x5e>;
>  		};
> -		rom_a4h {
> +		rom-a4h {
>  			rom-addr = /bits/ 8 <0xa4>;
>  			rom-val = /bits/ 8 <0x02>;
>  		};
> -		rom_a5h {
> +		rom-a5h {
>  			rom-addr = /bits/ 8 <0xa5>;
>  			rom-val = /bits/ 8 <0x04>;
>  		};
> -		rom_a6h {
> +		rom-a6h {
>  			rom-addr = /bits/ 8 <0xa6>;
>  			rom-val = /bits/ 8 <0x80>;
>  		};
> -		rom_a7h {
> +		rom-a7h {
>  			rom-addr = /bits/ 8 <0xa7>;
>  			rom-val = /bits/ 8 <0xf7>;
>  		};
> -		rom_a9h {
> +		rom-a9h {
>  			rom-addr = /bits/ 8 <0xa9>;
>  			rom-val = /bits/ 8 <0x80>;
>  		};
> -		rom_aah {
> +		rom-aah {
>  			rom-addr = /bits/ 8 <0xaa>;
>  			rom-val = /bits/ 8 <0x0f>;
>  		};
> -		rom_aeh {
> +		rom-aeh {
>  			rom-addr = /bits/ 8 <0xae>;
>  			rom-val = /bits/ 8 <0x0f>;
>  		};