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

[Arthur Heymans (Code Review)]

Attention is currently required from: Bill XIE, Nico Huber, Patrick Rudolph, Angel Pons, Julius Werner, Arthur Heymans, Michael Niewöhner, Kyösti Mälkki, Aaron Durbin.
Arthur Heymans 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 27:

(1 comment)

File src/ec/hp/kbc1126/Kconfig:

https://review.coreboot.org/c/coreboot/+/51671/comment/b66104db_2e60e017 
PS27, Line 30: if KBC1126_FIRMWARE
> I'd much prefer to always add the `.ecfw_ptr` section, even when not adding the actual EC firmware (pointers would simply be invalid). This way, Jenkins will build-test it, and will catch build errors related to `.ecfw_ptr`.

I second this. 'if EC_HP_KBC1126' is better idea.