On Mon, Oct 6, 2025 at 2:12 AM Alok Tiwari <[email protected]> wrote: > > virtio_vdpa_set_status() is declared as returning void, but it used > "return vdpa_set_status()" Since vdpa_set_status() also returns > void, the return statement is unnecessary and misleading. > Remove it. > > Fixes: c043b4a8cf3b ("virtio: introduce a vDPA based transport") > Signed-off-by: Alok Tiwari <[email protected]> > --- > v1 -> v2 > added fixes tag > --- > drivers/virtio/virtio_vdpa.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/virtio/virtio_vdpa.c b/drivers/virtio/virtio_vdpa.c > index 657b07a60788..1abecaf76465 100644 > --- a/drivers/virtio/virtio_vdpa.c > +++ b/drivers/virtio/virtio_vdpa.c > @@ -80,7 +80,7 @@ static void virtio_vdpa_set_status(struct virtio_device > *vdev, u8 status) > { > struct vdpa_device *vdpa = vd_get_vdpa(vdev); > > - return vdpa_set_status(vdpa, status); > + vdpa_set_status(vdpa, status); > } > > static void virtio_vdpa_reset(struct virtio_device *vdev) > -- > 2.50.1 >
Acked-by: Jason Wang <[email protected]> Thanks
