vfio: platform: No need to check res again
authorAngus Chen <angus.chen@jaguarmicro.com>
Sat, 7 Jan 2023 03:47:20 +0000 (11:47 +0800)
committerAlex Williamson <alex.williamson@redhat.com>
Mon, 23 Jan 2023 18:26:28 +0000 (11:26 -0700)
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>
Link: https://lore.kernel.org/r/20230107034721.2127-1-angus.chen@jaguarmicro.com
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
drivers/vfio/platform/vfio_platform_common.c

index 1a0a238ffa3543d7999c42b4d219ccfe7897fdb7..a9ad3f4d2613722c791106da9078264e5458b8fe 100644 (file)
@@ -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;