Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38368 )
Change subject: sb/intel/i82371eb: Enable upper NVRAM bank ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/38368/1/src/southbridge/intel/i8237... File src/southbridge/intel/i82371eb/bootblock.c:
https://review.coreboot.org/c/coreboot/+/38368/1/src/southbridge/intel/i8237... PS1, Line 67: RTC_POS_DECODE | UPPER_RAM_EN | RTC_ENABLE); Fits on the previous line (we switched to 96 characters not too long ago)
https://review.coreboot.org/c/coreboot/+/38368/1/src/southbridge/intel/i8237... PS1, Line 68: Empty line