* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-06-22 12:19 Lei Wen
0 siblings, 0 replies; 10+ messages in thread
From: Lei Wen @ 2010-06-22 12:19 UTC (permalink / raw)
To: linux-arm-kernel
The compile output as below:
WARNING: modpost: Found 1 section mismatch(es).
Signed-off-by: Lei Wen <leiwen@marvell.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 3ffcbcd..52cec28 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
this->chip_delay = 25;
}
-static int pxa3xx_nand_probe(struct platform_device *pdev)
+static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
{
struct pxa3xx_nand_platform_data *pdata;
struct pxa3xx_nand_info *info;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 02/29] pxa3xx_nand: fix compile warning
2010-07-28 7:23 ` Russell King - ARM Linux
@ 2010-07-28 7:38 ` Eric Miao
-1 siblings, 0 replies; 10+ messages in thread
From: Eric Miao @ 2010-07-28 7:38 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: David Woodhouse, Lei Wen, Haojian Zhuang, linux-mtd,
Marc Kleine-Budde, David Woodhouse, linux-arm-kernel
On Wed, Jul 28, 2010 at 3:23 PM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Wed, Jul 28, 2010 at 02:17:23PM +0800, Eric Miao wrote:
>> On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
>> <haojian.zhuang@gmail.com> wrote:
>> > From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
>> > From: Lei Wen <leiwen@marvell.com>
>> > Date: Tue, 22 Jun 2010 20:19:35 +0800
>> > Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
>> >
>> > The compile output as below:
>> > WARNING: modpost: Found 1 section mismatch(es).
>> >
>> > Signed-off-by: Lei Wen <leiwen@marvell.com>
>> > ---
>> > drivers/mtd/nand/pxa3xx_nand.c | 2 +-
>> > 1 files changed, 1 insertions(+), 1 deletions(-)
>> >
>> > diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
>> > index 3ffcbcd..52cec28 100644
>> > --- a/drivers/mtd/nand/pxa3xx_nand.c
>> > +++ b/drivers/mtd/nand/pxa3xx_nand.c
>> > @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
>> > this->chip_delay = 25;
>> > }
>> >
>> > -static int pxa3xx_nand_probe(struct platform_device *pdev)
>> > +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
>>
>> Ack
>
> __devinit is not always the right thing for this. Not having any
> attribute is the safest thing. Having any kind of init tag means
> that there's a circumstance when the function will be freed, and
> if the function pointer is called...
>
Should be quite safe if all the NAND devices are registered during
the initcall phase, which is normally true. Yet indeed will create
problems if they are registered later, e.g. in modules.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-07-28 7:38 ` Eric Miao
0 siblings, 0 replies; 10+ messages in thread
From: Eric Miao @ 2010-07-28 7:38 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 28, 2010 at 3:23 PM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Wed, Jul 28, 2010 at 02:17:23PM +0800, Eric Miao wrote:
>> On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
>> <haojian.zhuang@gmail.com> wrote:
>> > From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
>> > From: Lei Wen <leiwen@marvell.com>
>> > Date: Tue, 22 Jun 2010 20:19:35 +0800
>> > Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
>> >
>> > The compile output as below:
>> > WARNING: modpost: Found 1 section mismatch(es).
>> >
>> > Signed-off-by: Lei Wen <leiwen@marvell.com>
>> > ---
>> > ?drivers/mtd/nand/pxa3xx_nand.c | ? ?2 +-
>> > ?1 files changed, 1 insertions(+), 1 deletions(-)
>> >
>> > diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
>> > index 3ffcbcd..52cec28 100644
>> > --- a/drivers/mtd/nand/pxa3xx_nand.c
>> > +++ b/drivers/mtd/nand/pxa3xx_nand.c
>> > @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
>> > ? ? ? ?this->chip_delay = 25;
>> > ?}
>> >
>> > -static int pxa3xx_nand_probe(struct platform_device *pdev)
>> > +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
>>
>> Ack
>
> __devinit is not always the right thing for this. ?Not having any
> attribute is the safest thing. ?Having any kind of init tag means
> that there's a circumstance when the function will be freed, and
> if the function pointer is called...
>
Should be quite safe if all the NAND devices are registered during
the initcall phase, which is normally true. Yet indeed will create
problems if they are registered later, e.g. in modules.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 02/29] pxa3xx_nand: fix compile warning
2010-07-28 6:17 ` Eric Miao
@ 2010-07-28 7:23 ` Russell King - ARM Linux
-1 siblings, 0 replies; 10+ messages in thread
From: Russell King - ARM Linux @ 2010-07-28 7:23 UTC (permalink / raw)
To: Eric Miao
Cc: David Woodhouse, Lei Wen, Haojian Zhuang, linux-mtd,
Marc Kleine-Budde, David Woodhouse, linux-arm-kernel
On Wed, Jul 28, 2010 at 02:17:23PM +0800, Eric Miao wrote:
> On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
> <haojian.zhuang@gmail.com> wrote:
> > From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
> > From: Lei Wen <leiwen@marvell.com>
> > Date: Tue, 22 Jun 2010 20:19:35 +0800
> > Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
> >
> > The compile output as below:
> > WARNING: modpost: Found 1 section mismatch(es).
> >
> > Signed-off-by: Lei Wen <leiwen@marvell.com>
> > ---
> > drivers/mtd/nand/pxa3xx_nand.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> > index 3ffcbcd..52cec28 100644
> > --- a/drivers/mtd/nand/pxa3xx_nand.c
> > +++ b/drivers/mtd/nand/pxa3xx_nand.c
> > @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
> > this->chip_delay = 25;
> > }
> >
> > -static int pxa3xx_nand_probe(struct platform_device *pdev)
> > +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
>
> Ack
__devinit is not always the right thing for this. Not having any
attribute is the safest thing. Having any kind of init tag means
that there's a circumstance when the function will be freed, and
if the function pointer is called...
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-07-28 7:23 ` Russell King - ARM Linux
0 siblings, 0 replies; 10+ messages in thread
From: Russell King - ARM Linux @ 2010-07-28 7:23 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 28, 2010 at 02:17:23PM +0800, Eric Miao wrote:
> On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
> <haojian.zhuang@gmail.com> wrote:
> > From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
> > From: Lei Wen <leiwen@marvell.com>
> > Date: Tue, 22 Jun 2010 20:19:35 +0800
> > Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
> >
> > The compile output as below:
> > WARNING: modpost: Found 1 section mismatch(es).
> >
> > Signed-off-by: Lei Wen <leiwen@marvell.com>
> > ---
> > ?drivers/mtd/nand/pxa3xx_nand.c | ? ?2 +-
> > ?1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> > index 3ffcbcd..52cec28 100644
> > --- a/drivers/mtd/nand/pxa3xx_nand.c
> > +++ b/drivers/mtd/nand/pxa3xx_nand.c
> > @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
> > ? ? ? ?this->chip_delay = 25;
> > ?}
> >
> > -static int pxa3xx_nand_probe(struct platform_device *pdev)
> > +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
>
> Ack
__devinit is not always the right thing for this. Not having any
attribute is the safest thing. Having any kind of init tag means
that there's a circumstance when the function will be freed, and
if the function pointer is called...
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 02/29] pxa3xx_nand: fix compile warning
2010-07-28 5:56 ` Haojian Zhuang
@ 2010-07-28 6:17 ` Eric Miao
-1 siblings, 0 replies; 10+ messages in thread
From: Eric Miao @ 2010-07-28 6:17 UTC (permalink / raw)
To: Haojian Zhuang
Cc: David Woodhouse, Lei Wen, Marc Kleine-Budde, linux-mtd,
David Woodhouse, linux-arm-kernel
On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
<haojian.zhuang@gmail.com> wrote:
> From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
> From: Lei Wen <leiwen@marvell.com>
> Date: Tue, 22 Jun 2010 20:19:35 +0800
> Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
>
> The compile output as below:
> WARNING: modpost: Found 1 section mismatch(es).
>
> Signed-off-by: Lei Wen <leiwen@marvell.com>
> ---
> drivers/mtd/nand/pxa3xx_nand.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 3ffcbcd..52cec28 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
> this->chip_delay = 25;
> }
>
> -static int pxa3xx_nand_probe(struct platform_device *pdev)
> +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
Ack
> {
> struct pxa3xx_nand_platform_data *pdata;
> struct pxa3xx_nand_info *info;
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-07-28 6:17 ` Eric Miao
0 siblings, 0 replies; 10+ messages in thread
From: Eric Miao @ 2010-07-28 6:17 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 28, 2010 at 1:56 PM, Haojian Zhuang
<haojian.zhuang@gmail.com> wrote:
> From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
> From: Lei Wen <leiwen@marvell.com>
> Date: Tue, 22 Jun 2010 20:19:35 +0800
> Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
>
> The compile output as below:
> WARNING: modpost: Found 1 section mismatch(es).
>
> Signed-off-by: Lei Wen <leiwen@marvell.com>
> ---
> ?drivers/mtd/nand/pxa3xx_nand.c | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 3ffcbcd..52cec28 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
> ? ? ? ?this->chip_delay = 25;
> ?}
>
> -static int pxa3xx_nand_probe(struct platform_device *pdev)
> +static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
Ack
> ?{
> ? ? ? ?struct pxa3xx_nand_platform_data *pdata;
> ? ? ? ?struct pxa3xx_nand_info *info;
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-07-28 5:56 ` Haojian Zhuang
0 siblings, 0 replies; 10+ messages in thread
From: Haojian Zhuang @ 2010-07-28 5:56 UTC (permalink / raw)
To: Eric Miao, linux-arm-kernel, David Woodhouse, David Woodhouse,
Marc Kleine-Budde, linux-mtd, Lei Wen
>From c374a3f27c56f9112b97960eaa3194a08db6f4a4 Mon Sep 17 00:00:00 2001
From: Lei Wen <leiwen@marvell.com>
Date: Tue, 22 Jun 2010 20:19:35 +0800
Subject: [PATCH 02/29] pxa3xx_nand: fix compile warning
The compile output as below:
WARNING: modpost: Found 1 section mismatch(es).
Signed-off-by: Lei Wen <leiwen@marvell.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 3ffcbcd..52cec28 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
this->chip_delay = 25;
}
-static int pxa3xx_nand_probe(struct platform_device *pdev)
+static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
{
struct pxa3xx_nand_platform_data *pdata;
struct pxa3xx_nand_info *info;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-07-28 5:56 ` Haojian Zhuang
0 siblings, 0 replies; 10+ messages in thread
From: Haojian Zhuang @ 2010-07-28 5:56 UTC (permalink / raw)
To: linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 02/29] pxa3xx_nand: fix compile warning
@ 2010-06-22 12:19 Lei Wen
0 siblings, 0 replies; 10+ messages in thread
From: Lei Wen @ 2010-06-22 12:19 UTC (permalink / raw)
To: linux-mtd
The compile output as below:
WARNING: modpost: Found 1 section mismatch(es).
Signed-off-by: Lei Wen <leiwen@marvell.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 3ffcbcd..52cec28 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1069,7 +1069,7 @@ static void pxa3xx_nand_init_mtd(struct mtd_info *mtd,
this->chip_delay = 25;
}
-static int pxa3xx_nand_probe(struct platform_device *pdev)
+static int __devinit pxa3xx_nand_probe(struct platform_device *pdev)
{
struct pxa3xx_nand_platform_data *pdata;
struct pxa3xx_nand_info *info;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-07-28 7:38 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-22 12:19 [PATCH 02/29] pxa3xx_nand: fix compile warning Lei Wen
2010-06-22 12:19 Lei Wen
2010-07-28 5:56 Haojian Zhuang
2010-07-28 5:56 ` Haojian Zhuang
2010-07-28 6:17 ` Eric Miao
2010-07-28 6:17 ` Eric Miao
2010-07-28 7:23 ` Russell King - ARM Linux
2010-07-28 7:23 ` Russell King - ARM Linux
2010-07-28 7:38 ` Eric Miao
2010-07-28 7:38 ` Eric Miao
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.