2 comments:
This requires soc/intel target to
explicitly set this in the SOC dir
I see the sanity, but don't follow conclusion, why is it require by […]
It's a bit like the overuse of __weak in soc/intel to make stuff build without being correct. Requiring things to de explicitly done in the soc dir could avoid that? I'll try to better state that.
Patch Set #1, Line 12: wrong
How can it be wrong? does the FSP binary depend on a specific
location?
The integration guide recommends (I looked at kabylake) against using non default values and if FSP-T is used it needs to match what is programmed in that UPD, so I think it sufficiently safe to say there are values that should not be tried out.
To view, visit change 35884. To unsubscribe, or for help writing mail filters, visit settings.