* [PATCH 22/31] i2c: Auto-load i2c_dev module when device opened.
[not found] ` <cover.1236702228.git.scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
@ 2009-03-02 18:41 ` Scott James Remnant
[not found] ` <c9e8ec654db2ee34fc9fda29222aee9dd085ddaa.1236702228.git.scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Scott James Remnant @ 2009-03-02 18:41 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-i2c-u79uwXL29TY76Z2rM5mHXA
The i2c_dev module is missing the char-major-89-* alias that would cause
it to be auto-loaded when a device of that type is opened. This patch
adds the alias.
Signed-off-by: Scott James Remnant <scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
Signed-off-by: Tim Gardner <tim.gardner-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
---
drivers/i2c/i2c-dev.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/i2c/i2c-dev.c b/drivers/i2c/i2c-dev.c
index 7e13d2d..f3f83ba 100644
--- a/drivers/i2c/i2c-dev.c
+++ b/drivers/i2c/i2c-dev.c
@@ -618,6 +618,7 @@ MODULE_AUTHOR("Frodo Looijaard <frodol-B0qZmFHriGg@public.gmane.org> and "
"Simon G. Vogl <simon-nD9nYVNVf00W+b/DJNNodF6hYfS7NtTn@public.gmane.org>");
MODULE_DESCRIPTION("I2C /dev entries driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS_CHARDEV_MAJOR(I2C_MAJOR);
module_init(i2c_dev_init);
module_exit(i2c_dev_exit);
--
1.6.0.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 22/31] i2c: Auto-load i2c_dev module when device opened.
[not found] ` <c9e8ec654db2ee34fc9fda29222aee9dd085ddaa.1236702228.git.scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
@ 2009-03-10 16:56 ` Jean Delvare
[not found] ` <20090310175612.7e5b8272-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Jean Delvare @ 2009-03-10 16:56 UTC (permalink / raw)
To: Scott James Remnant
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA
Scott,
Your clock is off by a significant number of days, isn't it?
On Mon, 2 Mar 2009 18:41:51 +0000, Scott James Remnant wrote:
> The i2c_dev module is missing the char-major-89-* alias that would cause
> it to be auto-loaded when a device of that type is opened. This patch
> adds the alias.
Why not, however this is pretty pointless nowadays as /dev nodes tend
to be created dynamically by udev, so there is no way i2c-dev will
auto-load.
> Signed-off-by: Scott James Remnant <scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
> Signed-off-by: Tim Gardner <tim.gardner-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
> ---
> drivers/i2c/i2c-dev.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/i2c/i2c-dev.c b/drivers/i2c/i2c-dev.c
> index 7e13d2d..f3f83ba 100644
> --- a/drivers/i2c/i2c-dev.c
> +++ b/drivers/i2c/i2c-dev.c
> @@ -618,6 +618,7 @@ MODULE_AUTHOR("Frodo Looijaard <frodol-B0qZmFHriGg@public.gmane.org> and "
> "Simon G. Vogl <simon-nD9nYVNVf00W+b/DJNNodF6hYfS7NtTn@public.gmane.org>");
> MODULE_DESCRIPTION("I2C /dev entries driver");
> MODULE_LICENSE("GPL");
> +MODULE_ALIAS_CHARDEV_MAJOR(I2C_MAJOR);
>
> module_init(i2c_dev_init);
> module_exit(i2c_dev_exit);
--
Jean Delvare
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 22/31] i2c: Auto-load i2c_dev module when device opened.
[not found] ` <20090310175612.7e5b8272-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
@ 2009-03-10 17:00 ` Scott James Remnant
0 siblings, 0 replies; 3+ messages in thread
From: Scott James Remnant @ 2009-03-10 17:00 UTC (permalink / raw)
To: Jean Delvare
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 382 bytes --]
On Tue, 2009-03-10 at 17:56 +0100, Jean Delvare wrote:
> Your clock is off by a significant number of days, isn't it?
>
No, I've just been carefully testing them - and then waited until they
were in the jaunty kernel and a couple of thousand other users
not-so-carefully tested them too <g>
Scott
--
Scott James Remnant
scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-03-10 17:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1236702228.git.scott@canonical.com>
[not found] ` <cover.1236702228.git.scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
2009-03-02 18:41 ` [PATCH 22/31] i2c: Auto-load i2c_dev module when device opened Scott James Remnant
[not found] ` <c9e8ec654db2ee34fc9fda29222aee9dd085ddaa.1236702228.git.scott-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>
2009-03-10 16:56 ` Jean Delvare
[not found] ` <20090310175612.7e5b8272-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2009-03-10 17:00 ` Scott James Remnant
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox