[PATCH v2 1/1] pci: definition of pci_addr_t and pci_size_t

Tom Rini trini at konsulko.com
Thu Feb 13 23:12:04 CET 2020


On Wed, Feb 05, 2020 at 09:59:12PM +0100, Heinrich Schuchardt wrote:

> Currently the size of pci_addr_t and pci_size_t depends on
> CONFIG_SYS_PCI_64BIT. For qemu_arm64_defconfig with 4 GiB RAM this leads
> to an error
> 
>     pci_hose_phys_to_bus: invalid physical address
> 
> which is due to the truncation of the bus address in _dm_pci_phys_to_bus.
> 
> Defining CONFIG_SYS_PCI_64BIT is not a solution as this results in an error
> 
>    PCI: Failed autoconfig bar 10
> 
> So let's use unsigned long for pci_addr_t and pci_size_t if
> CONFIG_SYS_PCI_64BIT is not defined.
> 
> Considering that 32bit U-Boot is used to launch some 64bit x86 systems we
> cannot do without CONFIG_SYS_PCI_64BIT requiring u64 as type.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200213/847d9066/attachment.sig>


More information about the U-Boot mailing list