[PATCH] i2c: Drop i2c_driver.flags, 2 of 3

Just about every i2c chip driver sets the I2C_DF_NOTIFY flag, so we
can simply make it the default and drop the flag. If any driver really
doesn't want to be notified when i2c adapters are added, that driver
can simply omit to set .attach_adapter. This approach is also more
robust as it prevents accidental NULL pointer dereferences.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

3 files changed, 0 insertions, 3 deletions

diff --git a/sound/oss/dmasound/dac3550a.c b/sound/oss/dmasound/dac3550a.c
index 533895eba0eb..3402a663d07f 100644
--- a/sound/oss/dmasound/dac3550a.c
+++ b/sound/oss/dmasound/dac3550a.c
@@ -44,7 +44,6 @@ struct i2c_driver daca_driver = {
         .owner                  = THIS_MODULE,
         .name                   = "DAC3550A driver  V " DACA_VERSION,
         .id                     = I2C_DRIVERID_DACA,
-        .flags                  = I2C_DF_NOTIFY,
         .attach_adapter         = daca_attach_adapter,
         .detach_client          = daca_detach_client,
 };
diff --git a/sound/oss/dmasound/tas_common.c b/sound/oss/dmasound/tas_common.c
index d36a1fe2fcf3..7e3d517af4b9 100644
--- a/sound/oss/dmasound/tas_common.c
+++ b/sound/oss/dmasound/tas_common.c
@@ -49,7 +49,6 @@ static int tas_detach_client(struct i2c_client *);
 struct i2c_driver tas_driver = {
         .owner          = THIS_MODULE,
         .name           = "tas",
-        .flags          = I2C_DF_NOTIFY,
         .attach_adapter = tas_attach_adapter,
         .detach_client  = tas_detach_client,
 };
diff --git a/sound/ppc/keywest.c b/sound/ppc/keywest.c
index 097fbcfc5d45..fd8e2e6062f6 100644
--- a/sound/ppc/keywest.c
+++ b/sound/ppc/keywest.c
@@ -43,7 +43,6 @@ static int keywest_detach_client(struct i2c_client *client);
 struct i2c_driver keywest_driver = {  
         .name = "PMac Keywest Audio",
         .id = I2C_DRIVERID_KEYWEST,
-        .flags = I2C_DF_NOTIFY,
         .attach_adapter = &keywest_attach_adapter,
         .detach_client = &keywest_detach_client,
 };