linux-kernel - Re: [PATCH 4/4] ARM: dts: allwinner: Replace status "failed" with "fail"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAGb2v65LzhdXfX16HTgGbC7OYkuX=vqCzB65=BqD4i1SiCQeRw@mail.gmail.com>
Date: Thu, 8 Jan 2026 23:07:52 +0800
From: Chen-Yu Tsai <wens@...nel.org>
To: Jernej Škrabec <jernej.skrabec@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, 
	Conor Dooley <conor+dt@...nel.org>, Jernej Skrabec <jernej@...nel.org>, 
	Samuel Holland <samuel@...lland.org>, Linus Walleij <linusw@...nel.org>, 
	Mauro Carvalho Chehab <mchehab@...nel.org>, devicetree@...r.kernel.org, 
	linux-sunxi@...ts.linux.dev, linux-gpio@...r.kernel.org, 
	linux-media@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/4] ARM: dts: allwinner: Replace status "failed" with "fail"

On Fri, Dec 26, 2025 at 2:49 AM Jernej Škrabec <jernej.skrabec@...il.com> wrote:
>
> Dne četrtek, 25. december 2025 ob 11:36:15 Srednjeevropski standardni čas je Chen-Yu Tsai napisal(a):
> > The device tree bindings specify using "fail", not "failed".
> >
> > Fix up all the ones that are wrong.
>
> While patch indeed fixes the issue, I wonder if "fail" is proper value?
> According to specs, it means that device needs repair. What about just
> keep "disabled"?

"disabled" isn't right either, as that means its disconnected or not used.
These are definitely connected.

On the other hand we could fix these and describe the supplies for the
external as supplies for the pin banks? But I doubt anyone has these
devices around for testing.

ChenYu

> Best regards,
> Jernej
>
> >
> > Signed-off-by: Chen-Yu Tsai <wens@...nel.org>
> > ---
> >  arch/arm/boot/dts/allwinner/sun4i-a10-dserve-dsrv9703c.dts | 2 +-
> >  arch/arm/boot/dts/allwinner/sun4i-a10-pov-protab2-ips9.dts | 2 +-
> >  arch/arm/boot/dts/allwinner/sun6i-a31-hummingbird.dts      | 2 +-
> >  arch/arm/boot/dts/allwinner/sun6i-a31s-primo81.dts         | 2 +-
> >  4 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/allwinner/sun4i-a10-dserve-dsrv9703c.dts b/arch/arm/boot/dts/allwinner/sun4i-a10-dserve-dsrv9703c.dts
> > index 63e77c05bfda..f2413ba6a858 100644
> > --- a/arch/arm/boot/dts/allwinner/sun4i-a10-dserve-dsrv9703c.dts
> > +++ b/arch/arm/boot/dts/allwinner/sun4i-a10-dserve-dsrv9703c.dts
> > @@ -112,7 +112,7 @@ axp209: pmic@34 {
> >
> >  &i2c1 {
> >       /* pull-ups and devices require AXP209 LDO3 */
> > -     status = "failed";
> > +     status = "fail";
> >  };
> >
> >  &i2c2 {
> > diff --git a/arch/arm/boot/dts/allwinner/sun4i-a10-pov-protab2-ips9.dts b/arch/arm/boot/dts/allwinner/sun4i-a10-pov-protab2-ips9.dts
> > index c32596947647..e0c7099015da 100644
> > --- a/arch/arm/boot/dts/allwinner/sun4i-a10-pov-protab2-ips9.dts
> > +++ b/arch/arm/boot/dts/allwinner/sun4i-a10-pov-protab2-ips9.dts
> > @@ -96,7 +96,7 @@ axp209: pmic@34 {
> >
> >  &i2c1 {
> >       /* pull-ups and devices require AXP209 LDO3 */
> > -     status = "failed";
> > +     status = "fail";
> >  };
> >
> >  &i2c2 {
> > diff --git a/arch/arm/boot/dts/allwinner/sun6i-a31-hummingbird.dts b/arch/arm/boot/dts/allwinner/sun6i-a31-hummingbird.dts
> > index 5bce7a32651e..5dfd36e3a49d 100644
> > --- a/arch/arm/boot/dts/allwinner/sun6i-a31-hummingbird.dts
> > +++ b/arch/arm/boot/dts/allwinner/sun6i-a31-hummingbird.dts
> > @@ -170,7 +170,7 @@ hdmi_out_con: endpoint {
> >
> >  &i2c0 {
> >       /* pull-ups and devices require AXP221 DLDO3 */
> > -     status = "failed";
> > +     status = "fail";
> >  };
> >
> >  &i2c1 {
> > diff --git a/arch/arm/boot/dts/allwinner/sun6i-a31s-primo81.dts b/arch/arm/boot/dts/allwinner/sun6i-a31s-primo81.dts
> > index b32b70ada7fd..fefd887fbc39 100644
> > --- a/arch/arm/boot/dts/allwinner/sun6i-a31s-primo81.dts
> > +++ b/arch/arm/boot/dts/allwinner/sun6i-a31s-primo81.dts
> > @@ -90,7 +90,7 @@ hdmi_out_con: endpoint {
> >
> >  &i2c0 {
> >       /* pull-ups and device VDDIO use AXP221 DLDO3 */
> > -     status = "failed";
> > +     status = "fail";
> >  };
> >
> >  &i2c1 {
> >
>
>
>
>
>