All of lore.kernel.org
 help / color / mirror / Atom feed
* [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init
@ 2007-02-16  0:03 David Brownell
  2007-02-16  6:35 ` Jean Delvare
  2007-02-16  6:39 ` David Brownell
  0 siblings, 2 replies; 3+ messages in thread
From: David Brownell @ 2007-02-16  0:03 UTC (permalink / raw)
  To: lm-sensors

The hwmon subsystem registers itself excessively late when it's not linked
as a module.  That can cause problems when multifunction chips, that happen
to have hwmon capabilities and live outside the hwmon subdirectory, need to
initialize earlier than on various platforms.

This patch makes the hwmon subsystem code register itself appropriately early:
as a subsystem, like other subsystems.

Signed-off-by: David Brownell <dbrownell at users.sourceforge.net>

Index: osk/drivers/hwmon/hwmon.c
=================================--- osk.orig/drivers/hwmon/hwmon.c	2006-05-02 19:18:31.000000000 -0700
+++ osk/drivers/hwmon/hwmon.c	2006-05-02 19:18:46.000000000 -0700
@@ -102,7 +102,7 @@ static void __exit hwmon_exit(void)
 	class_destroy(hwmon_class);
 }
 
-module_init(hwmon_init);
+subsys_initcall(hwmon_init);
 module_exit(hwmon_exit);
 
 EXPORT_SYMBOL_GPL(hwmon_device_register);


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init
  2007-02-16  0:03 [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init David Brownell
@ 2007-02-16  6:35 ` Jean Delvare
  2007-02-16  6:39 ` David Brownell
  1 sibling, 0 replies; 3+ messages in thread
From: Jean Delvare @ 2007-02-16  6:35 UTC (permalink / raw)
  To: lm-sensors

Hi David,

On Thu, 15 Feb 2007 16:03:13 -0800, David Brownell wrote:
> The hwmon subsystem registers itself excessively late when it's not linked
> as a module.  That can cause problems when multifunction chips, that happen
> to have hwmon capabilities and live outside the hwmon subdirectory, need to
> initialize earlier than on various platforms.
> 
> This patch makes the hwmon subsystem code register itself appropriately early:
> as a subsystem, like other subsystems.

This patch is already in my tree and planned for inclusion in kernel
2.6.21. I asked Linus to pull the hwmon updates (including this one)
from my tree two days ago but he did not yet. I'll ask him again
tomorrow if needed.

Thanks,
-- 
Jean Delvare


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init
  2007-02-16  0:03 [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init David Brownell
  2007-02-16  6:35 ` Jean Delvare
@ 2007-02-16  6:39 ` David Brownell
  1 sibling, 0 replies; 3+ messages in thread
From: David Brownell @ 2007-02-16  6:39 UTC (permalink / raw)
  To: lm-sensors

> > This patch makes the hwmon subsystem code register itself appropriately early:
> > as a subsystem, like other subsystems.
>
> This patch is already in my tree and planned for inclusion in kernel
> 2.6.21.

OK ... I thought I'd already sent that along, but didn't have a
note that I'd done so.


>	I asked Linus to pull the hwmon updates (including this one)
> from my tree two days ago but he did not yet. I'll ask him again
> tomorrow if needed.

He has to pull ARM patches too.  Right now ARM can't build, so I
can't test the latest I2C stuff ... ;)

- Dave



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-02-16  6:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-02-16  0:03 [lm-sensors] [patch 2.6.20-git] hwmon should use subsys_init David Brownell
2007-02-16  6:35 ` Jean Delvare
2007-02-16  6:39 ` David Brownell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.