On Wed, Aug 23, 2017 at 03:16:53PM +0300, Baruch Siach wrote: > Hi Rob, > > On Mon, Aug 21, 2017 at 02:10:33PM -0500, Rob Herring wrote: > > On Sun, Aug 20, 2017 at 5:28 AM, Baruch Siach <baruch@xxxxxxxxxx> wrote: > > > Add device-tree binding documentation SFP transceivers. Support for SFP > > > transceivers has been recently introduced (drivers/net/phy/sfp.c). > > > > > > Signed-off-by: Baruch Siach <baruch@xxxxxxxxxx> > > > --- > > > > > > The SFP driver is on net-next. > > [...] > > > > +Optional Properties: > > > + > > > +- i2c-bus : phandle of an I2C bus controller for the SFP two wire serial > > > + interface > > > > Why not a child of the i2c bus it is on? IOW, what should this be a child of? > > As I understand form the code the ID of the SFP i2c slave is derived from the > Ethernet PHY 'reg' property. The PHY node's 'sfp' property points to a phandle > of the sff,sfp node. Hi Rob The SFP module uses multiple addresses on the i2c bus. 0x50 is something like an EEPROM, but some of the content is dynamic. Depending of the version of the standard, it can also use 0x51 for additional information. If the SPF module contains a copper PHY, it also uses another address on the i2c bus for the standard copper PHY registers. An SFP module does not fit the usual I2C client model, since it is a collection of interconnected i2c clients in one package. Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
