Hi Manorit
On 09/05/24 11:04, Manorit Chawdhry wrote:
Hi Neha,
On 10:37-20240509, Manorit Chawdhry wrote:
Hi Neha,
On 16:09-20240508, Neha Malcom Francis wrote:
Hi Manorit,
On 08/05/24 12:56, Manorit Chawdhry wrote:
Update the file with the required nodes from J721s2 R5 file to start
using k3-am68-sk-r5 file for AM68.
Signed-off-by: Manorit Chawdhry <m-chawd...@ti.com>
---
What's the motivation behind this patch vs. squashing it into patch 5/6?
Could've squashed it ig, I think developed it in this order so this
remained. Would squash it. Also realised that I should be putting that
patch before the config split otherwise am68 boot would break again.
Would take that up as well in v2. Thanks for the review!
Though on second thoughts.. I think it's good as it tells that AM68 R5
DT had been missing some changes. If someone wants to track what changed
then ig it's better that they don't have to debug the merge commit which
ends up altering the contents of AM68 R5 DT ( in-case this patch ain't
there ) and people will have to manually check the diff as to what
altered. Do you think it's better to keep this patch with the following
reasoning?
Yes you can do that but I think this commit message is confusing. The "start
using k3-am68-sk-r5 file for AM68" threw me off, maybe modify it to say that
AM68 R5 DT is missing these changes and needs them why? After that grabbing the
common bits into an SoC R5 file in patch 5/6 makes sense.
Regards,
Manorit
Regards,
Manorit
arch/arm/dts/k3-am68-sk-r5-base-board.dts | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/k3-am68-sk-r5-base-board.dts
b/arch/arm/dts/k3-am68-sk-r5-base-board.dts
index 695aadc287bd..038b08dc3e01 100644
--- a/arch/arm/dts/k3-am68-sk-r5-base-board.dts
+++ b/arch/arm/dts/k3-am68-sk-r5-base-board.dts
@@ -24,7 +24,8 @@
compatible = "ti,am654-rproc";
reg = <0x0 0x00a90000 0x0 0x10>;
power-domains = <&k3_pds 61 TI_SCI_PD_EXCLUSIVE>,
- <&k3_pds 202 TI_SCI_PD_EXCLUSIVE>;
+ <&k3_pds 202 TI_SCI_PD_EXCLUSIVE>,
+ <&k3_pds 4 TI_SCI_PD_EXCLUSIVE>;
resets = <&k3_reset 202 0>;
clocks = <&k3_clks 61 1>;
assigned-clocks = <&k3_clks 61 1>, <&k3_clks 202 0>;
@@ -54,10 +55,12 @@
&secure_proxy_mcu {
bootph-pre-ram;
+ status = "okay";
};
&secure_proxy_sa3 {
bootph-pre-ram;
+ status = "okay";
};
&cbass_mcu_wakeup {
--
Thanking You
Neha Malcom Francis
--
Thanking You
Neha Malcom Francis
