Messages in this thread | | | Date | Mon, 16 Jan 2023 21:37:14 +0100 | Subject | Re: [PATCH] vfio: platform: No need to check res again | From | Eric Auger <> |
| |
Hi, On 1/7/23 04:47, Angus Chen wrote: > In function vfio_platform_regions_init(),we did check res implied > by using while loop, > so no need to check whether res be null or not again. > > No functional change intended. > > Signed-off-by: Angus Chen <angus.chen@jaguarmicro.com> Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric > --- > drivers/vfio/platform/vfio_platform_common.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/vfio/platform/vfio_platform_common.c b/drivers/vfio/platform/vfio_platform_common.c > index 1a0a238ffa35..a9ad3f4d2613 100644 > --- a/drivers/vfio/platform/vfio_platform_common.c > +++ b/drivers/vfio/platform/vfio_platform_common.c > @@ -150,9 +150,6 @@ static int vfio_platform_regions_init(struct vfio_platform_device *vdev) > struct resource *res = > vdev->get_resource(vdev, i); > > - if (!res) > - goto err; > - > vdev->regions[i].addr = res->start; > vdev->regions[i].size = resource_size(res); > vdev->regions[i].flags = 0;
| |