Attention is currently required from: Felix Singer, 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: Code-Review+2
(1 comment)
File src/mainboard/starlabs/labtop/include/variants.h:
https://review.coreboot.org/c/coreboot/+/60212/comment/e6306954_fe6e7c08
PS1, Line 11: PP_PERFORMANCE = 2,
If I do this, then GCC will complain that `NUM_POWER_PROFILES` is not handled in the switch statemen […]
oh. that's a good point to keep the current implementation then
--
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: Felix Held
felix-coreboot@felixheld.de
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: Andy Pont
andy.pont@sdcsystems.com
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Felix Singer
felixsinger@posteo.net
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Michael Niewöhner
foss@mniewoehner.de
Gerrit-Comment-Date: Sat, 18 Dec 2021 15:07:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Comment-In-Reply-To: Felix Held
felix-coreboot@felixheld.de
Gerrit-MessageType: comment