gianfar: Don't enable the Filer w/o the Parser
authorClaudiu Manoil <claudiu.manoil@freescale.com>
Fri, 23 Oct 2015 08:41:59 +0000 (11:41 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Oct 2015 01:28:08 +0000 (18:28 -0700)
commit15bf176db1fb00333af7050c0c699fc7b4e4a960
tree94aab08f4231207558578e5b6a9f83b566421a8a
parent5188f7e5a7175975f8b943a4b25e499c98a7b9d6
gianfar: Don't enable the Filer w/o the Parser

Under one unusual circumstance it's possible to wrongly set
FILREN without enabling PRSDEP as well in the RCTRL register,
against the hardware specifications.  With the default config
this does not happen because the default Rx offloads (Rx csum
and Rx VLAN) properly enable PRSDEP.  But if anyone disables
all these offloads (via ethtool), we get a wrong configuration
were the Rx flow classification and hashing, and other Filer
based features (e.g. wake-on-filer interrupt) won't work.
This patch fixes the issue.
Also, account for Rx FCB insertion which happens every time
PRSDEP is set.

Signed-off-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/gianfar.c