Hi Sebastian,
Am Mittwoch, den 23.07.2014, 11:26 +0200 schrieb Sebastian Hesselbarth:
> 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.
>
> Signed-off-by: Sebastian Hesselbarth <[email protected]>
> ---
> To: [email protected]
> To: Sebastian Hesselbarth <[email protected]>
> Cc: Antony Pavlov <[email protected]>
> Cc: Jean-Christophe PLAGNIOL-VILLARD <[email protected]>
> Cc: Thomas Petazzoni <[email protected]>
> Cc: Ezequiel Garcia <[email protected]>
> ---
> drivers/pci/pci.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index 3d88b0ff5fd0..e5cd8a33b2be 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -196,6 +196,7 @@ unsigned int pci_scan_bus(struct pci_bus *bus)
>
> for (bar = 0; bar < 6; bar++) {
> resource_size_t last_addr;
> + bool found_bar64 = false;
I don't think we need this variable...
>
> pci_read_config_dword(dev, PCI_BASE_ADDRESS_0 + bar *
> 4, &old_bar);
> pci_write_config_dword(dev, PCI_BASE_ADDRESS_0 + bar *
> 4, 0xfffffffe);
> @@ -213,17 +214,22 @@ unsigned int pci_scan_bus(struct pci_bus *bus)
> pci_write_config_dword(dev, PCI_BASE_ADDRESS_0
> + bar * 4, last_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);
> last_addr = last_mem;
> last_mem += size;
> +
> + if ((mask & PCI_BASE_ADDRESS_MEM_TYPE_MASK) ==
> + PCI_BASE_ADDRESS_MEM_TYPE_64)
> + found_bar64 = true;
... if we set the dev->resource.flags to IORESOURCE_MEM_64 (bonus points
for setting the flags in other cases properly also) here...
> }
>
> dev->resource[bar].start = last_addr;
> dev->resource[bar].end = last_addr + size - 1;
> + if (found_bar64)
> + bar++;
... and check for this flag here.
> }
> }
>
Regards,
Lucas
--
Pengutronix e.K. | Lucas Stach |
Industrial Linux Solutions | http://www.pengutronix.de/ |
_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox
