diff options
| author | Mauro Carvalho Chehab <mchehab@infradead.org> | 2008-01-29 12:30:43 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2008-01-29 12:30:43 -0200 |
| commit | 0b1a2bc366328e9caa70b9f2cef95618b0d0271d (patch) | |
| tree | b25ada55875855817763ec4181a7b02e5f86c2aa | |
| parent | 5f4890d5ecb2b24267125ce096b0844a0ec976ee (diff) | |
| download | mediapointer-dvb-s2-0b1a2bc366328e9caa70b9f2cef95618b0d0271d.tar.gz mediapointer-dvb-s2-0b1a2bc366328e9caa70b9f2cef95618b0d0271d.tar.bz2 | |
tuner-xc3028: Don't check return code for clock reset
From: Mauro Carvalho Chehab <mchehab@infradead.org>
Only tm6000 needs to be aware when a frequency is being changed. This seems
to improve channel change detection. Other bridges don't need this.
So, better to discard any errors if this fails, and proceed changing the
channels.
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
---
linux/drivers/media/video/tuner-xc2028.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
| -rw-r--r-- | linux/drivers/media/video/tuner-xc2028.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/linux/drivers/media/video/tuner-xc2028.c b/linux/drivers/media/video/tuner-xc2028.c index 89bba71d9..bfa6ac740 100644 --- a/linux/drivers/media/video/tuner-xc2028.c +++ b/linux/drivers/media/video/tuner-xc2028.c @@ -919,9 +919,11 @@ static int generic_set_freq(struct dvb_frontend *fe, u32 freq /* in HZ */, if (rc < 0) goto ret; - rc = priv->tuner_callback(priv->video_dev, XC2028_RESET_CLK, 1); - if (rc < 0) - goto ret; + /* Return code shouldn't be checked. + The reset CLK is needed only with tm6000. + Driver should work fine even if this fails. + */ + priv->tuner_callback(priv->video_dev, XC2028_RESET_CLK, 1); msleep(10); |