Attention is currently required from: Felix Singer, Sean Rhodes, Angel Pons, Michael Niewöhner.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60212 )
Change subject: mb/starlabs/labtop: Add enum for `power_profile`
......................................................................
Patch Set 1:
(1 comment)
File src/mainboard/starlabs/labtop/include/variants.h:
https://review.coreboot.org/c/coreboot/+/60212/comment/409f4103_572bb314
PS1, Line 11: PP_PERFORMANCE = 2,
might be an idea to add a last entry without explicitly assigning a value to that and then just use that instead of NUM_POWER_PROFILES which might become wrong when an additional enum value would be added later. maybe also add a comment to that entry that it needs to be the last one and is only used for the total number of valid values
--
To view, visit
https://review.coreboot.org/c/coreboot/+/60212
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I47a453ea12d164d26908a9944a89a481757e753c
Gerrit-Change-Number: 60212
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Felix Singer
felixsinger@posteo.net
Gerrit-Reviewer: Michael Niewöhner
foss@mniewoehner.de
Gerrit-Reviewer: Sean Rhodes
admin@starlabs.systems
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Felix Held
felix-coreboot@felixheld.de
Gerrit-Attention: Felix Singer
felixsinger@posteo.net
Gerrit-Attention: Sean Rhodes
admin@starlabs.systems
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Michael Niewöhner
foss@mniewoehner.de
Gerrit-Comment-Date: Sat, 18 Dec 2021 13:11:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment