[coreboot-gerrit] [S] Change in coreboot[main]: mb/starlabs/starlite_adl: Allow controlling the power LED brightness

[Sean Rhodes (Code Review)]

Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86240?usp=email )

Change subject: mb/starlabs/starlite_adl: Allow controlling the power LED brightness
......................................................................

mb/starlabs/starlite_adl: Allow controlling the power LED brightness

Add an option to selected a reduced brightness for the power LED. The
EC code will use this option to write to the relavant offset
accordingly.

Change-Id: I4796e0572a48bca5f9c59e96466416e975cfe8ca
Signed-off-by: Sean Rhodes 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/86240
Tested-by: build bot (Jenkins) 
Reviewed-by: Matt DeVillier 
---
M src/mainboard/starlabs/starlite_adl/cfr.c
1 file changed, 13 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Matt DeVillier: Looks good to me, approved

diff --git a/src/mainboard/starlabs/starlite_adl/cfr.c b/src/mainboard/starlabs/starlite_adl/cfr.c
index 69635bb..58a09a3 100644
--- a/src/mainboard/starlabs/starlite_adl/cfr.c
+++ b/src/mainboard/starlabs/starlite_adl/cfr.c
@@ -193,6 +193,18 @@
 });
 #endif

+static const struct sm_object power_led = SM_DECLARE_ENUM({
+	.opt_name	= "power_led",
+	.ui_name	= "Power LED Brightness",
+	.ui_helptext	= "Control the maximum brightness of the power LED",
+	.default_value	= 0,
+	.values		= (const struct sm_enum_value[]) {
+				{ "Normal",		0		},
+				{ "Reduced",		1		},
+				SM_ENUM_VALUE_END,
+	},
+});
+
 static const struct sm_object reboot_counter = SM_DECLARE_NUMBER({
    .opt_name	= "reboot_counter",
    .ui_name	= "Reboot Counter",
@@ -254,6 +266,7 @@
    	#if CONFIG(EC_STARLABS_CHARGING_SPEED)
    	&charging_speed,
    	#endif
+		&power_led,
    	&power_on_after_fail,
    	NULL
    },