diff options
author | Stefan Chulski <stefanc@marvell.com> | 2017-08-09 10:37:46 +0300 |
---|---|---|
committer | Stefan Roese <sr@denx.de> | 2017-08-10 08:33:02 +0200 |
commit | bb915c843f28573bb31496644e7f7e2c0a066fd8 (patch) | |
tree | c877058871231c009c6489f11f737dbf08d8a478 | |
parent | 73f592fb724153f8afbd79098f918ff548fda7ef (diff) | |
download | u-boot-bb915c843f28573bb31496644e7f7e2c0a066fd8.zip u-boot-bb915c843f28573bb31496644e7f7e2c0a066fd8.tar.gz u-boot-bb915c843f28573bb31496644e7f7e2c0a066fd8.tar.bz2 |
net: mvpp2x: fix BM configuration overrun issue
Issue:
BM counters were overrun by probe that called per Network interface and
caused release of wrong number of buffers during remove procedure.
Fix:
Use probe_done and num_ports to call init and remove procedure
once per communication controller.
Signed-off-by: Stefan Chulski <stefanc@marvell.com>
Tested-by: iSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: Igal Liberman <igall@marvell.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Signed-off-by: Stefan Roese <sr@denx.de>
-rw-r--r-- | drivers/net/mvpp2.c | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/drivers/net/mvpp2.c b/drivers/net/mvpp2.c index 3083111..426b2c2 100644 --- a/drivers/net/mvpp2.c +++ b/drivers/net/mvpp2.c @@ -942,6 +942,7 @@ struct mvpp2 { struct mii_dev *bus; int probe_done; + u8 num_ports; }; struct mvpp2_pcpu_stats { @@ -4848,6 +4849,7 @@ static int mvpp2_port_probe(struct udevice *dev, #endif priv->port_list[port->id] = port; + priv->num_ports++; return 0; } @@ -5512,10 +5514,8 @@ static int mvpp2_probe(struct udevice *dev) int err; /* Only call the probe function for the parent once */ - if (!priv->probe_done) { + if (!priv->probe_done) err = mvpp2_base_probe(dev->parent); - priv->probe_done = 1; - } port->priv = dev_get_priv(dev->parent); @@ -5553,11 +5553,15 @@ static int mvpp2_probe(struct udevice *dev) gop_port_init(port); } - /* Initialize network controller */ - err = mvpp2_init(dev, priv); - if (err < 0) { - dev_err(&pdev->dev, "failed to initialize controller\n"); - return err; + if (!priv->probe_done) { + /* Initialize network controller */ + err = mvpp2_init(dev, priv); + if (err < 0) { + dev_err(&pdev->dev, "failed to initialize controller\n"); + return err; + } + priv->num_ports = 0; + priv->probe_done = 1; } err = mvpp2_port_probe(dev, port, dev_of_offset(dev), priv); @@ -5585,6 +5589,11 @@ static int mvpp2_remove(struct udevice *dev) struct mvpp2 *priv = port->priv; int i; + priv->num_ports--; + + if (priv->num_ports) + return 0; + for (i = 0; i < MVPP2_BM_POOLS_NUM; i++) mvpp2_bm_pool_destroy(dev, priv, &priv->bm_pools[i]); |