[coreboot-gerrit] Change in coreboot[master]: sb/intel/bd82x6x/usb_xhci: Lock OC config and refactoring

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34335 )

Change subject: sb/intel/bd82x6x/usb_xhci: Lock OC config and refactoring
......................................................................

Patch Set 3:

(4 comments)

https://review.coreboot.org/c/coreboot/+/34335/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34335/3//COMMIT_MSG@7 
PS3, Line 7: refactoring
*do refactoring* or *refactor code*

https://review.coreboot.org/c/coreboot/+/34335/3//COMMIT_MSG@19 
PS3, Line 19: Untested
Can the successful locking be verified from the OS?

https://review.coreboot.org/c/coreboot/+/34335/3/src/southbridge/intel/bd82x... 
File src/southbridge/intel/bd82x6x/usb_xhci.c:

https://review.coreboot.org/c/coreboot/+/34335/3/src/southbridge/intel/bd82x... 
PS3, Line 52: XHCC+2
Spaces around the operator?

XHCC + 2

https://review.coreboot.org/c/coreboot/+/34335/3/src/southbridge/intel/bd82x... 
PS3, Line 55: XHCC+2
Ditto.