diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2022-11-10 14:51:49 +0100 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2022-11-12 23:23:02 +0000 |
commit | 852ab7500565c7c5ff256b32c6373d38da1c8866 (patch) | |
tree | beea731ba5249b2290425624dfad0e67f9422abf /src/drivers | |
parent | 9df0fee8fab3dc7a79632122257845e8d5720095 (diff) |
drivers/ipmi/ocp: Fix building with clang
Fix the following warning:
error: use of logical '&&' with constant operand
[-Werror,-Wconstant-logical-operand]
Change-Id: I9a2f03a0e05088a780ce1e829859421b461032ca
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69437
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin L Roth <gaumless@gmail.com>
Diffstat (limited to 'src/drivers')
-rw-r--r-- | src/drivers/ipmi/ocp/ipmi_sel.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/ipmi/ocp/ipmi_sel.c b/src/drivers/ipmi/ocp/ipmi_sel.c index b10755fd97..f7605ec572 100644 --- a/src/drivers/ipmi/ocp/ipmi_sel.c +++ b/src/drivers/ipmi/ocp/ipmi_sel.c @@ -105,7 +105,7 @@ __weak uint8_t get_blade_id(void) { struct ipmi_config_rsp rsp = {.slot_id = UINT8_MAX}; - if (CONFIG(IPMI_KCS) && CONFIG_BMC_KCS_BASE) { + if (CONFIG(IPMI_KCS) && CONFIG_BMC_KCS_BASE != 0) { if (ipmi_get_board_config(CONFIG_BMC_KCS_BASE, &rsp) != CB_SUCCESS) return UINT8_MAX; } |