[coreboot-gerrit] [S] Change in coreboot[master]: drivers/pc80/rtc/option.c: Allow CMOS defaults to extend to bank 1

[Jonathon Hall (Code Review)]

Jonathon Hall has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74902 )

Change subject: drivers/pc80/rtc/option.c: Allow CMOS defaults to extend to bank 1
......................................................................

drivers/pc80/rtc/option.c: Allow CMOS defaults to extend to bank 1

CMOS defaults greater than 128 bytes long will extend to bank 1.

Change-Id: I9ee8364d01dd8520be101de3f83d2302d50c7283
Signed-off-by: Jonathon Hall 
---
M src/drivers/pc80/rtc/option.c
1 file changed, 14 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/74902/1

diff --git a/src/drivers/pc80/rtc/option.c b/src/drivers/pc80/rtc/option.c
index e8e2345..bb295cc 100644
--- a/src/drivers/pc80/rtc/option.c
+++ b/src/drivers/pc80/rtc/option.c
@@ -211,7 +211,8 @@
    		return;

u8 control_state = cmos_disable_rtc();
-		for (i = 14; i < MIN(128, length); i++)
+		/* Max length of 256 spans bank 0 and bank 1 */
+		for (i = 14; i < MIN(256, length); i++)
    		cmos_write_inner(cmos_default[i], i);
    	cmos_restore_rtc(control_state);
    }