[S] Change in coreboot[main]: ec/google: Assume LID open for platforms without VBOOT_LID_SWITCH

[Subrata Banik (Code Review)]

Attention is currently required from: Boris Mittelberg, Caveh Jalali, Julius Werner, Kapil Porwal, Karthik Ramasubramanian, Nick Vaccaro, Nick Vaccaro.

Hello Boris Mittelberg, Caveh Jalali, Julius Werner, Kapil Porwal, Nick Vaccaro, Nick Vaccaro, build bot (Jenkins), 

I'd like you to reexamine a change. Please visit

    https://review.coreboot.org/c/coreboot/+/78077?usp=email

to look at the new patch set (#3).


Change subject: ec/google: Assume LID open for platforms without VBOOT_LID_SWITCH
......................................................................

ec/google: Assume LID open for platforms without VBOOT_LID_SWITCH

This patch modifies the get_lid_switch() implementation to assume
the LID is open when VBOOT_LID_SWITCH is not enabled. This is
necessary for platforms that do not have a physical LID switch.

Additionally, change the return type of `get_lid_switch()` to
`uint32_t` to keep it aligned with value field in lb_gpio.

BUG=b:299137940
TEST=Able to build and boot google/ovis.

Change-Id: I34bfc21329660a4bfd1ffed939b6dfb7e7109204
Signed-off-by: Subrata Banik <[e-mail]>
---
M src/ec/google/chromeec/switches.c
M src/include/bootmode.h
M src/lib/bootmode.c
M src/mainboard/google/butterfly/chromeos.c
M src/mainboard/google/drallion/chromeos.c
M src/mainboard/google/parrot/chromeos.c
M src/mainboard/google/sarien/chromeos.c
M src/mainboard/google/stout/chromeos.c
M src/mainboard/intel/adlrvp/chromeos.c
M src/mainboard/intel/coffeelake_rvp/chromeos.c
M src/mainboard/intel/glkrvp/chromeos.c
M src/mainboard/intel/jasperlake_rvp/chromeos.c
M src/mainboard/intel/kblrvp/chromeos.c
M src/mainboard/intel/mtlrvp/chromeos.c
M src/mainboard/intel/tglrvp/chromeos.c
M src/mainboard/samsung/lumpy/chromeos.c
16 files changed, 18 insertions(+), 17 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/78077/3
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/78077?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I34bfc21329660a4bfd1ffed939b6dfb7e7109204
Gerrit-Change-Number: 78077
Gerrit-PatchSet: 3
Gerrit-Owner: Subrata Banik <[e-mail]>
Gerrit-Reviewer: Boris Mittelberg <[e-mail]>
Gerrit-Reviewer: Caveh Jalali <[e-mail]>
Gerrit-Reviewer: Julius Werner <[e-mail]>
Gerrit-Reviewer: Kapil Porwal <[e-mail]>
Gerrit-Reviewer: Nick Vaccaro <[e-mail]>
Gerrit-Reviewer: Nick Vaccaro <[e-mail]>
Gerrit-Reviewer: build bot (Jenkins) <[e-mail]>
Gerrit-CC: Karthik Ramasubramanian <[e-mail]>
Gerrit-Attention: Caveh Jalali <[e-mail]>
Gerrit-Attention: Kapil Porwal <[e-mail]>
Gerrit-Attention: Nick Vaccaro <[e-mail]>
Gerrit-Attention: Julius Werner <[e-mail]>
Gerrit-Attention: Nick Vaccaro <[e-mail]>
Gerrit-Attention: Karthik Ramasubramanian <[e-mail]>
Gerrit-Attention: Boris Mittelberg <[e-mail]>
Gerrit-MessageType: newpatchset