[PATCH 1/1] pci: definition of pci_addr_t and pci_size_t
sjg at chromium.org
Wed Feb 5 21:28:03 CET 2020
On Wed, 5 Feb 2020 at 13:18, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> On 2/5/20 9:05 PM, Simon Glass wrote:
> > Hi Heinrich,
> > On Wed, 5 Feb 2020 at 12:07, 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.
> > But how will this work on x86 where we might have 32-bit U-Boot but
> > need 64-bit PCI addresses?
> > Regards,
> > Simon
> So would you suggest to only change the code for CONFIG_SYS_PCI_64BIT=n?
I think that is safe, since nothing has a ulong less than 32 bits.
More information about the U-Boot