media: sta2x11: Use gpio_is_valid() and remove unnecessary check
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Fri, 27 Apr 2018 11:44:01 +0000 (07:44 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Wed, 9 May 2018 20:20:16 +0000 (16:20 -0400)
commit0bade8b54395200b6a08e143e2bdc5b992f883e7
treea9928b17692057824370be349130a2f0e2dbf624
parent03b765237b0f728c2586b557fbfaed86bff7bf07
media: sta2x11: Use gpio_is_valid() and remove unnecessary check

Replace the manual validity checks for the GPIO with the
gpio_is_valid().

In vip_gpio_reserve(), Error checking for gpio pin is not correct.
If pwr_pin = -1, It will return 0. This should be return an error.

In sta2x11_vip_init_one(), Error checking for gpio 'reset_pin'
is unnecessary. Because vip_gpio_reserve() is also checking for
valid gpio pin. So removed extra error checking for gpio 'reset_pin'.

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/pci/sta2x11/sta2x11_vip.c