@@ -284,7 +284,7 @@ int i2c_dw_amdpsp_probe_lock_support(struct dw_i2c_dev *dev)
if (!dev)
return -ENODEV;
- if (!(dev->flags & ARBITRATION_SEMAPHORE))
+ if ((dev->flags & SEMAPHORE_MASK) != SEMAPHORE_AMD_PSP)
return -ENODEV;
/* Allow to bind only one instance of a driver */
@@ -302,7 +302,9 @@ struct dw_i2c_dev {
#define ACCESS_INTR_MASK BIT(0)
#define ACCESS_NO_IRQ_SUSPEND BIT(1)
-#define ARBITRATION_SEMAPHORE BIT(2)
+
+#define SEMAPHORE_AMD_PSP (1 << 4)
+#define SEMAPHORE_MASK GENMASK(7, 4)
#define MODEL_MSCC_OCELOT (1 << 8)
#define MODEL_BAIKAL_BT1 (2 << 8)
@@ -50,7 +50,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = {
{ "808622C1", ACCESS_NO_IRQ_SUSPEND },
{ "AMD0010", ACCESS_INTR_MASK },
{ "AMDI0010", ACCESS_INTR_MASK },
- { "AMDI0019", ACCESS_INTR_MASK | ARBITRATION_SEMAPHORE },
+ { "AMDI0019", ACCESS_INTR_MASK | SEMAPHORE_AMD_PSP },
{ "AMDI0510", 0 },
{ "APMC0D0F", 0 },
{ "HISI02A1", 0 },
Make AMD PSP I2C bus arbitration semaphore flag as a type and add a related flag bit mask. Only one bus semaphore implementation can be active at a time so no need to reserve one flag bit for each of them. Chosen mask size of 4-bits is large considering currently there is only two semaphore implementations but there is no lack of free flag bits either. Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com> --- drivers/i2c/busses/i2c-designware-amdpsp.c | 2 +- drivers/i2c/busses/i2c-designware-core.h | 4 +++- drivers/i2c/busses/i2c-designware-platdrv.c | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-)