* [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
@ 2010-12-08 2:42 Bryan Wu
2010-12-09 14:59 ` Samuel Ortiz
0 siblings, 1 reply; 8+ messages in thread
From: Bryan Wu @ 2010-12-08 2:42 UTC (permalink / raw)
To: paul, linux-omap, sameo
Fix the following section mismatch warning when building omap2plus_defconfig:
WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
Signed-off-by: Paul Walmsley <paul@pwsan.com>
---
drivers/mfd/twl-core.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
index 35275ba..3aa46ff 100644
--- a/drivers/mfd/twl-core.c
+++ b/drivers/mfd/twl-core.c
@@ -969,7 +969,7 @@ static int twl_remove(struct i2c_client *client)
}
/* NOTE: this driver only handles a single twl4030/tps659x0 chip */
-static int __init
+static int __devinit
twl_probe(struct i2c_client *client, const struct i2c_device_id *id)
{
int status;
--
1.7.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-08 2:42 [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c Bryan Wu
@ 2010-12-09 14:59 ` Samuel Ortiz
0 siblings, 0 replies; 8+ messages in thread
From: Samuel Ortiz @ 2010-12-09 14:59 UTC (permalink / raw)
To: Bryan Wu; +Cc: paul, linux-omap
Hi Bryan,
On Wed, Dec 08, 2010 at 10:42:04AM +0800, Bryan Wu wrote:
> Fix the following section mismatch warning when building omap2plus_defconfig:
>
> WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
>
Patch applied, thanks.
Cheers,
Samuel.
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-07 16:12 ` Paul Walmsley
@ 2010-12-08 1:45 ` Bryan Wu
0 siblings, 0 replies; 8+ messages in thread
From: Bryan Wu @ 2010-12-08 1:45 UTC (permalink / raw)
To: Paul Walmsley; +Cc: Samuel Ortiz, linux-kernel, linux-omap
On Wed, Dec 8, 2010 at 12:12 AM, Paul Walmsley <paul@pwsan.com> wrote:
> Hello Samuel,
>
> On Tue, 7 Dec 2010, Samuel Ortiz wrote:
>
>> On Mon, Dec 06, 2010 at 06:40:38PM -0700, Paul Walmsley wrote:
>> >
>> > That's fine with me. Samuel et al, Bryan's already done a patch
>> > for this stuff:
>> >
>> > https://patchwork.kernel.org/patch/367011/
>> >
>> > so we should use that instead, if you're happy with it. Samuel, maybe we
>> > could get an ack from you on it?
>> The twl driver is not OMAP specific, so this should be a separate patch thatI
>> will merge to my mfd tree.
>> If you really insist in pushing this through Tony's tree, then please add my
>> Acked-by for the mfd part.
>
> I don't insist at all :-)
>
> Bryan, maybe split your patch into a mach-omap2 patch and a TWL
> driver-specific patch, and send the latter for Samuel?
>
>
No problem. I'll do soon.
Thanks,
--
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer +86.138-1617-6545 Mobile
Ubuntu Kernel Team
Canonical Ltd. www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-07 9:48 ` Samuel Ortiz
@ 2010-12-07 16:12 ` Paul Walmsley
2010-12-08 1:45 ` Bryan Wu
0 siblings, 1 reply; 8+ messages in thread
From: Paul Walmsley @ 2010-12-07 16:12 UTC (permalink / raw)
To: Samuel Ortiz; +Cc: Bryan Wu, linux-kernel, linux-omap
Hello Samuel,
On Tue, 7 Dec 2010, Samuel Ortiz wrote:
> On Mon, Dec 06, 2010 at 06:40:38PM -0700, Paul Walmsley wrote:
> >
> > That's fine with me. Samuel et al, Bryan's already done a patch
> > for this stuff:
> >
> > https://patchwork.kernel.org/patch/367011/
> >
> > so we should use that instead, if you're happy with it. Samuel, maybe we
> > could get an ack from you on it?
> The twl driver is not OMAP specific, so this should be a separate patch thatI
> will merge to my mfd tree.
> If you really insist in pushing this through Tony's tree, then please add my
> Acked-by for the mfd part.
I don't insist at all :-)
Bryan, maybe split your patch into a mach-omap2 patch and a TWL
driver-specific patch, and send the latter for Samuel?
- Paul
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-07 1:40 ` Paul Walmsley
@ 2010-12-07 9:48 ` Samuel Ortiz
2010-12-07 16:12 ` Paul Walmsley
0 siblings, 1 reply; 8+ messages in thread
From: Samuel Ortiz @ 2010-12-07 9:48 UTC (permalink / raw)
To: Paul Walmsley; +Cc: Bryan Wu, linux-kernel, linux-omap
Hi Paul,
On Mon, Dec 06, 2010 at 06:40:38PM -0700, Paul Walmsley wrote:
> On Tue, 7 Dec 2010, Bryan Wu wrote:
>
> > On Tue, Dec 7, 2010 at 8:35 AM, Paul Walmsley <paul@pwsan.com> wrote:
> > >
> > > Fix the following section mismatch warning when building omap2plus_defconfig:
> > >
> > > WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
> > >
> > > Signed-off-by: Paul Walmsley <paul@pwsan.com>
> > > Cc: Samuel Ortiz <sameo@linux.intel.com>
> > > ---
> > > drivers/mfd/twl-core.c | 2 +-
> > > 1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
> > > index 35275ba..615cf38 100644
> > > --- a/drivers/mfd/twl-core.c
> > > +++ b/drivers/mfd/twl-core.c
> > > @@ -969,7 +969,7 @@ static int twl_remove(struct i2c_client *client)
> > > }
> > >
> > > /* NOTE: this driver only handles a single twl4030/tps659x0 chip */
> > > -static int __init
> > > +static int
> >
> > I think we might change __init to __devinit.
> >
> > > twl_probe(struct i2c_client *client, const struct i2c_device_id *id)
> > > {
> > > int status;
> > > --
>
> That's fine with me. Samuel et al, Bryan's already done a patch
> for this stuff:
>
> https://patchwork.kernel.org/patch/367011/
>
> so we should use that instead, if you're happy with it. Samuel, maybe we
> could get an ack from you on it?
The twl driver is not OMAP specific, so this should be a separate patch thatI
will merge to my mfd tree.
If you really insist in pushing this through Tony's tree, then please add my
Acked-by for the mfd part.
Cheers,
Samuel.
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-07 1:36 ` Bryan Wu
@ 2010-12-07 1:40 ` Paul Walmsley
2010-12-07 9:48 ` Samuel Ortiz
0 siblings, 1 reply; 8+ messages in thread
From: Paul Walmsley @ 2010-12-07 1:40 UTC (permalink / raw)
To: Bryan Wu; +Cc: linux-kernel, linux-omap, Samuel Ortiz
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1378 bytes --]
On Tue, 7 Dec 2010, Bryan Wu wrote:
> On Tue, Dec 7, 2010 at 8:35 AM, Paul Walmsley <paul@pwsan.com> wrote:
> >
> > Fix the following section mismatch warning when building omap2plus_defconfig:
> >
> > WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
> >
> > Signed-off-by: Paul Walmsley <paul@pwsan.com>
> > Cc: Samuel Ortiz <sameo@linux.intel.com>
> > ---
> > drivers/mfd/twl-core.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
> > index 35275ba..615cf38 100644
> > --- a/drivers/mfd/twl-core.c
> > +++ b/drivers/mfd/twl-core.c
> > @@ -969,7 +969,7 @@ static int twl_remove(struct i2c_client *client)
> > }
> >
> > /* NOTE: this driver only handles a single twl4030/tps659x0 chip */
> > -static int __init
> > +static int
>
> I think we might change __init to __devinit.
>
> > twl_probe(struct i2c_client *client, const struct i2c_device_id *id)
> > {
> > int status;
> > --
That's fine with me. Samuel et al, Bryan's already done a patch
for this stuff:
https://patchwork.kernel.org/patch/367011/
so we should use that instead, if you're happy with it. Samuel, maybe we
could get an ack from you on it?
- Paul
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
2010-12-07 0:35 Paul Walmsley
@ 2010-12-07 1:36 ` Bryan Wu
2010-12-07 1:40 ` Paul Walmsley
0 siblings, 1 reply; 8+ messages in thread
From: Bryan Wu @ 2010-12-07 1:36 UTC (permalink / raw)
To: Paul Walmsley; +Cc: linux-kernel, linux-omap, Samuel Ortiz
On Tue, Dec 7, 2010 at 8:35 AM, Paul Walmsley <paul@pwsan.com> wrote:
>
> Fix the following section mismatch warning when building omap2plus_defconfig:
>
> WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
>
> Signed-off-by: Paul Walmsley <paul@pwsan.com>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> ---
> drivers/mfd/twl-core.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
> index 35275ba..615cf38 100644
> --- a/drivers/mfd/twl-core.c
> +++ b/drivers/mfd/twl-core.c
> @@ -969,7 +969,7 @@ static int twl_remove(struct i2c_client *client)
> }
>
> /* NOTE: this driver only handles a single twl4030/tps659x0 chip */
> -static int __init
> +static int
I think we might change __init to __devinit.
> twl_probe(struct i2c_client *client, const struct i2c_device_id *id)
> {
> int status;
> --
Thanks,
--
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer +86.138-1617-6545 Mobile
Ubuntu Kernel Team
Canonical Ltd. www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c
@ 2010-12-07 0:35 Paul Walmsley
2010-12-07 1:36 ` Bryan Wu
0 siblings, 1 reply; 8+ messages in thread
From: Paul Walmsley @ 2010-12-07 0:35 UTC (permalink / raw)
To: linux-kernel; +Cc: linux-omap, Samuel Ortiz
Fix the following section mismatch warning when building omap2plus_defconfig:
WARNING: vmlinux.o(.data+0x47d7c): Section mismatch in reference from the variable twl_driver to the function .init.text:twl_probe()
Signed-off-by: Paul Walmsley <paul@pwsan.com>
Cc: Samuel Ortiz <sameo@linux.intel.com>
---
drivers/mfd/twl-core.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c
index 35275ba..615cf38 100644
--- a/drivers/mfd/twl-core.c
+++ b/drivers/mfd/twl-core.c
@@ -969,7 +969,7 @@ static int twl_remove(struct i2c_client *client)
}
/* NOTE: this driver only handles a single twl4030/tps659x0 chip */
-static int __init
+static int
twl_probe(struct i2c_client *client, const struct i2c_device_id *id)
{
int status;
--
1.7.2.3
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2010-12-09 15:00 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-08 2:42 [PATCH] MFD: TWL/TPS: fix twl_probe section mismatch warning in mfd/twl-core.c Bryan Wu
2010-12-09 14:59 ` Samuel Ortiz
-- strict thread matches above, loose matches on Subject: below --
2010-12-07 0:35 Paul Walmsley
2010-12-07 1:36 ` Bryan Wu
2010-12-07 1:40 ` Paul Walmsley
2010-12-07 9:48 ` Samuel Ortiz
2010-12-07 16:12 ` Paul Walmsley
2010-12-08 1:45 ` Bryan Wu
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.