[PATCH] ARM: dts: Fix node status to "okay" on TI boards

Nishanth Menon nm at ti.com
Fri Aug 20 19:51:03 CEST 2021


On 13:41-20210820, Roger Quadros wrote:
> As per Device Tree core schema [1], status parameter of nodes can be either
> "okay" or "disabled".
> 
> U-boot Driver Model does not recognize status="ok" and treats
> the node as disabled.
> 

Did you miss:
[1] https://github.com/devicetree-org/devicetree-specification/releases/tag/v0.3

Also, I think it might be better to refer to the specification than the
schema implementation of the specification.

> Signed-off-by: Roger Quadros <rogerq at kernel.org>
> ---
>  arch/arm/dts/am3517-evm-ui.dtsi                 | 4 ++--
>  arch/arm/dts/am3517-evm.dts                     | 2 +-
>  arch/arm/dts/am437x-gp-evm.dts                  | 2 +-
>  arch/arm/dts/am43x-epos-evm.dts                 | 2 +-
>  arch/arm/dts/am57xx-beagle-x15-common.dtsi      | 6 +++---
>  arch/arm/dts/da850-evm.dts                      | 2 +-
>  arch/arm/dts/dra7-evm.dts                       | 2 +-
>  arch/arm/dts/dra72-evm-common.dtsi              | 6 +++---
>  arch/arm/dts/keystone-k2e-evm.dts               | 2 +-
>  arch/arm/dts/keystone-k2hk-evm.dts              | 2 +-
>  arch/arm/dts/keystone-k2l-evm.dts               | 2 +-
>  arch/arm/dts/omap3-beagle-xm.dts                | 4 ++--
>  arch/arm/dts/omap3-beagle.dts                   | 6 +++---
>  arch/arm/dts/omap3-igep0020-common.dtsi         | 2 +-
>  arch/arm/dts/omap3-panel-sharp-ls037v7dw01.dtsi | 2 +-
>  arch/arm/dts/omap34xx.dtsi                      | 2 +-
>  arch/arm/dts/omap36xx.dtsi                      | 2 +-
>  arch/arm/dts/omap4-panda-common.dtsi            | 6 +++---
>  arch/arm/dts/omap4-sdp.dts                      | 8 ++++----
>  arch/arm/dts/omap5-board-common.dtsi            | 4 ++--
>  20 files changed, 34 insertions(+), 34 deletions(-)
> 
> diff --git a/arch/arm/dts/am3517-evm-ui.dtsi b/arch/arm/dts/am3517-evm-ui.dtsi
> index e841918c1c..54aa2522aa 100644
> --- a/arch/arm/dts/am3517-evm-ui.dtsi
> +++ b/arch/arm/dts/am3517-evm-ui.dtsi
> @@ -186,14 +186,14 @@
>  };
>  
>  &mcbsp1 {
> -	status = "ok";
> +	status = "okay";
>  	#sound-dai-cells = <0>;
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&mcbsp1_pins>;
>  };
>  
>  &mcbsp2 {
> -	status = "ok";
> +	status = "okay";
>  	#sound-dai-cells = <0>;
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&mcbsp2_pins>;
> diff --git a/arch/arm/dts/am3517-evm.dts b/arch/arm/dts/am3517-evm.dts
> index 3527c0f2df..935c471c97 100644
> --- a/arch/arm/dts/am3517-evm.dts
> +++ b/arch/arm/dts/am3517-evm.dts
> @@ -193,7 +193,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&dss_dpi_pins>;
> diff --git a/arch/arm/dts/am437x-gp-evm.dts b/arch/arm/dts/am437x-gp-evm.dts
> index 3c500d52db..21f7691f49 100644
> --- a/arch/arm/dts/am437x-gp-evm.dts
> +++ b/arch/arm/dts/am437x-gp-evm.dts
> @@ -742,7 +742,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&dss_pins>;
> diff --git a/arch/arm/dts/am43x-epos-evm.dts b/arch/arm/dts/am43x-epos-evm.dts
> index 65f157ed59..b940bc6ccf 100644
> --- a/arch/arm/dts/am43x-epos-evm.dts
> +++ b/arch/arm/dts/am43x-epos-evm.dts
> @@ -752,7 +752,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&dss_pins>;
> diff --git a/arch/arm/dts/am57xx-beagle-x15-common.dtsi b/arch/arm/dts/am57xx-beagle-x15-common.dtsi
> index d6b94d528f..1912ea9a15 100644
> --- a/arch/arm/dts/am57xx-beagle-x15-common.dtsi
> +++ b/arch/arm/dts/am57xx-beagle-x15-common.dtsi
> @@ -528,13 +528,13 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	vdda_video-supply = <&ldoln_reg>;
>  };
>  
>  &hdmi {
> -	status = "ok";
> +	status = "okay";
>  	vdda-supply = <&ldo4_reg>;
>  
>  	port {
> @@ -545,7 +545,7 @@
>  };
>  
>  &pcie1_rc {
> -	status = "ok";
> +	status = "okay";
>  	gpios = <&gpio2 8 GPIO_ACTIVE_LOW>;
>  };
>  
> diff --git a/arch/arm/dts/da850-evm.dts b/arch/arm/dts/da850-evm.dts
> index f04bc3e153..b331cefd18 100644
> --- a/arch/arm/dts/da850-evm.dts
> +++ b/arch/arm/dts/da850-evm.dts
> @@ -405,7 +405,7 @@
>  &aemif {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&nand_pins>;
> -	status = "ok";
> +	status = "okay";
>  	cs3 {
>  		#address-cells = <2>;
>  		#size-cells = <1>;
> diff --git a/arch/arm/dts/dra7-evm.dts b/arch/arm/dts/dra7-evm.dts
> index 43de9638e3..8e9a1a80a8 100644
> --- a/arch/arm/dts/dra7-evm.dts
> +++ b/arch/arm/dts/dra7-evm.dts
> @@ -501,7 +501,7 @@
>  };
>  
>  &dcan1 {
> -	status = "ok";
> +	status = "okay";
>  	pinctrl-names = "default", "sleep", "active";
>  	pinctrl-0 = <&dcan1_pins_sleep>;
>  	pinctrl-1 = <&dcan1_pins_sleep>;
> diff --git a/arch/arm/dts/dra72-evm-common.dtsi b/arch/arm/dts/dra72-evm-common.dtsi
> index 2e485a13df..964e5e9b90 100644
> --- a/arch/arm/dts/dra72-evm-common.dtsi
> +++ b/arch/arm/dts/dra72-evm-common.dtsi
> @@ -430,7 +430,7 @@
>  };
>  
>  &dcan1 {
> -	status = "ok";
> +	status = "okay";
>  	pinctrl-names = "default", "sleep", "active";
>  	pinctrl-0 = <&dcan1_pins_sleep>;
>  	pinctrl-1 = <&dcan1_pins_sleep>;
> @@ -499,11 +499,11 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &hdmi {
> -	status = "ok";
> +	status = "okay";
>  
>  	port {
>  		hdmi_out: endpoint {
> diff --git a/arch/arm/dts/keystone-k2e-evm.dts b/arch/arm/dts/keystone-k2e-evm.dts
> index 9288df21ce..bb197e133e 100644
> --- a/arch/arm/dts/keystone-k2e-evm.dts
> +++ b/arch/arm/dts/keystone-k2e-evm.dts
> @@ -142,7 +142,7 @@
>  };
>  
>  &mdio {
> -	status = "ok";
> +	status = "okay";
>  	ethphy0: ethernet-phy at 0 {
>  		compatible = "marvell,88E1514", "marvell,88E1510", "ethernet-phy-ieee802.3-c22";
>  		reg = <0>;
> diff --git a/arch/arm/dts/keystone-k2hk-evm.dts b/arch/arm/dts/keystone-k2hk-evm.dts
> index 84c58d75ad..acfcaff0a8 100644
> --- a/arch/arm/dts/keystone-k2hk-evm.dts
> +++ b/arch/arm/dts/keystone-k2hk-evm.dts
> @@ -170,7 +170,7 @@
>  };
>  
>  &mdio {
> -	status = "ok";
> +	status = "okay";
>  	ethphy0: ethernet-phy at 0 {
>  		compatible = "marvell,88E1111", "ethernet-phy-ieee802.3-c22";
>  		reg = <0>;
> diff --git a/arch/arm/dts/keystone-k2l-evm.dts b/arch/arm/dts/keystone-k2l-evm.dts
> index 91cefdf2aa..ca049ba81b 100644
> --- a/arch/arm/dts/keystone-k2l-evm.dts
> +++ b/arch/arm/dts/keystone-k2l-evm.dts
> @@ -119,7 +119,7 @@
>  };
>  
>  &mdio {
> -	status = "ok";
> +	status = "okay";
>  	ethphy0: ethernet-phy at 0 {
>  		compatible = "marvell,88E1514", "marvell,88E1510", "ethernet-phy-ieee802.3-c22";
>  		reg = <0>;
> diff --git a/arch/arm/dts/omap3-beagle-xm.dts b/arch/arm/dts/omap3-beagle-xm.dts
> index 0349fcc9dc..8461159baf 100644
> --- a/arch/arm/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/dts/omap3-beagle-xm.dts
> @@ -379,7 +379,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	pinctrl-names = "default";
>  	pinctrl-0 = <
> @@ -396,7 +396,7 @@
>  };
>  
>  &venc {
> -	status = "ok";
> +	status = "okay";
>  
>  	vdda-supply = <&vdac>;
>  
> diff --git a/arch/arm/dts/omap3-beagle.dts b/arch/arm/dts/omap3-beagle.dts
> index 3ca8991a6c..4ceee2ba19 100644
> --- a/arch/arm/dts/omap3-beagle.dts
> +++ b/arch/arm/dts/omap3-beagle.dts
> @@ -353,7 +353,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&dss_dpi_pins>;
> @@ -367,7 +367,7 @@
>  };
>  
>  &venc {
> -	status = "ok";
> +	status = "okay";
>  
>  	vdda-supply = <&vdac>;
>  
> @@ -380,7 +380,7 @@
>  };
>  
>  &gpmc {
> -	status = "ok";
> +	status = "okay";
>  	ranges = <0 0 0x30000000 0x1000000>;	/* CS0 space, 16MB */
>  
>  	/* Chip select 0 */
> diff --git a/arch/arm/dts/omap3-igep0020-common.dtsi b/arch/arm/dts/omap3-igep0020-common.dtsi
> index ecbec23af4..d62481dacd 100644
> --- a/arch/arm/dts/omap3-igep0020-common.dtsi
> +++ b/arch/arm/dts/omap3-igep0020-common.dtsi
> @@ -248,7 +248,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	port {
>  		dpi_out: endpoint {
> diff --git a/arch/arm/dts/omap3-panel-sharp-ls037v7dw01.dtsi b/arch/arm/dts/omap3-panel-sharp-ls037v7dw01.dtsi
> index b8b9fcc41e..2dbb687d4d 100644
> --- a/arch/arm/dts/omap3-panel-sharp-ls037v7dw01.dtsi
> +++ b/arch/arm/dts/omap3-panel-sharp-ls037v7dw01.dtsi
> @@ -46,7 +46,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  	port {
>  		dpi_out: endpoint {
>  			remote-endpoint = <&lcd_in>;
> diff --git a/arch/arm/dts/omap34xx.dtsi b/arch/arm/dts/omap34xx.dtsi
> index ac4f8795b7..a703d09736 100644
> --- a/arch/arm/dts/omap34xx.dtsi
> +++ b/arch/arm/dts/omap34xx.dtsi
> @@ -69,7 +69,7 @@
>  };
>  
>  &ssi {
> -	status = "ok";
> +	status = "okay";
>  
>  	clocks = <&ssi_ssr_fck>,
>  		 <&ssi_sst_fck>,
> diff --git a/arch/arm/dts/omap36xx.dtsi b/arch/arm/dts/omap36xx.dtsi
> index 6fb23ada1f..52e1b8ce0f 100644
> --- a/arch/arm/dts/omap36xx.dtsi
> +++ b/arch/arm/dts/omap36xx.dtsi
> @@ -153,7 +153,7 @@
>  };
>  
>  &ssi {
> -	status = "ok";
> +	status = "okay";
>  
>  	clocks = <&ssi_ssr_fck>,
>  		 <&ssi_sst_fck>,
> diff --git a/arch/arm/dts/omap4-panda-common.dtsi b/arch/arm/dts/omap4-panda-common.dtsi
> index 55ea8b6189..c124b20d46 100644
> --- a/arch/arm/dts/omap4-panda-common.dtsi
> +++ b/arch/arm/dts/omap4-panda-common.dtsi
> @@ -546,7 +546,7 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  
>  	port {
>  		dpi_out: endpoint {
> @@ -557,12 +557,12 @@
>  };
>  
>  &dsi2 {
> -	status = "ok";
> +	status = "okay";
>  	vdd-supply = <&vcxio>;
>  };
>  
>  &hdmi {
> -	status = "ok";
> +	status = "okay";
>  	vdda-supply = <&vdac>;
>  
>  	port {
> diff --git a/arch/arm/dts/omap4-sdp.dts b/arch/arm/dts/omap4-sdp.dts
> index 91480ac1f3..28b989cfdb 100644
> --- a/arch/arm/dts/omap4-sdp.dts
> +++ b/arch/arm/dts/omap4-sdp.dts
> @@ -648,11 +648,11 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &dsi1 {
> -	status = "ok";
> +	status = "okay";
>  	vdd-supply = <&vcxio>;
>  
>  	port {
> @@ -677,7 +677,7 @@
>  };
>  
>  &dsi2 {
> -	status = "ok";
> +	status = "okay";
>  	vdd-supply = <&vcxio>;
>  
>  	port {
> @@ -702,7 +702,7 @@
>  };
>  
>  &hdmi {
> -	status = "ok";
> +	status = "okay";
>  	vdda-supply = <&vdac>;
>  
>  	port {
> diff --git a/arch/arm/dts/omap5-board-common.dtsi b/arch/arm/dts/omap5-board-common.dtsi
> index 68ac04641b..1eedd8d706 100644
> --- a/arch/arm/dts/omap5-board-common.dtsi
> +++ b/arch/arm/dts/omap5-board-common.dtsi
> @@ -743,11 +743,11 @@
>  };
>  
>  &dss {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &hdmi {
> -	status = "ok";
> +	status = "okay";
>  
>  	/* vdda-supply populated in board specific dts file */
>  
> -- 
> 2.17.1
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list