On Mon, Nov 21, 2016 at 9:14 PM, Christophe JAILLET <christophe.jail...@wanadoo.fr> wrote: > Knowing that: > #define TUNNEL_DST_PORT_FREE_REQ_TUNNEL_TYPE_VXLAN (0x1UL << 0) > #define TUNNEL_DST_PORT_FREE_REQ_TUNNEL_TYPE_GENEVE (0x5UL << 0) > and that 'bnxt_hwrm_tunnel_dst_port_alloc()' is only called with one of > these 2 constants, the TUNNEL_DST_PORT_ALLOC_REQ_TUNNEL_TYPE_GENEVE can not > trigger. > > Replace the bit test that overlap by an equality test, just as in > 'bnxt_hwrm_tunnel_dst_port_free()' above. > > Signed-off-by: Christophe JAILLET <christophe.jail...@wanadoo.fr>
Thanks. Acked-by: Michael Chan <michael.c...@broadcom.com>
