From f42da176de685ebc6432da1008c408876060dced Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 1 Dec 2020 21:29:59 +0100 Subject: soc/amd/common/smbus: remove misleading definition SMBHST_STAT_NOERROR was a redefinition of SMBHST_STAT_INTERRUPT that was used in smbus_wait_until_done. Remove the misleading bit definition that also didn't correspond with the register definitions and replace it with the definition of the actual bit that gets checked. Also add a comment that the code actually checks the IRQ status flag to see if the last command is already completed. Change-Id: I1a58fe0d58d3887dd2e83320e977a57e271685b3 Signed-off-by: Felix Held Reviewed-on: https://review.coreboot.org/c/coreboot/+/48219 Tested-by: build bot (Jenkins) Reviewed-by: Marshall Dawson --- src/soc/amd/common/block/include/amdblocks/smbus.h | 1 - 1 file changed, 1 deletion(-) (limited to 'src/soc/amd/common/block/include') diff --git a/src/soc/amd/common/block/include/amdblocks/smbus.h b/src/soc/amd/common/block/include/amdblocks/smbus.h index 773daf796c..73f63b02eb 100644 --- a/src/soc/amd/common/block/include/amdblocks/smbus.h +++ b/src/soc/amd/common/block/include/amdblocks/smbus.h @@ -11,7 +11,6 @@ #define SMBHST_STAT_INTERRUPT (1 << 1) #define SMBHST_STAT_BUSY (1 << 0) #define SMBHST_STAT_CLEAR 0xff -#define SMBHST_STAT_NOERROR (1 << 1) /* TODO: this one looks odd */ #define SMBHST_STAT_VAL_BITS 0x1f #define SMBHST_STAT_ERROR_BITS 0x1c -- cgit v1.2.3