On Fri, May 08, 2020 at 11:40:26PM +0100, Colin King wrote:
> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> The variable ret is being assigned with a value that is never read,
> the assignment is redundant and can be removed.
>
> Addresses-Coverity: ("Unused value")
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
We used to return negative error codes until commit 23021c21055f ("cnic:
Improve error recovery on bnx2x devices").
To be honest, I like the deliberate "ret = 0;" because this code will
trigger a static checker warning about wrong error codes. Also it looks
wrong to human reviewers. We should probably add a comment:
/* Deliberately returning success */
return 0;
regards,
dan carpenter
