Hi Tomasz

-----Original Message-----
From: Tomasz Nowicki [mailto:tn at semihalf.com]
Sent: 11 May 2016 12:08
To: Gabriele Paoloni; helgaas at kernel.org; arnd at arndb.de;
will.deacon at arm.com; catalin.marinas at arm.com; rafael at kernel.org;
hanjun.guo at linaro.org; Lorenzo.Pieralisi at arm.com; okaya at codeaurora.org;
jchandra at broadcom.com
Cc: robert.richter at caviumnetworks.com; mw at semihalf.com;
Liviu.Dudau at arm.com; ddaney at caviumnetworks.com; Wangyijing;
Suravee.Suthikulpanit at amd.com; msalter at redhat.com; linux-
pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-
acpi at vger.kernel.org; linux-kernel at vger.kernel.org; linaro-
acpi at lists.linaro.org; jcm at redhat.com; andrea.gallo at linaro.org;
dhdang at apm.com; jeremy.linton at arm.com; liudongdong (C);
cov at codeaurora.org
Subject: Re: [PATCH V7 00/11] Support for generic ACPI based PCI host
controller

Hi Gabriele,

On 11.05.2016 12:41, Gabriele Paoloni wrote:

quoted

Hi Tomasz

quoted

-----Original Message-----
From: linux-kernel-owner at vger.kernel.org [mailto:linux-kernel-
owner at vger.kernel.org] On Behalf Of Tomasz Nowicki
Sent: 10 May 2016 16:20
To: helgaas at kernel.org; arnd at arndb.de; will.deacon at arm.com;
catalin.marinas at arm.com; rafael at kernel.org; hanjun.guo at linaro.org;
Lorenzo.Pieralisi at arm.com; okaya at codeaurora.org;

jchandra at broadcom.com

quoted

quoted

Cc: robert.richter at caviumnetworks.com; mw at semihalf.com;
Liviu.Dudau at arm.com; ddaney at caviumnetworks.com; Wangyijing;
Suravee.Suthikulpanit at amd.com; msalter at redhat.com; linux-
pci at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-
acpi at vger.kernel.org; linux-kernel at vger.kernel.org; linaro-
acpi at lists.linaro.org; jcm at redhat.com; andrea.gallo at linaro.org;
dhdang at apm.com; jeremy.linton at arm.com; liudongdong (C);
cov at codeaurora.org; Tomasz Nowicki
Subject: [PATCH V7 00/11] Support for generic ACPI based PCI host
controller

 From the functionality point of view this series may be split into

the

quoted

quoted

following logic parts:
1. New ECAM API and update for users of the pci-host-common API
2. Necessary fixes as the preparation for using driver on ARM64.
3. Use new MCFG interface and implement generic ACPI based PCI host
controller driver.
4. Enable above driver on ARM64

Patches has been built on top of 4.6-rc7 and can be found here:
git at github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v7)

This has been tested on Cavium ThunderX server. Any help in

reviewing

quoted

quoted

and
testing is very appreciated.

v6 -> v7
- drop quirks handling

Maybe I missed something in the v6 discussion thread; when was it
decided to drop quirk handling?

I had such requests in previous series.

quoted

I think it is important to have this in place to accommodate

different

quoted

vendors. If the intention is to keep this patchset "clean" maybe
we can add it as a separate patch on top later on...

What's your view?

Yes, keeping these things separated should help in review. Obviously I
agree that we need quirk handling but currently there is no
implementation which we all agree upon. For the test, you can use quirk
handling approach from the previous series until we sort out final
solution.

Great

This explains perfectly

We will apply previous series quirks and try to test the patchset
from Huawei side

Many Thanks

Gab

Thanks,
Tomasz