diff options
author | David Brownell <david-b@pacbell.net> | 2007-05-06 17:48:42 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-07 15:12:50 -0400 |
commit | 02c83595b86480ee4d61665beb13f76685d40239 (patch) | |
tree | 5bd0dd2352a5d9c0df260d0eaa5a84a4ea861d4c | |
parent | fd76bab2fa6d8f3ef6b326a4c6ae442fa21d30a4 (diff) |
at91_cf, minor fix
This is a minor correctness fix: since the at91_cf driver probe() routine
is in the init section, it should use platform_driver_probe() instead of
leaving that pointer around in the driver struct after init section
removal.
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Cc: Dominik Brodowski <linux@dominikbrodowski.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/pcmcia/at91_cf.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pcmcia/at91_cf.c b/drivers/pcmcia/at91_cf.c index 99baabc23599..948efc775a78 100644 --- a/drivers/pcmcia/at91_cf.c +++ b/drivers/pcmcia/at91_cf.c | |||
@@ -360,7 +360,6 @@ static struct platform_driver at91_cf_driver = { | |||
360 | .name = (char *) driver_name, | 360 | .name = (char *) driver_name, |
361 | .owner = THIS_MODULE, | 361 | .owner = THIS_MODULE, |
362 | }, | 362 | }, |
363 | .probe = at91_cf_probe, | ||
364 | .remove = __exit_p(at91_cf_remove), | 363 | .remove = __exit_p(at91_cf_remove), |
365 | .suspend = at91_cf_suspend, | 364 | .suspend = at91_cf_suspend, |
366 | .resume = at91_cf_resume, | 365 | .resume = at91_cf_resume, |
@@ -370,7 +369,7 @@ static struct platform_driver at91_cf_driver = { | |||
370 | 369 | ||
371 | static int __init at91_cf_init(void) | 370 | static int __init at91_cf_init(void) |
372 | { | 371 | { |
373 | return platform_driver_register(&at91_cf_driver); | 372 | return platform_driver_probe(&at91_cf_driver, at91_cf_probe); |
374 | } | 373 | } |
375 | module_init(at91_cf_init); | 374 | module_init(at91_cf_init); |
376 | 375 | ||