* [patch 2.6.25-rc8] leds: fix platform driver hotplug/coldplug
@ 2008-04-11 17:38 David Brownell
2008-04-12 4:54 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: David Brownell @ 2008-04-11 17:38 UTC (permalink / raw)
To: Richard Purdie, lkml; +Cc: Andrew Morton, Kay Sievers
From: Kay Sievers <kay.sievers@vrfy.org>
Since 43cc71eed1250755986da4c0f9898f9a635cb3bf, the platform
modalias is prefixed with "platform:". Add MODULE_ALIAS() to the
hotpluggable platform LED drivers, to re-enable auto loading.
[ dbrownell@users.sourceforge.net: more drivers, registration fixes ]
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
---
drivers/leds/leds-ams-delta.c | 2 ++
drivers/leds/leds-atmel-pwm.c | 3 +++
drivers/leds/leds-clevo-mail.c | 1 +
drivers/leds/leds-cm-x270.c | 2 ++
drivers/leds/leds-cobalt-qube.c | 3 +++
drivers/leds/leds-corgi.c | 2 ++
drivers/leds/leds-gpio.c | 1 +
drivers/leds/leds-h1940.c | 2 ++
drivers/leds/leds-hp6xx.c | 4 ++++
drivers/leds/leds-s3c24xx.c | 1 +
drivers/leds/leds-spitz.c | 2 ++
drivers/leds/leds-tosa.c | 2 ++
12 files changed, 25 insertions(+)
--- g26.orig/drivers/leds/leds-ams-delta.c 2008-04-11 10:22:44.000000000 -0700
+++ g26/drivers/leds/leds-ams-delta.c 2008-04-11 10:26:21.000000000 -0700
@@ -140,6 +140,7 @@ static struct platform_driver ams_delta_
.resume = ams_delta_led_resume,
.driver = {
.name = "ams-delta-led",
+ .owner = THIS_MODULE,
},
};
@@ -159,3 +160,4 @@ module_exit(ams_delta_led_exit);
MODULE_AUTHOR("Jonathan McDowell <noodles@earth.li>");
MODULE_DESCRIPTION("Amstrad Delta LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:ams-delta-led");
--- g26.orig/drivers/leds/leds-cm-x270.c 2008-04-11 10:22:43.000000000 -0700
+++ g26/drivers/leds/leds-cm-x270.c 2008-04-11 10:28:11.000000000 -0700
@@ -101,6 +101,7 @@ static struct platform_driver cmx270led_
#endif
.driver = {
.name = "cm-x270-led",
+ .owner = THIS_MODULE,
},
};
@@ -120,3 +121,4 @@ module_exit(cmx270led_exit);
MODULE_AUTHOR("Mike Rapoport <mike@compulab.co.il>");
MODULE_DESCRIPTION("CM-x270 LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:cm-x270-led");
--- g26.orig/drivers/leds/leds-corgi.c 2008-04-11 10:22:43.000000000 -0700
+++ g26/drivers/leds/leds-corgi.c 2008-04-11 10:29:14.000000000 -0700
@@ -99,6 +99,7 @@ static struct platform_driver corgiled_d
#endif
.driver = {
.name = "corgi-led",
+ .owner = THIS_MODULE,
},
};
@@ -118,3 +119,4 @@ module_exit(corgiled_exit);
MODULE_AUTHOR("Richard Purdie <rpurdie@openedhand.com>");
MODULE_DESCRIPTION("Corgi LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:corgi-led");
--- g26.orig/drivers/leds/leds-gpio.c 2008-04-11 10:22:43.000000000 -0700
+++ g26/drivers/leds/leds-gpio.c 2008-04-11 10:25:27.000000000 -0700
@@ -199,3 +199,4 @@ module_exit(gpio_led_exit);
MODULE_AUTHOR("Raphael Assenat <raph@8d.com>");
MODULE_DESCRIPTION("GPIO LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:leds-gpio");
--- g26.orig/drivers/leds/leds-h1940.c 2008-04-11 10:22:44.000000000 -0700
+++ g26/drivers/leds/leds-h1940.c 2008-04-11 10:29:43.000000000 -0700
@@ -139,6 +139,7 @@ static int h1940leds_remove(struct platf
static struct platform_driver h1940leds_driver = {
.driver = {
.name = "h1940-leds",
+ .owner = THIS_MODULE,
},
.probe = h1940leds_probe,
.remove = h1940leds_remove,
@@ -161,3 +162,4 @@ module_exit(h1940leds_exit);
MODULE_AUTHOR("Arnaud Patard <arnaud.patard@rtp-net.org>");
MODULE_DESCRIPTION("LED driver for the iPAQ H1940");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:h1940-leds");
--- g26.orig/drivers/leds/leds-s3c24xx.c 2008-04-11 10:22:44.000000000 -0700
+++ g26/drivers/leds/leds-s3c24xx.c 2008-04-11 10:25:28.000000000 -0700
@@ -160,3 +160,4 @@ module_exit(s3c24xx_led_exit);
MODULE_AUTHOR("Ben Dooks <ben@simtec.co.uk>");
MODULE_DESCRIPTION("S3C24XX LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:s3c24xx_led");
--- g26.orig/drivers/leds/leds-spitz.c 2008-04-11 10:22:44.000000000 -0700
+++ g26/drivers/leds/leds-spitz.c 2008-04-11 10:30:29.000000000 -0700
@@ -105,6 +105,7 @@ static struct platform_driver spitzled_d
#endif
.driver = {
.name = "spitz-led",
+ .owner = THIS_MODULE,
},
};
@@ -124,3 +125,4 @@ module_exit(spitzled_exit);
MODULE_AUTHOR("Richard Purdie <rpurdie@openedhand.com>");
MODULE_DESCRIPTION("Spitz LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:spitz-led");
--- g26.orig/drivers/leds/leds-tosa.c 2008-04-11 10:22:44.000000000 -0700
+++ g26/drivers/leds/leds-tosa.c 2008-04-11 10:30:39.000000000 -0700
@@ -109,6 +109,7 @@ static struct platform_driver tosaled_dr
.resume = tosaled_resume,
.driver = {
.name = "tosa-led",
+ .owner = THIS_MODULE,
},
};
@@ -128,3 +129,4 @@ module_exit(tosaled_exit);
MODULE_AUTHOR("Dirk Opfer <Dirk@Opfer-Online.de>");
MODULE_DESCRIPTION("Tosa LED driver");
MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:tosa-led");
--- g26.orig/drivers/leds/leds-atmel-pwm.c 2008-04-11 10:25:34.000000000 -0700
+++ g26/drivers/leds/leds-atmel-pwm.c 2008-04-11 10:27:12.000000000 -0700
@@ -132,6 +132,9 @@ static int __exit pwmled_remove(struct p
return 0;
}
+/* work with hotplug and coldplug */
+MODULE_ALIAS("platform:leds-atmel-pwm");
+
static struct platform_driver pwmled_driver = {
.driver = {
.name = "leds-atmel-pwm",
--- g26.orig/drivers/leds/leds-clevo-mail.c 2008-04-11 10:25:34.000000000 -0700
+++ g26/drivers/leds/leds-clevo-mail.c 2008-04-11 10:27:54.000000000 -0700
@@ -170,6 +170,7 @@ static struct platform_driver clevo_mail
.resume = clevo_mail_led_resume,
.driver = {
.name = KBUILD_MODNAME,
+ .owner = THIS_MODULE,
},
};
--- g26.orig/drivers/leds/leds-cobalt-qube.c 2008-04-11 10:25:34.000000000 -0700
+++ g26/drivers/leds/leds-cobalt-qube.c 2008-04-11 10:28:33.000000000 -0700
@@ -75,6 +75,9 @@ static int __devexit cobalt_qube_led_rem
return 0;
}
+/* work with hotplug and coldplug */
+MODULE_ALIAS("platform:cobalt-qube-leds");
+
static struct platform_driver cobalt_qube_led_driver = {
.probe = cobalt_qube_led_probe,
.remove = __devexit_p(cobalt_qube_led_remove),
--- g26.orig/drivers/leds/leds-hp6xx.c 2008-04-11 10:25:34.000000000 -0700
+++ g26/drivers/leds/leds-hp6xx.c 2008-04-11 10:30:06.000000000 -0700
@@ -90,6 +90,9 @@ static int hp6xxled_remove(struct platfo
return 0;
}
+/* work with hotplug and coldplug */
+MODULE_ALIAS("platform:hp6xx-led");
+
static struct platform_driver hp6xxled_driver = {
.probe = hp6xxled_probe,
.remove = hp6xxled_remove,
@@ -99,6 +102,7 @@ static struct platform_driver hp6xxled_d
#endif
.driver = {
.name = "hp6xx-led",
+ .owner = THIS_MODULE,
},
};
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [patch 2.6.25-rc8] leds: fix platform driver hotplug/coldplug
2008-04-11 17:38 [patch 2.6.25-rc8] leds: fix platform driver hotplug/coldplug David Brownell
@ 2008-04-12 4:54 ` Andrew Morton
2008-04-12 8:13 ` Richard Purdie
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2008-04-12 4:54 UTC (permalink / raw)
To: David Brownell; +Cc: Richard Purdie, lkml, Kay Sievers
On Fri, 11 Apr 2008 10:38:34 -0700 David Brownell <david-b@pacbell.net> wrote:
> From: Kay Sievers <kay.sievers@vrfy.org>
>
> Since 43cc71eed1250755986da4c0f9898f9a635cb3bf, the platform
> modalias is prefixed with "platform:". Add MODULE_ALIAS() to the
> hotpluggable platform LED drivers, to re-enable auto loading.
Richard, I'm pushing as many of these into 2.6.25 as I reasonably can. A
swift ack would be good, please.
Of course, we can always go the backport-it-to-2.6.25.1 route as well.
Apparently these problems are also present in 2.6.24.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch 2.6.25-rc8] leds: fix platform driver hotplug/coldplug
2008-04-12 4:54 ` Andrew Morton
@ 2008-04-12 8:13 ` Richard Purdie
0 siblings, 0 replies; 3+ messages in thread
From: Richard Purdie @ 2008-04-12 8:13 UTC (permalink / raw)
To: Andrew Morton; +Cc: David Brownell, lkml, Kay Sievers
On Fri, 2008-04-11 at 21:54 -0700, Andrew Morton wrote:
> On Fri, 11 Apr 2008 10:38:34 -0700 David Brownell <david-b@pacbell.net> wrote:
>
> > From: Kay Sievers <kay.sievers@vrfy.org>
> >
> > Since 43cc71eed1250755986da4c0f9898f9a635cb3bf, the platform
> > modalias is prefixed with "platform:". Add MODULE_ALIAS() to the
> > hotpluggable platform LED drivers, to re-enable auto loading.
>
> Richard, I'm pushing as many of these into 2.6.25 as I reasonably can. A
> swift ack would be good, please.
>
> Of course, we can always go the backport-it-to-2.6.25.1 route as well.
> Apparently these problems are also present in 2.6.24.
Acked-by: Richard Purdie <rpurdie@rpsys.net>
Cheers,
Richard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-04-12 8:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-11 17:38 [patch 2.6.25-rc8] leds: fix platform driver hotplug/coldplug David Brownell
2008-04-12 4:54 ` Andrew Morton
2008-04-12 8:13 ` Richard Purdie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox