Re: [PATCH v3 4/6] i2c: npcm: Modify the client address assignment
From: Andy Shevchenko
Date: Fri Sep 13 2024 - 06:36:57 EST
On Fri, Sep 13, 2024 at 06:14:44PM +0800, warp5tw@xxxxxxxxx wrote:
> From: Tyrone Ting <kfting@xxxxxxxxxxx>
>
> Store the client address earlier since it might get called in
> the i2c_recover_bus logic flow at the early stage of the function
> npcm_i2c_master_xfer.
We refer to the functions as func().
...
> + /*
> + * Previously, the address was stored w/o left-shift by one bit and
> + * with that shift in the following call to npcm_i2c_master_start_xmit.
> + *
> + * Since there are cases that the i2c_recover_bus gets called at the
> + * early stage of the function npcm_i2c_master_xfer, the address is
Ditto.
> + * stored with the shift and used in the i2c_recover_bus call.
> + *
> + * The address is stored from bit 1 to bit 7 in the register for
> + * sending the i2c address later so it's left-shifted by 1 bit.
> + */
--
With Best Regards,
Andy Shevchenko