Re: [PATCH v2 0/2] pci: xgene: Add multiple memory ranges support

From: Arnd Bergmann
Date: Thu Jul 09 2015 - 07:48:32 EST


On Monday 06 July 2015 16:28:43 Duc Dang wrote:
> On Tue, Jun 30, 2015 at 11:22 AM, Duc Dang <dhdang@xxxxxxx> wrote:
> > This patch set adds 1 large (up to 64GB) memory window for each PCIe
> > controller nodes in X-Gene device tree and fix PCIe controller driver
> > to handle multiple memory ranges correctly. These changes are required
> > to support PCIe devices that has huge BAR.
> >
> > v2 changes:
> > 1. Separate device-tree changes and driver changes into different
> > patches
> > 2. Explicitly define new large window as 64-bit prefetchable in dts
> > 3. Use IORESOURCE_PREFETCH flag to determine which PCIe controller
> > register to be used to configure the memory ranges.
> >
> > arch/arm64/boot/dts/apm/apm-storm.dtsi | 23 ++++++++++++++---------
> > drivers/pci/host/pci-xgene.c | 12 ++++++++++--
> > 2 files changed, 24 insertions(+), 11 deletions(-)
>
> Hi Arnd, Bjorn,
>
> Do you have additional comment on this v2 patch set?
>
>

The changes look ok to me now, but I'd mention in the changelog about
the fact that one of the windows is prefetchable and the other one
is not, and that only prefetchable BARs are now using the 64-bit
window.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/