In PCI 64-bit BARs span two 32-bit BARs, therefore if BAR type
indicates a 64-bit BAR we have to skip the next BAR register.
Note that this does not add proper support for 64-bit BARs and
64-bit addresses but still picks the lower 32-bit address.

While at it, also set proper IORESOURCE flags for I/O and
32b MEM.

Signed-off-by: Sebastian Hesselbarth <[email protected]>
Acked-by: Lucas Stach <[email protected]>
---
Changelog:
v1->v2:
- set resource flags for all resources found (Suggested by Lucas Stach)
- use MEM_64 resource flag for BAR64 detection (Suggested by Lucas Stach)

Cc: [email protected]
Cc: Antony Pavlov <[email protected]>
Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
Cc: Lucas Stach <[email protected]>
Cc: Thomas Petazzoni <[email protected]>
Cc: Ezequiel Garcia <[email protected]>
---
 drivers/pci/pci.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 3d88b0ff5fd0..501a36cc8356 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -211,19 +211,26 @@ unsigned int pci_scan_bus(struct pci_bus *bus)
                                size = -(mask & 0xfffffffe);
                                DBG("  PCI: pbar%d: mask=%08x io %d bytes\n", 
bar, mask, size);
                                pci_write_config_dword(dev, PCI_BASE_ADDRESS_0 
+ bar * 4, last_io);
+                               dev->resource[bar].flags = IORESOURCE_IO;
                                last_addr = last_io;
                                last_io += size;
-
                        } else { /* MEM */
                                size = -(mask & 0xfffffff0);
                                DBG("  PCI: pbar%d: mask=%08x memory %d 
bytes\n", bar, mask, size);
                                pci_write_config_dword(dev, PCI_BASE_ADDRESS_0 
+ bar * 4, last_mem);
+                               dev->resource[bar].flags = IORESOURCE_MEM;
                                last_addr = last_mem;
                                last_mem += size;
+
+                               if ((mask & PCI_BASE_ADDRESS_MEM_TYPE_MASK) ==
+                                   PCI_BASE_ADDRESS_MEM_TYPE_64)
+                                       dev->resource[bar].flags = 
IORESOURCE_MEM_64;
                        }
 
                        dev->resource[bar].start = last_addr;
                        dev->resource[bar].end = last_addr + size - 1;
+                       if (dev->resource[bar].flags & IORESOURCE_MEM_64)
+                               bar++;
                }
        }
 
-- 
2.0.0


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to