[PATCH 2/2] Fix x86 baytrail boot flow, UPD/VPD Updates

Tom Rini trini at konsulko.com
Tue Feb 3 19:14:44 CET 2026


On Mon, Jul 14, 2025 at 04:02:49PM -0500, Eric Schikschneit wrote:

> The VPD has been updated to include additional values found in the Intel
> FSP_MR5 Integration Guide.
> 
> Disable MRC_CACHE for the baytrail platform as this will break the boot flow
> as well. This issue has not been diagnosed further, but I suspect it is similar
> to my previous patch and related to how modern GCC generates the low level
> op-codes.
> 
> Patch 2 of 2
> 
> Upstream-Status: Pending
> 
> Signed-off-by: Eric Schikschneit <eric.schikschneit at novatechautomation.com>
[snip]
> +	/* Set all values to known good defaults as documented in 	*
> +	 * Intel FSP_MR5 file: BAYTRAIL_FSP.bsf				*/
> +
> +	/* NOTE: This breaks u-boot ability to set board options via	*
> +	 * device tree, but guarantees specific known values..		*
> +	 * you wouldnt use an uninitialized variable would you??	*/

/*
 * Multi-line comments like this please per
 * https://docs.u-boot.org/en/latest/develop/codingstyle.html
 */

And we should spell out that device tree can't set these any more in the
commit message, along with spelling out why we can't do that anymore
safely (why can't we?). Thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20260203/328df1e2/attachment-0001.sig>


More information about the U-Boot mailing list