Hi Sergei, On Mon, Oct 24, 2022 at 08:58:23PM +0300, Sergei Antonov wrote: > The ftmac100 controller considers packets >1518 (1500 + Ethernet + FCS) > FTL (frame too long) and drops them. That is fine with mtu 1500 or less > and it saves CPU time. When DSA is present, mtu is bigger (for VLAN > tagging) and the controller's built-in behavior is not desired then. We > can make the controller deliver FTL packets to the driver by setting > FTMAC100_MACCR_RX_FTL. Then we have to check ftmac100_rxdes_frame_length() > (packet length sans FCS) on packets marked with FTMAC100_RXDES0_FTL flag. > > Check for mtu > 1500 in .ndo_open() and set FTMAC100_MACCR_RX_FTL to let > the driver FTL packets. Implement .ndo_change_mtu() and check for > mtu > 1500 to set/clear FTMAC100_MACCR_RX_FTL dynamically. > > Fixes: 8d77c036b57c ("net: add Faraday FTMAC100 10/100 Ethernet driver") > Signed-off-by: Sergei Antonov > --- I think it's clear there are problems in communication between us, so let me try differently. Does the attached series of 3 patches work for you? I only compile tested them. I tried to keep as much of your work and authorship as possible, the intention was to rewrite the justification in the commit message and to fix the things which your patches didn't do (as separate patches).