Some device vendors has forgotten set correct slave demod I2C address
to eeprom. Use default I2C address when eeprom has no address at all.

Signed-off-by: Antti Palosaari <cr...@iki.fi>
---
 drivers/media/usb/dvb-usb-v2/af9035.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/drivers/media/usb/dvb-usb-v2/af9035.c 
b/drivers/media/usb/dvb-usb-v2/af9035.c
index 49e8360..1434d37 100644
--- a/drivers/media/usb/dvb-usb-v2/af9035.c
+++ b/drivers/media/usb/dvb-usb-v2/af9035.c
@@ -575,11 +575,11 @@ static int af9035_download_firmware(struct dvb_usb_device 
*d,
                if (ret < 0)
                        goto err;
 
-               if (state->chip_type == 0x9135) {
-                       if (!tmp)
-                               /* default 0x9135 slave I2C address */
-                               tmp = 0x3a;
+               /* use default I2C address if eeprom has no address set */
+               if (!tmp)
+                       tmp = 0x3a;
 
+               if (state->chip_type == 0x9135) {
                        ret = af9035_wr_reg(d, 0x004bfb, tmp);
                        if (ret < 0)
                                goto err;
@@ -641,6 +641,7 @@ static int af9035_read_config(struct dvb_usb_device *d)
 
        /* demod I2C "address" */
        state->af9033_config[0].i2c_addr = 0x38;
+       state->af9033_config[1].i2c_addr = 0x3a;
        state->af9033_config[0].adc_multiplier = AF9033_ADC_MULTIPLIER_2X;
        state->af9033_config[1].adc_multiplier = AF9033_ADC_MULTIPLIER_2X;
        state->af9033_config[0].ts_mode = AF9033_TS_MODE_USB;
@@ -688,11 +689,9 @@ static int af9035_read_config(struct dvb_usb_device *d)
                if (ret < 0)
                        goto err;
 
-               if (!tmp && state->chip_type == 0x9135)
-                       /* default 0x9135 slave I2C address */
-                       tmp = 0x3a;
+               if (tmp)
+                       state->af9033_config[1].i2c_addr = tmp;
 
-               state->af9033_config[1].i2c_addr = tmp;
                dev_dbg(&d->udev->dev, "%s: 2nd demod I2C addr=%02x\n",
                                __func__, tmp);
        }
-- 
1.8.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to