[coreboot-gerrit] Change in coreboot[master]: [WIP] arch/x86: Add symbols for CAR MTRRs in linker script

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30872 )

Change subject: [WIP] arch/x86: Add symbols for CAR MTRRs in linker script
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/30872/5/src/arch/x86/car.ld 
File src/arch/x86/car.ld:

https://review.coreboot.org/c/coreboot/+/30872/5/src/arch/x86/car.ld@90 
PS5, Line 90: _xip_mtrr_mask = ~(MAX(4096, _xip_program_sz_log2) - 1);
> There's only a few chipsets that would benefit from these symbols. […]

The references are in couple cache_as_ram.S files and I don't have a single good guard ready at hand. It would be either new Kconfig or logical or of selected CPU_INTEL_SLOT_x and CPU_INTEL_SOCKET_x.

Would you accept _booblock_mtrr_mask exposed unconditionally?