Re: [PATCH net-next] net: bgmac: stop advertising TX checksum offload

From: Florian Fainelli

Date: Fri Nov 06 2026 - 19:02:58 EST




On 11/2/2026 1:58 PM, Rosen Penev wrote:
The driver advertises NETIF_F_IP_CSUM and NETIF_F_IPV6_CSUM, but
bgmac_dma_tx_add() calls skb_checksum_help() on every CHECKSUM_PARTIAL
skb, so the checksum is always computed in software anyway. The return
value is ignored too, so if skb_checksum_help() fails the frame is sent
with a bad checksum.

Remove the feature flags and the fallback. The stack then computes the
checksum before handing the skb to the driver.

Tested on an ASUS RT-N16.

Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx>

Reviewed-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>

There is no transmit checksum offload through the DMA descriptors, it can only be done through FA/FA+ which is not supported yet (working on that).
--
Florian