Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86322?usp=email )
Change subject: ec/starlabs/merlin: Only include lid switch ACPI when needed ......................................................................
ec/starlabs/merlin: Only include lid switch ACPI when needed
Desktop devices don't need and shouldn't define a lid switch.
TEST=build/boot starlabs/byte,starlite_adl
Change-Id: Iecf3e2558e244cc0bec301a505c0bc86684954aa Signed-off-by: Matt DeVillier matt.devillier@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/86322 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Sean Rhodes sean@starlabs.systems --- M src/ec/starlabs/merlin/acpi/ec.asl 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Sean Rhodes: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/ec/starlabs/merlin/acpi/ec.asl b/src/ec/starlabs/merlin/acpi/ec.asl index f3fdb07..ae50927 100644 --- a/src/ec/starlabs/merlin/acpi/ec.asl +++ b/src/ec/starlabs/merlin/acpi/ec.asl @@ -112,11 +112,11 @@ #include "ac.asl" #if CONFIG(SYSTEM_TYPE_LAPTOP) || CONFIG(SYSTEM_TYPE_DETACHABLE) #include "battery.asl" + #include "lid.asl" #endif #if !CONFIG(EC_STARLABS_MERLIN) #include "events.asl" #endif - #include "lid.asl" #if CONFIG(SYSTEM_TYPE_DETACHABLE) #include "dock.asl" #endif