V4L/DVB: si470x: -EINVAL overwritten in si470x_vidioc_s_tuner()
authorTobias Lorenz <tobias.lorenz@gmx.net>
Sat, 10 Jul 2010 16:01:33 +0000 (13:01 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 2 Aug 2010 19:42:44 +0000 (16:42 -0300)
This patch to the si470x_vidioc_s_tuner function was developed in
cooperation with Roel Kluin <roel.kluin@gmail.com>. It sets the default
retval to 0 instead of -EINVAL, identical to what is done in all other
set/get functions of v4l2_ioctl_ops. This is just as cosmetic change, as
retval is directly overwritten by the si470x_disconnect_check() anyway.

Signed-off-by: Tobias Lorenz <tobias.lorenz@gmx.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/si470x/radio-si470x-common.c

index 47075fc71f112a8e7be6b6dfac8133a1ad9ab490..9927a595b426a285f2755d608631200a51a3a0e7 100644 (file)
@@ -748,7 +748,7 @@ static int si470x_vidioc_s_tuner(struct file *file, void *priv,
                struct v4l2_tuner *tuner)
 {
        struct si470x_device *radio = video_drvdata(file);
-       int retval = -EINVAL;
+       int retval = 0;
 
        /* safety checks */
        retval = si470x_disconnect_check(radio);