diff mbox

[28/38] don't use __exit_p to wrap ds1302_rtc_remove

Message ID 1259096853-18909-28-git-send-email-u.kleine-koenig@pengutronix.de
State Accepted, archived
Headers show

Commit Message

Uwe Kleine-König Nov. 24, 2009, 9:07 p.m. UTC
The function ds1302_rtc_remove is defined using __devexit, so don't use
__exit_p but __devexit_p to wrap it.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: Paul Mundt <lethal@linux-sh.org>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: linux-kernel@vger.kernel.org
Cc: Paul Gortmaker <p_gortmaker@yahoo.com>
Cc: rtc-linux@googlegroups.com
---
 drivers/rtc/rtc-ds1302.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Paul Mundt Nov. 25, 2009, 1:36 a.m. UTC | #1
On Tue, Nov 24, 2009 at 10:07:23PM +0100, Uwe Kleine-K??nig wrote:
> The function ds1302_rtc_remove is defined using __devexit, so don't use
> __exit_p but __devexit_p to wrap it.
> 
> Signed-off-by: Uwe Kleine-K??nig <u.kleine-koenig@pengutronix.de>
> Cc: Paul Mundt <lethal@linux-sh.org>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: linux-kernel@vger.kernel.org
> Cc: Paul Gortmaker <p_gortmaker@yahoo.com>
> Cc: rtc-linux@googlegroups.com

On Tue, Nov 24, 2009 at 10:07:32PM +0100, Uwe Kleine-K??nig wrote:
> The function sci_remove is defined without any section modifier, so
> don't use __devexit_p to wrap it.
> 
> Signed-off-by: Uwe Kleine-K??nig <u.kleine-koenig@pengutronix.de>
> Cc: Paul Mundt <lethal@linux-sh.org>
> Cc: Magnus Damm <damm@opensource.se>
> Cc: linux-kernel@vger.kernel.org

I've applied both of these, thanks.
diff mbox

Patch

diff --git a/drivers/rtc/rtc-ds1302.c b/drivers/rtc/rtc-ds1302.c
index d490628..1e73c8f 100644
--- a/drivers/rtc/rtc-ds1302.c
+++ b/drivers/rtc/rtc-ds1302.c
@@ -201,7 +201,7 @@  static struct platform_driver ds1302_platform_driver = {
 		.name	= DRV_NAME,
 		.owner	= THIS_MODULE,
 	},
-	.remove		= __exit_p(ds1302_rtc_remove),
+	.remove		= __devexit_p(ds1302_rtc_remove),
 };
 
 static int __init ds1302_rtc_init(void)