Hi, i2c_add_driver() may actually fail, but this driver returns 0 regardless. Against 2.6.7. Thanks. Signed-off-by: Arthur Othieno <a.othieno at bluewin.ch> tda9887.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-) --- a/drivers/media/video/tda9887.c 2004-06-17 17:29:45.000000000 +0200 +++ b/drivers/media/video/tda9887.c 2004-06-17 17:31:21.000000000 +0200 @@ -458,13 +458,12 @@ static struct i2c_client client_template .driver = &driver, }; -static int tda9887_init_module(void) +static int __init tda9887_init_module(void) { - i2c_add_driver(&driver); - return 0; + return i2c_add_driver(&driver); } -static void tda9887_cleanup_module(void) +static void __exit tda9887_cleanup_module(void) { i2c_del_driver(&driver); }