Change in coreboot[master]: mb/starlabs/labtop: Store serial number in CMOS - coreboot-gerrit

9 Dec 2021


      Attention is currently required from: Sean Rhodes.
Andy Pont has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59979 )
Change subject: mb/starlabs/labtop: Store serial number in CMOS
......................................................................
Patch Set 4:
(1 comment)
File src/mainboard/starlabs/labtop/cmos.layout:
https://review.coreboot.org/c/coreboot/+/59979/comment/4ed511d0_5f391d9d 
PS4, Line 54: 2000	16	h	0	serial_number
16bits only gives serial numbers 0 -> 65535. Either more bits need allocating here or the check in mainboard.c needs amending.
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/59979
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I70c35c8f99f7e3ac8e45e742156dca741a6c6933
Gerrit-Change-Number: 59979
Gerrit-PatchSet: 4
Gerrit-Owner: Sean Rhodes admin@starlabs.systems
Gerrit-Reviewer: Andy Pont andy.pont@sdcsystems.com
Gerrit-Reviewer: Paul Menzel paulepanter@mailbox.org
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Sean Rhodes admin@starlabs.systems
Gerrit-Comment-Date: Thu, 09 Dec 2021 11:17:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment