[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <EE11001F9E5DDD47B7634E2F8A612F2E1F7D730E@lhreml507-mbx>
Date: Tue, 12 Jul 2016 10:32:33 +0000
From: Gabriele Paoloni <gabriele.paoloni@...wei.com>
To: Gabriele Paoloni <gabriele.paoloni@...wei.com>,
Arnd Bergmann <arnd@...db.de>,
"liudongdong (C)" <liudongdong3@...wei.com>
CC: "Lorenzo.Pieralisi@....com" <Lorenzo.Pieralisi@....com>,
"Chenxin (Charles)" <charles.chenxin@...wei.com>,
"rafael@...nel.org" <rafael@...nel.org>,
"tn@...ihalf.com" <tn@...ihalf.com>,
"pratyush.anand@...il.com" <pratyush.anand@...il.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Linuxarm <linuxarm@...wei.com>,
"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
"helgaas@...nel.org" <helgaas@...nel.org>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"jcm@...hat.com" <jcm@...hat.com>
Subject: RE: [RFC PATCH 3/3] PCI/ACPI: hisi: Add ACPI support for HiSilicon
SoCs Host Controllers
> -----Original Message-----
> From: linuxarm-bounces@...wei.com [mailto:linuxarm-bounces@...wei.com]
> On Behalf Of Gabriele Paoloni
> Sent: 12 July 2016 18:22
> To: Arnd Bergmann; liudongdong (C)
> Cc: Lorenzo.Pieralisi@....com; Chenxin (Charles); rafael@...nel.org;
> tn@...ihalf.com; pratyush.anand@...il.com; linux-kernel@...r.kernel.org;
> Linuxarm; linux-acpi@...r.kernel.org; helgaas@...nel.org; linux-
> pci@...r.kernel.org; jcm@...hat.com
> Subject: RE: [RFC PATCH 3/3] PCI/ACPI: hisi: Add ACPI support for
> HiSilicon SoCs Host Controllers
>
> > -----Original Message-----
> > From: Arnd Bergmann [mailto:arnd@...db.de]
> > Sent: 12 July 2016 16:35
> > To: liudongdong (C)
> > Cc: helgaas@...nel.org; rafael@...nel.org; Lorenzo.Pieralisi@....com;
> > tn@...ihalf.com; Wangzhou (B); pratyush.anand@...il.com; linux-
> > pci@...r.kernel.org; linux-acpi@...r.kernel.org; linux-
> > kernel@...r.kernel.org; jcm@...hat.com; Gabriele Paoloni; Chenxin
> > (Charles); Linuxarm
> > Subject: Re: [RFC PATCH 3/3] PCI/ACPI: hisi: Add ACPI support for
> > HiSilicon SoCs Host Controllers
> >
> > On Tuesday, July 12, 2016 10:42:24 AM CEST Dongdong Liu wrote:
> > > MAINTAINERS | 1 +
> > > drivers/pci/host/Kconfig | 7 ++
> > > drivers/pci/host/Makefile | 1 +
> > > drivers/pci/host/mcfg-quirks.c | 8 ++
> > > drivers/pci/host/mcfg-quirks.h | 8 ++
> > > drivers/pci/host/pcie-hisi-acpi.c | 151
> > ++++++++++++++++++++++++++++++++++++++
> > > drivers/pci/host/pcie-hisi.c | 2 -
> > > drivers/pci/host/pcie-hisi.h | 2 +
> > > 8 files changed, 178 insertions(+), 2 deletions(-)
> > > create mode 100644 drivers/pci/host/pcie-hisi-acpi.c
> > >
> > >
> >
> > Please keep the quirks separate from the PCI host drivers.
> >
> > The ACPI code includes its own host driver and the only thing
> > you need to override here is the config space access, so just put
> that
> > into drivers/acpi/
>
> Hi Arnd, thanks for replying.
>
> Basically we just followed what Tomasz has done in
> "[RFC PATCH v4 0/5] ECAM quirks handling for ARM64 platforms"
>
> In this case the ThunderX quirks have been defined in
> "drivers/pci/host/pci-thunder-pem.c".
>
Also on top of this I think that from
"Re: [RFC PATCH v3 1/2] ACPI/PCI: Check platform specific ECAM quirks"
Lorenzo suggested to add the static array of quirks (and therefore the
respective quirk mechanisms in "drivers/pci/host").
See:
[...]
It should be easier to implement (provided we find a place where
to add this static array of hooks matching MCFG, I suspect it is
going to be a file in drivers/pci/host but Tomasz and I need
input on that) and prevent abuse (since it is a static array of
hooks in a single place, it is easier to manage than section
entries).
[...]
Thanks
Gab
> Cheers
> Gab
>
> >
> > Arnd
>
> _______________________________________________
> linuxarm mailing list
> linuxarm@...wei.com
> http://rnd-openeuler.huawei.com/mailman/listinfo/linuxarm
Powered by blists - more mailing lists