[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4f2f2f95-d0f4-4d2f-8873-b1026340a8c9@kernel.org>
Date: Tue, 8 Jul 2025 08:27:50 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Shawn Guo <shawnguo2@...h.net>, Wojciech Dubowik <Wojciech.Dubowik@...com>
Cc: linux-kernel@...r.kernel.org, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>, Kees Cook <kees@...nel.org>,
Tony Luck <tony.luck@...el.com>, "Guilherme G. Piccoli"
<gpiccoli@...lia.com>, Alexander Stein <alexander.stein@...tq-group.com>,
Marek Vasut <marex@...x.de>, Frank Li <Frank.Li@....com>,
Markus Niebel <Markus.Niebel@...group.com>,
João Paulo Gonçalves <joao.goncalves@...adex.com>,
Frieder Schrempf <frieder.schrempf@...tron.de>,
Michael Walle <mwalle@...nel.org>, Heiko Schocher <hs@...x.de>,
Francesco Dolcini <francesco.dolcini@...adex.com>,
Max Merchel <Max.Merchel@...tq-group.com>,
Primoz Fiser <primoz.fiser@...ik.com>, Tim Harvey <tharvey@...eworks.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Heiko Stuebner <heiko@...ech.de>, Caleb James DeLisle <cjd@...ns.fr>,
Junhao Xie <bigfoot@...ssfun.cn>, Andre Przywara <andre.przywara@....com>,
Rafał Miłecki <rafal@...ecki.pl>,
Kever Yang <kever.yang@...k-chips.com>,
Manivannan Sadhasivam <mani@...nel.org>, devicetree@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
linux-hardening@...r.kernel.org, Tobias Graemer <Tobias.Graemer@...com>
Subject: Re: [PATCH v3 3/4] arm64: dts: freescale: Add Mettler-Toledo
Snowflake V2 support
On 08/07/2025 08:20, Shawn Guo wrote:
>> +
>> + backlight: backlight {
>> + compatible = "pwm-backlight";
>> + pwms = <&pwm1 0 500000 0>;
>> + power-supply = <®_5v>;
>> + brightness-levels = <0 100>;
>> + num-interpolated-steps = <100>;
>> + default-brightness-level = <100>;
>> + status = "okay";
>
> We usually use "okay" status to enable "disabled" devices. It doesn't
> seem to be needed here.
I already asked for this in v2. Changelog claims it was done, but I
believe nothing here improved and they just sent the same patchset
wasting review effort.
Best regards,
Krzysztof
Powered by blists - more mailing lists