On 05/22/2018 03:17 AM, Antoine Tenart wrote: > Hi Russell, David, > > A small series of patches improving the SFP support by adding a warning > when no Tx disable pin is available, and making the i2c-bus property > mandatory. > > Thanks! > Antoine
Antoine, can you please do CC the people who worked on that code before, arguably, send an update to MAINTAINERS file to create a specific section for PHYLINK. Thank you > > Since v1: > - Removed the patch fixing the sfp driver when no i2c bus was described. > - Made two new patches to make the i2c-bus property mandatory for sfp > modules. > > Since the phylink series: > - s/-EOPNOTSUPP/-ENODEV/ in patch 1/2. > - I added the acked-by tag in patch 2/2. > > Antoine Tenart (3): > net: phy: sfp: warn the user when no tx_disable pin is available > net: phy: sfp: make the i2c-bus dt property mandatory > Documentation/bindings: net: the sfp i2c-bus property is now mandatory > > .../devicetree/bindings/net/sff,sfp.txt | 4 +- > drivers/net/phy/sfp.c | 37 ++++++++++++------- > 2 files changed, 26 insertions(+), 15 deletions(-) > -- Florian