From 73c994e4fb69b7bb85663e5175432c307657d207 Mon Sep 17 00:00:00 2001 From: Tobias Lorenz Date: Sat, 10 Jul 2010 13:01:33 -0300 Subject: [PATCH] V4L/DVB: si470x: -EINVAL overwritten in si470x_vidioc_s_tuner() This patch to the si470x_vidioc_s_tuner function was developed in cooperation with Roel Kluin . 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 Signed-off-by: Mauro Carvalho Chehab --- drivers/media/radio/si470x/radio-si470x-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/radio/si470x/radio-si470x-common.c b/drivers/media/radio/si470x/radio-si470x-common.c index 47075fc71f11..9927a595b426 100644 --- a/drivers/media/radio/si470x/radio-si470x-common.c +++ b/drivers/media/radio/si470x/radio-si470x-common.c @@ -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); -- 2.30.2