On Jun 19, 2024 at 15:44:41 -0500, Andrew Davis wrote:
> On 6/19/24 2:12 PM, Bryan Brattlof wrote:
> > Unfortunately when enabling FDT fixups for the AM62x family of SoCs and
> > moving TF-A to the bottom of RAM we missed the BeaglePlay. This is
> > causing Linux's memory allocator to clobber TF-A and break its boot.
> > 
> > Enable OF_SYSTEM_SETUP to fixup the kernel's FDT to inform it of the
> > actual location of the firmware
> > 
> > CC: Andrew Davis <a...@ti.com>
> > CC: Nishanth Menon <n...@ti.com>
> > CC: Robert Nelson <robertcnel...@gmail.com>
> > Reported-by: Dhruva Gole <d-g...@ti.com>
> > Signed-off-by: Bryan Brattlof <b...@ti.com>
> > ---
> 
> Acked-by: Andrew Davis <a...@ti.com>
> 
> > Hello everyone,
> > 
> > Fair warning, this may turn into a philosophical discussion about the
> > role of device-tree with SystemReady and U-Boot's role in enabling true
> > distribution to be completely agnostic of the board it's running on.
> > 
> > However substantively this is simply fixing a boot regression Dhruva
> > found while testing out the beagleplay.
> > 
> > Happy reviewing
> > ~Bryan
> > ---
> >   board/beagle/beagleplay/Kconfig | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/board/beagle/beagleplay/Kconfig 
> > b/board/beagle/beagleplay/Kconfig
> > index 7dbd833acb4cc..896a1c1be3010 100644
> > --- a/board/beagle/beagleplay/Kconfig
> > +++ b/board/beagle/beagleplay/Kconfig
> > @@ -12,6 +12,7 @@ config TARGET_AM625_A53_BEAGLEPLAY
> >     bool "BeagleBoard.org AM625 BeaglePlay running on A53"
> >     select ARM64
> >     select BINMAN
> > +   select OF_SYSTEM_SETUP

Thanks for the quick fix,
Tested-by: Dhruva Gole <d-g...@ti.com>

Boot logs upto kernel prompt:
https://gist.github.com/DhruvaG2000/4dc1c1e42207dd98a144f27cc9dff177

-- 
Best regards,
Dhruva Gole <d-g...@ti.com>

Reply via email to