[coreboot-gerrit] [S] Change in coreboot[main]: drivers/pc80/rtc/option.c: Reset only checked CMOS range during resume

[Bill XIE (Code Review)]

Attention is currently required from: Jonathon Hall, Jérémy Compostella, Nico Huber.

Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78906?usp=email )

Change subject: drivers/pc80/rtc/option.c: Reset only checked CMOS range during resume
......................................................................

Patch Set 6:

(2 comments)

Patchset:

PS4:
> I agree to the patch, just want to mention that this will most […]

Then we could restore the checked range only during s3 resume, while restore the whole CMOS after 14 during normal boot, as did in the Patchset 6.

File src/drivers/pc80/rtc/option.c:

https://review.coreboot.org/c/coreboot/+/78906/comment/b6cb2f20_3e77fecf :
PS4, Line 217: 		for (i = LB_CKS_RANGE_START; i < MIN(LB_CKS_RANGE_END, length); i++)
> @Bill It looks like the range end is inclusive, so `LB_CKS_RANGE_END+1` here. […]

Done