Attention is currently required from: Bill XIE, Jérémy Compostella.
Jonathon Hall has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78906?usp=email )
Change subject: drivers/pc80/rtc/option.c: Reset only CMOS range covered by checksum ......................................................................
Patch Set 4:
(2 comments)
Patchset:
PS4: Thank you Bill! Looks great
File src/drivers/pc80/rtc/option.c:
https://review.coreboot.org/c/coreboot/+/78906/comment/5841d187_d48de62a : 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.
Mini v2's checksum only covers one byte and the option_table.h has:
``` #define LB_CKS_RANGE_START 169 #define LB_CKS_RANGE_END 169 ```
Thanks for putting this together!