On Fri, May 31, 2024 at 11:17:48AM +0200, Jean Delvare wrote:
> When an I2C adapter acts only as a slave, it should not claim to
> support I2C master capabilities.
>
> Fixes: 5b6d721b266a ("i2c: designware: enable SLAVE in platform module")
> Signed-off-by: Jean Delvare <jdelvare@xxxxxxx>
The below can be separated by
---
to avoid noise in the commit messages. The effect, namely Cc'ing to people,
will stay the same.
> Cc: Luis Oliveira <lolivei@xxxxxxxxxxxx>
> Cc: Jarkko Nikula <jarkko.nikula@xxxxxxxxxxxxxxx>
> Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Cc: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> Cc: Jan Dabros <jsd@xxxxxxxxxxxx>
> Cc: Andi Shyti <andi.shyti@xxxxxxxxxx>
> ---
Other than that, LGTM,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> drivers/i2c/busses/i2c-designware-slave.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-6.9.orig/drivers/i2c/busses/i2c-designware-slave.c
> +++ linux-6.9/drivers/i2c/busses/i2c-designware-slave.c
> @@ -220,7 +220,7 @@ static const struct i2c_algorithm i2c_dw
>
> void i2c_dw_configure_slave(struct dw_i2c_dev *dev)
> {
> - dev->functionality = I2C_FUNC_SLAVE | DW_IC_DEFAULT_FUNCTIONALITY;
> + dev->functionality = I2C_FUNC_SLAVE;
>
> dev->slave_cfg = DW_IC_CON_RX_FIFO_FULL_HLD_CTRL |
> DW_IC_CON_RESTART_EN | DW_IC_CON_STOP_DET_IFADDRESSED;
--
With Best Regards,
Andy Shevchenko
