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> tda7432.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) --- a/drivers/media/video/tda7432.c 2004-06-17 17:29:45.000000000 +0200 +++ b/drivers/media/video/tda7432.c 2004-06-17 17:31:21.000000000 +0200 @@ -532,17 +532,17 @@ static struct i2c_client client_template .driver = &driver, }; -static int tda7432_init(void) +static int __init tda7432_init(void) { if ( (loudness < 0) || (loudness > 15) ) { printk(KERN_ERR "tda7432: loudness parameter must be between 0 and 15\n"); return -EINVAL; } - i2c_add_driver(&driver); - return 0; + + return i2c_add_driver(&driver); } -static void tda7432_fini(void) +static void __exit tda7432_fini(void) { i2c_del_driver(&driver); }