Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48571 )
Change subject: soc/intel/skylake: Drop unreferenced devicetree settings ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48571/3/src/soc/intel/skylake/chip.... File src/soc/intel/skylake/chip.h:
https://review.coreboot.org/c/coreboot/+/48571/3/src/soc/intel/skylake/chip.... PS3, Line 311: ApertureSize Not referenced on Skylake.
https://review.coreboot.org/c/coreboot/+/48571/3/src/soc/intel/skylake/chip.... PS3, Line 398: PmConfigPciClockRun Duplicates `lpc_enable_pci_clk_cntl()` in `soc/intel/common/block/lpc/lpc_lib.c` which is unconditionally called. Should I create a follow-up?