Attention is currently required from: Patrick Rudolph. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51332 )
Change subject: soc/intel/skylake: detect if IGD is present ......................................................................
Patch Set 1:
(5 comments)
File src/soc/intel/skylake/chip.c:
https://review.coreboot.org/c/coreboot/+/51332/comment/352c18b1_bd462baa PS1, Line 536: if (CONFIG(RUN_FSP_GOP) && dev && dev->enabled && \ Avoid unnecessary line continuations
File src/soc/intel/skylake/romstage/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/51332/comment/a9f1dd67_67c04a43 PS1, Line 118: igd = 0x8086 == pci_read_config16(SA_DEVFN_IGD, PCI_VENDOR_ID); Comparisons should place the constant on the right side of the test
File src/soc/intel/skylake/romstage/systemagent.c:
https://review.coreboot.org/c/coreboot/+/51332/comment/034c3c00_04585797 PS1, Line 32: if (igd_dev && igd_dev->enabled && pci_read_config16(SA_DEVFN_IGD, PCI_VENDOR_ID) == 0x8086) { line over 96 characters
https://review.coreboot.org/c/coreboot/+/51332/comment/2fde6556_429e08a6 PS1, Line 32: if (igd_dev && igd_dev->enabled && pci_read_config16(SA_DEVFN_IGD, PCI_VENDOR_ID) == 0x8086) { braces {} are not necessary for single statement blocks
File src/soc/intel/skylake/systemagent.c:
https://review.coreboot.org/c/coreboot/+/51332/comment/ce545ecb_5b128aa2 PS1, Line 49: if (igd_dev && igd_dev->enabled && pci_read_config16(igd_dev, PCI_VENDOR_ID) == 0x8086) line over 96 characters