Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62126 )
Change subject: ec/starlabs: Correct Keyboard Backlight offsets for GLK ......................................................................
ec/starlabs: Correct Keyboard Backlight offsets for GLK
Correct the offsets used for the keyboard backlight control: ECRAM_KBL_STATE 0x19 ECRAM_KBL_BRIGHTNESS 0x18
Signed-off-by: Sean Rhodes sean@starlabs.systems Change-Id: I23bac43301635e6b18f1cbd28311e7210b049c70 Reviewed-on: https://review.coreboot.org/c/coreboot/+/62126 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Andy Pont andy.pont@sdcsystems.com Reviewed-by: Angel Pons th3fanbus@gmail.com --- M src/ec/starlabs/merlin/variants/glk/ecdefs.h M src/ec/starlabs/merlin/variants/glk/emem.asl 2 files changed, 3 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Andy Pont: Looks good to me, but someone else must approve
diff --git a/src/ec/starlabs/merlin/variants/glk/ecdefs.h b/src/ec/starlabs/merlin/variants/glk/ecdefs.h index 5f43991..b09ffc7 100644 --- a/src/ec/starlabs/merlin/variants/glk/ecdefs.h +++ b/src/ec/starlabs/merlin/variants/glk/ecdefs.h @@ -15,8 +15,8 @@
/* EC RAM offsets. */ #define ECRAM_TRACKPAD_STATE 0x14 -#define ECRAM_KBL_STATE 0x18 -#define ECRAM_KBL_BRIGHTNESS 0x19 +#define ECRAM_KBL_STATE 0x19 +#define ECRAM_KBL_BRIGHTNESS 0x18 #define ECRAM_KBL_TIMEOUT 0x1a #define ECRAM_FN_LOCK_STATE 0x2c #define ECRAM_FN_CTRL_REVERSE 0x2d diff --git a/src/ec/starlabs/merlin/variants/glk/emem.asl b/src/ec/starlabs/merlin/variants/glk/emem.asl index d30b58c..9173745 100644 --- a/src/ec/starlabs/merlin/variants/glk/emem.asl +++ b/src/ec/starlabs/merlin/variants/glk/emem.asl @@ -20,8 +20,8 @@ AFG3, 8, // After G3 CLTP, 8, // Close Trackpad WKOL, 8, // Wake on Lid - KLSE, 8, // Keyboard Backlight State KLBE, 8, // Keyboard Backlight Brightness + KLSE, 8, // Keyboard Backlight State
Offset(0x1a), KLTE, 8, // Keyboard Backlight Timeout