Commit 9bcdef32 authored Feb 07, 2017 by Jiri Pirko Committed by David S. Miller Feb 07, 2017

spectrum: acl_tcam: Fix catchall prio value



This fixes an issue reported by smatch:
mlxsw_sp_acl_tcam_chunk_create() warn: impossible condition '(priority == (-1)) => (0-u32max == u64max)'

Reported-by: Or Gerlitz <ogerlitz@mellanox.com>
Reported-by: Ido Schimmel <idosch@mellanox.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: 22a67766 ("mlxsw: spectrum: Introduce ACL core with simple TCAM implementation")
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Acked-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent a8cab863

drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -561,7 +561,7 @@ mlxsw_sp_acl_tcam_region_entry_remove(struct mlxsw_sp *mlxsw_sp,
		mlxsw_reg_write(mlxsw_sp->core, MLXSW_REG(ptce2), ptce2_pl);
		}

		#define MLXSW_SP_ACL_TCAM_CATCHALL_PRIO (-1UL)
		#define MLXSW_SP_ACL_TCAM_CATCHALL_PRIO (~0U)

		static int
		mlxsw_sp_acl_tcam_region_catchall_add(struct mlxsw_sp *mlxsw_sp,