[coreboot-gerrit] [S] Change in coreboot[master]: ec/google/chromeec: Use either EC or MB lid switch state

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Raul Rangel, Martin L Roth, Subrata Banik, Caveh Jalali, Matt DeVillier, Nick Vaccaro, Eric Lai, Tim Van Patten, Felix Held, Boris Mittelberg.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74332 )

Change subject: ec/google/chromeec: Use either EC or MB lid switch state
......................................................................

Patch Set 1:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74332/comment/481356db_37ffb6c5 
PS1, Line 19: TEST=Build Skyrim BIOS image and boot to OS. Trigger lid open/close
            : events and ensure that they are detected properly through
            : /proc/acpi/button/lid/LID0/state.
            : localhost ~ # cat /proc/acpi/button/lid/LID0/state
            : state:      open
            : localhost ~ # cat /proc/acpi/button/lid/LID0/state
            : state:      closed
            : 
            : Ensure that the system behaves as expected based on powerd
            : configuration. After signin, system suspends/resumes for lid close/open.
            : On signin screen, system shuts down/boots for lid close/open.
> Was `EC_ENABLE_LID_SWITCH` enabled or disabled for this testing? […]

I have tested with both configuration. The system behavior is not impacted. The only thing with disabling EC_ENABLE_LID_SWITCH is /proc/acpi/button/lid/LID0/state will not be present since the mainboard does not implement Lid Switch in Skyrim. That has got nothing to do with this change though.