- 22 Mar, 2013 4 commits
-
-
Mauro Carvalho Chehab authored
It used to be a code that returns arror at ov7660_init. However, this was removed by changeset c84e412f: @@ -231,33 +116,40 @@ int ov7660_init(struct sd *sd) if (dump_sensor) ov7660_dump_registers(sd); - err = ov7660_set_gain(&sd->gspca_dev, sensor_settings[GAIN_IDX]); - if (err < 0) - return err; + return 0; +} - err = ov7660_set_auto_white_balance(&sd->gspca_dev, - sensor_settings[AUTO_WHITE_BALANCE_IDX]); - if (err < 0) - return err; As complained by gcc: drivers/media/usb/gspca/m5602/m5602_ov7660.c: In function 'ov7660_init': drivers/media/usb/gspca/m5602/m5602_ov7660.c:99:9: warning: variable 'err' set but not used [-Wunused-but-set-variable] It should be noticed that the original error code was crappy, as it wasn't returning any error if sensor init fails. Fix it by returning an error if the sensor can't be initialized. Cc: Hans de Goede <hdegoede@redhat.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab@redhat.com>
-
Mauro Carvalho Chehab authored
This field is unused. Remove it. Signed-off-by: -
Mauro Carvalho Chehab authored
Changeset 8f3741e0 accidentally broke the CNR estimation. It should be calculated as "a + b - c". However, previously, the subtraction by c only occurred if SNR would be positive, due to a bad binding to DVBv3 API. This also fixes the following warning: drivers/media/dvb-frontends/drxk_hard.c:2556:6: warning: variable 'c' set but not used [-Wunused-but-set-variable] Signed-off-by:
-
Mauro Carvalho Chehab authored
As reported by Frank Schäfer <fschaefer.oss@googlemail.com>: ERROR: "__divdi3" [drivers/media/common/siano/smsdvb.ko] undefined! Reported-by:
Frank Schäfer <fschaefer.oss@googlemail.com> Tested-by:
Gianluca Gennari <gennarone@gmail.com> Signed-off-by:
-
- 21 Mar, 2013 36 commits
-
-
Antti Palosaari authored
Signed-off-by:
Antti Palosaari <crope@iki.fi> Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Use more careful error checks. Signed-off-by:
-
Antti Palosaari authored
Use register address auto increment to reduce I/O when large register / values tables are written. Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
That register is property of demodulator so move it correct place. Signed-off-by:
-
Antti Palosaari authored
Those are demodulator init tables according to used tuner tuner config. af9033 demod driver does those inits currently and due to that these duplicate inits could be removed. Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
Dumped out from the Windows driver version 12.07.06.1 Signed-off-by:
-
Antti Palosaari authored
For some reason there seems to be value 0x03 in eeprom for dual mode (and 0x00 for single mode). Boolean is not always 1 bit wide - it could be 8 bit wide too. Storing number 0x03 to boolean causes driver to thing there is 4 tuners in some cases :o Add also some comments regarding to eeprom. Signed-off-by:
-
Antti Palosaari authored
There is no difference towards AF9035 remote controller, so enable it. Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
eeprom memory mapped base address is detected at the very first. Use it everywhere. Signed-off-by:
-
Antti Palosaari authored
AF9035 and IT9135 supports two different firmware format. Select correct loader according to first byte of firmware file. Signed-off-by:
-
Antti Palosaari authored
These are demod inits and are already done by af9033 demod driver. Signed-off-by:
-
Antti Palosaari authored
Dumped out from Windows driver version 12.07.06.1, 07/06/2012. Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
We don't need it. Tuner ID and device address are enough. Signed-off-by:
-
Antti Palosaari authored
Copy forgotten power up registers from it913x-fe driver. Remove two demod registers as those are already written by af9033 driver. Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Merge those functions to one and disable sleep. Signed-off-by:
-
Antti Palosaari authored
Now it supports IT9135 based dual tuner devices. Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
Signed-off-by:
-
Antti Palosaari authored
That same USB ID is used both AF9015 and AF9035 driver. iManufacturer is only thing we can select correct driver without a I/O. Signed-off-by:
-
Antti Palosaari authored
That same USB ID is used both AF9015 and AF9035 driver. iManufacturer is only thing we can select correct driver without a I/O. Signed-off-by:
-
