aboutsummaryrefslogtreecommitdiffstats
path: root/Documentation/i2c
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2005-11-26 14:28:06 -0500
committerGreg Kroah-Hartman <gregkh@suse.de>2006-01-06 01:16:21 -0500
commit8a9947552d43b0d20d5fa23ac0ba435d526be454 (patch)
tree086844a62f549e488ebeecc196e1d4726a1d4598 /Documentation/i2c
parentff179c8cf5caa17bf3d407edbb5872aa2eee6900 (diff)
[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>
Diffstat (limited to 'Documentation/i2c')
-rw-r--r--Documentation/i2c/porting-clients3
-rw-r--r--Documentation/i2c/writing-clients5
2 files changed, 3 insertions, 5 deletions
diff --git a/Documentation/i2c/porting-clients b/Documentation/i2c/porting-clients
index 184fac2377aa..64c610bf2fbc 100644
--- a/Documentation/i2c/porting-clients
+++ b/Documentation/i2c/porting-clients
@@ -109,6 +109,9 @@ Technical changes:
109 there is a MODULE_LICENSE() line, at the bottom of the file 109 there is a MODULE_LICENSE() line, at the bottom of the file
110 (after MODULE_AUTHOR() and MODULE_DESCRIPTION(), in this order). 110 (after MODULE_AUTHOR() and MODULE_DESCRIPTION(), in this order).
111 111
112* [Driver] The flags field of the i2c_driver structure is gone.
113 I2C_DF_NOTIFY is now the default behavior.
114
112Coding policy: 115Coding policy:
113 116
114* [Copyright] Use (C), not (c), for copyright. 117* [Copyright] Use (C), not (c), for copyright.
diff --git a/Documentation/i2c/writing-clients b/Documentation/i2c/writing-clients
index d19993cc0604..59d2c169cd61 100644
--- a/Documentation/i2c/writing-clients
+++ b/Documentation/i2c/writing-clients
@@ -27,7 +27,6 @@ address.
27static struct i2c_driver foo_driver = { 27static struct i2c_driver foo_driver = {
28 .owner = THIS_MODULE, 28 .owner = THIS_MODULE,
29 .name = "Foo version 2.3 driver", 29 .name = "Foo version 2.3 driver",
30 .flags = I2C_DF_NOTIFY,
31 .attach_adapter = &foo_attach_adapter, 30 .attach_adapter = &foo_attach_adapter,
32 .detach_client = &foo_detach_client, 31 .detach_client = &foo_detach_client,
33 .command = &foo_command /* may be NULL */ 32 .command = &foo_command /* may be NULL */
@@ -36,10 +35,6 @@ static struct i2c_driver foo_driver = {
36The name field must match the driver name, including the case. It must not 35The name field must match the driver name, including the case. It must not
37contain spaces, and may be up to 31 characters long. 36contain spaces, and may be up to 31 characters long.
38 37
39Don't worry about the flags field; just put I2C_DF_NOTIFY into it. This
40means that your driver will be notified when new adapters are found.
41This is almost always what you want.
42
43All other fields are for call-back functions which will be explained 38All other fields are for call-back functions which will be explained
44below. 39below.
45 40