From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Nowicki Subject: Re: [PATCH V7 00/11] Support for generic ACPI based PCI host controller Date: Wed, 11 May 2016 13:08:00 +0200 Message-ID: <57331290.7070104@semihalf.com> References: <1462893601-8937-1-git-send-email-tn@semihalf.com> Mime-Version: 1.0 Content-Type: text/plain; charset=gbk; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-pci-owner@vger.kernel.org To: Gabriele Paoloni , "helgaas@kernel.org" , "arnd@arndb.de" , "will.deacon@arm.com" , "catalin.marinas@arm.com" , "rafael@kernel.org" , "hanjun.guo@linaro.org" , "Lorenzo.Pieralisi@arm.com" , "okaya@codeaurora.org" , "jchandra@broadcom.com" Cc: "robert.richter@caviumnetworks.com" , "mw@semihalf.com" , "Liviu.Dudau@arm.com" , "ddaney@caviumnetworks.com" , Wangyijing , "Suravee.Suthikulpanit@amd.com" , "msalter@redhat.com" , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linaro-acpi@lists.linaro.org" , "jcm@redhat.com" , "andrea.gallo@linaro.org" , "dhdang@apm.com" , "jeremy.linton@arm.com" , "liudongdong (C)" List-Id: linux-acpi@vger.kernel.org Hi Gabriele, On 11.05.2016 12:41, Gabriele Paoloni wrote: > Hi Tomasz > >> -----Original Message----- >> From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel- >> owner@vger.kernel.org] On Behalf Of Tomasz Nowicki >> Sent: 10 May 2016 16:20 >> To: helgaas@kernel.org; arnd@arndb.de; will.deacon@arm.com; >> catalin.marinas@arm.com; rafael@kernel.org; hanjun.guo@linaro.org; >> Lorenzo.Pieralisi@arm.com; okaya@codeaurora.org; jchandra@broadcom.c= om >> Cc: robert.richter@caviumnetworks.com; mw@semihalf.com; >> Liviu.Dudau@arm.com; ddaney@caviumnetworks.com; Wangyijing; >> Suravee.Suthikulpanit@amd.com; msalter@redhat.com; linux- >> pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux- >> acpi@vger.kernel.org; linux-kernel@vger.kernel.org; linaro- >> acpi@lists.linaro.org; jcm@redhat.com; andrea.gallo@linaro.org; >> dhdang@apm.com; jeremy.linton@arm.com; liudongdong (C); >> cov@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 >> 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@github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v7) >> >> This has been tested on Cavium ThunderX server. Any help in reviewin= g >> 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. > > I think it is important to have this in place to accommodate differen= t > vendors. If the intention is to keep this patchset "clean" maybe > we can add it as a separate patch on top later on... > > What=A1=AFs your view? Yes, keeping these things separated should help in review. Obviously I=20 agree that we need quirk handling but currently there is no=20 implementation which we all agree upon. For the test, you can use quirk= =20 handling approach from the previous series until we sort out final solu= tion. Thanks, Tomasz From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752567AbcEKLIJ (ORCPT ); Wed, 11 May 2016 07:08:09 -0400 Received: from mail-lf0-f54.google.com ([209.85.215.54]:35922 "EHLO mail-lf0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751552AbcEKLIF (ORCPT ); Wed, 11 May 2016 07:08:05 -0400 Subject: Re: [PATCH V7 00/11] Support for generic ACPI based PCI host controller To: Gabriele Paoloni , "helgaas@kernel.org" , "arnd@arndb.de" , "will.deacon@arm.com" , "catalin.marinas@arm.com" , "rafael@kernel.org" , "hanjun.guo@linaro.org" , "Lorenzo.Pieralisi@arm.com" , "okaya@codeaurora.org" , "jchandra@broadcom.com" References: <1462893601-8937-1-git-send-email-tn@semihalf.com> Cc: "robert.richter@caviumnetworks.com" , "mw@semihalf.com" , "Liviu.Dudau@arm.com" , "ddaney@caviumnetworks.com" , Wangyijing , "Suravee.Suthikulpanit@amd.com" , "msalter@redhat.com" , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linaro-acpi@lists.linaro.org" , "jcm@redhat.com" , "andrea.gallo@linaro.org" , "dhdang@apm.com" , "jeremy.linton@arm.com" , "liudongdong (C)" , "cov@codeaurora.org" From: Tomasz Nowicki Message-ID: <57331290.7070104@semihalf.com> Date: Wed, 11 May 2016 13:08:00 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=gbk; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Gabriele, On 11.05.2016 12:41, Gabriele Paoloni wrote: > Hi Tomasz > >> -----Original Message----- >> From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel- >> owner@vger.kernel.org] On Behalf Of Tomasz Nowicki >> Sent: 10 May 2016 16:20 >> To: helgaas@kernel.org; arnd@arndb.de; will.deacon@arm.com; >> catalin.marinas@arm.com; rafael@kernel.org; hanjun.guo@linaro.org; >> Lorenzo.Pieralisi@arm.com; okaya@codeaurora.org; jchandra@broadcom.com >> Cc: robert.richter@caviumnetworks.com; mw@semihalf.com; >> Liviu.Dudau@arm.com; ddaney@caviumnetworks.com; Wangyijing; >> Suravee.Suthikulpanit@amd.com; msalter@redhat.com; linux- >> pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux- >> acpi@vger.kernel.org; linux-kernel@vger.kernel.org; linaro- >> acpi@lists.linaro.org; jcm@redhat.com; andrea.gallo@linaro.org; >> dhdang@apm.com; jeremy.linton@arm.com; liudongdong (C); >> cov@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 >> 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@github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v7) >> >> This has been tested on Cavium ThunderX server. Any help in reviewing >> 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. > > I think it is important to have this in place to accommodate different > 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. Thanks, Tomasz From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Subject: Re: [PATCH V7 00/11] Support for generic ACPI based PCI host controller To: Gabriele Paoloni , "helgaas@kernel.org" , "arnd@arndb.de" , "will.deacon@arm.com" , "catalin.marinas@arm.com" , "rafael@kernel.org" , "hanjun.guo@linaro.org" , "Lorenzo.Pieralisi@arm.com" , "okaya@codeaurora.org" , "jchandra@broadcom.com" References: <1462893601-8937-1-git-send-email-tn@semihalf.com> Cc: "robert.richter@caviumnetworks.com" , "mw@semihalf.com" , "Liviu.Dudau@arm.com" , "ddaney@caviumnetworks.com" , Wangyijing , "Suravee.Suthikulpanit@amd.com" , "msalter@redhat.com" , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linaro-acpi@lists.linaro.org" , "jcm@redhat.com" , "andrea.gallo@linaro.org" , "dhdang@apm.com" , "jeremy.linton@arm.com" , "liudongdong (C)" , "cov@codeaurora.org" From: Tomasz Nowicki Message-ID: <57331290.7070104@semihalf.com> Date: Wed, 11 May 2016 13:08:00 +0200 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=gbk; format=flowed List-ID: Hi Gabriele, On 11.05.2016 12:41, Gabriele Paoloni wrote: > Hi Tomasz > >> -----Original Message----- >> From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel- >> owner@vger.kernel.org] On Behalf Of Tomasz Nowicki >> Sent: 10 May 2016 16:20 >> To: helgaas@kernel.org; arnd@arndb.de; will.deacon@arm.com; >> catalin.marinas@arm.com; rafael@kernel.org; hanjun.guo@linaro.org; >> Lorenzo.Pieralisi@arm.com; okaya@codeaurora.org; jchandra@broadcom.com >> Cc: robert.richter@caviumnetworks.com; mw@semihalf.com; >> Liviu.Dudau@arm.com; ddaney@caviumnetworks.com; Wangyijing; >> Suravee.Suthikulpanit@amd.com; msalter@redhat.com; linux- >> pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux- >> acpi@vger.kernel.org; linux-kernel@vger.kernel.org; linaro- >> acpi@lists.linaro.org; jcm@redhat.com; andrea.gallo@linaro.org; >> dhdang@apm.com; jeremy.linton@arm.com; liudongdong (C); >> cov@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 >> 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@github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v7) >> >> This has been tested on Cavium ThunderX server. Any help in reviewing >> 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. > > I think it is important to have this in place to accommodate different > 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. Thanks, Tomasz From mboxrd@z Thu Jan 1 00:00:00 1970 From: tn@semihalf.com (Tomasz Nowicki) Date: Wed, 11 May 2016 13:08:00 +0200 Subject: [PATCH V7 00/11] Support for generic ACPI based PCI host controller In-Reply-To: References: <1462893601-8937-1-git-send-email-tn@semihalf.com> Message-ID: <57331290.7070104@semihalf.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Gabriele, On 11.05.2016 12:41, Gabriele Paoloni wrote: > Hi Tomasz > >> -----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 >> 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 >> 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 >> 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. > > I think it is important to have this in place to accommodate different > 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. Thanks, Tomasz