From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH net-next v2 0/3] net: sfp: small improvements Date: Wed, 23 May 2018 11:40:50 -0700 Message-ID: References: <20180522101801.18947-1-antoine.tenart@bootlin.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, thomas.petazzoni@bootlin.com, maxime.chevallier@bootlin.com, gregory.clement@bootlin.com, miquel.raynal@bootlin.com, nadavh@marvell.com, stefanc@marvell.com, ymarkman@marvell.com, mw@semihalf.com To: Antoine Tenart , davem@davemloft.net, linux@armlinux.org.uk Return-path: In-Reply-To: <20180522101801.18947-1-antoine.tenart@bootlin.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org 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