Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752910AbcLFUSk (ORCPT ); Tue, 6 Dec 2016 15:18:40 -0500 Received: from mail.kernel.org ([198.145.29.136]:43598 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751556AbcLFUSj (ORCPT ); Tue, 6 Dec 2016 15:18:39 -0500 Date: Tue, 6 Dec 2016 14:18:35 -0600 From: Bjorn Helgaas To: Jon Masters Cc: Duc Dang , Mark Salter , Rafael Wysocki , Lorenzo Pieralisi , Arnd Bergmann , linux-pci@vger.kernel.org, linux-arm , Linux Kernel Mailing List , Tomasz Nowicki , patches Subject: Re: [PATCH v3] PCI/ACPI: xgene: Add ECAM quirk for X-Gene PCIe controller Message-ID: <20161206201835.GA19700@bhelgaas-glaptop.roam.corp.google.com> References: <1480620053.4751.30.camel@redhat.com> <20161201194114.GA8263@bhelgaas-glaptop.roam.corp.google.com> <20161201230736.GA17948@bhelgaas-glaptop.roam.corp.google.com> <20161202233943.GF9903@bhelgaas-glaptop.roam.corp.google.com> <62dd5462-4d68-b89f-2b45-0d58bbd96bcd@redhat.com> <20161205212140.GA18259@bhelgaas-glaptop.roam.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1227 Lines: 29 On Tue, Dec 06, 2016 at 02:46:00PM -0500, Jon Masters wrote: > On 12/05/2016 04:21 PM, Bjorn Helgaas wrote: > > On Fri, Dec 02, 2016 at 07:33:46PM -0500, Jon Masters wrote: > > >>> Even without this patch, I don't think it's a show-stopper to have > >>> Linux mistakenly thinking this region is routed to PCI, because the > >>> driver does reserve it and the PCI core will never try to use it. > >> > >> Ok. So are you happy with pulling in Duc's v4 patch and retaining > >> status quo on the bridge resources for 4.10? > > > > Yes, I think it looks good. I'll finish packaging things up and > > repost the current series. > > Ok, great. So you're still pretty confident we'll have "out of the box" > booting on these machines for 4.10? :) I just merged pci/ecam into my "next" branch, so as long as tomorrow's linux-next boots out of the box, we should be set. I made the following changes compared to v11: - dropped the x86 change to use acpi_resource_consumer() - added parens around macro args used in arithmetic expressions - renamed "seg" to "node" in THUNDER_PEM_RES and THUNDER_PEM_QUIRK - incorporated (u64) cast and dropped "UL" postfix for THUNDER_PEM_QUIRK Let me know if you see any issues. Bjorn