diff options
author | Jean Delvare <khali@linux-fr.org> | 2005-08-11 17:51:10 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-09-05 12:14:32 -0400 |
commit | 1684a984303abbfc39aa8b59b0fe825c717811a9 (patch) | |
tree | 5ba646c7c81f1ed3a2bbc032a412dcff21fc7c37 /drivers/media/video | |
parent | c7a46533ff7ef9e1c51bae6e54208527c5275b24 (diff) |
[PATCH] I2C: Kill i2c_algorithm.id (6/7)
In theory, there should be no more users of I2C_ALGO_* at this point.
However, it happens that several drivers were using I2C_ALGO_* for
adapter ids, so we need to correct these before we can get rid of all
the I2C_ALGO_* definitions.
Note that this also fixes a bug in media/video/tvaudio.c:
/* don't attach on saa7146 based cards,
because dedicated drivers are used */
if ((adap->id & I2C_ALGO_SAA7146))
return 0;
This test was plain broken, as it would succeed for many more adapters
than just the saa7146: any those id would share at least one bit with
the saa7146 id. We are really lucky that the few other adapters we want
this driver to work with did not fulfill that condition.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/media/video')
-rw-r--r-- | drivers/media/video/ir-kbd-i2c.c | 2 | ||||
-rw-r--r-- | drivers/media/video/saa7134/saa7134-i2c.c | 2 | ||||
-rw-r--r-- | drivers/media/video/tda9840.c | 2 | ||||
-rw-r--r-- | drivers/media/video/tda9887.c | 2 | ||||
-rw-r--r-- | drivers/media/video/tea6415c.c | 2 | ||||
-rw-r--r-- | drivers/media/video/tea6420.c | 2 | ||||
-rw-r--r-- | drivers/media/video/tvaudio.c | 4 |
7 files changed, 8 insertions, 8 deletions
diff --git a/drivers/media/video/ir-kbd-i2c.c b/drivers/media/video/ir-kbd-i2c.c index c2f32d522668..feccf08bc0e3 100644 --- a/drivers/media/video/ir-kbd-i2c.c +++ b/drivers/media/video/ir-kbd-i2c.c | |||
@@ -432,7 +432,7 @@ static int ir_probe(struct i2c_adapter *adap) | |||
432 | case I2C_HW_B_BT848: | 432 | case I2C_HW_B_BT848: |
433 | probe = probe_bttv; | 433 | probe = probe_bttv; |
434 | break; | 434 | break; |
435 | case I2C_ALGO_SAA7134: | 435 | case I2C_HW_SAA7134: |
436 | probe = probe_saa7134; | 436 | probe = probe_saa7134; |
437 | break; | 437 | break; |
438 | } | 438 | } |
diff --git a/drivers/media/video/saa7134/saa7134-i2c.c b/drivers/media/video/saa7134/saa7134-i2c.c index 0bb1073d94bd..238ac3fdeb4a 100644 --- a/drivers/media/video/saa7134/saa7134-i2c.c +++ b/drivers/media/video/saa7134/saa7134-i2c.c | |||
@@ -381,7 +381,7 @@ static struct i2c_adapter saa7134_adap_template = { | |||
381 | .class = I2C_CLASS_TV_ANALOG, | 381 | .class = I2C_CLASS_TV_ANALOG, |
382 | #endif | 382 | #endif |
383 | I2C_DEVNAME("saa7134"), | 383 | I2C_DEVNAME("saa7134"), |
384 | .id = I2C_ALGO_SAA7134, | 384 | .id = I2C_HW_SAA7134, |
385 | .algo = &saa7134_algo, | 385 | .algo = &saa7134_algo, |
386 | .client_register = attach_inform, | 386 | .client_register = attach_inform, |
387 | }; | 387 | }; |
diff --git a/drivers/media/video/tda9840.c b/drivers/media/video/tda9840.c index c29bdfc3244e..c0594e09d63e 100644 --- a/drivers/media/video/tda9840.c +++ b/drivers/media/video/tda9840.c | |||
@@ -205,7 +205,7 @@ static int detect(struct i2c_adapter *adapter, int address, int kind) | |||
205 | static int attach(struct i2c_adapter *adapter) | 205 | static int attach(struct i2c_adapter *adapter) |
206 | { | 206 | { |
207 | /* let's see whether this is a know adapter we can attach to */ | 207 | /* let's see whether this is a know adapter we can attach to */ |
208 | if (adapter->id != I2C_ALGO_SAA7146) { | 208 | if (adapter->id != I2C_HW_SAA7146) { |
209 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); | 209 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); |
210 | return -ENODEV; | 210 | return -ENODEV; |
211 | } | 211 | } |
diff --git a/drivers/media/video/tda9887.c b/drivers/media/video/tda9887.c index f1b1bb5acf70..abb96ce464c7 100644 --- a/drivers/media/video/tda9887.c +++ b/drivers/media/video/tda9887.c | |||
@@ -620,7 +620,7 @@ static int tda9887_probe(struct i2c_adapter *adap) | |||
620 | switch (adap->id) { | 620 | switch (adap->id) { |
621 | case I2C_HW_B_BT848: | 621 | case I2C_HW_B_BT848: |
622 | case I2C_HW_B_RIVA: | 622 | case I2C_HW_B_RIVA: |
623 | case I2C_ALGO_SAA7134: | 623 | case I2C_HW_SAA7134: |
624 | return i2c_probe(adap, &addr_data, tda9887_attach); | 624 | return i2c_probe(adap, &addr_data, tda9887_attach); |
625 | break; | 625 | break; |
626 | } | 626 | } |
diff --git a/drivers/media/video/tea6415c.c b/drivers/media/video/tea6415c.c index b44db8a7b94d..8334d6ccd0a0 100644 --- a/drivers/media/video/tea6415c.c +++ b/drivers/media/video/tea6415c.c | |||
@@ -86,7 +86,7 @@ static int detect(struct i2c_adapter *adapter, int address, int kind) | |||
86 | static int attach(struct i2c_adapter *adapter) | 86 | static int attach(struct i2c_adapter *adapter) |
87 | { | 87 | { |
88 | /* let's see whether this is a know adapter we can attach to */ | 88 | /* let's see whether this is a know adapter we can attach to */ |
89 | if (adapter->id != I2C_ALGO_SAA7146) { | 89 | if (adapter->id != I2C_HW_SAA7146) { |
90 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); | 90 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); |
91 | return -ENODEV; | 91 | return -ENODEV; |
92 | } | 92 | } |
diff --git a/drivers/media/video/tea6420.c b/drivers/media/video/tea6420.c index 48d4db7d507b..9d09d2d23c2e 100644 --- a/drivers/media/video/tea6420.c +++ b/drivers/media/video/tea6420.c | |||
@@ -135,7 +135,7 @@ static int tea6420_detect(struct i2c_adapter *adapter, int address, int kind) | |||
135 | static int attach(struct i2c_adapter *adapter) | 135 | static int attach(struct i2c_adapter *adapter) |
136 | { | 136 | { |
137 | /* let's see whether this is a know adapter we can attach to */ | 137 | /* let's see whether this is a know adapter we can attach to */ |
138 | if (adapter->id != I2C_ALGO_SAA7146) { | 138 | if (adapter->id != I2C_HW_SAA7146) { |
139 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); | 139 | dprintk("refusing to probe on unknown adapter [name='%s',id=0x%x]\n", adapter->name, adapter->id); |
140 | return -ENODEV; | 140 | return -ENODEV; |
141 | } | 141 | } |
diff --git a/drivers/media/video/tvaudio.c b/drivers/media/video/tvaudio.c index 9420b5f52915..f6aab89593ea 100644 --- a/drivers/media/video/tvaudio.c +++ b/drivers/media/video/tvaudio.c | |||
@@ -1548,7 +1548,7 @@ static int chip_probe(struct i2c_adapter *adap) | |||
1548 | { | 1548 | { |
1549 | /* don't attach on saa7146 based cards, | 1549 | /* don't attach on saa7146 based cards, |
1550 | because dedicated drivers are used */ | 1550 | because dedicated drivers are used */ |
1551 | if ((adap->id & I2C_ALGO_SAA7146)) | 1551 | if (adap->id == I2C_HW_SAA7146) |
1552 | return 0; | 1552 | return 0; |
1553 | #ifdef I2C_CLASS_TV_ANALOG | 1553 | #ifdef I2C_CLASS_TV_ANALOG |
1554 | if (adap->class & I2C_CLASS_TV_ANALOG) | 1554 | if (adap->class & I2C_CLASS_TV_ANALOG) |
@@ -1557,7 +1557,7 @@ static int chip_probe(struct i2c_adapter *adap) | |||
1557 | switch (adap->id) { | 1557 | switch (adap->id) { |
1558 | case I2C_HW_B_BT848: | 1558 | case I2C_HW_B_BT848: |
1559 | case I2C_HW_B_RIVA: | 1559 | case I2C_HW_B_RIVA: |
1560 | case I2C_ALGO_SAA7134: | 1560 | case I2C_HW_SAA7134: |
1561 | return i2c_probe(adap, &addr_data, chip_attach); | 1561 | return i2c_probe(adap, &addr_data, chip_attach); |
1562 | } | 1562 | } |
1563 | #endif | 1563 | #endif |