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?
and the last memcpy out of the loop need access to old_hotplug_map.