Re: [PATCH v3 1/1] schemas: i2c: Introduce I2C bus extensions

[Wolfram Sang <wsa+renesas@sang-engineering.com>]

[-- Attachment #1: Type: text/plain, Size: 1286 bytes --]

Hi,

> > A different option is to only have the "i2c-parent" phandle in the
> > extension node and nothing else in DT (no bidirectional link, no
> > compatible string), without any full-tree searches.
> >
> > On the implementation side, the connector driver when probing would
> > register the extension nodes at the I2C core, which would maintain a
> > list of extension nodes. This is important when the connector probes
> > first. Then when any adapter probes the core would iterate over the
> > list to check whether the newly-probed adapter is pointed to by one of
> > the registered bus extensions, and then start populating the devices on
> > the matching bus extension(s).
> >
> > A lot of care would have to be put in the disconnection path and while
> > removing any bus extension from the global list, which could race with
> > the I2C core using the list itself. The drive core wouldn't do it for
> > us for free.
> 
> I'll defer to Wolfram on I2C core implementation...

One input already before we dive into the unconference. I don't want to
maintain the above solution, i.e. handling lists with sublte race issues
which could be (and should be IMO) handled by the driver core anyhow.

See you soon, may the A/V-setup be with us,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]