build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35762 )
Change subject: sb/intel/ibexpeak: Implement USB current settings ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35762/1/src/southbridge/intel/ibexp... File src/southbridge/intel/ibexpeak/early_usb.c:
https://review.coreboot.org/c/coreboot/+/35762/1/src/southbridge/intel/ibexp... PS1, Line 44: RCBA32_AND_OR(USBIRB, ~0x617f0, (3 << 17) | (1 << 12) | (1 << 10) | (1 << 8) | (4 << 4)); line over 96 characters
https://review.coreboot.org/c/coreboot/+/35762/1/src/southbridge/intel/ibexp... File src/southbridge/intel/ibexpeak/pch.h:
https://review.coreboot.org/c/coreboot/+/35762/1/src/southbridge/intel/ibexp... PS1, Line 86: { open brace '{' following struct go on the same line