Skip to content

Commit

Permalink
i2c: at91: Fix the functionality flags of the slave-only interface
Browse files Browse the repository at this point in the history
[ Upstream commit d6d5645 ]

When an I2C adapter acts only as a slave, it should not claim to
support I2C master capabilities.

Fixes: 9d3ca54 ("i2c: at91: added slave mode support")
Signed-off-by: Jean Delvare <[email protected]>
Cc: Juergen Fitschen <[email protected]>
Cc: Ludovic Desroches <[email protected]>
Cc: Codrin Ciubotariu <[email protected]>
Cc: Andi Shyti <[email protected]>
Cc: Nicolas Ferre <[email protected]>
Cc: Alexandre Belloni <[email protected]>
Cc: Claudiu Beznea <[email protected]>
Signed-off-by: Andi Shyti <[email protected]>
Signed-off-by: Sasha Levin <[email protected]>
  • Loading branch information
jdelvare authored and gregkh committed Jun 21, 2024
1 parent a4cd607 commit 572afd4
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions drivers/i2c/busses/i2c-at91-slave.c
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,7 @@ static int at91_unreg_slave(struct i2c_client *slave)

static u32 at91_twi_func(struct i2c_adapter *adapter)
{
return I2C_FUNC_SLAVE | I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL
| I2C_FUNC_SMBUS_READ_BLOCK_DATA;
return I2C_FUNC_SLAVE;
}

static const struct i2c_algorithm at91_twi_algorithm_slave = {
Expand Down

0 comments on commit 572afd4

Please sign in to comment.