diff options
author | Saqeb Akhter <johoja@gmail.com> | 2006-06-29 19:29:29 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-06-30 14:59:45 -0400 |
commit | ccd214b27341485bd471e4e031c33d2ba1a9aaac (patch) | |
tree | d02bb9244fd5e3ccd77369925cc68659155e7b6c | |
parent | 70d906354fd7fe3956366ade15ab36d6b7aed971 (diff) |
V4L/DVB (4284): Cx24123: fix set_voltage function according to the specs
The set_voltage function in cx24123.c was corrected to match how it is
described in the CX24123 specs, producing the correct behaviour for cards
that require it.
Acked-by: Andrew de Quincey <adq_dvb@lidskialf.net>
Signed-off-by: Saqeb Akhter <johoja@gmail.com>
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
-rw-r--r-- | drivers/media/dvb/frontends/cx24123.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/dvb/frontends/cx24123.c b/drivers/media/dvb/frontends/cx24123.c index f2f795cba56a..274a87b7a5d5 100644 --- a/drivers/media/dvb/frontends/cx24123.c +++ b/drivers/media/dvb/frontends/cx24123.c | |||
@@ -670,10 +670,10 @@ static int cx24123_set_voltage(struct dvb_frontend* fe, fe_sec_voltage_t voltage | |||
670 | switch (voltage) { | 670 | switch (voltage) { |
671 | case SEC_VOLTAGE_13: | 671 | case SEC_VOLTAGE_13: |
672 | dprintk("%s: setting voltage 13V\n", __FUNCTION__); | 672 | dprintk("%s: setting voltage 13V\n", __FUNCTION__); |
673 | return cx24123_writereg(state, 0x29, val | 0x80); | 673 | return cx24123_writereg(state, 0x29, val & 0x7f); |
674 | case SEC_VOLTAGE_18: | 674 | case SEC_VOLTAGE_18: |
675 | dprintk("%s: setting voltage 18V\n", __FUNCTION__); | 675 | dprintk("%s: setting voltage 18V\n", __FUNCTION__); |
676 | return cx24123_writereg(state, 0x29, val & 0x7f); | 676 | return cx24123_writereg(state, 0x29, val | 0x80); |
677 | default: | 677 | default: |
678 | return -EINVAL; | 678 | return -EINVAL; |
679 | }; | 679 | }; |