michael/zephyr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

drivers: i2c: microchip: Add mutex to transaction implementation

Browse Source 
Currently, I2C transfer implementation does not have a mutex to
prevent multiple users from sending data at the same time, this could
lead to devices malfunction due to mixing I2C transaction data.

This patch adds mutex into transfer implementation to allow only one
user to send data at the time.

Signed-off-by: Bernardo Perez Priego <bernardo.perez.priego@intel.com>
This commit is contained in:
Bernardo Perez Priego
2025-10-21 16:42:20 -07:00
committed by Fabio Baltieri
parent cbc881cc51
commit 42fcca480b
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/i2c/i2c_mchp_xec_v2.c
View File

@@ -106,6 +106,7 @@ struct i2c_xec_data {
uint8_t i2c_ctrl;
uint8_t i2c_addr;
uint8_t i2c_status;
struct k_mutex mux;
};
/* Recommended programming values based on 16MHz
@@ -783,6 +784,7 @@ static int i2c_xec_transfer(const struct device *dev, struct i2c_msg *msgs,
}
#endif
k_mutex_lock(&data->mux, K_FOREVER);
for (uint8_t i = 0; i < num_msgs; i++) {
struct i2c_msg *m = &msgs[i];
@@ -798,6 +800,7 @@ static int i2c_xec_transfer(const struct device *dev, struct i2c_msg *msgs,
break;
}
}
k_mutex_unlock(&data->mux);
return ret;
}
@@ -1074,6 +1077,7 @@ static int i2c_xec_init(const struct device *dev)
return ret;
}
k_mutex_init(&data->mux);
#ifdef CONFIG_I2C_TARGET
const struct i2c_xec_config *config =
(const struct i2c_xec_config *const) (dev->config);