[PATCH v4 2/2] arm: dts: j7200: dts sync with Linux 6.6-rc1

[reidt]

On 06:33-20231005, Nishanth Menon wrote:
> On 11:56-20231004, Reid Tonking wrote:
> > Sync j7200 dts with Linux 6.6-rc1
> [..]
> > 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 e62f9218e8..9469dca39f 100644
> > --- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
> > +++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
> > @@ -1,13 +1,14 @@
> >  // SPDX-License-Identifier: GPL-2.0
> >  /*
> > - * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/
> > + * Copyright (C) 2020-2023 Texas Instruments Incorporated - https://www.ti.com/
> >   */
> >  
> >  /dts-v1/;
> >  
> > -#include "k3-j7200-som-p0.dtsi"
> > +#include "k3-j7200-common-proc-board.dts"
> >  #include "k3-j7200-ddr-evm-lp4-2666.dtsi"
> >  #include "k3-j721e-ddr.dtsi"
> > +#include "k3-j7200-common-proc-board-u-boot.dtsi"
> >  
> >  / {
> >  	aliases {
> > @@ -15,17 +16,6 @@
> >  		remoteproc1 = &a72_0;
> >  	};
> >  
> > -	chosen {
> > -		stdout-path = &main_uart0;
> > -		tick-timer = &timer1;
> > -		firmware-loader = &fs_loader0;
> > -	};
> > -
> > -	fs_loader0: fs_loader at 0 {
> > -		bootph-all;
> > -		compatible = "u-boot,fs-loader";
> > -	};
> > -
> >  	a72_0: a72 at 0 {
> >  		compatible = "ti,am654-rproc";
> >  		reg = <0x0 0x00a90000 0x0 0x10>;
> > @@ -39,21 +29,17 @@
> >  		ti,sci = <&dmsc>;
> >  		ti,sci-proc-id = <32>;
> >  		ti,sci-host-id = <10>;
> > -		bootph-pre-ram;
> > -	};
> > -
> > -	clk_200mhz: dummy_clock_200mhz {
> > -		compatible = "fixed-clock";
> > -		#clock-cells = <0>;
> > -		clock-frequency = <200000000>;
> > -		bootph-pre-ram;
> > +		bootph-all;
> 
> Here and else where in the r5 file: why switch from pre-ram to bootph-all
> ? dont we need these prior to ddr initialization?
>

Due to the change in how booth-pre-ram works [0], bootph-pre-ram alone no
longer works. U-boot docs Pre-Relocation Support section says some-ram
can be used for u-boot prior to reloc, but not spl. So like Massimo
mentioned in the linked discussion, some-ram + pre-ram can work, as well
as -all. Outside of affecting the TPL phase, I'm not sure I know of a
difference in regards to how it affects pre-ddr.


[0]: https://lore.kernel.org/u-boot/CAPnjgZ3MgWX8T0A0SofphEr_Xd77pE3hte9DNye1RuBVeB9N8Q@mail.gmail.com/

> [...]
> 
> -- 
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D