diff options
author | Rob Barnes <robbarnes@google.com> | 2021-06-14 10:22:21 -0600 |
---|---|---|
committer | Werner Zeh <werner.zeh@siemens.com> | 2021-06-30 04:57:16 +0000 |
commit | f1ade489c85c95c572ef907bdb3b7f4835c1a9c7 (patch) | |
tree | ea5c369beecca0d625b0f440af932dbc03a1dad5 /src/mainboard | |
parent | 8bc5fa9f0a36514c2a7ebc5a375b18cb467a57e7 (diff) |
ec/google: Use EC_HOST_EVENT_NONE
google_chromeec_get_event returns 0 for no event. Return
EC_HOST_EVENT_NONE=0 to improve readability.
BUG=b:184074997
TEST=Build and boot guybrush without error
Signed-off-by: Rob Barnes <robbarnes@google.com>
Change-Id: Ic08ed9ccdd7c0023d0fe8b641fcf60dca495a242
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55547
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Diffstat (limited to 'src/mainboard')
-rw-r--r-- | src/mainboard/google/auron/ec.c | 2 | ||||
-rw-r--r-- | src/mainboard/google/auron/smihandler.c | 6 | ||||
-rw-r--r-- | src/mainboard/google/cyan/smihandler.c | 6 | ||||
-rw-r--r-- | src/mainboard/google/link/smihandler.c | 9 | ||||
-rw-r--r-- | src/mainboard/google/rambi/smihandler.c | 9 | ||||
-rw-r--r-- | src/mainboard/google/slippy/ec.c | 2 | ||||
-rw-r--r-- | src/mainboard/google/slippy/smihandler.c | 9 | ||||
-rw-r--r-- | src/mainboard/intel/strago/smihandler.c | 6 |
8 files changed, 29 insertions, 20 deletions
diff --git a/src/mainboard/google/auron/ec.c b/src/mainboard/google/auron/ec.c index 5be20c01d2..d4482b01ba 100644 --- a/src/mainboard/google/auron/ec.c +++ b/src/mainboard/google/auron/ec.c @@ -23,7 +23,7 @@ void mainboard_ec_init(void) google_chromeec_events_init(&info, s3_wakeup); if (s3_wakeup) { /* Clear pending events. */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; } diff --git a/src/mainboard/google/auron/smihandler.c b/src/mainboard/google/auron/smihandler.c index 34ed171f8a..6fa95e8dff 100644 --- a/src/mainboard/google/auron/smihandler.c +++ b/src/mainboard/google/auron/smihandler.c @@ -97,7 +97,7 @@ void mainboard_smi_sleep(u8 slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; } @@ -107,14 +107,14 @@ int mainboard_smi_apmc(u8 apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_sci_mask(MAINBOARD_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_smi_mask(MAINBOARD_EC_SMI_EVENTS); break; diff --git a/src/mainboard/google/cyan/smihandler.c b/src/mainboard/google/cyan/smihandler.c index d9121e16d0..9083a8773b 100644 --- a/src/mainboard/google/cyan/smihandler.c +++ b/src/mainboard/google/cyan/smihandler.c @@ -126,7 +126,7 @@ void mainboard_smi_sleep(uint8_t slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; /* Set LPC lines to low power in S3/S5. */ @@ -141,14 +141,14 @@ int mainboard_smi_apmc(uint8_t apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_sci_mask(MAINBOARD_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_smi_mask(MAINBOARD_EC_SMI_EVENTS); break; diff --git a/src/mainboard/google/link/smihandler.c b/src/mainboard/google/link/smihandler.c index 719076b3f8..2f42b7da18 100644 --- a/src/mainboard/google/link/smihandler.c +++ b/src/mainboard/google/link/smihandler.c @@ -69,7 +69,8 @@ void mainboard_smi_sleep(u8 slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; /* Enable wake events */ google_chromeec_set_wake_mask(LINK_EC_S3_WAKE_EVENTS); @@ -81,13 +82,15 @@ int mainboard_smi_apmc(u8 apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_sci_mask(LINK_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_smi_mask(LINK_EC_SMI_EVENTS); break; } diff --git a/src/mainboard/google/rambi/smihandler.c b/src/mainboard/google/rambi/smihandler.c index c16a0f525b..358a92f967 100644 --- a/src/mainboard/google/rambi/smihandler.c +++ b/src/mainboard/google/rambi/smihandler.c @@ -84,7 +84,8 @@ void mainboard_smi_sleep(uint8_t slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; } int mainboard_smi_apmc(uint8_t apmc) @@ -93,13 +94,15 @@ int mainboard_smi_apmc(uint8_t apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_sci_mask(MAINBOARD_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_smi_mask(MAINBOARD_EC_SMI_EVENTS); break; } diff --git a/src/mainboard/google/slippy/ec.c b/src/mainboard/google/slippy/ec.c index 5761a7c579..a3ee95bd0b 100644 --- a/src/mainboard/google/slippy/ec.c +++ b/src/mainboard/google/slippy/ec.c @@ -22,7 +22,7 @@ void mainboard_ec_init(void) google_chromeec_events_init(&info, s3_wakeup); if (s3_wakeup) { /* Clear pending events. */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; } diff --git a/src/mainboard/google/slippy/smihandler.c b/src/mainboard/google/slippy/smihandler.c index 81fcb0aafa..16a1b18b7a 100644 --- a/src/mainboard/google/slippy/smihandler.c +++ b/src/mainboard/google/slippy/smihandler.c @@ -100,7 +100,8 @@ void mainboard_smi_sleep(u8 slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; } int mainboard_smi_apmc(u8 apmc) @@ -109,13 +110,15 @@ int mainboard_smi_apmc(u8 apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_sci_mask(MAINBOARD_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0); + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) + ; google_chromeec_set_smi_mask(MAINBOARD_EC_SMI_EVENTS); break; } diff --git a/src/mainboard/intel/strago/smihandler.c b/src/mainboard/intel/strago/smihandler.c index fc77103537..e953ccd531 100644 --- a/src/mainboard/intel/strago/smihandler.c +++ b/src/mainboard/intel/strago/smihandler.c @@ -112,7 +112,7 @@ void mainboard_smi_sleep(uint8_t slp_typ) google_chromeec_set_sci_mask(0); /* Clear pending events that may trigger immediate wake */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; /* Set LPC lines to low power in S3/S5. */ @@ -126,14 +126,14 @@ int mainboard_smi_apmc(uint8_t apmc) case APM_CNT_ACPI_ENABLE: google_chromeec_set_smi_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_sci_mask(MAINBOARD_EC_SCI_EVENTS); break; case APM_CNT_ACPI_DISABLE: google_chromeec_set_sci_mask(0); /* Clear all pending events */ - while (google_chromeec_get_event() != 0) + while (google_chromeec_get_event() != EC_HOST_EVENT_NONE) ; google_chromeec_set_smi_mask(MAINBOARD_EC_SMI_EVENTS); break; |