net: mscc: fix the injection header
authorAntoine Tenart <antoine.tenart@bootlin.com>
Wed, 20 Jun 2018 08:50:46 +0000 (10:50 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 21 Jun 2018 01:29:46 +0000 (10:29 +0900)
commit08d02364b12faa54d76dbfea2090321fd27996f2
treea40b3a5e1608afe22717b80d4a92c4ff950326c1
parentea0820bb771175c7d4192fc6f5b5c56b3c6d5239
net: mscc: fix the injection header

When injecting frames in the Ocelot switch driver an injection header
(IFH) should be used to configure various parameters related to a given
frame, such as the port onto which the frame should be departed or its
vlan id. Other parameters in the switch configuration can led to an
injected frame being sent without an IFH but this led to various issues
as the per-frame parameters are then not used. This is especially true
when using multiple ports for injection.

The IFH was injected with the wrong endianness which led to the switch
not taking it into account as the IFH_INJ_BYPASS bit was then unset.
(The bit tells the switch to use the IFH over its internal
configuration). This patch fixes it.

In addition to the endianness fix, the IFH is also fixed. As it was
(unwillingly) unused, some of its fields were not configured the right
way.

Fixes: a556c76adc05 ("net: mscc: Add initial Ocelot switch support")
Signed-off-by: Antoine Tenart <antoine.tenart@bootlin.com>
Reviewed-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mscc/ocelot.c