Re: [PATCH v3 2/4] leds: flash: Add support for the TI LM3643 dual LED flash driver

[Roman Vivchar <rva333@protonmail.com>]

Hi Rillian,

On 10/11/26 22:09, Rillian Grant wrote:
> The LM3643 is an I2C dual LED flash driver. Its two high-side current
> sources share a synchronous boost converter and each supports torch and
> flash modes with programmable current and flash timeout alongside fault
> reporting.
> 
> It also supports connecting both sources together to supply one LED with
> current up to the boost converter's 1.5A maximum. In that case the current
> will be split between each source symmetrically and the sources will be
> enabled and disabled at the same time.
> 
> As strobing is toggled chip-wide the driver returns -EBUSY when a sibling
> LED is active.
> 
> Flash timeouts do not have a constant step. The driver advertises the most
> granular step and rounds down to the nearest supported value.
> 
> Register behavior was taken from the datasheet (SNVS967A).
> 
> Co-developed-by: Ruslan Koreev <koreev.r@gmail.com>
> Signed-off-by: Ruslan Koreev <koreev.r@gmail.com>
> Signed-off-by: Rillian Grant <rillian.grant@gmail.com>

I have tested this series on a Realme C3 phone, which uses Orient Chip 
OCP81373 IC (register-compatible clone of the LM3643, except the device 
ID is 0x3A instead of 0x00). I had to modify the driver to support 
multiple chips, since there's at least one more clone IC (Awinic 
AW3643). After applying modifications, flash and torch modes were 
confirmed to work correctly. Once this series is merged, I will send a 
small series to add support for the OCP81373.

Tested-by: Roman Vivchar <rva333@protonmail.com>

Best regards,
Roman