diff options
author | Michael Ellerman <michael@ellerman.id.au> | 2007-01-25 03:34:07 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-02-07 18:50:06 -0500 |
commit | 88187dfa4d8bb565df762f272511d2c91e427e0d (patch) | |
tree | 451f53265c52ea50bcb43bb4e5854fe9c3359182 /drivers/pci/msi.c | |
parent | 89298c7a41e71ecb1e0c3f793655e9ce09662ce0 (diff) |
MSI: Replace pci_msi_quirk with calls to pci_no_msi()
I don't see any reason why we need pci_msi_quirk, quirk code can just
call pci_no_msi() instead.
Remove the check of pci_msi_quirk in msi_init(). This is safe as all
calls to msi_init() are protected by calls to pci_msi_supported(),
which checks pci_msi_enable, which is disabled by pci_no_msi().
The pci_disable_msi routines didn't check pci_msi_quirk, only
pci_msi_enable, but as far as I can see that was a bug not a feature.
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/pci/msi.c')
-rw-r--r-- | drivers/pci/msi.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c index e87e8efb9bb4..3776531586d1 100644 --- a/drivers/pci/msi.c +++ b/drivers/pci/msi.c | |||
@@ -170,13 +170,6 @@ static int msi_init(void) | |||
170 | if (!status) | 170 | if (!status) |
171 | return status; | 171 | return status; |
172 | 172 | ||
173 | if (pci_msi_quirk) { | ||
174 | pci_msi_enable = 0; | ||
175 | printk(KERN_WARNING "PCI: MSI quirk detected. MSI disabled.\n"); | ||
176 | status = -EINVAL; | ||
177 | return status; | ||
178 | } | ||
179 | |||
180 | status = msi_cache_init(); | 173 | status = msi_cache_init(); |
181 | if (status < 0) { | 174 | if (status < 0) { |
182 | pci_msi_enable = 0; | 175 | pci_msi_enable = 0; |