On 20/06/17 15:54, Andrew Lunn wrote:
On Tue, Jun 20, 2017 at 10:06:55AM +0200, John Crispin wrote:
The MT7530 inserts the 4 magic header in between the 802.3 address and
protocol field. The patch defines these header such that the flow_disector
can properly parse the packet and thus allows hashing to function properly.
This is to do with tagging, not the switch driver. The Marvell switch
driver can be used with two different tagging protocols.
So i would put these fields in the dsa_device_ops.
Andrew
Hi Andrew,
i originally did so but struct dsa_device_ops is defined inside
net/dsa/dsa_priv.h so flow_dissector.c would need a
#include "../dsa/dsa_priv.h"
I was not sure if this is ok or if we would need to move the struct
definition to include/net/dsa.h in that case
John
