[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/xeon_sp: Handle GPIO SMIs

[Patrick Rudolph (Code Review)]

Attention is currently required from: Alicja Michalska, Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Shuo Liu, Tim Chu.

Patrick Rudolph has posted comments on this change by Alicja Michalska. ( https://review.coreboot.org/c/coreboot/+/84301?usp=email )

Change subject: soc/intel/xeon_sp: Handle GPIO SMIs
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84301/comment/7053d723_d922661d?usp... :
PS2, Line 9: GPIO_SMI_STS
Why do you need to handle GPIO_SMIs? Why are they enabled when there are no mainboard_smi_gpi_handler() in any of the boards supporting Lewisburg.

Maybe your GPIO config is just wrong or copy&pasted from some other firmware?