diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2024-03-13 12:07:03 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2024-03-13 12:38:56 +0100 |
commit | cbb8be4784295a1192d908ac969c5bd1a757ca98 (patch) | |
tree | 37302f716b00f9ebea0da6b333c550afae01ff4a /drivers/net | |
parent | 41b8dc7a59d90015e6a8e50bb8f3352a19f23272 (diff) | |
download | barebox-cbb8be4784295a1192d908ac969c5bd1a757ca98.tar.gz barebox-cbb8be4784295a1192d908ac969c5bd1a757ca98.tar.xz |
net: cpsw: replace global NetRxPackets with per-interface allocation
NetRxPackets is a remnant of times, where a board had at most one
Ethernet controller. This is outdated and we should drop NetRxPackets.
Switch over the driver to allocate the receive buffers needed.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20240313110704.1095554-7-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/cpsw.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/net/cpsw.c b/drivers/net/cpsw.c index 42fb32bde2..3f24c21594 100644 --- a/drivers/net/cpsw.c +++ b/drivers/net/cpsw.c @@ -215,6 +215,8 @@ struct cpsw_priv { unsigned int slave_size; unsigned int sliver_ofs; + void *rx_buffer[PKTBUFSRX - 2]; + struct cpdma_desc *descs; struct cpdma_desc *desc_free; struct cpdma_chan rx_chan, tx_chan; @@ -980,7 +982,7 @@ static int cpsw_setup(struct device *dev) /* submit rx descs */ for (i = 0; i < PKTBUFSRX - 2; i++) { - ret = cpdma_submit(priv, &priv->rx_chan, NetRxPackets[i], + ret = cpdma_submit(priv, &priv->rx_chan, priv->rx_buffer[i], PKTSIZE, 0); if (ret < 0) { dev_err(dev, "error %d submitting rx desc\n", ret); @@ -1350,6 +1352,10 @@ static int cpsw_probe(struct device *dev) priv = xzalloc(sizeof(*priv)); priv->dev = dev; + ret = net_alloc_packets(priv->rx_buffer, ARRAY_SIZE(priv->rx_buffer)); + if (ret) + goto out; + if (dev->of_node) { ret = cpsw_probe_dt(priv); if (ret) |