[PATCH 4/6] arch: arm: dts: k3-am68-sk-r5: Sync with J721s2 R5 file

[Neha Malcom Francis]

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 at 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