Re: [PATCH v8 2/3] i2c: mux: Set adapter physical device

[Herve Codina <herve.codina@bootlin.com>]

Hi Peter,

On Sat, 4 Jul 2026 11:28:21 +0200
Peter Rosin <peda@lysator.liu.se> wrote:

> On 2026-06-30 12:30, Herve Codina wrote:
> > For i2c muxes, the parent of the mux adapter device is the adapter
> > device the mux is connected to.
> > 
> > This parent is not the physical device related to the mux adapter.
> > Indeed, the physical device of the mux adapter is the mux device itself.
> > 
> > Fill the adap.physdev with the mux device.
> > 
> > Signed-off-by: Herve Codina <herve.codina@bootlin.com>
> > Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
> > ---
> >   drivers/i2c/i2c-mux.c | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/i2c/i2c-mux.c b/drivers/i2c/i2c-mux.c
> > index 681a201c239b..f4e73e7a0e33 100644
> > --- a/drivers/i2c/i2c-mux
> > +++ b/drivers/i2c/i2c-mux.c
> > @@ -315,6 +315,7 @@ int i2c_mux_add_adapter(struct i2c_mux_core *muxc,
> >   	priv->adap.algo = &priv->algo;
> >   	priv->adap.algo_data = priv;
> >   	priv->adap.dev.parent = &parent->dev;
> > +	priv->adap.physdev = muxc->dev;
> >   	priv->adap.retries = parent->retries;
> >   	priv->adap.timeout = parent->timeout;
> >   	priv->adap.quirks = parent->quirks;  
> 
> Given how similar the implementation details are for address translators
> and muxes, I expect that this pattern is present there as well.
> 
> I.e. I think a line with "chan->adap.physdev = dev;" should be added to
> the i2c_atr_add_adapter() function (in the obvious place).
> 

Yes, I can add the physdev setting for i2c address translators.

I think also that you will expect the same kind of modification done for
muxes by patch 3 (create missing devlink) to be done for atr.

On my side, I don't have any address translators devices on my boards
I won't be able to test modifications for atr.

Do you think I should add both new patches related to atr (setting physdev
and creating missing devlink) in this series (again, I cannot test atr
modifications on my side) ?

Best regards,
Hervé