Re: [PATCH 1/1] RDMA/irdma: Remove the redundant return

[Leon Romanovsky <leon@kernel.org>]

On Mon, Jan 10, 2022 at 08:54:07AM -0400, Jason Gunthorpe wrote:
> On Sun, Jan 09, 2022 at 08:44:55PM +0200, Leon Romanovsky wrote:
> > On Mon, Jan 10, 2022 at 02:37:33AM -0500, yanjun.zhu@linux.dev wrote:
> > > From: Zhu Yanjun <yanjun.zhu@linux.dev>
> > > 
> > > The type of the function i40iw_remove is void. So remove
> > > the unnecessary return.
> > > 
> > > Signed-off-by: Zhu Yanjun <yanjun.zhu@linux.dev>
> > >  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
> > > +++ 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.
> 
> It is odd, but valid, C to return void like this..

Any idea where such C expression can be useful?

Thanks

> 
> Jason