Bill XIE 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 3:
(1 comment)
https://review.coreboot.org/#/c/21178/2/src/southbridge/intel/bd82x6x/pch.c File src/southbridge/intel/bd82x6x/pch.c:
https://review.coreboot.org/#/c/21178/2/src/southbridge/intel/bd82x6x/pch.c@... PS2, Line 266: struct southbridge_intel_bd82x6x_config *config = NULL; : u8 *old_hotplug_map = NULL;
you could initialise them here instead of in the loop?
for I have to get access to sizeof(config->pcie_hotplug_map) in order to define new_hotplug_map.