Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936053AbcKWOJg convert rfc822-to-8bit (ORCPT ); Wed, 23 Nov 2016 09:09:36 -0500 Received: from lhrrgout.huawei.com ([194.213.3.17]:4402 "EHLO lhrrgout.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934916AbcKWOJe (ORCPT ); Wed, 23 Nov 2016 09:09:34 -0500 From: Gabriele Paoloni To: Graeme Gregory CC: Tomasz Nowicki , "liudongdong (C)" , "helgaas@kernel.org" , "arnd@arndb.de" , "rafael@kernel.org" , "Lorenzo.Pieralisi@arm.com" , "Wangzhou (B)" , "pratyush.anand@gmail.com" , "linux-pci@vger.kernel.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "jcm@redhat.com" , "Chenxin (Charles)" , "hanjun.guo@linaro.org" , Linuxarm Subject: RE: [PATCH V6 2/2] PCI/ACPI: hisi: Add ACPI support for HiSilicon SoCs Host Controllers Thread-Topic: [PATCH V6 2/2] PCI/ACPI: hisi: Add ACPI support for HiSilicon SoCs Host Controllers Thread-Index: AQHSRLbhXeBs/9iThEqerpPKSqr65aDlB4CAgAAR+jCAATl1AIAAReuQ Date: Wed, 23 Nov 2016 14:02:11 +0000 Message-ID: References: <1479816529-97410-1-git-send-email-liudongdong3@huawei.com> <1479816529-97410-3-git-send-email-liudongdong3@huawei.com> <026faf89-93de-4238-f69a-ea02c57e5ee0@semihalf.com> <20161123094428.GA878@linaro-xps13.localdomain> In-Reply-To: <20161123094428.GA878@linaro-xps13.localdomain> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.203.181.151] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.5835A173.0387,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 166872ce1d3a2c8dee614845ff850f66 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5509 Lines: 136 Hi Graeme > -----Original Message----- > From: Graeme Gregory [mailto:gg@slimlogic.co.uk] > Sent: 23 November 2016 09:44 > To: Gabriele Paoloni > Cc: Tomasz Nowicki; liudongdong (C); helgaas@kernel.org; arnd@arndb.de; > rafael@kernel.org; Lorenzo.Pieralisi@arm.com; Wangzhou (B); > pratyush.anand@gmail.com; linux-pci@vger.kernel.org; linux- > acpi@vger.kernel.org; linux-kernel@vger.kernel.org; jcm@redhat.com; > Chenxin (Charles); hanjun.guo@linaro.org; Linuxarm > Subject: Re: [PATCH V6 2/2] PCI/ACPI: hisi: Add ACPI support for > HiSilicon SoCs Host Controllers > > On Tue, Nov 22, 2016 at 03:05:29PM +0000, Gabriele Paoloni wrote: > > Hi Tomasz > > > > > -----Original Message----- > > > From: Tomasz Nowicki [mailto:tn@semihalf.com] > > > Sent: 22 November 2016 13:58 > > > To: liudongdong (C); helgaas@kernel.org; arnd@arndb.de; > > > rafael@kernel.org; Lorenzo.Pieralisi@arm.com; Wangzhou (B); > > > pratyush.anand@gmail.com > > > Cc: linux-pci@vger.kernel.org; linux-acpi@vger.kernel.org; linux- > > > kernel@vger.kernel.org; jcm@redhat.com; Gabriele Paoloni; Chenxin > > > (Charles); hanjun.guo@linaro.org; Linuxarm > > > Subject: Re: [PATCH V6 2/2] PCI/ACPI: hisi: Add ACPI support for > > > HiSilicon SoCs Host Controllers > > > > > > On 22.11.2016 13:08, Dongdong Liu wrote: > > > > PCIe controller in Hip05/HIP06/HIP07 SoCs is not ECAM compliant. > > > > It is non ECAM only for the RC bus config space;for any other bus > > > > underneath the root bus we support ECAM access. > > > > Add specific quirks for PCI config space accessors.This involves: > > > > 1. New initialization call hisi_pcie_init() to obtain rc base > > > > addresses from PNP0C02 at the root of the ACPI namespace (under > > > \_SB). > > > > 2. New entry in common quirk array. > > > > > > > > Signed-off-by: Dongdong Liu > > > > Signed-off-by: Gabriele Paoloni > > > > --- > > > > MAINTAINERS | 1 + > > > > drivers/acpi/pci_mcfg.c | 13 +++++ > > > > drivers/pci/host/Kconfig | 7 +++ > > > > drivers/pci/host/Makefile | 1 + > > > > drivers/pci/host/pcie-hisi-acpi.c | 119 > > > ++++++++++++++++++++++++++++++++++++++ > > > > include/linux/pci-ecam.h | 5 ++ > > > > 6 files changed, 146 insertions(+) > > > > create mode 100644 drivers/pci/host/pcie-hisi-acpi.c > > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > index 1cd38a7..b224caa 100644 > > > > --- a/MAINTAINERS > > > > +++ b/MAINTAINERS > > > > @@ -9358,6 +9358,7 @@ L: linux-pci@vger.kernel.org > > > > S: Maintained > > > > F: Documentation/devicetree/bindings/pci/hisilicon-pcie.txt > > > > F: drivers/pci/host/pcie-hisi.c > > > > +F: drivers/pci/host/pcie-hisi-acpi.c > > > > > > > > PCIE DRIVER FOR ROCKCHIP > > > > M: Shawn Lin > > > > diff --git a/drivers/acpi/pci_mcfg.c b/drivers/acpi/pci_mcfg.c > > > > index ac21db3..3297c5a 100644 > > > > --- a/drivers/acpi/pci_mcfg.c > > > > +++ b/drivers/acpi/pci_mcfg.c > > > > @@ -57,6 +57,19 @@ struct mcfg_fixup { > > > > { "QCOM ", "QDF2432 ", 1, 5, MCFG_BUS_ANY, &pci_32b_ops }, > > > > { "QCOM ", "QDF2432 ", 1, 6, MCFG_BUS_ANY, &pci_32b_ops }, > > > > { "QCOM ", "QDF2432 ", 1, 7, MCFG_BUS_ANY, &pci_32b_ops }, > > > > +#ifdef CONFIG_PCI_ECAM_QUIRKS > > > > + #define PCI_ACPI_QUIRK_QUAD_DOM(table_id, seg, ops) \ > > > > + { "HISI ", table_id, 0, seg + 0, MCFG_BUS_ANY, ops }, \ > > > > + { "HISI ", table_id, 0, seg + 1, MCFG_BUS_ANY, ops }, \ > > > > + { "HISI ", table_id, 0, seg + 2, MCFG_BUS_ANY, ops }, \ > > > > + { "HISI ", table_id, 0, seg + 3, MCFG_BUS_ANY, ops } > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP05 ", 0, &hisi_pcie_ops), > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP06 ", 0, &hisi_pcie_ops), > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP07 ", 0, &hisi_pcie_ops), > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP07 ", 4, &hisi_pcie_ops), > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP07 ", 8, &hisi_pcie_ops), > > > > + PCI_ACPI_QUIRK_QUAD_DOM("HIP07 ", 12, &hisi_pcie_ops), > > > > +#endif > > > > }; > > > > > > > > static char mcfg_oem_id[ACPI_OEM_ID_SIZE]; > > > > diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig > > > > index ae98644..1fbade5 100644 > > > > --- a/drivers/pci/host/Kconfig > > > > +++ b/drivers/pci/host/Kconfig > > > > @@ -301,4 +301,11 @@ config VMD > > > > To compile this driver as a module, choose M here: the > > > > module will be called vmd. > > > > > > > > +config PCI_ECAM_QUIRKS > > > > + bool "PCI ECAM quirks for ARM64 platform" > > > > + depends on PCI_ECAM && ARM64 && ACPI > > > > + help > > > > + Say y here to enable your platform-specific config access > that > > > > + is not ECAM compliant. > > > > + > > > > > > ARM64 selects PCI_ECAM for ACPI anyway so we can skip it here. > > > > Yes sure > > > > > > > > How about selecting PCI_ECAM_QUIRKS from ARM64 Kconfig: > > > > I don't think we should assume that ARM64 platforms should > > support quirks by default... > > i.e. from my perspective the quirk module should go in only > > if strictly required by the platform you're working on. > > > > Thoughts? > > > > Thanks > > > No, same kernel multiple machines is the model, code should go in > unconditional. What about selecting it in "arch/arm64/Kconfig.platforms" ? So it seems more logical (as the quirk is not selected directly by the ARM64 architecture) and would solve the "same kernel multiple machine" issue... Thanks Gab > > Graeme