From: Roger Quadros <rogerq@kernel.org>
To: Lokesh Vutla <lokeshvutla@ti.com>
Cc: nm@ti.com, u-boot@lists.denx.de
Subject: Re: [PATCH] ARM: dts: Fix node status to "okay" on TI boards
Date: Mon, 23 Aug 2021 09:44:30 +0300 [thread overview]
Message-ID: <cee0bbea-7367-8a26-5b69-ed8254e7b71c@kernel.org> (raw)
In-Reply-To: <4f6f20c0-e81d-22df-c043-4deb3c88a4c6@ti.com>
Hi Lokesh,
On 23/08/2021 09:23, Lokesh Vutla wrote:
> Hi Roger,
>
> On 20/08/21 4:11 pm, 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.
>>
>
> Is Kernel DTS updated? The idea is to not deviate from Kernel dts as much as
> possible.
Yes, on kernel side, most nodes have been updated. There were a few stray left overs which I have sent patches for. I will send v2 of this series to address Nishanth's comment.
cheers,
-roger
>
> Thanks and regards,
> Lokesh
>
>> 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 */
>>
>>
prev parent reply other threads:[~2021-08-23 6:44 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
2021-08-23 6:23 ` Lokesh Vutla
2021-08-23 6:44 ` Roger Quadros [this message]
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=cee0bbea-7367-8a26-5b69-ed8254e7b71c@kernel.org \
--to=rogerq@kernel.org \
--cc=lokeshvutla@ti.com \
--cc=nm@ti.com \
--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