[coreboot-gerrit] [S] Change in coreboot[main]: sb/intel/bd82x6x: Make space for USB port config in devicetree

[Patrick Rudolph (Code Review)]

Attention is currently required from: Keith Hui.

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81878?usp=email )

Change subject: sb/intel/bd82x6x: Make space for USB port config in devicetree
......................................................................

Patch Set 2: Code-Review+2

(1 comment)

File src/southbridge/intel/bd82x6x/chip.h:

https://review.coreboot.org/c/coreboot/+/81878/comment/9b20fccd_3da6fd97 :
PS2, Line 12: USB_GAIN_1
This can become confusing when trying to port a new mainboard. How about a single enum? You could parse the enum value in native raminit code directly and convert it for MRC in southbridge_fill_pei_data().