diff options
author | Arnaud Lacombe <lacombar@gmail.com> | 2011-08-04 10:39:44 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-08-04 06:32:51 -1000 |
commit | 82de9a0cc34bc1640c4f133f13d62a765596b2b9 (patch) | |
tree | 122ecd6e6b0662fcfaacd04ce0a5d37b7f62146b | |
parent | 288d5abec8314ae50fe6692f324b0444acae8486 (diff) | |
download | linux-3.10-82de9a0cc34bc1640c4f133f13d62a765596b2b9.tar.gz linux-3.10-82de9a0cc34bc1640c4f133f13d62a765596b2b9.tar.bz2 linux-3.10-82de9a0cc34bc1640c4f133f13d62a765596b2b9.zip |
eisa/pci_eisa.c: fix BUG introduced by 005bdad7b80
While `pci_eisa_driver' still refer `pci_eisa_init', the .probe() function
should not be called after init memory release, as pointed out by commit
74b9a297. The structure is still referenced in the drivers subsystem, and can
be accesseed through sysfs, so the modpost warning is a false positive. Mark
it as such.
In the same time, the warning referenced in 005bdad7b80 did only mention
`pci_eisa_driver', not `pci_eisa_pci_tbl', so remove its marking.
Broken-by: Arnaud Lacombe <lacombar@gmail.com> (in 005bdad7b80)
Reported-by: Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>
Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/eisa/pci_eisa.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/eisa/pci_eisa.c b/drivers/eisa/pci_eisa.c index 30da70d06a6..cdae207028a 100644 --- a/drivers/eisa/pci_eisa.c +++ b/drivers/eisa/pci_eisa.c @@ -45,13 +45,13 @@ static int __init pci_eisa_init(struct pci_dev *pdev, return 0; } -static struct pci_device_id __initdata pci_eisa_pci_tbl[] = { +static struct pci_device_id pci_eisa_pci_tbl[] = { { PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, PCI_CLASS_BRIDGE_EISA << 8, 0xffff00, 0 }, { 0, } }; -static struct pci_driver __initdata pci_eisa_driver = { +static struct pci_driver __refdata pci_eisa_driver = { .name = "pci_eisa", .id_table = pci_eisa_pci_tbl, .probe = pci_eisa_init, |