ARM: dts: allwinner: Fix remaining DT validation errors

[PATCH 4/4] ARM: dts: allwinner: Replace status "failed" with "fail"

Posted by Chen-Yu Tsai 1 month, 2 weeks ago

The device tree bindings specify using "fail", not "failed".

Fix up all the ones that are wrong.

Signed-off-by: Chen-Yu Tsai <wens@kernel.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 {
-- 
2.47.3

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

Posted by Jernej Škrabec 1 month, 2 weeks ago

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"?

Best regards,
Jernej

> 
> Signed-off-by: Chen-Yu Tsai <wens@kernel.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 {
>

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

Posted by Chen-Yu Tsai 1 month ago

On Fri, Dec 26, 2025 at 2:49 AM Jernej Škrabec <jernej.skrabec@gmail.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@kernel.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 {
> >
>
>
>
>
>

[PATCH 1/4] dt-bindings: media: sun4i-a10-video-engine: Add interconnect properties
[PATCH 2/4] dt-bindings: pinctrl: sunxi: Allow pinmux sub-pattern with leading numbers
[PATCH 3/4] ARM: dts: allwinner: sun5i-a13-utoo-p66: delete "power-gpios" property
[PATCH 4/4] ARM: dts: allwinner: Replace status "failed" with "fail"