Attention is currently required from: Paul Menzel, Christian Walter, Angel Pons. Lean Sheng Tan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63464 )
Change subject: mb/prodrive/atlas: Update Kconfig ......................................................................
Patch Set 5:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63464/comment/314f7b07_ea401cc4 PS4, Line 7: KConfig
Kconfig
Done
https://review.coreboot.org/c/coreboot/+/63464/comment/f33cdb06_f6f18f16 PS4, Line 9: KConfig
Kconfig
Done
https://review.coreboot.org/c/coreboot/+/63464/comment/d8a64126_82a9269f PS4, Line 9: Update KConfig per Atlas usages.
Please summarize that usage, but maybe you could also split the patch up.
Done
File src/mainboard/prodrive/atlas/Kconfig:
https://review.coreboot.org/c/coreboot/+/63464/comment/75de7a28_b218c076 PS2, Line 5: select DRIVERS_I2C_GENERIC : select DRIVERS_INTEL_DPTF : select DRIVERS_SPI_ACPI : select DRIVERS_USB_ACPI
But the chip drivers aren't used. […]
Done
https://review.coreboot.org/c/coreboot/+/63464/comment/f100ae9f_2a7b8f27 PS2, Line 17: : config IGNORE_IASL_MISSING_DEPENDENCY : def_bool y
The ChromeOS ASL should be fixed by CB:63521
Done
-
Lean Sheng Tan (Code Review)