Attention is currently required from: Arthur Heymans, Evgeny Zinoviev, Alexander Couzens, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56912 )
Change subject: nb/intel/sandybridge: Add a chipset devicetree ......................................................................
Patch Set 5:
(4 comments)
File src/mainboard/kontron/ktqm77/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/56912/comment/5075e3eb_b8ded2c6 PS5, Line 11: register "acpi_c1" = "1" # ACPI(C1) = MWAIT(C1) Might be better to leave this one in place for consistency. Not a big deal, though.
File src/northbridge/intel/sandybridge/Kconfig:
https://review.coreboot.org/c/coreboot/+/56912/comment/34343bef_b3d64129 PS5, Line 21: string nit: Type not needed
File src/northbridge/intel/sandybridge/chipset.cb:
https://review.coreboot.org/c/coreboot/+/56912/comment/3c0ebd2e_1a09e26a PS5, Line 17: subsystemid 0x17aa 0x21db inherit What is this?
https://review.coreboot.org/c/coreboot/+/56912/comment/2fd663ff_d0231486 PS5, Line 19: device pci 00.0 alias hb on end # host bridge : device pci 01.0 alias peg10 off end # PEG10 : device pci 01.1 alias peg11 off end # PEG11 : device pci 01.2 alias peg12 off end # PEG12 : device pci 02.0 alias igd off end # vga controller : device pci 04.0 alias dev4 off end # Device 4 : device pci 06.0 alias peg60 off end # PEG60 : : chip southbridge/intel/bd82x6x # Intel Series 6/7 PCH : device pci 14.0 alias xhci off end # USB 3.0 Controller (only on 7 series) : device pci 16.0 alias mei1 off end # Management Engine Interface 1 : device pci 16.1 alias mei2 off end # Management Engine Interface 2 : device pci 16.2 alias me_ide_r off end # Management Engine IDE-R : device pci 16.3 alias me_kt off end # Management Engine KT : device pci 19.0 alias gbe off end # Intel Gigabit Ethernet : device pci 1a.0 alias ehci2 off end # USB2 EHCI #2 : device pci 1b.0 alias hda off end # High Definition Audio : device pci 1c.0 alias rp1 off end # PCIe Port #1 : device pci 1c.1 alias rp2 off end # PCIe Port #2 : device pci 1c.2 alias rp3 off end # PCIe Port #3 : device pci 1c.3 alias rp4 off end # PCIe Port #4 : device pci 1c.4 alias rp5 off end # PCIe Port #5 : device pci 1c.5 alias rp6 off end # PCIe Port #6 : device pci 1c.6 alias rp7 off end # PCIe Port #7 : device pci 1c.7 alias rp8 off end # PCIe Port #8 : device pci 1d.0 alias ehci1 off end # USB2 EHCI #1 : device pci 1e.0 alias pci_b off end # PCI bridge : device pci 1f.0 alias lpc on end # LPC bridge : device pci 1f.2 alias sata1 off end # SATA Controller 1 : device pci 1f.3 alias smbus on end # SMBus : device pci 1f.5 alias sata2 off end # SATA Controller 2 : device pci 1f.6 alias thermal off end # Thermal : end Do you *need* to define the PCI devices in the chipset devicetree in this patch? It would seem that the aliases could break static.c overrides.