lkml.org 
[lkml]   [2004]   [Sep]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH 2.4] Update Documentation/i2c/writing-clients
Hi Marcelo,

This is a quick update to the Documentation/i2c/writing-clients file. A
similar change was accepted by Greg KH in 2.6 and was also applied to
the i2c CVS repository.

The changes are about i2c client driver IDs. It used to say that chip
driver writers should ask for a unique ID. It now explains that such an
ID is not required and they can go without it. The patch additionally
features CodingStyle updates.

Fell free to apply it if you want,
thanks.

Signed-off-by: Jean Delvare <khali@linux-fr.org>

--- linux-2.4.28-pre3/Documentation/i2c/writing-clients.orig 2004-09-14 21:46:41.000000000 +0200
+++ linux-2.4.28-pre3/Documentation/i2c/writing-clients 2004-09-14 22:15:14.000000000 +0200
@@ -24,24 +24,25 @@
routines, a client structure specific information like the actual I2C
address.

- struct i2c_driver foo_driver
- {
- /* name */ "Foo version 2.3 and later driver",
- /* id */ I2C_DRIVERID_FOO,
- /* flags */ I2C_DF_NOTIFY,
- /* attach_adapter */ &foo_attach_adapter,
- /* detach_client */ &foo_detach_client,
- /* command */ &foo_command, /* May be NULL */
- /* inc_use */ &foo_inc_use, /* May be NULL */
- /* dec_use */ &foo_dec_use /* May be NULL */
- }
+static struct i2c_driver foo_driver = {
+ .name = "Foo version 2.3 driver",
+ .id = I2C_DRIVERID_FOO, /* from i2c-id.h, optional */
+ .flags = I2C_DF_NOTIFY,
+ .attach_adapter = &foo_attach_adapter,
+ .detach_client = &foo_detach_client,
+ .command = &foo_command, /* may be NULL */
+ .inc_use = &foo_inc_use, /* May be NULL */
+ .dec_use = &foo_dec_use, /* May be NULL */
+}

The name can be chosen freely, and may be upto 40 characters long. Please
use something descriptive here.

-The id should be a unique ID. The range 0xf000 to 0xffff is reserved for
-local use, and you can use one of those until you start distributing the
-driver. Before you do that, contact the i2c authors to get your own ID(s).
+If used, the id should be a unique ID. The range 0xf000 to 0xffff is
+reserved for local use, and you can use one of those until you start
+distributing the driver, at which time you should contact the i2c authors
+to get your own ID(s). Note that most of the time you don't need an ID
+at all so you can just omit it.

Don't worry about the flags field; just put I2C_DF_NOTIFY into it. This
means that your driver will be notified when new adapters are found.

--
Jean "Khali" Delvare
http://khali.linux-fr.org/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:06    [W:0.107 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site