Re: [PATCH v1 0/4] k3: migrate SPL_TEXT_BASE to new address

[Nishanth Menon <nm@ti.com>]

On 11:38-20250416, Andrew Davis wrote:
> > > > 	How about u-boot documentation?
> > > 
> > > I will update that in the next revision along with any feedback I
> > > receive on this patch.
> > 
> > IMHO, This change is too intrusive and impacts downstream customers
> > including inflight production devices.
> > 
> 
> Could you elaborate a little?
> 
> They do not even need to update their TF-A version, if they want to
> use their current version they simply recompile it with a single
> extra build flag:
> 
> PRELOADED_BL33_BASE=0x822000000

For folks who are trying to boot linux from TFA, they could do the same.
> 
> They should have no reason not to be able to do this. But even *if*
> they cannot do that, the whole point of the jump-stub here is to
> save them even having to do a simple compile.
> 
> IMHO "downstream customers including inflight production devices"
> should not be a reason to block good and useful changes here in
> upstream. But again, this change doesn't break anything for them in
> the first place, so why bring that up?

I dont see a specific value here. U-boot just works. For folks who want
direct TFA to kernel jump (which is a niche fast boot usecase), go ahead
and use TFA with the mentioned build option.

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