diff options
author | Tobias Lorenz <tobias.lorenz@gmx.net> | 2010-07-10 12:01:33 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-08-02 15:42:44 -0400 |
commit | 73c994e4fb69b7bb85663e5175432c307657d207 (patch) | |
tree | a7654313065ded316c25bbeba7af3d6caa11bb43 /drivers/media/radio | |
parent | 22b0766b62d517a597d8155d828c5f93e3523cfa (diff) |
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 <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>
Diffstat (limited to 'drivers/media/radio')
-rw-r--r-- | drivers/media/radio/si470x/radio-si470x-common.c | 2 |
1 files changed, 1 insertions, 1 deletions
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, | |||
748 | struct v4l2_tuner *tuner) | 748 | struct v4l2_tuner *tuner) |
749 | { | 749 | { |
750 | struct si470x_device *radio = video_drvdata(file); | 750 | struct si470x_device *radio = video_drvdata(file); |
751 | int retval = -EINVAL; | 751 | int retval = 0; |
752 | 752 | ||
753 | /* safety checks */ | 753 | /* safety checks */ |
754 | retval = si470x_disconnect_check(radio); | 754 | retval = si470x_disconnect_check(radio); |