Attention is currently required from: Julius Werner. Bill XIE has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/51671 )
Change subject: arch/x86: Reserve space for EC SMSC KBC1098/KBC1126 ......................................................................
arch/x86: Reserve space for EC SMSC KBC1098/KBC1126
According to util/kbc1126/README.md, for these ECs to work, the address and size of their two firmware should be written to $s-0x100` (`$s` means the image size, done with kbc1126_ec_insert), which means that every existing section (especially those used to store code) should not overlap this address, otherwise the bootblock will get damaged when inserting firmwares of the EC.
Signed-off-by: Bill XIE persmule@hardenedlinux.org Change-Id: I4f0de0c4d7283e630242fbe84a46e0547783c49e --- M src/arch/x86/bootblock.ld 1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/51671/1
diff --git a/src/arch/x86/bootblock.ld b/src/arch/x86/bootblock.ld index 3cd0900..37367c9 100644 --- a/src/arch/x86/bootblock.ld +++ b/src/arch/x86/bootblock.ld @@ -30,8 +30,12 @@ * address gets applied. */ PROGRAM_SZ = SIZEOF(.text) + 512; - +#if CONFIG(EC_HP_KBC1126) + _KBC1126_FW_LOCATIONS = 0xffffff00; + . = MIN(_KBC1126_FW_LOCATIONS, MIN(_ID_SECTION, _FIT_POINTER)) - EARLYASM_SZ; +#else . = MIN(_ID_SECTION, _FIT_POINTER) - EARLYASM_SZ; +#endif . = CONFIG(SIPI_VECTOR_IN_ROM) ? ALIGN(4096) : ALIGN(16); BOOTBLOCK_TOP = .; .init (.) : {