Re: [PATCH net] net: lan743x: permit VLAN-tagged packets up to configured MTU

From: Jakub Kicinski

Date: Tue May 19 2026 - 21:31:08 EST


On Fri, 15 May 2026 19:23:57 +0000 David Thompson wrote:
> The driver must set the MAC_RX.FSE bit before setting MAC_RX.RXEN to allow
> VLAN-tagged frames up to the interface MTU

AFAIU this sentence indicates that the double write
is intentional but could you make that more explicit?
Maybe mention whether this is based on experience or the AN?

It's going to be invaluable if someone ever sends a "cleanup"
to fold the writes into one..

> temp = lan743x_csr_read(adapter, MAC_RX);
> + temp |= MAC_RX_FSE_;
> + lan743x_csr_write(adapter, MAC_RX, temp);
> lan743x_csr_write(adapter, MAC_RX, temp | MAC_RX_RXEN_);
--
pw-bot: cr