diff options
author | Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> | 2016-06-02 15:14:52 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-06-04 01:35:06 +0200 |
commit | 8478b6cdc10e8a7735deeb9d9e46ad5b157c84d0 (patch) | |
tree | 192067885b73a0641ef11cb28d0e283e24295918 /drivers | |
parent | gianfar: fix the last transmit buffer descriptor (diff) | |
download | linux-8478b6cdc10e8a7735deeb9d9e46ad5b157c84d0.tar.xz linux-8478b6cdc10e8a7735deeb9d9e46ad5b157c84d0.zip |
net: ethernet: ti: cpsw: fix rx-usecs interrupt pacing consistency
The rx-usecs shouldn't be changed while interface down/up.
Currently, for instance, if it's set to 100us, after interface
down/up it's 500us. It's a hidden bug that can lead to lavish
interrupt pacing time increasing while "down/up" up to max value.
Steps to reproduce:
- set rx-usecs to be 100us
- down/up interface
- read new unexpected rx-usecs
Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/ethernet/ti/cpsw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 4b08a2f52b3e..e6bb0ecb12c7 100644 --- a/drivers/net/ethernet/ti/cpsw.c +++ b/drivers/net/ethernet/ti/cpsw.c @@ -1339,7 +1339,7 @@ static int cpsw_ndo_open(struct net_device *ndev) if (priv->coal_intvl != 0) { struct ethtool_coalesce coal; - coal.rx_coalesce_usecs = (priv->coal_intvl << 4); + coal.rx_coalesce_usecs = priv->coal_intvl; cpsw_set_coalesce(ndev, &coal); } |