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 6:
(2 comments)
https://review.coreboot.org/#/c/21178/6/src/southbridge/intel/bd82x6x/pch.c File src/southbridge/intel/bd82x6x/pch.c:
https://review.coreboot.org/#/c/21178/6/src/southbridge/intel/bd82x6x/pch.c@... PS6, Line 259: southbridge_intel_bd82x6x_config
In order to make the 80-char-per-line limit happy, I have to write it like
just write it like this:
static void pch_pcie_devicetree_update( struct southbridge_intel_bd82x6x_config *config)
(two tabs indent is common in this case)
https://review.coreboot.org/#/c/21178/6/src/southbridge/intel/bd82x6x/pch.c@... PS6, Line 312: return; Not that I care, but others demand that code follows declarations. So this should come after the `u32 reg32;`.