On Mon, Jan 10, 2022 at 02:37:33AM -0500, yanjun.zhu@xxxxxxxxx wrote: > From: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > > The type of the function i40iw_remove is void. So remove > the unnecessary return. > > Signed-off-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > --- > drivers/infiniband/hw/irdma/i40iw_if.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/infiniband/hw/irdma/i40iw_if.c b/drivers/infiniband/hw/irdma/i40iw_if.c > index d219f64b2c3d..43e962b97d6a 100644 > --- a/drivers/infiniband/hw/irdma/i40iw_if.c > +++ b/drivers/infiniband/hw/irdma/i40iw_if.c > @@ -198,7 +198,7 @@ static void i40iw_remove(struct auxiliary_device *aux_dev) > aux_dev); > struct i40e_info *cdev_info = i40e_adev->ldev; > > - return i40e_client_device_unregister(cdev_info); > + i40e_client_device_unregister(cdev_info); I'm surprised that compiler didn't warn about extra parameter to return. Thanks, Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxx>
