Hi, thanks for the patches! I have a high level question about them, first. > +- bus-supply > + phandle to the regulator that provides power to i2c. I think this is not clear enough. I assume it means the regulator for driving SCL/SDA? In the coverletter you mention also an EEPROM. In your case, this is driven by the same regulator? I am skeptical we can abstract it like this because I2C devices could be driven by various regulators in the system, so there couldn't be one "bus regulator". The regulator for the EEPROM should be described in the EEPROM node. So, this "bus supply" is only for driving SCL/SDA? Kind regards, Wolfram
Attachment:
signature.asc
Description: PGP signature