Re: [PATCH v2 0/4] i2c: Support i2c_register_spd() on multiplexed bus segments

Wolfram Sang
2024-02-21T17:19:55Z

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

Hi Heiner,

> Note: i801 creates the mux platform device, loading and probing of the
> mux driver may be asynchronous. Therefore we can't call i2c_register_spd()
> for the muxed segments from i801. Instead we have to add a flag to the
> platform data, so that the mux driver knows it's supposed to call
> i2c_register_spd().

Has it been considered to use a bus_notifier and check for
BUS_NOTIFY_BOUND_DRIVER?

I'd really like to keep it inside i801 if possible. First, all these
flags in mux drivers only for this corner case are relatively intrusive.
Second, selecting SMBUS for I2C_MUX is also a tad too much for my taste.

I understand that removing CLASS_SPD is a worthy goal. So, if all fails
we could still try this. But I'd think with bus_notifiers it should be
possible to keep it all in i801.

Do you think this could work?

Happy hacking,

Wolfram

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