Commit 432b5942 authored by Maxime Chevallier's avatar Maxime Chevallier Committed by David S. Miller
Browse files

net: mvpp2: prs: Drop unnecessary swab16 in vlan detection



Vlan IDs must not be swapped when creating Header Parser entries. This
has no effect on little-endian systems, but is wrong for big-endian.

Signed-off-by: default avatarMaxime Chevallier <maxime.chevallier@bootlin.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent bd43d1ba
Loading
Loading
Loading
Loading
+3 −3
Original line number Original line Diff line number Diff line
@@ -647,7 +647,7 @@ static int mvpp2_prs_vlan_find(struct mvpp2 *priv, unsigned short tpid, int ai)
			continue;
			continue;


		mvpp2_prs_init_from_hw(priv, &pe, tid);
		mvpp2_prs_init_from_hw(priv, &pe, tid);
		match = mvpp2_prs_tcam_data_cmp(&pe, 0, swab16(tpid));
		match = mvpp2_prs_tcam_data_cmp(&pe, 0, tpid);
		if (!match)
		if (!match)
			continue;
			continue;


@@ -775,8 +775,8 @@ static int mvpp2_prs_double_vlan_find(struct mvpp2 *priv, unsigned short tpid1,


		mvpp2_prs_init_from_hw(priv, &pe, tid);
		mvpp2_prs_init_from_hw(priv, &pe, tid);


		match = mvpp2_prs_tcam_data_cmp(&pe, 0, swab16(tpid1)) &&
		match = mvpp2_prs_tcam_data_cmp(&pe, 0, tpid1) &&
			mvpp2_prs_tcam_data_cmp(&pe, 4, swab16(tpid2));
			mvpp2_prs_tcam_data_cmp(&pe, 4, tpid2);


		if (!match)
		if (!match)
			continue;
			continue;