From mboxrd@z Thu Jan  1 00:00:00 1970
From: jeremy.linton@arm.com (Jeremy Linton)
Date: Thu, 14 Jan 2016 16:51:16 -0600
Subject: [PATCH V3 00/21] MMCONFIG refactoring and support for ARM64 PCI
 hostbridge init based on ACPI
In-Reply-To: <1452785393.28109.16.camel@redhat.com>
References: <1452691267-32240-1-git-send-email-tn@semihalf.com>
 <1452785393.28109.16.camel@redhat.com>
Message-ID: <56982664.5040009@arm.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On 01/14/2016 09:29 AM, Mark Salter wrote:
> IO resources on Mustang get disabled unless I do:
>
> @@ -126,9 +126,10 @@ static void acpi_dev_ioresource_flags(struct resource *res, u64 len,
>          if (!acpi_dev_resource_len_valid(res->start, res->end, len, true))
>                  res->flags |= IORESOURCE_DISABLED | IORESOURCE_UNSET;
>
> +#if 0
>          if (res->end >= 0x10003)
>                  res->flags |= IORESOURCE_DISABLED | IORESOURCE_UNSET;
> -
> +#endif
>
> res->end is way beyond 0x10003 on Mustang:
>
>    pci_bus 0000:00: root bus resource [io  0x0000-0xffff window] (bus address [0x10000000-0x1000ffff])

Juno is throwing "No Space" errors too with the following acpi definition.

DWordIo ( // IO window
	ResourceProducer,
	MinFixed,
	MaxFixed,
	PosDecode,
	EntireRange,
	0x00000000,// Granularity
	0x00000000,// Min Base Address
	0x007fffff,// Max Base Address
	0x5f800000, // Translate
	0x00800000, // Length
	,
	,
	,
	TypeTranslation
)

Although, my reading of the ACPI dword section is suspect...