The function gem_recv implements a buffer "ring" that stops at filling level 10. That means, when the driver is used as gem, it stops receiving after exactly 10 packets. Instead of implementing macb_recv twice, use it also for the gem part. If there needs to be an extra recv function for the gigabit case, it can be added than. Also, first set the type of device (macb or gem) and then use functions that use this info.
Tested on a Zynq7000. Signed-off-by: Steffen Trumtrar <[email protected]> Cc: Nicolas Ferre <[email protected]> Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]> --- drivers/net/macb.c | 47 +++++++---------------------------------------- 1 file changed, 7 insertions(+), 40 deletions(-) diff --git a/drivers/net/macb.c b/drivers/net/macb.c index 8602437..a12eea7 100644 --- a/drivers/net/macb.c +++ b/drivers/net/macb.c @@ -169,37 +169,6 @@ static void reclaim_rx_buffers(struct macb_device *macb, macb->rx_tail = new_tail; } -static int gem_recv(struct eth_device *edev) -{ - struct macb_device *macb = edev->priv; - unsigned int rx_tail = macb->rx_tail; - void *buffer; - int length; - u32 status; - - dev_dbg(macb->dev, "%s\n", __func__); - - for (;;) { - barrier(); - if (!(macb->rx_ring[rx_tail].addr & MACB_BIT(RX_USED))) - return -1; - - barrier(); - status = macb->rx_ring[rx_tail].ctrl; - length = MACB_BFEXT(RX_FRMLEN, status); - if (status & MACB_BIT(RX_SOF)) { - buffer = macb->rx_buffer + macb->rx_buffer_size * macb->rx_tail; - net_receive(buffer, length); - macb->rx_ring[rx_tail].ctrl &= ~MACB_BIT(RX_USED); - barrier(); - } - rx_tail++; - macb->rx_tail++; - } - - return 0; -} - static int macb_recv(struct eth_device *edev) { struct macb_device *macb = edev->priv; @@ -619,11 +588,6 @@ static int macb_probe(struct device_d *dev) macb->phy_flags = pdata->phy_flags; - macb_init_rx_buffer_size(macb, PKTSIZE); - macb->rx_buffer = dma_alloc_coherent(macb->rx_buffer_size * macb->rx_ring_size); - macb->rx_ring = dma_alloc_coherent(RX_RING_BYTES(macb)); - macb->tx_ring = dma_alloc_coherent(TX_RING_BYTES); - macb->regs = dev_request_mem_region(dev, 0); /* @@ -638,12 +602,15 @@ static int macb_probe(struct device_d *dev) clk_enable(macb->pclk); - if (macb_is_gem(macb)) - edev->recv = gem_recv; - else - edev->recv = macb_recv; macb->is_gem = read_is_gem(macb); + macb_init_rx_buffer_size(macb, PKTSIZE); + macb->rx_buffer = dma_alloc_coherent(macb->rx_buffer_size * macb->rx_ring_size); + macb->rx_ring = dma_alloc_coherent(RX_RING_BYTES(macb)); + macb->tx_ring = dma_alloc_coherent(TX_RING_BYTES); + + edev->recv = macb_recv; + macb_reset_hw(macb); ncfgr = macb_mdc_clk_div(macb); ncfgr |= MACB_BIT(PAE); /* PAuse Enable */ -- 1.8.2.rc2 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
