Commit acebc70d authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

V4L/DVB (11372): v4l2: use old-style i2c API for kernels < 2.6.26 instead of < 2.6.22

Originally the intention was to switch to the new style i2c API starting with
the introduction of the API in 2.6.22. However, the i2c_new_probed_device()
function has a lethal bug that wasn't fixed until 2.6.25. Or more accurately,
it was only fixed in the stable series of 2.6.25 and 2.6.26.

Given the fact that the new i2c API also changed starting with 2.6.26 (the
addition of i2c_device_id), it is easiest to switch APIs starting with
2.6.26.

This patch updates all the legacy code accordingly.
Signed-off-by: default avatarHans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 0c846743
...@@ -192,7 +192,6 @@ static int tda9840_remove(struct i2c_client *client) ...@@ -192,7 +192,6 @@ static int tda9840_remove(struct i2c_client *client)
return 0; return 0;
} }
static const struct i2c_device_id tda9840_id[] = { static const struct i2c_device_id tda9840_id[] = {
{ "tda9840", 0 }, { "tda9840", 0 },
{ } { }
......
...@@ -170,7 +170,6 @@ static int tea6415c_remove(struct i2c_client *client) ...@@ -170,7 +170,6 @@ static int tea6415c_remove(struct i2c_client *client)
return 0; return 0;
} }
static const struct i2c_device_id tea6415c_id[] = { static const struct i2c_device_id tea6415c_id[] = {
{ "tea6415c", 0 }, { "tea6415c", 0 },
{ } { }
......
...@@ -156,7 +156,6 @@ static int tea6420_remove(struct i2c_client *client) ...@@ -156,7 +156,6 @@ static int tea6420_remove(struct i2c_client *client)
return 0; return 0;
} }
static const struct i2c_device_id tea6420_id[] = { static const struct i2c_device_id tea6420_id[] = {
{ "tea6420", 0 }, { "tea6420", 0 },
{ } { }
......
...@@ -1164,7 +1164,6 @@ static int tuner_probe(struct i2c_client *client, ...@@ -1164,7 +1164,6 @@ static int tuner_probe(struct i2c_client *client,
return 0; return 0;
} }
static int tuner_remove(struct i2c_client *client) static int tuner_remove(struct i2c_client *client)
{ {
struct tuner *t = to_tuner(i2c_get_clientdata(client)); struct tuner *t = to_tuner(i2c_get_clientdata(client));
......
...@@ -2069,7 +2069,6 @@ static int tvaudio_remove(struct i2c_client *client) ...@@ -2069,7 +2069,6 @@ static int tvaudio_remove(struct i2c_client *client)
return 0; return 0;
} }
/* This driver supports many devices and the idea is to let the driver /* This driver supports many devices and the idea is to let the driver
detect which device is present. So rather than listing all supported detect which device is present. So rather than listing all supported
devices here, we pretend to support a single, fake device type. */ devices here, we pretend to support a single, fake device type. */
......
...@@ -51,7 +51,7 @@ static struct v4l2_i2c_driver_data v4l2_i2c_data; ...@@ -51,7 +51,7 @@ static struct v4l2_i2c_driver_data v4l2_i2c_data;
static struct i2c_driver v4l2_i2c_driver; static struct i2c_driver v4l2_i2c_driver;
/* Bus-based I2C implementation for kernels >= 2.6.22 */ /* Bus-based I2C implementation for kernels >= 2.6.26 */
static int __init v4l2_i2c_drv_init(void) static int __init v4l2_i2c_drv_init(void)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment