From: Manoil Claudiu <[email protected]> Date: Tue, 15 Dec 2015 10:12:36 +0000
>>-----Original Message----- >>From: Hamish Martin [mailto:[email protected]] >>Sent: Tuesday, December 15, 2015 3:15 AM >>To: Manoil Claudiu-B08782 <[email protected]> >>Cc: [email protected]; Hamish Martin >><[email protected]> >>Subject: [PATCH] gianfar: Don't enable RX Filer if not supported >> >>After commit 15bf176db1fb ("gianfar: Don't enable the Filer w/o the >>Parser"), 'TSEC' model controllers (for example as seen on MPC8541E) >>always have 8 bytes stripped from the front of received frames. >>Only 'eTSEC' gianfar controllers have the RX Filer capability (amongst >>other enhancements). Previously this was treated as always enabled >>for both 'TSEC' and 'eTSEC' controllers. >>In commit 15bf176db1fb ("gianfar: Don't enable the Filer w/o the Parser") >>a subtle change was made to the setting of 'uses_rxfcb' to effectively >>always set it (since 'rx_filer_enable' was always true). This had the >>side-effect of always stripping 8 bytes from the front of received frames >>on 'TSEC' type controllers. >> >>We now only enable the RX Filer capability on controller types that >>support it, thereby avoiding the issue for 'TSEC' type controllers. >> > > > Reviewed-by: Claudiu Manoil <[email protected]> Applied and queued up for -stable, thanks everyone. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
