Attention is currently required from: Sean Rhodes, Andy Pont.
Paul Menzel 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: Code-Review+1
(1 comment)
File src/mainboard/starlabs/labtop/mainboard.c:
https://review.coreboot.org/c/coreboot/+/59979/comment/236d51cb_246ac8f4
PS4, Line 50: if (num > 99999)
Print an error, if the number is invalid?
--
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-Attention: Andy Pont
andy.pont@sdcsystems.com
Gerrit-Comment-Date: Thu, 09 Dec 2021 11:13:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment