[PATCH v7 2/8] i2c: muxes: add support for tsd,mule-i2c multiplexer
Peter Rosin
peda at axentia.se
Wed Sep 4 01:59:47 PDT 2024
Hi!
2024-09-04 at 10:35, Farouk Bouabid wrote:
> Hi Andi,
>
> On 03.09.24 17:13, Andi Shyti wrote:
>
> [...]
>
>>> + ret = i2c_mux_add_adapter(muxc, 0, reg);
>>> + if (ret)
>>> + return ret;
>> do we need to delete the adapters we added in previous cycles?
>>
>
> We calldevm_action_or_reset() before the loop to add adapter-removal to the error path. I think that does the job
>
> for us or am I missing something ?
I missed that too, but it LGTM. It's safe to call i2c_mux_del_adapters() as
soon the mux core has been allocated, so there is no risk it is called too
early or something. With that said, I agree with Andi on the naming and the
nitpicks.
Cheers,
Peter
More information about the Linux-rockchip
mailing list