u-boot.git/drivers/i2c, branch v2023.07

I2C: cdns: Fix broken retry mechanism on arbitration lost.

2023-06-05T04:54:25+00:00

In the current implementation, in case of I2C arbitration lost, a retry is
attempted; the message counter and pointer are reset to the original values
and the I2C xfer process is restart from the beginning.

However the message counter and message pointer are respectively
decremented and incremented by one before attempting any transfer, causing
the 1st transfer not to be actually retried (in case of a single transfer,
nothing is actually retried at all).

This patch fixes this: in case of retry, the 1st transfer is also retried.

Tested on a ZynqMP Kria board, with upstream older u-boot, but the involved
file and underlying logic seem basically the same.

Signed-off-by: Andrea Merello

i2c: rockchip: De-initialize the bus after start bit failure

2023-06-05T04:52:46+00:00

Failure can happen when i2c is used without initializing pinctrl properly,
which U-Boot happily allows in SPL. Without this fix, further I2C access would
fail, even after proper pinctrl initialization.

Signed-off-by: Ondrej Jirman 
Cc: Heiko Schocher 
Reviewed-by: Kever Yang 
Reviewed-by: Heiko Schocher

drivers: use dev_read_addr_ptr when cast to pointer

2023-05-06T09:28:18+00:00

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU
can expect 64-bit data from the device tree parser, so use
dev_read_addr_ptr instead of the dev_read_addr function in the
various files in the drivers directory that cast to a pointer.
As we are there also streamline the error response to -EINVAL on return.

Signed-off-by: Johan Jonker 
Reviewed-by: Simon Glass

i2c: fsl_i2c: fix m68k transferts

2023-04-11T04:51:56+00:00

This driver is actually used for powerpc and m68k/ColdFire.

On ColdFire SoC's, interrupt flag get not set if IIEN flag (mbcr bit6,
interrupt enabled) is not set appropriately before each transfert.
As a result, the transfert hangs forever waiting for IIEN.
This patch set IIEN before each transfert, while considering this fix
as not harming powerpc arch.

Signed-off-by: Angelo Dureghello

i2c: imx_lpi2c: Fix misuse the IS_ENABLED for DM clock

2023-04-11T04:45:29+00:00

The IS_ENABLED, which does not consider SPL build, should be replaced
by CONFIG_IS_ENABLED.
For the case that we only enable DM CLK for u-boot but not in SPL, the
IS_ENABLED(CONFIG_CLK) still returns true, then cause clock failure.

Reviewed-by: Peng Fan 
Signed-off-by: Ye Li 
Signed-off-by: Peng Fan

i2c: correct I2C deblock logic

2023-04-11T04:44:39+00:00

Current code use dm_gpio_get_value() to get SDA and SCL value, and the
value depends on whether DTS file config the GPIO_ACTIVE_LOW. In ususal
case for i2c GPIO, DTS need to set GPIO_ACTIVE_LOW for SCL/SDA pins. So
here the logic is not correct.

And we must not use GPIOD_ACTIVE_LOW in client code include the
dm_gpio_set_dir_flags(), it is DTS's responsibility for this flag. So
remove GPIOD_ACTIVE_LOW here.

Fixes: aa54192d4a87 ("dm: i2c: implement gpio-based I2C deblock")
Signed-off-by: Haibo Chen 
Reviewed-by: Alexander Kochetkov >

i2c: designware_i2c: remove apparently redundant read of 'i2c, speeds' DT property

2023-04-11T04:43:33+00:00

This code first figures out if there is an i2c,speeds property, if so
its size in u32s, and then reads the value into the local speeds[]
array. Both 'size' and 'speeds' are completely unused thereafter.

It's not at all clear what this is supposed to do. Of course, it could
be seen as a sanity check that the DT node does have an i2c,speeds
property with an appropriate number of elements, but for that one
wouldn't actually need to read it into speeds[]. Also, I can't find
anywhere else in the U-Boot code which makes use of values from that
property (this is is the only C code referencing "i2c,speeds"), so it
seems pointless to insist that it's there.

Signed-off-by: Rasmus Villemoes

i2c: rcar_i2c: Add R-Car Gen4 support

2023-04-07T15:13:27+00:00

Add support for R-Car Gen4 SoCs into the driver.

While I2C on R-Car Gen4 does support some extra features (Slave Clock
Stretch Select), for now it is treated the same as I2C on R-Car Gen3,
which let us share the same driver.

Reviewed-by: Marek Vasut 
Signed-off-by: Hai Pham 
Signed-off-by: Marek Vasut  # Use RCAR_64 Kconfig
Reviewed-by: Heiko Schocher

i2c: rcar_i2c: Sort Kconfig depends list ascending

2023-03-18T11:02:38+00:00

Sort the list of "depends" symbols in ascending order.
No functional change.

Signed-off-by: Marek Vasut 
Reviewed-by: Heiko Schocher

i2c: rcar_iic: Sort Kconfig depends list ascending

2023-03-18T11:02:38+00:00

Sort the list of "depends" symbols in ascending order.
No functional change.

Signed-off-by: Marek Vasut 
Reviewed-by: Heiko Schocher