Hi, On 29.11.23 12:33, Steffen Trumtrar wrote: > dma_sync_single_for_cpu(macb->dev, > macb->rx_buffer_phys + macb->rx_buffer_size * macb->rx_tail, > length, DMA_FROM_DEVICE); > @@ -198,6 +190,7 @@ static int gem_recv(struct eth_device *edev) > dma_sync_single_for_device(macb->dev, > macb->rx_buffer_phys + macb->rx_buffer_size * macb->rx_tail, > length, DMA_FROM_DEVICE); > + clrbits_le32(&macb->rx_ring[macb->rx_tail].addr, MACB_BIT(RX_USED)); This is being moved around in the previous patch and moved back here, so that move seems to have been unintentional. By the way, did you see my comment about increasing PKTSIZE to be a multiple of 64 bytes? Thanks, Ahmad -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
