net: dsa: seville: fix VCAP IS2 action width
authorVladimir Oltean <vladimir.oltean@nxp.com>
Tue, 29 Sep 2020 11:20:25 +0000 (14:20 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Sep 2020 20:24:17 +0000 (13:24 -0700)
Since the actions are packed together in the action RAM, an incorrect
action width means that no action except the first one would behave
correctly.

The tc-flower offload has probably not been tested on this hardware
since its introduction.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/ocelot/seville_vsc9953.c

index 29df079..110070a 100644 (file)
@@ -706,7 +706,7 @@ static const struct vcap_props vsc9953_vcap_props[] = {
                .action_type_width = 1,
                .action_table = {
                        [IS2_ACTION_TYPE_NORMAL] = {
-                               .width = 44,
+                               .width = 50, /* HIT_CNT not included */
                                .count = 2
                        },
                        [IS2_ACTION_TYPE_SMAC_SIP] = {