Change in coreboot[master]: arch/x86: Init firmware pointer for EC SMSC KBC1098/KBC1126 at build ...

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans, Bill XIE, Patrick Rudolph, Angel Pons, Julius Werner, Michael Niewöhner, Kyösti Mälkki, Aaron Durbin.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51671 )

Change subject: arch/x86: Init firmware pointer for EC SMSC KBC1098/KBC1126 at build time
......................................................................

Patch Set 37: Code-Review+2

(2 comments)

Patchset:

PS37: 
Looks good, sorry for the delay. I've tested that the linker
complains if a bad location is chosen, e.g. overlap with .id
or the reset vector.

File src/arch/x86/bootblock.ld:

https://review.coreboot.org/c/coreboot/+/51671/comment/6fcd47fe_28f47784 
PS22, Line 57: _ID_SECTION_END - _ECFW_PTR
> Nico: Is the current approach okay? Is there any further changes needing to be made?

Seems good, yes. I still don't understand the code of this PS22, but
doesn't matter anymore, this line is gone :)