media: platform: Use gpio_is_valid()
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Fri, 27 Apr 2018 11:44:00 +0000 (07:44 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 9 May 2018 20:17:15 +0000 (16:17 -0400)
Replace the manual validity checks for the GPIO with the
gpio_is_valid().

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/via-camera.c

index e9a0263..f01c3e8 100644 (file)
@@ -178,7 +178,7 @@ static int via_sensor_power_setup(struct via_camera *cam)
 
        cam->power_gpio = viafb_gpio_lookup("VGPIO3");
        cam->reset_gpio = viafb_gpio_lookup("VGPIO2");
-       if (cam->power_gpio < 0 || cam->reset_gpio < 0) {
+       if (!gpio_is_valid(cam->power_gpio) || !gpio_is_valid(cam->reset_gpio)) {
                dev_err(&cam->platdev->dev, "Unable to find GPIO lines\n");
                return -EINVAL;
        }