summaryrefslogtreecommitdiffstats
path: root/src/mainboard/google/zork/variants/trembyle
diff options
context:
space:
mode:
authorEric Lai <ericr_lai@compal.corp-partner.google.com>2020-10-20 02:44:53 +0800
committerPatrick Georgi <pgeorgi@google.com>2020-10-26 06:44:02 +0000
commitf209b18df3b66d78a9838787182d40fc72eee010 (patch)
tree69f27fbfb9c36cb15efb827bf9959b53ea57cd8b /src/mainboard/google/zork/variants/trembyle
parent7d1a137b845ad0b4d1aa1553070b26411b899cf6 (diff)
downloadcoreboot-f209b18df3b66d78a9838787182d40fc72eee010.tar.gz
coreboot-f209b18df3b66d78a9838787182d40fc72eee010.tar.bz2
coreboot-f209b18df3b66d78a9838787182d40fc72eee010.zip
mb/google/zork: Update style of check on cbi return values
Since google_chromeec_cbi_get_board_version and google_chromeec_cbi_get_fw_config both call cbi_get_unit32 and return 0 as success, non-zero as failure. Let's add more readability for the false condition. BUG=None TEST=check with empty CBI value Signed-off-by: Eric Lai <ericr_lai@compal.corp-partner.google.com> Change-Id: Ia49ac1ee35302f8f6afe8c0eb8e13afdf36c5b2b Reviewed-on: https://review.coreboot.org/c/coreboot/+/46566 Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/zork/variants/trembyle')
-rw-r--r--src/mainboard/google/zork/variants/trembyle/gpio.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mainboard/google/zork/variants/trembyle/gpio.c b/src/mainboard/google/zork/variants/trembyle/gpio.c
index 9931479cc016..4d73ea01224a 100644
--- a/src/mainboard/google/zork/variants/trembyle/gpio.c
+++ b/src/mainboard/google/zork/variants/trembyle/gpio.c
@@ -60,7 +60,7 @@ const struct soc_amd_gpio *variant_override_gpio_table(size_t *size)
* and so apply overrides. If board version is provided by the EC, then apply overrides
* if version < 2.
*/
- if (google_chromeec_cbi_get_board_version(&board_version))
+ if (google_chromeec_cbi_get_board_version(&board_version) != 0)
board_version = 1;
if (board_version <= 2) {