Attention is currently required from: Bora Guvendik, Michał Żygowski, Michał Żygowski, Name of user not set #1004406, Subrata Banik, Tim Crawford.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69798?usp=email )
Change subject: soc/intel/alderlake: Add support for Raptor Lake S CPUs ......................................................................
Patch Set 11: Code-Review+1
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69798/comment/d168ed85_118fb7ce : PS11, Line 10: DOCs docs
https://review.coreboot.org/c/coreboot/+/69798/comment/1d51c2ed_26bbdb66 : PS11, Line 13: LinuxBoot Does that mean Linux kernel and u-root?
https://review.coreboot.org/c/coreboot/+/69798/comment/770d58d0_ca20e427 : PS11, Line 14: What CPU did you test with?
File src/soc/intel/alderlake/Kconfig:
https://review.coreboot.org/c/coreboot/+/69798/comment/0b16c141_3323f26b : PS11, Line 159: default 32 Please mention this in the commit message.
File src/soc/intel/common/block/lpc/lpc.c:
https://review.coreboot.org/c/coreboot/+/69798/comment/b94b366e_db5cf738 : PS11, Line 185: PCI_DID_INTEL_RPP_S_ESPI_Z790, : PCI_DID_INTEL_RPP_S_ESPI_H770, : PCI_DID_INTEL_RPP_S_ESPI_B760, Please mention this in the commit message, or even make the rename a separate commit.