* [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup()
@ 2014-10-03 19:25 Petri Gynther
2014-10-04 4:18 ` Florian Fainelli
2014-10-06 4:18 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Petri Gynther @ 2014-10-03 19:25 UTC (permalink / raw)
To: netdev; +Cc: davem, f.fainelli
bcmgenet_mii_setup() is called from the PHY state machine every 1-2 seconds
when the PHYs are in PHY_POLL mode.
Improve bcmgenet_mii_setup() so that it touches the MAC registers only when
the link is up and there was a change to link, speed, duplex, or pause status.
Signed-off-by: Petri Gynther <pgynther@google.com>
---
drivers/net/ethernet/broadcom/genet/bcmgenet.c | 3 +-
drivers/net/ethernet/broadcom/genet/bcmgenet.h | 3 +-
drivers/net/ethernet/broadcom/genet/bcmmii.c | 73 ++++++++++++++++----------
3 files changed, 48 insertions(+), 31 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
index d51729c..e0a6238 100644
--- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c
+++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
@@ -2162,9 +2162,10 @@ static void bcmgenet_netif_stop(struct net_device *dev)
*/
cancel_work_sync(&priv->bcmgenet_irq_work);
- priv->old_pause = -1;
priv->old_link = -1;
+ priv->old_speed = -1;
priv->old_duplex = -1;
+ priv->old_pause = -1;
}
static int bcmgenet_close(struct net_device *dev)
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.h b/drivers/net/ethernet/broadcom/genet/bcmgenet.h
index ad95fe5..321b1db 100644
--- a/drivers/net/ethernet/broadcom/genet/bcmgenet.h
+++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.h
@@ -548,8 +548,9 @@ struct bcmgenet_priv {
u16 gphy_rev;
/* PHY device variables */
- int old_duplex;
int old_link;
+ int old_speed;
+ int old_duplex;
int old_pause;
phy_interface_t phy_interface;
int phy_addr;
diff --git a/drivers/net/ethernet/broadcom/genet/bcmmii.c b/drivers/net/ethernet/broadcom/genet/bcmmii.c
index 75b26cba..9ff799a 100644
--- a/drivers/net/ethernet/broadcom/genet/bcmmii.c
+++ b/drivers/net/ethernet/broadcom/genet/bcmmii.c
@@ -82,24 +82,33 @@ static void bcmgenet_mii_setup(struct net_device *dev)
struct bcmgenet_priv *priv = netdev_priv(dev);
struct phy_device *phydev = priv->phydev;
u32 reg, cmd_bits = 0;
- unsigned int status_changed = 0;
+ bool status_changed = false;
if (priv->old_link != phydev->link) {
- status_changed = 1;
+ status_changed = true;
priv->old_link = phydev->link;
}
if (phydev->link) {
- /* program UMAC and RGMII block based on established link
- * speed, pause, and duplex.
- * the speed set in umac->cmd tell RGMII block which clock
- * 25MHz(100Mbps)/125MHz(1Gbps) to use for transmit.
- * receive clock is provided by PHY.
- */
- reg = bcmgenet_ext_readl(priv, EXT_RGMII_OOB_CTRL);
- reg &= ~OOB_DISABLE;
- reg |= RGMII_LINK;
- bcmgenet_ext_writel(priv, reg, EXT_RGMII_OOB_CTRL);
+ /* check speed/duplex/pause changes */
+ if (priv->old_speed != phydev->speed) {
+ status_changed = true;
+ priv->old_speed = phydev->speed;
+ }
+
+ if (priv->old_duplex != phydev->duplex) {
+ status_changed = true;
+ priv->old_duplex = phydev->duplex;
+ }
+
+ if (priv->old_pause != phydev->pause) {
+ status_changed = true;
+ priv->old_pause = phydev->pause;
+ }
+
+ /* done if nothing has changed */
+ if (!status_changed)
+ return;
/* speed */
if (phydev->speed == SPEED_1000)
@@ -110,36 +119,39 @@ static void bcmgenet_mii_setup(struct net_device *dev)
cmd_bits = UMAC_SPEED_10;
cmd_bits <<= CMD_SPEED_SHIFT;
- if (priv->old_duplex != phydev->duplex) {
- status_changed = 1;
- priv->old_duplex = phydev->duplex;
- }
-
/* duplex */
if (phydev->duplex != DUPLEX_FULL)
cmd_bits |= CMD_HD_EN;
- if (priv->old_pause != phydev->pause) {
- status_changed = 1;
- priv->old_pause = phydev->pause;
- }
-
/* pause capability */
if (!phydev->pause)
cmd_bits |= CMD_RX_PAUSE_IGNORE | CMD_TX_PAUSE_IGNORE;
- }
- if (!status_changed)
- return;
+ /*
+ * Program UMAC and RGMII block based on established
+ * link speed, duplex, and pause. The speed set in
+ * umac->cmd tell RGMII block which clock to use for
+ * transmit -- 25MHz(100Mbps) or 125MHz(1Gbps).
+ * Receive clock is provided by the PHY.
+ */
+ reg = bcmgenet_ext_readl(priv, EXT_RGMII_OOB_CTRL);
+ reg &= ~OOB_DISABLE;
+ reg |= RGMII_LINK;
+ bcmgenet_ext_writel(priv, reg, EXT_RGMII_OOB_CTRL);
- if (phydev->link) {
reg = bcmgenet_umac_readl(priv, UMAC_CMD);
reg &= ~((CMD_SPEED_MASK << CMD_SPEED_SHIFT) |
CMD_HD_EN |
CMD_RX_PAUSE_IGNORE | CMD_TX_PAUSE_IGNORE);
reg |= cmd_bits;
bcmgenet_umac_writel(priv, reg, UMAC_CMD);
+ } else {
+ /* done if nothing has changed */
+ if (!status_changed)
+ return;
+ /* needed for MoCA fixed PHY to reflect correct link status */
+ netif_carrier_off(dev);
}
phy_print_status(phydev);
@@ -318,6 +330,12 @@ static int bcmgenet_mii_probe(struct net_device *dev)
/* Communicate the integrated PHY revision */
phy_flags = priv->gphy_rev;
+ /* Initialize link state variables that bcmgenet_mii_setup() uses */
+ priv->old_link = -1;
+ priv->old_speed = -1;
+ priv->old_duplex = -1;
+ priv->old_pause = -1;
+
phydev = of_phy_connect(dev, priv->phy_dn, bcmgenet_mii_setup,
phy_flags, priv->phy_interface);
if (!phydev) {
@@ -325,9 +343,6 @@ static int bcmgenet_mii_probe(struct net_device *dev)
return -ENODEV;
}
- priv->old_link = -1;
- priv->old_duplex = -1;
- priv->old_pause = -1;
priv->phydev = phydev;
/* Configure port multiplexer based on what the probed PHY device since
--
2.1.0.rc2.206.gedb03e5
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup()
2014-10-03 19:25 [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup() Petri Gynther
@ 2014-10-04 4:18 ` Florian Fainelli
2014-10-06 4:18 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2014-10-04 4:18 UTC (permalink / raw)
To: Petri Gynther; +Cc: netdev, David Miller
2014-10-03 12:25 GMT-07:00 Petri Gynther <pgynther@google.com>:
> bcmgenet_mii_setup() is called from the PHY state machine every 1-2 seconds
> when the PHYs are in PHY_POLL mode.
At some point, I would to make sure we can avoid polling the PHY
completely and just rely on link interrupts, and use that scheme for
the MoCA PHYs too.
>
> Improve bcmgenet_mii_setup() so that it touches the MAC registers only when
> the link is up and there was a change to link, speed, duplex, or pause status.
>
> Signed-off-by: Petri Gynther <pgynther@google.com>
Tested-by: Florian Fainelli <f.fainelli@gmail.com>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
> drivers/net/ethernet/broadcom/genet/bcmgenet.c | 3 +-
> drivers/net/ethernet/broadcom/genet/bcmgenet.h | 3 +-
> drivers/net/ethernet/broadcom/genet/bcmmii.c | 73 ++++++++++++++++----------
> 3 files changed, 48 insertions(+), 31 deletions(-)
>
> diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
> index d51729c..e0a6238 100644
> --- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c
> +++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
> @@ -2162,9 +2162,10 @@ static void bcmgenet_netif_stop(struct net_device *dev)
> */
> cancel_work_sync(&priv->bcmgenet_irq_work);
>
> - priv->old_pause = -1;
> priv->old_link = -1;
> + priv->old_speed = -1;
> priv->old_duplex = -1;
> + priv->old_pause = -1;
> }
>
> static int bcmgenet_close(struct net_device *dev)
> diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.h b/drivers/net/ethernet/broadcom/genet/bcmgenet.h
> index ad95fe5..321b1db 100644
> --- a/drivers/net/ethernet/broadcom/genet/bcmgenet.h
> +++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.h
> @@ -548,8 +548,9 @@ struct bcmgenet_priv {
> u16 gphy_rev;
>
> /* PHY device variables */
> - int old_duplex;
> int old_link;
> + int old_speed;
> + int old_duplex;
> int old_pause;
> phy_interface_t phy_interface;
> int phy_addr;
> diff --git a/drivers/net/ethernet/broadcom/genet/bcmmii.c b/drivers/net/ethernet/broadcom/genet/bcmmii.c
> index 75b26cba..9ff799a 100644
> --- a/drivers/net/ethernet/broadcom/genet/bcmmii.c
> +++ b/drivers/net/ethernet/broadcom/genet/bcmmii.c
> @@ -82,24 +82,33 @@ static void bcmgenet_mii_setup(struct net_device *dev)
> struct bcmgenet_priv *priv = netdev_priv(dev);
> struct phy_device *phydev = priv->phydev;
> u32 reg, cmd_bits = 0;
> - unsigned int status_changed = 0;
> + bool status_changed = false;
>
> if (priv->old_link != phydev->link) {
> - status_changed = 1;
> + status_changed = true;
> priv->old_link = phydev->link;
> }
>
> if (phydev->link) {
> - /* program UMAC and RGMII block based on established link
> - * speed, pause, and duplex.
> - * the speed set in umac->cmd tell RGMII block which clock
> - * 25MHz(100Mbps)/125MHz(1Gbps) to use for transmit.
> - * receive clock is provided by PHY.
> - */
> - reg = bcmgenet_ext_readl(priv, EXT_RGMII_OOB_CTRL);
> - reg &= ~OOB_DISABLE;
> - reg |= RGMII_LINK;
> - bcmgenet_ext_writel(priv, reg, EXT_RGMII_OOB_CTRL);
> + /* check speed/duplex/pause changes */
> + if (priv->old_speed != phydev->speed) {
> + status_changed = true;
> + priv->old_speed = phydev->speed;
> + }
> +
> + if (priv->old_duplex != phydev->duplex) {
> + status_changed = true;
> + priv->old_duplex = phydev->duplex;
> + }
> +
> + if (priv->old_pause != phydev->pause) {
> + status_changed = true;
> + priv->old_pause = phydev->pause;
> + }
> +
> + /* done if nothing has changed */
> + if (!status_changed)
> + return;
>
> /* speed */
> if (phydev->speed == SPEED_1000)
> @@ -110,36 +119,39 @@ static void bcmgenet_mii_setup(struct net_device *dev)
> cmd_bits = UMAC_SPEED_10;
> cmd_bits <<= CMD_SPEED_SHIFT;
>
> - if (priv->old_duplex != phydev->duplex) {
> - status_changed = 1;
> - priv->old_duplex = phydev->duplex;
> - }
> -
> /* duplex */
> if (phydev->duplex != DUPLEX_FULL)
> cmd_bits |= CMD_HD_EN;
>
> - if (priv->old_pause != phydev->pause) {
> - status_changed = 1;
> - priv->old_pause = phydev->pause;
> - }
> -
> /* pause capability */
> if (!phydev->pause)
> cmd_bits |= CMD_RX_PAUSE_IGNORE | CMD_TX_PAUSE_IGNORE;
> - }
>
> - if (!status_changed)
> - return;
> + /*
> + * Program UMAC and RGMII block based on established
> + * link speed, duplex, and pause. The speed set in
> + * umac->cmd tell RGMII block which clock to use for
> + * transmit -- 25MHz(100Mbps) or 125MHz(1Gbps).
> + * Receive clock is provided by the PHY.
> + */
> + reg = bcmgenet_ext_readl(priv, EXT_RGMII_OOB_CTRL);
> + reg &= ~OOB_DISABLE;
> + reg |= RGMII_LINK;
> + bcmgenet_ext_writel(priv, reg, EXT_RGMII_OOB_CTRL);
>
> - if (phydev->link) {
> reg = bcmgenet_umac_readl(priv, UMAC_CMD);
> reg &= ~((CMD_SPEED_MASK << CMD_SPEED_SHIFT) |
> CMD_HD_EN |
> CMD_RX_PAUSE_IGNORE | CMD_TX_PAUSE_IGNORE);
> reg |= cmd_bits;
> bcmgenet_umac_writel(priv, reg, UMAC_CMD);
> + } else {
> + /* done if nothing has changed */
> + if (!status_changed)
> + return;
>
> + /* needed for MoCA fixed PHY to reflect correct link status */
> + netif_carrier_off(dev);
> }
>
> phy_print_status(phydev);
> @@ -318,6 +330,12 @@ static int bcmgenet_mii_probe(struct net_device *dev)
> /* Communicate the integrated PHY revision */
> phy_flags = priv->gphy_rev;
>
> + /* Initialize link state variables that bcmgenet_mii_setup() uses */
> + priv->old_link = -1;
> + priv->old_speed = -1;
> + priv->old_duplex = -1;
> + priv->old_pause = -1;
> +
> phydev = of_phy_connect(dev, priv->phy_dn, bcmgenet_mii_setup,
> phy_flags, priv->phy_interface);
> if (!phydev) {
> @@ -325,9 +343,6 @@ static int bcmgenet_mii_probe(struct net_device *dev)
> return -ENODEV;
> }
>
> - priv->old_link = -1;
> - priv->old_duplex = -1;
> - priv->old_pause = -1;
> priv->phydev = phydev;
>
> /* Configure port multiplexer based on what the probed PHY device since
> --
> 2.1.0.rc2.206.gedb03e5
>
--
Florian
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup()
2014-10-03 19:25 [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup() Petri Gynther
2014-10-04 4:18 ` Florian Fainelli
@ 2014-10-06 4:18 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-10-06 4:18 UTC (permalink / raw)
To: pgynther; +Cc: netdev, f.fainelli
From: Petri Gynther <pgynther@google.com>
Date: Fri, 3 Oct 2014 12:25:01 -0700 (PDT)
> bcmgenet_mii_setup() is called from the PHY state machine every 1-2 seconds
> when the PHYs are in PHY_POLL mode.
>
> Improve bcmgenet_mii_setup() so that it touches the MAC registers only when
> the link is up and there was a change to link, speed, duplex, or pause status.
>
> Signed-off-by: Petri Gynther <pgynther@google.com>
Yeah that's ugly reprogramming so much stuff unnecessarily every
second or two, applied thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-10-06 4:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-03 19:25 [PATCH net-next] net: bcmgenet: improve bcmgenet_mii_setup() Petri Gynther
2014-10-04 4:18 ` Florian Fainelli
2014-10-06 4:18 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox