* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 6:52 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 6:52 UTC (permalink / raw) To: David S. Miller Cc: Russell King, Markus Pargmann, Tony Lindgren, devicetree, netdev, linux-omap, linux-arm-kernel, Nishanth Menon Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russel here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 30 +++++++++++++++++++- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..e0c92e0 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,21 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg_en; + } + } + + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1526,14 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg_en: + if (!IS_ERR(ks->vdd_reg)) + regulator_put(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1547,10 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) { + regulator_disable(priv->vdd_reg); + regulator_put(priv->vdd_reg); + } free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 6:52 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 6:52 UTC (permalink / raw) To: David S. Miller Cc: Russell King, Markus Pargmann, Tony Lindgren, devicetree, netdev, linux-omap, linux-arm-kernel, Nishanth Menon Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russel here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 30 +++++++++++++++++++- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..e0c92e0 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,21 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg_en; + } + } + + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1526,14 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg_en: + if (!IS_ERR(ks->vdd_reg)) + regulator_put(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1547,10 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) { + regulator_disable(priv->vdd_reg); + regulator_put(priv->vdd_reg); + } free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 6:52 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 6:52 UTC (permalink / raw) To: linux-arm-kernel Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russel here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 30 +++++++++++++++++++- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..e0c92e0 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,21 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg_en; + } + } + + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1526,14 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg_en: + if (!IS_ERR(ks->vdd_reg)) + regulator_put(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1547,10 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) { + regulator_disable(priv->vdd_reg); + regulator_put(priv->vdd_reg); + } free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
[parent not found: <1395384768-8461-1-git-send-email-nm-l0cyMroinI0@public.gmane.org>]
* Re: [PATCH] net: micrel : ks8851-ml: add vdd-supply support 2014-03-21 6:52 ` Nishanth Menon @ 2014-03-21 8:51 ` Markus Pargmann -1 siblings, 0 replies; 15+ messages in thread From: Markus Pargmann @ 2014-03-21 8:51 UTC (permalink / raw) To: Nishanth Menon Cc: David S. Miller, Russell King, Tony Lindgren, devicetree-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA, linux-omap-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r [-- Attachment #1: Type: text/plain, Size: 4236 bytes --] Hi, On Fri, Mar 21, 2014 at 01:52:48AM -0500, Nishanth Menon wrote: > Few platforms use external regulator to keep the ethernet MAC supplied. > So, request and enable the regulator for driver functionality. > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > Reported-by: Russell King <rmk+kernel-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org> > Suggested-by: Markus Pargmann <mpa-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> > Signed-off-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> > --- > > This fixes a regression in SDP4430 platform as reported by Russel here: > http://marc.info/?t=139509918200014&r=1&w=2 > > Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost > rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git > > .../devicetree/bindings/net/micrel-ks8851.txt | 1 + > drivers/net/ethernet/micrel/ks8851.c | 30 +++++++++++++++++++- > 2 files changed, 30 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt > index 11ace3c..4fc3927 100644 > --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt > +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt > @@ -7,3 +7,4 @@ Required properties: > > Optional properties: > - local-mac-address : Ethernet mac address to use > +- vdd-supply: supply for Ethernet mac > diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c > index 727b546a..e0c92e0 100644 > --- a/drivers/net/ethernet/micrel/ks8851.c > +++ b/drivers/net/ethernet/micrel/ks8851.c > @@ -23,6 +23,7 @@ > #include <linux/crc32.h> > #include <linux/mii.h> > #include <linux/eeprom_93cx6.h> > +#include <linux/regulator/consumer.h> > > #include <linux/spi/spi.h> > > @@ -83,6 +84,7 @@ union ks8851_tx_hdr { > * @rc_rxqcr: Cached copy of KS_RXQCR. > * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom > * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. > + * @vdd_reg: Optional regulator supplying the chip > * > * The @lock ensures that the chip is protected when certain operations are > * in progress. When the read or write packet transfer is in progress, most > @@ -130,6 +132,7 @@ struct ks8851_net { > struct spi_transfer spi_xfer2[2]; > > struct eeprom_93cx6 eeprom; > + struct regulator *vdd_reg; > }; > > static int msg_enable; > @@ -1414,6 +1417,21 @@ static int ks8851_probe(struct spi_device *spi) > ks->spidev = spi; > ks->tx_space = 6144; > > + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); You could use devm_regulator_get_optional here and remove the regulator_put. Regards, Markus > + if (IS_ERR(ks->vdd_reg)) { > + ret = PTR_ERR(ks->vdd_reg); > + if (ret == -EPROBE_DEFER) > + goto err_reg; > + } else { > + ret = regulator_enable(ks->vdd_reg); > + if (ret) { > + dev_err(&spi->dev, "regulator enable fail: %d\n", > + ret); > + goto err_reg_en; > + } > + } > + > + > mutex_init(&ks->lock); > spin_lock_init(&ks->statelock); > > @@ -1508,8 +1526,14 @@ static int ks8851_probe(struct spi_device *spi) > err_netdev: > free_irq(ndev->irq, ks); > > -err_id: > err_irq: > +err_id: > + if (!IS_ERR(ks->vdd_reg)) > + regulator_disable(ks->vdd_reg); > +err_reg_en: > + if (!IS_ERR(ks->vdd_reg)) > + regulator_put(ks->vdd_reg); > +err_reg: > free_netdev(ndev); > return ret; > } > @@ -1523,6 +1547,10 @@ static int ks8851_remove(struct spi_device *spi) > > unregister_netdev(priv->netdev); > free_irq(spi->irq, priv); > + if (!IS_ERR(priv->vdd_reg)) { > + regulator_disable(priv->vdd_reg); > + regulator_put(priv->vdd_reg); > + } > free_netdev(priv->netdev); > > return 0; > -- > 1.7.9.5 > > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 8:51 ` Markus Pargmann 0 siblings, 0 replies; 15+ messages in thread From: Markus Pargmann @ 2014-03-21 8:51 UTC (permalink / raw) To: linux-arm-kernel Hi, On Fri, Mar 21, 2014 at 01:52:48AM -0500, Nishanth Menon wrote: > Few platforms use external regulator to keep the ethernet MAC supplied. > So, request and enable the regulator for driver functionality. > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > Suggested-by: Markus Pargmann <mpa@pengutronix.de> > Signed-off-by: Nishanth Menon <nm@ti.com> > --- > > This fixes a regression in SDP4430 platform as reported by Russel here: > http://marc.info/?t=139509918200014&r=1&w=2 > > Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost > rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git > > .../devicetree/bindings/net/micrel-ks8851.txt | 1 + > drivers/net/ethernet/micrel/ks8851.c | 30 +++++++++++++++++++- > 2 files changed, 30 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt > index 11ace3c..4fc3927 100644 > --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt > +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt > @@ -7,3 +7,4 @@ Required properties: > > Optional properties: > - local-mac-address : Ethernet mac address to use > +- vdd-supply: supply for Ethernet mac > diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c > index 727b546a..e0c92e0 100644 > --- a/drivers/net/ethernet/micrel/ks8851.c > +++ b/drivers/net/ethernet/micrel/ks8851.c > @@ -23,6 +23,7 @@ > #include <linux/crc32.h> > #include <linux/mii.h> > #include <linux/eeprom_93cx6.h> > +#include <linux/regulator/consumer.h> > > #include <linux/spi/spi.h> > > @@ -83,6 +84,7 @@ union ks8851_tx_hdr { > * @rc_rxqcr: Cached copy of KS_RXQCR. > * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom > * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. > + * @vdd_reg: Optional regulator supplying the chip > * > * The @lock ensures that the chip is protected when certain operations are > * in progress. When the read or write packet transfer is in progress, most > @@ -130,6 +132,7 @@ struct ks8851_net { > struct spi_transfer spi_xfer2[2]; > > struct eeprom_93cx6 eeprom; > + struct regulator *vdd_reg; > }; > > static int msg_enable; > @@ -1414,6 +1417,21 @@ static int ks8851_probe(struct spi_device *spi) > ks->spidev = spi; > ks->tx_space = 6144; > > + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); You could use devm_regulator_get_optional here and remove the regulator_put. Regards, Markus > + if (IS_ERR(ks->vdd_reg)) { > + ret = PTR_ERR(ks->vdd_reg); > + if (ret == -EPROBE_DEFER) > + goto err_reg; > + } else { > + ret = regulator_enable(ks->vdd_reg); > + if (ret) { > + dev_err(&spi->dev, "regulator enable fail: %d\n", > + ret); > + goto err_reg_en; > + } > + } > + > + > mutex_init(&ks->lock); > spin_lock_init(&ks->statelock); > > @@ -1508,8 +1526,14 @@ static int ks8851_probe(struct spi_device *spi) > err_netdev: > free_irq(ndev->irq, ks); > > -err_id: > err_irq: > +err_id: > + if (!IS_ERR(ks->vdd_reg)) > + regulator_disable(ks->vdd_reg); > +err_reg_en: > + if (!IS_ERR(ks->vdd_reg)) > + regulator_put(ks->vdd_reg); > +err_reg: > free_netdev(ndev); > return ret; > } > @@ -1523,6 +1547,10 @@ static int ks8851_remove(struct spi_device *spi) > > unregister_netdev(priv->netdev); > free_irq(spi->irq, priv); > + if (!IS_ERR(priv->vdd_reg)) { > + regulator_disable(priv->vdd_reg); > + regulator_put(priv->vdd_reg); > + } > free_netdev(priv->netdev); > > return 0; > -- > 1.7.9.5 > > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140321/57d3cc26/attachment-0001.sig> ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH V2] net: micrel : ks8851-ml: add vdd-supply support 2014-03-21 8:51 ` Markus Pargmann @ 2014-03-21 9:28 ` Nishanth Menon -1 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:28 UTC (permalink / raw) To: David S. Miller Cc: Nishanth Menon, devicetree, Tony Lindgren, netdev, Markus Pargmann, Russell King, linux-omap, linux-arm-kernel Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russel here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git Changes in v2: based on Marcus's comment, switched to managed regulator_get, however retained the enable/disable as balanced. v1: https://patchwork.kernel.org/patch/3871401/ .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 24 +++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..44923e0 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,20 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg; + } + } + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1525,11 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1543,8 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) + regulator_disable(priv->vdd_reg); free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH V2] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 9:28 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:28 UTC (permalink / raw) To: linux-arm-kernel Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russel here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git Changes in v2: based on Marcus's comment, switched to managed regulator_get, however retained the enable/disable as balanced. v1: https://patchwork.kernel.org/patch/3871401/ .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 24 +++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..44923e0 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,20 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg; + } + } + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1525,11 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1543,8 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) + regulator_disable(priv->vdd_reg); free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH V2] net: micrel : ks8851-ml: add vdd-supply support 2014-03-21 9:28 ` Nishanth Menon @ 2014-03-21 9:30 ` Nishanth Menon -1 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:30 UTC (permalink / raw) To: David S. Miller Cc: devicetree, Tony Lindgren, netdev, Markus Pargmann, Russell King, linux-omap, linux-arm-kernel On 03/21/2014 04:28 AM, Nishanth Menon wrote: > + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); ok, that is unbelievable.. no git commit --amend :(.. Apologies on the spam.. will resent. -- Regards, Nishanth Menon ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH V2] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 9:30 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:30 UTC (permalink / raw) To: linux-arm-kernel On 03/21/2014 04:28 AM, Nishanth Menon wrote: > + ks->vdd_reg = regulator_get_optional(&spi->dev, "vdd"); ok, that is unbelievable.. no git commit --amend :(.. Apologies on the spam.. will resent. -- Regards, Nishanth Menon ^ permalink raw reply [flat|nested] 15+ messages in thread
* [RESEND PATCH V2] net: micrel : ks8851-ml: add vdd-supply support 2014-03-21 8:51 ` Markus Pargmann @ 2014-03-21 9:41 ` Nishanth Menon -1 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:41 UTC (permalink / raw) To: David S. Miller Cc: Nishanth Menon, devicetree, Tony Lindgren, netdev, Markus Pargmann, Russell King, linux-omap, linux-arm-kernel Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russell here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git Changes in v2(resend): based on Marcus's comment, switched to managed regulator get, however retained the enable/disable as balanced. v1: https://patchwork.kernel.org/patch/3871401/ .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 24 +++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..674a58a 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,20 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = devm_regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg; + } + } + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1525,11 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1543,8 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) + regulator_disable(priv->vdd_reg); free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [RESEND PATCH V2] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-21 9:41 ` Nishanth Menon 0 siblings, 0 replies; 15+ messages in thread From: Nishanth Menon @ 2014-03-21 9:41 UTC (permalink / raw) To: linux-arm-kernel Few platforms use external regulator to keep the ethernet MAC supplied. So, request and enable the regulator for driver functionality. Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> Suggested-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Nishanth Menon <nm@ti.com> --- This fixes a regression in SDP4430 platform as reported by Russell here: http://marc.info/?t=139509918200014&r=1&w=2 Patch is based on [v3.14-rc7] tag, if it is too late to submit, I can repost rebased to git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git Changes in v2(resend): based on Marcus's comment, switched to managed regulator get, however retained the enable/disable as balanced. v1: https://patchwork.kernel.org/patch/3871401/ .../devicetree/bindings/net/micrel-ks8851.txt | 1 + drivers/net/ethernet/micrel/ks8851.c | 24 +++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/micrel-ks8851.txt b/Documentation/devicetree/bindings/net/micrel-ks8851.txt index 11ace3c..4fc3927 100644 --- a/Documentation/devicetree/bindings/net/micrel-ks8851.txt +++ b/Documentation/devicetree/bindings/net/micrel-ks8851.txt @@ -7,3 +7,4 @@ Required properties: Optional properties: - local-mac-address : Ethernet mac address to use +- vdd-supply: supply for Ethernet mac diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 727b546a..674a58a 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c @@ -23,6 +23,7 @@ #include <linux/crc32.h> #include <linux/mii.h> #include <linux/eeprom_93cx6.h> +#include <linux/regulator/consumer.h> #include <linux/spi/spi.h> @@ -83,6 +84,7 @@ union ks8851_tx_hdr { * @rc_rxqcr: Cached copy of KS_RXQCR. * @eeprom_size: Companion eeprom size in Bytes, 0 if no eeprom * @eeprom: 93CX6 EEPROM state for accessing on-board EEPROM. + * @vdd_reg: Optional regulator supplying the chip * * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most @@ -130,6 +132,7 @@ struct ks8851_net { struct spi_transfer spi_xfer2[2]; struct eeprom_93cx6 eeprom; + struct regulator *vdd_reg; }; static int msg_enable; @@ -1414,6 +1417,20 @@ static int ks8851_probe(struct spi_device *spi) ks->spidev = spi; ks->tx_space = 6144; + ks->vdd_reg = devm_regulator_get_optional(&spi->dev, "vdd"); + if (IS_ERR(ks->vdd_reg)) { + ret = PTR_ERR(ks->vdd_reg); + if (ret == -EPROBE_DEFER) + goto err_reg; + } else { + ret = regulator_enable(ks->vdd_reg); + if (ret) { + dev_err(&spi->dev, "regulator enable fail: %d\n", + ret); + goto err_reg; + } + } + mutex_init(&ks->lock); spin_lock_init(&ks->statelock); @@ -1508,8 +1525,11 @@ static int ks8851_probe(struct spi_device *spi) err_netdev: free_irq(ndev->irq, ks); -err_id: err_irq: +err_id: + if (!IS_ERR(ks->vdd_reg)) + regulator_disable(ks->vdd_reg); +err_reg: free_netdev(ndev); return ret; } @@ -1523,6 +1543,8 @@ static int ks8851_remove(struct spi_device *spi) unregister_netdev(priv->netdev); free_irq(spi->irq, priv); + if (!IS_ERR(priv->vdd_reg)) + regulator_disable(priv->vdd_reg); free_netdev(priv->netdev); return 0; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] net: micrel : ks8851-ml: add vdd-supply support 2014-03-21 6:52 ` Nishanth Menon @ 2014-03-24 4:37 ` David Miller -1 siblings, 0 replies; 15+ messages in thread From: David Miller @ 2014-03-24 4:37 UTC (permalink / raw) To: nm; +Cc: rmk+kernel, mpa, tony, devicetree, netdev, linux-omap, linux-arm-kernel From: Nishanth Menon <nm@ti.com> Date: Fri, 21 Mar 2014 01:52:48 -0500 > Few platforms use external regulator to keep the ethernet MAC supplied. > So, request and enable the regulator for driver functionality. > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > Suggested-by: Markus Pargmann <mpa@pengutronix.de> > Signed-off-by: Nishanth Menon <nm@ti.com> Applied, thanks. ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-24 4:37 ` David Miller 0 siblings, 0 replies; 15+ messages in thread From: David Miller @ 2014-03-24 4:37 UTC (permalink / raw) To: linux-arm-kernel From: Nishanth Menon <nm@ti.com> Date: Fri, 21 Mar 2014 01:52:48 -0500 > Few platforms use external regulator to keep the ethernet MAC supplied. > So, request and enable the regulator for driver functionality. > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > Suggested-by: Markus Pargmann <mpa@pengutronix.de> > Signed-off-by: Nishanth Menon <nm@ti.com> Applied, thanks. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] net: micrel : ks8851-ml: add vdd-supply support 2014-03-24 4:37 ` David Miller @ 2014-03-25 10:06 ` Markus Pargmann -1 siblings, 0 replies; 15+ messages in thread From: Markus Pargmann @ 2014-03-25 10:06 UTC (permalink / raw) To: David Miller Cc: nm, rmk+kernel, tony, devicetree, netdev, linux-omap, linux-arm-kernel [-- Attachment #1: Type: text/plain, Size: 1052 bytes --] Hi, On Mon, Mar 24, 2014 at 12:37:58AM -0400, David Miller wrote: > From: Nishanth Menon <nm@ti.com> > Date: Fri, 21 Mar 2014 01:52:48 -0500 > > > Few platforms use external regulator to keep the ethernet MAC supplied. > > So, request and enable the regulator for driver functionality. > > > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > > Suggested-by: Markus Pargmann <mpa@pengutronix.de> > > Signed-off-by: Nishanth Menon <nm@ti.com> > > Applied, thanks. > The two regulator patches for enable and disable are going into the stable trees so this should also be submitted to stable for 3.10 to 3.13. Regards, Markus -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH] net: micrel : ks8851-ml: add vdd-supply support @ 2014-03-25 10:06 ` Markus Pargmann 0 siblings, 0 replies; 15+ messages in thread From: Markus Pargmann @ 2014-03-25 10:06 UTC (permalink / raw) To: linux-arm-kernel Hi, On Mon, Mar 24, 2014 at 12:37:58AM -0400, David Miller wrote: > From: Nishanth Menon <nm@ti.com> > Date: Fri, 21 Mar 2014 01:52:48 -0500 > > > Few platforms use external regulator to keep the ethernet MAC supplied. > > So, request and enable the regulator for driver functionality. > > > > Fixes: 66fda75f47dc (regulator: core: Replace direct ops->disable usage) > > Reported-by: Russell King <rmk+kernel@arm.linux.org.uk> > > Suggested-by: Markus Pargmann <mpa@pengutronix.de> > > Signed-off-by: Nishanth Menon <nm@ti.com> > > Applied, thanks. > The two regulator patches for enable and disable are going into the stable trees so this should also be submitted to stable for 3.10 to 3.13. Regards, Markus -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140325/9409b7fe/attachment.sig> ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2014-03-25 10:06 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-21 6:52 [PATCH] net: micrel : ks8851-ml: add vdd-supply support Nishanth Menon
2014-03-21 6:52 ` Nishanth Menon
2014-03-21 6:52 ` Nishanth Menon
[not found] ` <1395384768-8461-1-git-send-email-nm-l0cyMroinI0@public.gmane.org>
2014-03-21 8:51 ` Markus Pargmann
2014-03-21 8:51 ` Markus Pargmann
2014-03-21 9:28 ` [PATCH V2] " Nishanth Menon
2014-03-21 9:28 ` Nishanth Menon
2014-03-21 9:30 ` Nishanth Menon
2014-03-21 9:30 ` Nishanth Menon
2014-03-21 9:41 ` [RESEND PATCH " Nishanth Menon
2014-03-21 9:41 ` Nishanth Menon
2014-03-24 4:37 ` [PATCH] " David Miller
2014-03-24 4:37 ` David Miller
2014-03-25 10:06 ` Markus Pargmann
2014-03-25 10:06 ` Markus Pargmann
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.