u-boot.git/drivers/i2c, branch v2022.10

i2c: stm32: fix usage of rise/fall device tree properties

2022-09-15T12:59:29Z

These two device tree properties were not being applied. Fixes: 1fd9eb68d6 ("i2c: stm32f7: move driver data of each instance in a privdata") Signed-off-by: Jorge Ramirez-Ortiz Reviewed-by: Alain Volmat Reviewed-by: Patrice Chotard Reviewed-by: Heiko Schocher Reviewed-by: Patrick Delaunay Tested-by: Patrick Delaunay

i2c: stm32: do not set the STOP condition on error

2022-09-15T12:59:22Z

Current function stm32_i2c_message_xfer is sending a STOP whatever the result of the transaction is. This can cause issues such as making the bus busy since the controller itself is already sending automatically a STOP when a NACK is generated. Thanks to Jorge Ramirez-Ortiz for diagnosing and proposing a first fix for this. [1] [1] https://lore.kernel.org/u-boot/20220815145211.31342-2-jorge@foundries.io/ Reported-by: Jorge Ramirez-Ortiz, Foundries Signed-off-by: Jorge Ramirez-Ortiz Signed-off-by: Alain Volmat Reviewed-by: Patrice Chotard Reviewed-by: Heiko Schocher Reviewed-by: Patrick Delaunay Tested-by: Patrick Delaunay

i2c: stm32: remove unused stop parameter in start & reload handling

2022-09-15T12:58:57Z

Functions stm32_i2c_message_start and stm32_i2c_handle_reload both get a stop boolean indicating if the transfer should end with a STOP or not. However no specific handling is needed in those functions hence remove the parameter. Signed-off-by: Alain Volmat Reviewed-by: Patrick Delaunay Reviewed-by: Heiko Schocher Reviewed-by: Patrice Chotard

i2c: stm32: fix comment and remove unused AUTOEND bit

2022-09-15T12:58:52Z

Comment within stm32_i2c_message_start is misleading, indicating that AUTOEND bit is setted while it is actually cleared. Moreover, the bit is actually never setted so there is no need to clear it hence get rid of this bit clear and the bit macro as well. Signed-off-by: Alain Volmat Reviewed-by: Patrick Delaunay Reviewed-by: Heiko Schocher Reviewed-by: Patrice Chotard

i2c: stm32f7: fix clearing the control register

2022-09-15T12:58:49Z

Bits should be set to 0, not 1. Signed-off-by: Jorge Ramirez-Ortiz Reviewed-by: Patrice Chotard Reviewed-by: Patrick Delaunay

i2c: avoid dynamic stack use in dm_i2c_write

2022-07-19T11:46:28Z

The size of the dynamic stack allocation here is bounded by the if() statement. However, just allocating the maximum size up-front and doing malloc() if necessary avoids code duplication (the i2c_setup_offset() until the invocation of ->xfer), and generates much better (smaller) code: bloat-o-meter drivers/i2c/i2c-uclass.o.{0,1} add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-144 (-144) Function old new delta dm_i2c_write 552 408 -144 Total: Before=3828, After=3684, chg -3.76% It also makes static analysis of maximum stack usage (using the .su files that are automatically generated during build) easier if there are no lines saying "dynamic". [This is not entirely equivalent to the existing code; this now uses the stack for len <= 64 rather than len <= 63, but that seems like a more natural limit.] Signed-off-by: Rasmus Villemoes Reviewed-by: Heiko Schocher

i2c: nuvoton: Add NPCM7xx i2c driver

2022-07-19T11:46:28Z

Add Nuvoton BMC NPCM750 i2c driver Signed-off-by: Jim Liu Reviewed-by: Heiko Schocher

i2c: ast_i2c: Remove SCL direct drive mode

2022-07-19T11:46:28Z

SCL direct drive mode prevents communication with devices that do clock stretching, so disable. The Linux driver doesn't use this mode, and the engine can handle clock stretching. Signed-off-by: Eddie James Reviewed-by: Heiko Schocher Reviewed-by: Joel Stanley Reviewed-by: ryan_chen

i2c: stm32: add support for the st,stm32mp13 SOC

2022-07-12T09:47:34Z

The stm32mp13 soc differs from the stm32mp15 in terms of clear register offset for controlling the FMP (Fast Mode Plus). Signed-off-by: Patrick Delaunay Reviewed-by: Patrice Chotard

i2c: Remove non-DM_I2C support from davinci_i2c.c

2022-07-08T21:57:34Z

As the migration deadline has passed, and all platforms have been migrated, remove the non-DM code here. Signed-off-by: Tom Rini