Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44865 )
Change subject: soc/intel/common/block/smbus: Add config to use ACPI ......................................................................
Patch Set 2:
Patch Set 1:
Thanks for the review.
where are you planning to use this? The only use I see in this patch train is the one on top of it, but nothing ever selects this Kconfig
I am setting this config for kontron mal10 (Apollo Lake) [1] board to ignore the warnings in CB:44866 and to build the acpi code for smbus (CB:44507). This is needed to access Nuvoton nct7802 HWM, which is connected to the SMBus.
I think this also needs to be done for the kontron/bsl6 (Sky/Kaby Lake) board as it also uses nct7802.
[1] https://review.coreboot.org/c/coreboot/+/39133/42/src/mainboard/kontron/mal1...