> > diff --git a/drivers/net/bnxt/bnxt_ethdev.c > b/drivers/net/bnxt/bnxt_ethdev.c > > index aa7e7fdc85..1a6e0128ff 100644 > > --- a/drivers/net/bnxt/bnxt_ethdev.c > > +++ b/drivers/net/bnxt/bnxt_ethdev.c > > @@ -1009,6 +1009,7 @@ static int bnxt_dev_info_get_op(struct rte_eth_dev > *eth_dev, > > dev_info->speed_capa = bnxt_get_speed_capabilities(bp); > > dev_info->dev_capa = RTE_ETH_DEV_CAPA_RUNTIME_RX_QUEUE_SETUP | > > RTE_ETH_DEV_CAPA_RUNTIME_TX_QUEUE_SETUP; > > + dev_info->dev_capa &= ~RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP; > > Sorry, but here and everywhere below I see no point to cleanup > the bit explicitly when it is not actually set. > > [snip]
(removed maintainers from CC) As Ferruh explained, this line serves as a TODO item for maintainers to either remove the useless line or to support the capability and advertise it. If deemed unnecessary in the end, this patch can be safely dropped. http://inbox.dpdk.org/dev/6e148fd8-c7b9-dd90-d286-a54ff0faf...@intel.com/
