* [PATCH 1/3] ARM: dts: ti: Drop usage of ti,am33xx-ecap from DT nodes
2021-06-01 9:24 [PATCH 0/3] ARM: dts: ti: Fix DT warnings for ecap nodes Lokesh Vutla
@ 2021-06-01 9:24 ` Lokesh Vutla
2021-06-01 13:05 ` Grygorii Strashko
2021-06-01 9:24 ` [PATCH 2/3] ARM: dts: ti: Fix node name for all ecap dt nodes Lokesh Vutla
` (2 subsequent siblings)
3 siblings, 1 reply; 8+ messages in thread
From: Lokesh Vutla @ 2021-06-01 9:24 UTC (permalink / raw)
To: Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List, Lokesh Vutla
ti,am33xx-ecap is used to represent device nodes using ti,hwmod data.
Now that hwmod entries are entirely removed, drop usage of
ti,am33xx-ecap compatible from all DT nodes.
Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
arch/arm/boot/dts/am33xx-l4.dtsi | 9 +++------
arch/arm/boot/dts/am437x-l4.dtsi | 9 +++------
arch/arm/boot/dts/da850.dtsi | 9 +++------
3 files changed, 9 insertions(+), 18 deletions(-)
diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
index fbf3458ab246..457d1b0fe510 100644
--- a/arch/arm/boot/dts/am33xx-l4.dtsi
+++ b/arch/arm/boot/dts/am33xx-l4.dtsi
@@ -1996,8 +1996,7 @@ epwmss0: epwmss@0 {
ranges = <0 0 0x1000>;
ecap0: ecap@100 {
- compatible = "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
@@ -2056,8 +2055,7 @@ epwmss1: epwmss@0 {
ranges = <0 0 0x1000>;
ecap1: ecap@100 {
- compatible = "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
@@ -2116,8 +2114,7 @@ epwmss2: epwmss@0 {
ranges = <0 0 0x1000>;
ecap2: ecap@100 {
- compatible = "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
diff --git a/arch/arm/boot/dts/am437x-l4.dtsi b/arch/arm/boot/dts/am437x-l4.dtsi
index e796b9bcc54e..4fa746c186be 100644
--- a/arch/arm/boot/dts/am437x-l4.dtsi
+++ b/arch/arm/boot/dts/am437x-l4.dtsi
@@ -1749,8 +1749,7 @@ epwmss0: epwmss@0 {
ecap0: ecap@100 {
compatible = "ti,am4372-ecap",
- "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
@@ -1800,8 +1799,7 @@ epwmss1: epwmss@0 {
ecap1: ecap@100 {
compatible = "ti,am4372-ecap",
- "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
@@ -1851,8 +1849,7 @@ epwmss2: epwmss@0 {
ecap2: ecap@100 {
compatible = "ti,am4372-ecap",
- "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index afdf3d3747ce..8e5748e6b270 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -592,8 +592,7 @@ ehrpwm1: pwm@302000 {
status = "disabled";
};
ecap0: ecap@306000 {
- compatible = "ti,da850-ecap", "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x306000 0x80>;
clocks = <&psc1 20>;
@@ -602,8 +601,7 @@ ecap0: ecap@306000 {
status = "disabled";
};
ecap1: ecap@307000 {
- compatible = "ti,da850-ecap", "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x307000 0x80>;
clocks = <&psc1 20>;
@@ -612,8 +610,7 @@ ecap1: ecap@307000 {
status = "disabled";
};
ecap2: ecap@308000 {
- compatible = "ti,da850-ecap", "ti,am3352-ecap",
- "ti,am33xx-ecap";
+ compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x308000 0x80>;
clocks = <&psc1 20>;
--
2.31.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 1/3] ARM: dts: ti: Drop usage of ti,am33xx-ecap from DT nodes
2021-06-01 9:24 ` [PATCH 1/3] ARM: dts: ti: Drop usage of ti,am33xx-ecap from DT nodes Lokesh Vutla
@ 2021-06-01 13:05 ` Grygorii Strashko
0 siblings, 0 replies; 8+ messages in thread
From: Grygorii Strashko @ 2021-06-01 13:05 UTC (permalink / raw)
To: Lokesh Vutla, Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List
On 01/06/2021 12:24, Lokesh Vutla wrote:
> ti,am33xx-ecap is used to represent device nodes using ti,hwmod data.
> Now that hwmod entries are entirely removed, drop usage of
> ti,am33xx-ecap compatible from all DT nodes.
>
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
> arch/arm/boot/dts/am33xx-l4.dtsi | 9 +++------
> arch/arm/boot/dts/am437x-l4.dtsi | 9 +++------
> arch/arm/boot/dts/da850.dtsi | 9 +++------
> 3 files changed, 9 insertions(+), 18 deletions(-)
>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
--
Best regards,
grygorii
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/3] ARM: dts: ti: Fix node name for all ecap dt nodes
2021-06-01 9:24 [PATCH 0/3] ARM: dts: ti: Fix DT warnings for ecap nodes Lokesh Vutla
2021-06-01 9:24 ` [PATCH 1/3] ARM: dts: ti: Drop usage of ti,am33xx-ecap from DT nodes Lokesh Vutla
@ 2021-06-01 9:24 ` Lokesh Vutla
2021-06-01 13:06 ` Grygorii Strashko
2021-06-01 9:24 ` [PATCH 3/3] ARM: dts: am33xx: Drop interrupt property from ecap nodes Lokesh Vutla
2021-06-11 7:29 ` [PATCH 0/3] ARM: dts: ti: Fix DT warnings for " Tony Lindgren
3 siblings, 1 reply; 8+ messages in thread
From: Lokesh Vutla @ 2021-06-01 9:24 UTC (permalink / raw)
To: Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List, Lokesh Vutla
dtbs_check produces the following warning for ecap node name:
ecap@100: $nodename:0: 'ecap@100' does not match '^pwm(@.*|-[0-9a-f])*$'
Fix this by replacing ecap with pwm in node name
Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
arch/arm/boot/dts/am335x-cm-t335.dts | 2 +-
arch/arm/boot/dts/am335x-evm.dts | 2 +-
arch/arm/boot/dts/am335x-evmsk.dts | 2 +-
arch/arm/boot/dts/am33xx-l4.dtsi | 6 +++---
arch/arm/boot/dts/am437x-l4.dtsi | 6 +++---
arch/arm/boot/dts/da850.dtsi | 6 +++---
arch/arm/boot/dts/dra7-l4.dtsi | 6 +++---
7 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/arch/arm/boot/dts/am335x-cm-t335.dts b/arch/arm/boot/dts/am335x-cm-t335.dts
index 36d963db4026..a306b0ccd06f 100644
--- a/arch/arm/boot/dts/am335x-cm-t335.dts
+++ b/arch/arm/boot/dts/am335x-cm-t335.dts
@@ -333,7 +333,7 @@ tlv320aic23: codec@1a {
&epwmss0 {
status = "okay";
- ecap0: ecap@100 {
+ ecap0: pwm@100 {
status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&ecap0_pins>;
diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
index 902e295b309e..9cf39c93defb 100644
--- a/arch/arm/boot/dts/am335x-evm.dts
+++ b/arch/arm/boot/dts/am335x-evm.dts
@@ -495,7 +495,7 @@ &elm {
&epwmss0 {
status = "okay";
- ecap0: ecap@100 {
+ ecap0: pwm@100 {
status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&ecap0_pins>;
diff --git a/arch/arm/boot/dts/am335x-evmsk.dts b/arch/arm/boot/dts/am335x-evmsk.dts
index d5f8d5e2eb5d..a35f727dcd12 100644
--- a/arch/arm/boot/dts/am335x-evmsk.dts
+++ b/arch/arm/boot/dts/am335x-evmsk.dts
@@ -510,7 +510,7 @@ &usb1 {
&epwmss2 {
status = "okay";
- ecap2: ecap@100 {
+ ecap2: pwm@100 {
status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&ecap2_pins>;
diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
index 457d1b0fe510..6359416b0fa7 100644
--- a/arch/arm/boot/dts/am33xx-l4.dtsi
+++ b/arch/arm/boot/dts/am33xx-l4.dtsi
@@ -1995,7 +1995,7 @@ epwmss0: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap0: ecap@100 {
+ ecap0: pwm@100 {
compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
@@ -2054,7 +2054,7 @@ epwmss1: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap1: ecap@100 {
+ ecap1: pwm@100 {
compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
@@ -2113,7 +2113,7 @@ epwmss2: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap2: ecap@100 {
+ ecap2: pwm@100 {
compatible = "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x100 0x80>;
diff --git a/arch/arm/boot/dts/am437x-l4.dtsi b/arch/arm/boot/dts/am437x-l4.dtsi
index 4fa746c186be..f9345e948fca 100644
--- a/arch/arm/boot/dts/am437x-l4.dtsi
+++ b/arch/arm/boot/dts/am437x-l4.dtsi
@@ -1747,7 +1747,7 @@ epwmss0: epwmss@0 {
ranges = <0 0 0x1000>;
status = "disabled";
- ecap0: ecap@100 {
+ ecap0: pwm@100 {
compatible = "ti,am4372-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
@@ -1797,7 +1797,7 @@ epwmss1: epwmss@0 {
ranges = <0 0 0x1000>;
status = "disabled";
- ecap1: ecap@100 {
+ ecap1: pwm@100 {
compatible = "ti,am4372-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
@@ -1847,7 +1847,7 @@ epwmss2: epwmss@0 {
ranges = <0 0 0x1000>;
status = "disabled";
- ecap2: ecap@100 {
+ ecap2: pwm@100 {
compatible = "ti,am4372-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 8e5748e6b270..c3942b4e82ad 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -591,7 +591,7 @@ ehrpwm1: pwm@302000 {
power-domains = <&psc1 17>;
status = "disabled";
};
- ecap0: ecap@306000 {
+ ecap0: pwm@306000 {
compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x306000 0x80>;
@@ -600,7 +600,7 @@ ecap0: ecap@306000 {
power-domains = <&psc1 20>;
status = "disabled";
};
- ecap1: ecap@307000 {
+ ecap1: pwm@307000 {
compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x307000 0x80>;
@@ -609,7 +609,7 @@ ecap1: ecap@307000 {
power-domains = <&psc1 20>;
status = "disabled";
};
- ecap2: ecap@308000 {
+ ecap2: pwm@308000 {
compatible = "ti,da850-ecap", "ti,am3352-ecap";
#pwm-cells = <3>;
reg = <0x308000 0x80>;
diff --git a/arch/arm/boot/dts/dra7-l4.dtsi b/arch/arm/boot/dts/dra7-l4.dtsi
index 149144cdff35..896ea110523a 100644
--- a/arch/arm/boot/dts/dra7-l4.dtsi
+++ b/arch/arm/boot/dts/dra7-l4.dtsi
@@ -2561,7 +2561,7 @@ epwmss0: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap0: ecap@100 {
+ ecap0: pwm@100 {
compatible = "ti,dra746-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
@@ -2607,7 +2607,7 @@ epwmss1: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap1: ecap@100 {
+ ecap1: pwm@100 {
compatible = "ti,dra746-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
@@ -2653,7 +2653,7 @@ epwmss2: epwmss@0 {
status = "disabled";
ranges = <0 0 0x1000>;
- ecap2: ecap@100 {
+ ecap2: pwm@100 {
compatible = "ti,dra746-ecap",
"ti,am3352-ecap";
#pwm-cells = <3>;
--
2.31.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 2/3] ARM: dts: ti: Fix node name for all ecap dt nodes
2021-06-01 9:24 ` [PATCH 2/3] ARM: dts: ti: Fix node name for all ecap dt nodes Lokesh Vutla
@ 2021-06-01 13:06 ` Grygorii Strashko
0 siblings, 0 replies; 8+ messages in thread
From: Grygorii Strashko @ 2021-06-01 13:06 UTC (permalink / raw)
To: Lokesh Vutla, Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List
On 01/06/2021 12:24, Lokesh Vutla wrote:
> dtbs_check produces the following warning for ecap node name:
> ecap@100: $nodename:0: 'ecap@100' does not match '^pwm(@.*|-[0-9a-f])*$'
>
> Fix this by replacing ecap with pwm in node name
>
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
> arch/arm/boot/dts/am335x-cm-t335.dts | 2 +-
> arch/arm/boot/dts/am335x-evm.dts | 2 +-
> arch/arm/boot/dts/am335x-evmsk.dts | 2 +-
> arch/arm/boot/dts/am33xx-l4.dtsi | 6 +++---
> arch/arm/boot/dts/am437x-l4.dtsi | 6 +++---
> arch/arm/boot/dts/da850.dtsi | 6 +++---
> arch/arm/boot/dts/dra7-l4.dtsi | 6 +++---
> 7 files changed, 15 insertions(+), 15 deletions(-)
>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
--
Best regards,
grygorii
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 3/3] ARM: dts: am33xx: Drop interrupt property from ecap nodes
2021-06-01 9:24 [PATCH 0/3] ARM: dts: ti: Fix DT warnings for ecap nodes Lokesh Vutla
2021-06-01 9:24 ` [PATCH 1/3] ARM: dts: ti: Drop usage of ti,am33xx-ecap from DT nodes Lokesh Vutla
2021-06-01 9:24 ` [PATCH 2/3] ARM: dts: ti: Fix node name for all ecap dt nodes Lokesh Vutla
@ 2021-06-01 9:24 ` Lokesh Vutla
2021-06-01 13:07 ` Grygorii Strashko
2021-06-11 7:29 ` [PATCH 0/3] ARM: dts: ti: Fix DT warnings for " Tony Lindgren
3 siblings, 1 reply; 8+ messages in thread
From: Lokesh Vutla @ 2021-06-01 9:24 UTC (permalink / raw)
To: Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List, Lokesh Vutla
Interrupts were never supported by ecap driver and the bindings
were never documented.Therefore drop the entries in preparation
to moving bindings to YAML schema.
Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
---
arch/arm/boot/dts/am33xx-l4.dtsi | 6 ------
1 file changed, 6 deletions(-)
diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
index 6359416b0fa7..50c79e59eecb 100644
--- a/arch/arm/boot/dts/am33xx-l4.dtsi
+++ b/arch/arm/boot/dts/am33xx-l4.dtsi
@@ -2001,8 +2001,6 @@ ecap0: pwm@100 {
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
clock-names = "fck";
- interrupts = <31>;
- interrupt-names = "ecap0";
status = "disabled";
};
@@ -2060,8 +2058,6 @@ ecap1: pwm@100 {
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
clock-names = "fck";
- interrupts = <47>;
- interrupt-names = "ecap1";
status = "disabled";
};
@@ -2119,8 +2115,6 @@ ecap2: pwm@100 {
reg = <0x100 0x80>;
clocks = <&l4ls_gclk>;
clock-names = "fck";
- interrupts = <61>;
- interrupt-names = "ecap2";
status = "disabled";
};
--
2.31.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 3/3] ARM: dts: am33xx: Drop interrupt property from ecap nodes
2021-06-01 9:24 ` [PATCH 3/3] ARM: dts: am33xx: Drop interrupt property from ecap nodes Lokesh Vutla
@ 2021-06-01 13:07 ` Grygorii Strashko
0 siblings, 0 replies; 8+ messages in thread
From: Grygorii Strashko @ 2021-06-01 13:07 UTC (permalink / raw)
To: Lokesh Vutla, Tony Lindgren, Rob Herring
Cc: Sekhar Nori, Vignesh R, Nishanth Menon, Linux OMAP Mailing List,
Device Tree Mailing List, Linux ARM Mailing List
On 01/06/2021 12:24, Lokesh Vutla wrote:
> Interrupts were never supported by ecap driver and the bindings
> were never documented.Therefore drop the entries in preparation
> to moving bindings to YAML schema.
>
> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
> arch/arm/boot/dts/am33xx-l4.dtsi | 6 ------
> 1 file changed, 6 deletions(-)
>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
--
Best regards,
grygorii
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/3] ARM: dts: ti: Fix DT warnings for ecap nodes
2021-06-01 9:24 [PATCH 0/3] ARM: dts: ti: Fix DT warnings for ecap nodes Lokesh Vutla
` (2 preceding siblings ...)
2021-06-01 9:24 ` [PATCH 3/3] ARM: dts: am33xx: Drop interrupt property from ecap nodes Lokesh Vutla
@ 2021-06-11 7:29 ` Tony Lindgren
3 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2021-06-11 7:29 UTC (permalink / raw)
To: Lokesh Vutla
Cc: Rob Herring, Sekhar Nori, Vignesh R, Nishanth Menon,
Linux OMAP Mailing List, Device Tree Mailing List,
Linux ARM Mailing List
* Lokesh Vutla <lokeshvutla@ti.com> [210601 12:25]:
> Fix DT warnings for all the ecap nodes on TI platforms. This is in
> preparation for converting pwm-tiecap documentation to YAML.
Applying into omap-for-v5.14/dt thanks.
Tony
^ permalink raw reply [flat|nested] 8+ messages in thread