* [PATCH] ksz884x: Fix section mismatch derived from pci_device_driver variable
@ 2011-01-03 2:01 Sedat Dilek
2011-01-03 19:52 ` David Miller
0 siblings, 1 reply; 3+ messages in thread
From: Sedat Dilek @ 2011-01-03 2:01 UTC (permalink / raw)
To: netdev, linux-kernel; +Cc: Sedat Dilek
WARNING: drivers/net/ksz884x.o(.data+0x18): Section mismatch in reference from the variable pci_device_driver to the function .init.text:pcidev_init()
The variable pci_device_driver references
the function __init pcidev_init()
If the reference is valid then annotate the
variable with __init* or __refdata (see linux/init.h) or name the variable:
*_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console,
This patch fixes the warning.
Tested with linux-next (next-20101231)
Signed-off-by: Sedat Dilek <sedat.dilek@gmail.com>
---
drivers/net/ksz884x.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/net/ksz884x.c b/drivers/net/ksz884x.c
index 49ea870..515b0d7 100644
--- a/drivers/net/ksz884x.c
+++ b/drivers/net/ksz884x.c
@@ -7277,7 +7277,7 @@ static struct pci_device_id pcidev_table[] = {
MODULE_DEVICE_TABLE(pci, pcidev_table);
-static struct pci_driver pci_device_driver = {
+static struct pci_driver pci_device_driver __refdata = {
#ifdef CONFIG_PM
.suspend = pcidev_suspend,
.resume = pcidev_resume,
--
1.7.4.rc0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ksz884x: Fix section mismatch derived from pci_device_driver variable
2011-01-03 2:01 [PATCH] ksz884x: Fix section mismatch derived from pci_device_driver variable Sedat Dilek
@ 2011-01-03 19:52 ` David Miller
2011-01-03 21:07 ` Sedat Dilek
0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2011-01-03 19:52 UTC (permalink / raw)
To: sedat.dilek; +Cc: netdev, linux-kernel, sedat.dilek
From: Sedat Dilek <sedat.dilek@googlemail.com>
Date: Mon, 3 Jan 2011 03:01:44 +0100
> WARNING: drivers/net/ksz884x.o(.data+0x18): Section mismatch in reference from the variable pci_device_driver to the function .init.text:pcidev_init()
> The variable pci_device_driver references
> the function __init pcidev_init()
> If the reference is valid then annotate the
> variable with __init* or __refdata (see linux/init.h) or name the variable:
> *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console,
>
> This patch fixes the warning.
>
> Tested with linux-next (next-20101231)
>
> Signed-off-by: Sedat Dilek <sedat.dilek@gmail.com>
pcidev_init() should be marked __devinit instead of __init to fix this
bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ksz884x: Fix section mismatch derived from pci_device_driver variable
2011-01-03 19:52 ` David Miller
@ 2011-01-03 21:07 ` Sedat Dilek
0 siblings, 0 replies; 3+ messages in thread
From: Sedat Dilek @ 2011-01-03 21:07 UTC (permalink / raw)
To: David Miller; +Cc: netdev, linux-kernel
On Mon, Jan 3, 2011 at 8:52 PM, David Miller <davem@davemloft.net> wrote:
> From: Sedat Dilek <sedat.dilek@googlemail.com>
> Date: Mon, 3 Jan 2011 03:01:44 +0100
>
>> WARNING: drivers/net/ksz884x.o(.data+0x18): Section mismatch in reference from the variable pci_device_driver to the function .init.text:pcidev_init()
>> The variable pci_device_driver references
>> the function __init pcidev_init()
>> If the reference is valid then annotate the
>> variable with __init* or __refdata (see linux/init.h) or name the variable:
>> *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console,
>>
>> This patch fixes the warning.
>>
>> Tested with linux-next (next-20101231)
>>
>> Signed-off-by: Sedat Dilek <sedat.dilek@gmail.com>
>
> pcidev_init() should be marked __devinit instead of __init to fix this
> bug.
>
I have sent a new fixed version.
- Sedat -
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-01-03 21:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-03 2:01 [PATCH] ksz884x: Fix section mismatch derived from pci_device_driver variable Sedat Dilek
2011-01-03 19:52 ` David Miller
2011-01-03 21:07 ` Sedat Dilek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).