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

[Simon Glass]

On Wed, 5 Feb 2020 at 13:59, Heinrich Schuchardt <xypron.glpk at gmx.de> 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>
> ---
> v2
>         Do not change CONFIG_SYS_PCI_64BIT=y case.
> ---
>  include/pci.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>