summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorJiri Benc <jbenc@suse.cz>2005-05-06 21:30:42 -0700
committerLinus Torvalds <torvalds@ppc970.osdl.org>2005-05-06 22:09:28 -0700
commite99d3438e4db1ef9e4cae5ad3946c76f4ffd268d (patch)
tree50118dee29963e6720f1dda7734ef6f3616fdb4e /drivers
parentb272125273103458b9727df1868b81bae64f44cb (diff)
downloadlwn-e99d3438e4db1ef9e4cae5ad3946c76f4ffd268d.tar.gz
lwn-e99d3438e4db1ef9e4cae5ad3946c76f4ffd268d.zip
[PATCH] video/tuner: fix tuner->freq updating
In VIDIOC_S_FREQUENCY command in tuner-core.c, t->freq is set to a new value before calling set_freq(). This is not necessary, as set_freq() sets t->freq itself. Moreover, it causes problems with Philips tuners, as they need to take into consideration difference between previous and new frequency. Signed-off-by: Jiri Benc <jbenc@suse.cz> Cc: Gerd Knorr <kraxel@bytesex.org> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/video/tuner-core.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/video/tuner-core.c b/drivers/media/video/tuner-core.c
index 881a0539fc17..c7d0c9f093fa 100644
--- a/drivers/media/video/tuner-core.c
+++ b/drivers/media/video/tuner-core.c
@@ -357,8 +357,7 @@ tuner_command(struct i2c_client *client, unsigned int cmd, void *arg)
V4L2_TUNER_RADIO != t->mode)
set_tv_freq(client,400*16);
t->mode = f->type;
- t->freq = f->frequency;
- set_freq(client,t->freq);
+ set_freq(client,f->frequency);
break;
}
case VIDIOC_G_TUNER: