On 2020-11-19 6:59 a.m., Ferruh Yigit wrote:
There is already a break above for the case "count >= 1", so at this stage 'count' should be always '0'.Fixes: 1a85922369c4 ("net/avp: add device configuration") Cc: [email protected] Signed-off-by: Ferruh Yigit <[email protected]> --- drivers/net/avp/avp_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/avp/avp_ethdev.c b/drivers/net/avp/avp_ethdev.c index 5f8187b905..f531e03c02 100644 --- a/drivers/net/avp/avp_ethdev.c +++ b/drivers/net/avp/avp_ethdev.c @@ -267,7 +267,7 @@ avp_dev_process_request(struct avp_dev *avp, struct rte_avp_request *request) break; } - if ((count < 1) && (retry == 0)) { + if (retry == 0) { PMD_DRV_LOG(ERR, "Timeout while waiting for a response for %u\n", request->req_id); ret = -ETIME; -- 2.26.2
Acked-by: Steven Webster <[email protected]>
