Nico Huber has posted comments on this change. ( https://review.coreboot.org/21178 )
Change subject: sb/intel/bd82x6x: make hotplug map consistent to remapped ports ......................................................................
Patch Set 7:
(1 comment)
Looks good now. Alas, I spotted a semantical problem.
https://review.coreboot.org/#/c/21178/7/src/southbridge/intel/bd82x6x/pch.c File src/southbridge/intel/bd82x6x/pch.c:
https://review.coreboot.org/#/c/21178/7/src/southbridge/intel/bd82x6x/pch.c@... PS7, Line 294: [PCI_FUNC(dev->path.pci.devfn)]; This should be done before the `if`. Slots that didn't move need the hotplug setting copied too. Alternatively you could initialize `new_hotplug_map` with the values of the old map.