Skip to content
Snippets Groups Projects
Commit b7598a43 authored by Sergei Shtylyov's avatar Sergei Shtylyov Committed by Wolfgang Denk
Browse files

[PATCH] Avoid assigning PCI resources from zero address


If a PCI IDE card happens to get a zero address assigned to it, the Linux IDE
core complains and IDE drivers fails to work.  Also, assigning zero to a BAR
was illegal according to PCI 2.1 (the later revisions seem to have excluded the
sentence about "0" being considered an invalid address) -- so, use a reasonable
starting value of 0x1000 (that's what the most Linux archs are using).

Alternatively, one might have fixed the calls to pci_set_region() individually
(some code even seems to have taken care of this issue) but that would have
been a lot more work. :-)

Signed-off-by: default avatarSergei Shtylyov <sshtylyov@ru.mvista.com>
Acked-by: default avatarStefan Roese <sr@denx.de>
parent 9ffd451a
No related branches found
No related tags found
No related merge requests found
...@@ -34,7 +34,12 @@ ...@@ -34,7 +34,12 @@
void pciauto_region_init(struct pci_region* res) void pciauto_region_init(struct pci_region* res)
{ {
res->bus_lower = res->bus_start; /*
* Avoid allocating PCI resources from address 0 -- this is illegal
* according to PCI 2.1 and moreover, this is known to cause Linux IDE
* drivers to fail. Use a reasonable starting value of 0x1000 instead.
*/
res->bus_lower = res->bus_start ? res->bus_start : 0x1000;
} }
void pciauto_region_align(struct pci_region *res, unsigned long size) void pciauto_region_align(struct pci_region *res, unsigned long size)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment