From: Nishanth Menon <nm@ti.com>
To: Roger Quadros <rogerq@kernel.org>
Cc: <lokeshvutla@ti.com>, <u-boot@lists.denx.de>
Subject: Re: [PATCH] ARM: dts: Fix node status to "okay" on TI boards
Date: Fri, 20 Aug 2021 12:51:03 -0500 [thread overview]
Message-ID: <20210820175103.c7phsyewcr6yy6sy@elastic> (raw)
In-Reply-To: <20210820104131.13188-1-rogerq@kernel.org>
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@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@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@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@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
next prev parent reply other threads:[~2021-08-20 17:51 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-20 10:41 [PATCH] ARM: dts: Fix node status to "okay" on TI boards Roger Quadros
2021-08-20 17:51 ` Nishanth Menon [this message]
2021-08-23 6:23 ` Lokesh Vutla
2021-08-23 6:44 ` Roger Quadros
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210820175103.c7phsyewcr6yy6sy@elastic \
--to=nm@ti.com \
--cc=lokeshvutla@ti.com \
--cc=rogerq@kernel.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox