Hi Aniket

On 17/10/24 11:59, Aniket Limaye wrote:
From: Reid Tonking <re...@ti.com>

Define the MSMC clk in the a72 node

The usage of MSMC and A72SS interchangeably in this series is confusing. Could you expand on it in the cover letter why in J7200 this clock is defined as part of the A72 node as opposed to devices that have MSMC as a separate module altogether with its own clock (like J721S2 and J784S4)?

Signed-off-by: Reid Tonking <re...@ti.com>
Signed-off-by: Aniket Limaye <a-lim...@ti.com>
---
  arch/arm/dts/k3-j7200-r5-common-proc-board.dts | 10 +++++-----
  1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts 
b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
index f096b102793..759a1e83456 100644
--- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
+++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
@@ -23,11 +23,11 @@
                                <&k3_pds 202 TI_SCI_PD_EXCLUSIVE>,
                                <&k3_pds 4 TI_SCI_PD_EXCLUSIVE>;
                resets = <&k3_reset 202 0>;
-               clocks = <&k3_clks 61 1>, <&k3_clks 202 2>;
-               clock-names = "gtc", "core";
-               assigned-clocks = <&k3_clks 202 2>, <&k3_clks 61 1>, <&k3_clks 323 
0>;
-               assigned-clock-parents= <0>, <0>, <&k3_clks 323 2>;
-               assigned-clock-rates = <2000000000>, <200000000>;
+               clocks = <&k3_clks 61 1>, <&k3_clks 4 1>, <&k3_clks 202 2> ;
+               clock-names = "gtc", "msmc", "core";
+               assigned-clocks = <&k3_clks 202 2>, <&k3_clks 61 1>, <&k3_clks 4 1>, 
<&k3_clks 323 0>;
+               assigned-clock-parents= <0>, <0>, <0>, <&k3_clks 323 2>;
+               assigned-clock-rates = <2000000000>, <200000000>, <1000000000>;
                ti,sci = <&dmsc>;
                ti,sci-proc-id = <32>;
                ti,sci-host-id = <10>;

--
Thanking You
Neha Malcom Francis

Reply via email to