Hi Neha,
On 11:29-20240509, Neha Malcom Francis wrote:
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-chawdhry@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.
Ah okay, I think I can explain better "start using k3-am68-sk-r5 file" in the commit message itself along with telling the reasoning as to why it wasn't failing previously as well along with your suggestions. Thanks!
Regards, Manorit
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