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
