linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml
@ 2021-05-10 14:50 Nishanth Menon
  2021-05-10 14:50 ` [PATCH 1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name Nishanth Menon
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-10 14:50 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel, Nishanth Menon

Hi,

Series of minor fixups for v5.13-rc1 for compliance exposed by yaml
conversion in the series:
https://lore.kernel.org/linux-arm-kernel/20210503190824.GA2192378@robh.at.kernel.org/#t

Minimal testing performed on k3 platforms.

If possible, will be good to get in 5.13 window.

Nishanth Menon (4):
  arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name
  arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node
  arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc
    node
  arm64: dts: ti: k3-*: Rename the TI-SCI node

 arch/arm64/boot/dts/ti/k3-am64-main.dtsi        |  4 ++--
 arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi      | 10 +++++-----
 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi |  4 ++--
 arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi |  4 ++--
 4 files changed, 11 insertions(+), 11 deletions(-)

-- 
2.31.0


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
@ 2021-05-10 14:50 ` Nishanth Menon
  2021-05-10 14:50 ` [PATCH 2/4] arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node Nishanth Menon
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-10 14:50 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel, Nishanth Menon

We currently use clocks as the node name for the node representing
TI-SCI clock nodes. This is better renamed to being clock-controller
as that is a better representative of the system controller function
as a clock controller for the SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am64-main.dtsi        | 2 +-
 arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi      | 2 +-
 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
 arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index b2bcbf23eefd..e1216073e3df 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -148,7 +148,7 @@ k3_pds: power-controller {
 			#power-domain-cells = <2>;
 		};
 
-		k3_clks: clocks {
+		k3_clks: clock-controller {
 			compatible = "ti,k2g-sci-clk";
 			#clock-cells = <2>;
 		};
diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
index ed42f13e7663..2ae1f9214b8a 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
@@ -23,7 +23,7 @@ k3_pds: power-controller {
 			#power-domain-cells = <2>;
 		};
 
-		k3_clks: clocks {
+		k3_clks: clock-controller {
 			compatible = "ti,k2g-sci-clk";
 			#clock-cells = <2>;
 		};
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 5e74e43822c3..9dba2df3569f 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -23,7 +23,7 @@ k3_pds: power-controller {
 			#power-domain-cells = <2>;
 		};
 
-		k3_clks: clocks {
+		k3_clks: clock-controller {
 			compatible = "ti,k2g-sci-clk";
 			#clock-cells = <2>;
 		};
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
index d56e3475aee7..b83801feeb10 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -23,7 +23,7 @@ k3_pds: power-controller {
 			#power-domain-cells = <2>;
 		};
 
-		k3_clks: clocks {
+		k3_clks: clock-controller {
 			compatible = "ti,k2g-sci-clk";
 			#clock-cells = <2>;
 		};
-- 
2.31.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/4] arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
  2021-05-10 14:50 ` [PATCH 1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name Nishanth Menon
@ 2021-05-10 14:50 ` Nishanth Menon
  2021-05-10 14:50 ` [PATCH 3/4] arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc node Nishanth Menon
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-10 14:50 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel, Nishanth Menon

Lets add the TISCI debug region to TI-SCI region in line with TI-SCI
documentation[1]. While at it, lets rename the node to indicate the
address usage.

[1] http://downloads.ti.com/tisci/esd/latest/4_trace/trace.html

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
index 2ae1f9214b8a..444842a2d556 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
@@ -6,7 +6,7 @@
  */
 
 &cbass_wakeup {
-	dmsc: dmsc {
+	dmsc: dmsc@44083000 {
 		compatible = "ti,am654-sci";
 		ti,host-id = <12>;
 		#address-cells = <1>;
@@ -18,6 +18,9 @@ dmsc: dmsc {
 		mboxes= <&secure_proxy_main 11>,
 			<&secure_proxy_main 13>;
 
+		reg-names = "debug_messages";
+		reg = <0x44083000 0x1000>;
+
 		k3_pds: power-controller {
 			compatible = "ti,sci-pm-domain";
 			#power-domain-cells = <2>;
-- 
2.31.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/4] arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc node
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
  2021-05-10 14:50 ` [PATCH 1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name Nishanth Menon
  2021-05-10 14:50 ` [PATCH 2/4] arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node Nishanth Menon
@ 2021-05-10 14:50 ` Nishanth Menon
  2021-05-10 14:50 ` [PATCH 4/4] arm64: dts: ti: k3-*: Rename the TI-SCI node Nishanth Menon
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-10 14:50 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel, Nishanth Menon

The DMSC node does'nt require any of "#address-cells", "#size-cells"
or "ranges" property as the child nodes are representations of SoC's
system controller itself, so align it with the bindings.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
index 444842a2d556..80d4df775f43 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
@@ -9,9 +9,6 @@ &cbass_wakeup {
 	dmsc: dmsc@44083000 {
 		compatible = "ti,am654-sci";
 		ti,host-id = <12>;
-		#address-cells = <1>;
-		#size-cells = <1>;
-		ranges;
 
 		mbox-names = "rx", "tx";
 
-- 
2.31.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 4/4] arm64: dts: ti: k3-*: Rename the TI-SCI node
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
                   ` (2 preceding siblings ...)
  2021-05-10 14:50 ` [PATCH 3/4] arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc node Nishanth Menon
@ 2021-05-10 14:50 ` Nishanth Menon
  2021-05-11  7:17 ` [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Tero Kristo
  2021-05-14 17:52 ` Nishanth Menon
  5 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-10 14:50 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel, Nishanth Menon

Lets rename the node name of TI-SCI node to be system-controller as it
is a better standardized name for the function that TI-SCI plays in the
SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am64-main.dtsi        | 2 +-
 arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi      | 2 +-
 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
 arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index e1216073e3df..dc52178d9b64 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -134,7 +134,7 @@ main_pktdma: dma-controller@485c0000 {
 		};
 	};
 
-	dmsc: dmsc@44043000 {
+	dmsc: system-controller@44043000 {
 		compatible = "ti,k2g-sci";
 		ti,host-id = <12>;
 		mbox-names = "rx", "tx";
diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
index 80d4df775f43..822f4cff1db4 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
@@ -6,7 +6,7 @@
  */
 
 &cbass_wakeup {
-	dmsc: dmsc@44083000 {
+	dmsc: system-controller@44083000 {
 		compatible = "ti,am654-sci";
 		ti,host-id = <12>;
 
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 9dba2df3569f..65f3fabda114 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -6,7 +6,7 @@
  */
 
 &cbass_mcu_wakeup {
-	dmsc: dmsc@44083000 {
+	dmsc: system-controller@44083000 {
 		compatible = "ti,k2g-sci";
 		ti,host-id = <12>;
 
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
index b83801feeb10..94e821467eaa 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -6,7 +6,7 @@
  */
 
 &cbass_mcu_wakeup {
-	dmsc: dmsc@44083000 {
+	dmsc: system-controller@44083000 {
 		compatible = "ti,k2g-sci";
 		ti,host-id = <12>;
 
-- 
2.31.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
                   ` (3 preceding siblings ...)
  2021-05-10 14:50 ` [PATCH 4/4] arm64: dts: ti: k3-*: Rename the TI-SCI node Nishanth Menon
@ 2021-05-11  7:17 ` Tero Kristo
  2021-05-14 17:52 ` Nishanth Menon
  5 siblings, 0 replies; 7+ messages in thread
From: Tero Kristo @ 2021-05-11  7:17 UTC (permalink / raw)
  To: Nishanth Menon; +Cc: Rob Herring, linux-kernel, devicetree, linux-arm-kernel

On 10/05/2021 17:50, Nishanth Menon wrote:
> Hi,
> 
> Series of minor fixups for v5.13-rc1 for compliance exposed by yaml
> conversion in the series:
> https://lore.kernel.org/linux-arm-kernel/20210503190824.GA2192378@robh.at.kernel.org/#t
> 
> Minimal testing performed on k3 platforms.
> 
> If possible, will be good to get in 5.13 window.
> 
> Nishanth Menon (4):
>    arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name
>    arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node
>    arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc
>      node
>    arm64: dts: ti: k3-*: Rename the TI-SCI node
> 
>   arch/arm64/boot/dts/ti/k3-am64-main.dtsi        |  4 ++--
>   arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi      | 10 +++++-----
>   arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi |  4 ++--
>   arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi |  4 ++--
>   4 files changed, 11 insertions(+), 11 deletions(-)
> 

For the whole series:

Reviewed-by: Tero Kristo <kristo@kernel.org>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml
  2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
                   ` (4 preceding siblings ...)
  2021-05-11  7:17 ` [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Tero Kristo
@ 2021-05-14 17:52 ` Nishanth Menon
  5 siblings, 0 replies; 7+ messages in thread
From: Nishanth Menon @ 2021-05-14 17:52 UTC (permalink / raw)
  To: Tero Kristo, Nishanth Menon
  Cc: linux-arm-kernel, devicetree, Rob Herring, linux-kernel

On Mon, 10 May 2021 09:50:29 -0500, Nishanth Menon wrote:
> Series of minor fixups for v5.13-rc1 for compliance exposed by yaml
> conversion in the series:
> https://lore.kernel.org/linux-arm-kernel/20210503190824.GA2192378@robh.at.kernel.org/#t
> 
> Minimal testing performed on k3 platforms.
> 
> If possible, will be good to get in 5.13 window.
> 
> [...]

Hi Nishanth Menon,

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name
      commit: a0812885fa7a1074c8003484b8176ffe28d5df68
[2/4] arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node
      commit: 830454bbd628330c3779c3de637b709dae790da0
[3/4] arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc node
      commit: 421c06b8761abd7d953148f5b955b4149df9846e
[4/4] arm64: dts: ti: k3-*: Rename the TI-SCI node
      commit: 9d3c9378f96a95f15881ee3373d2c2f773273fc2


All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-05-14 17:52 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-10 14:50 [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Nishanth Menon
2021-05-10 14:50 ` [PATCH 1/4] arm64: dts: ti: k3-*: Rename the TI-SCI clocks node name Nishanth Menon
2021-05-10 14:50 ` [PATCH 2/4] arm64: dts: ti: k3-am65-wakeup: Add debug region to TI-SCI node Nishanth Menon
2021-05-10 14:50 ` [PATCH 3/4] arm64: dts: ti: k3-am65-wakeup: Drop un-necessary properties from dmsc node Nishanth Menon
2021-05-10 14:50 ` [PATCH 4/4] arm64: dts: ti: k3-*: Rename the TI-SCI node Nishanth Menon
2021-05-11  7:17 ` [PATCH 0/4] arm64: dts: ti: k3-*: Fixup nodes to exposed to be non-compliant with yaml Tero Kristo
2021-05-14 17:52 ` Nishanth Menon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).