Skip to content

Commit cbf3fb5

Browse files
jdelvareAndi Shyti
authored andcommitted
i2c: designware: Fix the functionality flags of the slave-only interface
When an I2C adapter acts only as a slave, it should not claim to support I2C master capabilities. Fixes: 5b6d721 ("i2c: designware: enable SLAVE in platform module") Signed-off-by: Jean Delvare <[email protected]> Cc: Luis Oliveira <[email protected]> Cc: Jarkko Nikula <[email protected]> Cc: Andy Shevchenko <[email protected]> Cc: Mika Westerberg <[email protected]> Cc: Jan Dabros <[email protected]> Cc: Andi Shyti <[email protected]> Reviewed-by: Andy Shevchenko <[email protected]> Acked-by: Jarkko Nikula <[email protected]> Tested-by: Jarkko Nikula <[email protected]> Signed-off-by: Andi Shyti <[email protected]>
1 parent d6d5645 commit cbf3fb5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/i2c/busses/i2c-designware-slave.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ static const struct i2c_algorithm i2c_dw_algo = {
220220

221221
void i2c_dw_configure_slave(struct dw_i2c_dev *dev)
222222
{
223-
dev->functionality = I2C_FUNC_SLAVE | DW_IC_DEFAULT_FUNCTIONALITY;
223+
dev->functionality = I2C_FUNC_SLAVE;
224224

225225
dev->slave_cfg = DW_IC_CON_RX_FIFO_FULL_HLD_CTRL |
226226
DW_IC_CON_RESTART_EN | DW_IC_CON_STOP_DET_IFADDRESSED;

0 commit comments

Comments
 (0)