[PATCH 4/6] arch: arm: dts: k3-am68-sk-r5: Sync with J721s2 R5 file
Manorit Chawdhry
m-chawdhry at ti.com
Thu May 9 08:27:07 CEST 2024
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 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.
>
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
More information about the U-Boot
mailing list