[media] drxk: Convert an #ifdef logic as a new config parameter
authorMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 9 Jul 2011 16:06:12 +0000 (13:06 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 27 Jul 2011 20:55:48 +0000 (17:55 -0300)
Instead of using #ifdef I2C_LONG_ADR for some devices, convert
it into a parameter. Terratec H5 logs from the original driver
seems to need this mode.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/frontends/drxk.h
drivers/media/dvb/frontends/drxk_hard.c
drivers/media/dvb/frontends/drxk_hard.h

index a7b295f82a861eb6489b202df1f7015d93b8f20d..d5b6f9fe4d7c9c7ac627d9e4f65650f9a9a6cb00 100644 (file)
@@ -6,6 +6,7 @@
 
 struct drxk_config {
        u8 adr;
+       u32 single_master : 1;
 };
 
 extern struct dvb_frontend *drxk_attach(const struct drxk_config *config,
index d351e6a630fc748abe628014aca3874e5f3720ce..c4b35a5638cddf45246938104b9e323efc1dbbc4 100644 (file)
@@ -375,9 +375,10 @@ static int i2c_read(struct i2c_adapter *adap,
 static int read16_flags(struct drxk_state *state, u32 reg, u16 *data, u8 flags)
 {
        u8 adr = state->demod_address, mm1[4], mm2[2], len;
-#ifdef I2C_LONG_ADR
-       flags |= 0xC0;
-#endif
+
+       if (state->single_master)
+               flags |= 0xC0;
+
        if (DRXDAP_FASI_LONG_FORMAT(reg) || (flags != 0)) {
                mm1[0] = (((reg << 1) & 0xFF) | 0x01);
                mm1[1] = ((reg >> 16) & 0xFF);
@@ -406,9 +407,10 @@ static int read16(struct drxk_state *state, u32 reg, u16 *data)
 static int read32_flags(struct drxk_state *state, u32 reg, u32 *data, u8 flags)
 {
        u8 adr = state->demod_address, mm1[4], mm2[4], len;
-#ifdef I2C_LONG_ADR
-       flags |= 0xC0;
-#endif
+
+       if (state->single_master)
+               flags |= 0xC0;
+
        if (DRXDAP_FASI_LONG_FORMAT(reg) || (flags != 0)) {
                mm1[0] = (((reg << 1) & 0xFF) | 0x01);
                mm1[1] = ((reg >> 16) & 0xFF);
@@ -438,9 +440,9 @@ static int read32(struct drxk_state *state, u32 reg, u32 *data)
 static int write16_flags(struct drxk_state *state, u32 reg, u16 data, u8 flags)
 {
        u8 adr = state->demod_address, mm[6], len;
-#ifdef I2C_LONG_ADR
-       flags |= 0xC0;
-#endif
+
+       if (state->single_master)
+               flags |= 0xC0;
        if (DRXDAP_FASI_LONG_FORMAT(reg) || (flags != 0)) {
                mm[0] = (((reg << 1) & 0xFF) | 0x01);
                mm[1] = ((reg >> 16) & 0xFF);
@@ -469,9 +471,9 @@ static int write16(struct drxk_state *state, u32 reg, u16 data)
 static int write32_flags(struct drxk_state *state, u32 reg, u32 data, u8 flags)
 {
        u8 adr = state->demod_address, mm[8], len;
-#ifdef I2C_LONG_ADR
-       flags |= 0xC0;
-#endif
+
+       if (state->single_master)
+               flags |= 0xC0;
        if (DRXDAP_FASI_LONG_FORMAT(reg) || (flags != 0)) {
                mm[0] = (((reg << 1) & 0xFF) | 0x01);
                mm[1] = ((reg >> 16) & 0xFF);
@@ -503,9 +505,10 @@ static int write_block(struct drxk_state *state, u32 Address,
 {
        int status = 0, BlkSize = BlockSize;
        u8 Flags = 0;
-#ifdef I2C_LONG_ADR
-       Flags |= 0xC0;
-#endif
+
+       if (state->single_master)
+               Flags |= 0xC0;
+
        while (BlkSize > 0) {
                int Chunk = BlkSize > state->m_ChunkSize ?
                    state->m_ChunkSize : BlkSize;
@@ -6355,6 +6358,7 @@ struct dvb_frontend *drxk_attach(const struct drxk_config *config,
 
        state->i2c = i2c;
        state->demod_address = adr;
+       state->single_master = config->single_master;
 
        mutex_init(&state->mutex);
        mutex_init(&state->ctlock);
index 700f40c12632b97d09cc5ade95b619a70d24ca88..b7093e931582ab2d1e4830ed7884810572f745e9 100644 (file)
@@ -326,6 +326,11 @@ struct drxk_state {
        u16               m_AntennaSwitchDVBTDVBC;
 
        enum DRXPowerMode m_currentPowerMode;
+
+       /* Configurable parameters at the driver */
+
+       u32 single_master : 1;          /* Use single master i2c mode */
+
 };
 
 #define NEVER_LOCK 0