On Fri, Dec 01, 2023 at 02:51:26PM +0100, Steffen Trumtrar wrote:
> The rx_buffer is only dma_alloc'ed but never properly flushed.
> Fix that.
>
> While at it, also use proper volatile access instead of sw barriers.
>
> Also, redefine PKTSIZE to a sensible multiple of 64 bytes.
>
> Signed-off-by: Steffen Trumtrar
> ---
> Changes in v4:
> - align PKTSIZE with cacheline size
> - minor style changes
> - Link to v3:
> https://lore.barebox.org/20231129-v2023-08-0-topic-macb-v3-0-65ad6db83...@pengutronix.de
>
> Changes in v3:
> - fix dma_unmap_single direction
> - dma_map_single packet in macb_send()
> - Link to v2:
> https://lore.barebox.org/20231129-v2023-08-0-topic-macb-v2-0-4dc2cb4d5...@pengutronix.de
>
> Changes in v2:
> - change dma_map_single to DMA_FROM_DEVICE
> - drop (unsigned long) casts in dma_sync_*
> - rework writel/setbits/clearbits to keep semantics
> - Link to v1:
> https://lore.barebox.org/20231128-v2023-08-0-topic-macb-v1-0-9faff73bc...@pengutronix.de
>
> ---
> Steffen Trumtrar (3):
> net: macb: fix dma_alloc for rx_buffer
> net: macb: convert to volatile accesses
> include: net: align PKTSIZE to 64 bytes
>
> drivers/net/macb.c | 132
> +
> include/net.h | 4 +-
> 2 files changed, 75 insertions(+), 61 deletions(-)
Applied, thanks
Sascha
> ---
> base-commit: 5f200dd534c848dfa5d948334b6373f0310b8f73
> change-id: 20231128-v2023-08-0-topic-macb-0c13ed91179d
>
> Best regards,
> --
> Steffen Trumtrar
>
>
>
--
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- |