* [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 read-only IMAP folder(s) and NNTP newsgroup(s).