From: Dongdong Liu <liudongdong3@huawei.com>
To: Tomasz Nowicki <tn@semihalf.com>,
helgaas@kernel.org, arnd@arndb.de, will.deacon@arm.com,
catalin.marinas@arm.com, rafael@kernel.org,
Lorenzo.Pieralisi@arm.com
Cc: hanjun.guo@linaro.org, okaya@codeaurora.org,
jchandra@broadcom.com, cov@codeaurora.org, dhdang@apm.com,
ard.biesheuvel@linaro.org, robert.richter@caviumnetworks.com,
mw@semihalf.com, Liviu.Dudau@arm.com, ddaney@caviumnetworks.com,
wangyijing@huawei.com, msalter@redhat.com,
linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linaro-acpi@lists.linaro.org, jcm@redhat.com,
andrea.gallo@linaro.org, jeremy.linton@arm.com,
gabriele.paoloni@huawei.com, jhugo@codeaurora.org,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
Linuxarm <linuxarm@huawei.com>
Subject: Re: [RFC PATCH V5 0/5] ECAM quirks handling for ARM64 platforms
Date: Tue, 9 Aug 2016 17:20:42 +0800 [thread overview]
Message-ID: <57A9A06A.30907@huawei.com> (raw)
In-Reply-To: <1470661541-26270-1-git-send-email-tn@semihalf.com>
在 2016/8/8 21:05, Tomasz Nowicki 写道:
> Quirk handling relies on an idea of matching MCFG OEM ID, TABLE ID and
> revision (the ones from standard header of MCFG table).
>
> Static array is used to keep quirk entries. Each entry consists of
> mentioned MCFG IDs along with custom pci_ops structure and initialization call.
>
> As an example, the last patch presents quirk handling mechanism usage for
> ThunderX PEM driver.
>
> v4 -> v5
> - rebase against v4.8-rc1
> - rework to exact MCFG OEM ID, TABLE ID, rev match
> - use memcmp instead of strncmp
> - no substring match
> - fix typos and dmesg message
>
> Tomasz Nowicki (5):
> PCI: Embed pci_ecam_ops in pci_config_window structure
> PCI/ACPI: Move ACPI ECAM mapping to generic MCFG driver
> PCI: Check platform specific ECAM quirks
> ARM64/PCI: Start using quirks handling for ACPI based PCI host
> controller
> PCI: thunder-pem: Support quirky configuration space access for ACPI
> based PCI host controller
>
> arch/arm64/kernel/pci.c | 42 +----------------
> drivers/acpi/pci_mcfg.c | 40 ++++++++++++++++
> drivers/pci/ecam.c | 6 +--
> drivers/pci/host/Makefile | 1 +
> drivers/pci/host/mcfg-quirks.c | 93 ++++++++++++++++++++++++++++++++++++
> drivers/pci/host/mcfg-quirks.h | 24 ++++++++++
> drivers/pci/host/pci-thunder-pem.c | 96 ++++++++++++++++++++++++++++++++------
> include/linux/pci-acpi.h | 5 ++
> include/linux/pci-ecam.h | 2 +-
> 9 files changed, 252 insertions(+), 57 deletions(-)
> create mode 100644 drivers/pci/host/mcfg-quirks.c
> create mode 100644 drivers/pci/host/mcfg-quirks.h
>
Based on the patchset, tested on HiSilicon D03 board with intel 82599 net card. It worked OK.
Tested-by: Dongdong Liu <liudongdong3@huawei.com>
Thanks
Dongdong
WARNING: multiple messages have this Message-ID (diff)
From: Dongdong Liu <liudongdong3@huawei.com>
To: Tomasz Nowicki <tn@semihalf.com>, <helgaas@kernel.org>,
<arnd@arndb.de>, <will.deacon@arm.com>, <catalin.marinas@arm.com>,
<rafael@kernel.org>, <Lorenzo.Pieralisi@arm.com>
Cc: <hanjun.guo@linaro.org>, <okaya@codeaurora.org>,
<jchandra@broadcom.com>, <cov@codeaurora.org>, <dhdang@apm.com>,
<ard.biesheuvel@linaro.org>, <robert.richter@caviumnetworks.com>,
<mw@semihalf.com>, <Liviu.Dudau@arm.com>,
<ddaney@caviumnetworks.com>, <wangyijing@huawei.com>,
<msalter@redhat.com>, <linux-pci@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linaro-acpi@lists.linaro.org>, <jcm@redhat.com>,
<andrea.gallo@linaro.org>, <jeremy.linton@arm.com>,
<gabriele.paoloni@huawei.com>, <jhugo@codeaurora.org>,
<linux-acpi@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Linuxarm <linuxarm@huawei.com>
Subject: Re: [RFC PATCH V5 0/5] ECAM quirks handling for ARM64 platforms
Date: Tue, 9 Aug 2016 17:20:42 +0800 [thread overview]
Message-ID: <57A9A06A.30907@huawei.com> (raw)
In-Reply-To: <1470661541-26270-1-git-send-email-tn@semihalf.com>
在 2016/8/8 21:05, Tomasz Nowicki 写道:
> Quirk handling relies on an idea of matching MCFG OEM ID, TABLE ID and
> revision (the ones from standard header of MCFG table).
>
> Static array is used to keep quirk entries. Each entry consists of
> mentioned MCFG IDs along with custom pci_ops structure and initialization call.
>
> As an example, the last patch presents quirk handling mechanism usage for
> ThunderX PEM driver.
>
> v4 -> v5
> - rebase against v4.8-rc1
> - rework to exact MCFG OEM ID, TABLE ID, rev match
> - use memcmp instead of strncmp
> - no substring match
> - fix typos and dmesg message
>
> Tomasz Nowicki (5):
> PCI: Embed pci_ecam_ops in pci_config_window structure
> PCI/ACPI: Move ACPI ECAM mapping to generic MCFG driver
> PCI: Check platform specific ECAM quirks
> ARM64/PCI: Start using quirks handling for ACPI based PCI host
> controller
> PCI: thunder-pem: Support quirky configuration space access for ACPI
> based PCI host controller
>
> arch/arm64/kernel/pci.c | 42 +----------------
> drivers/acpi/pci_mcfg.c | 40 ++++++++++++++++
> drivers/pci/ecam.c | 6 +--
> drivers/pci/host/Makefile | 1 +
> drivers/pci/host/mcfg-quirks.c | 93 ++++++++++++++++++++++++++++++++++++
> drivers/pci/host/mcfg-quirks.h | 24 ++++++++++
> drivers/pci/host/pci-thunder-pem.c | 96 ++++++++++++++++++++++++++++++++------
> include/linux/pci-acpi.h | 5 ++
> include/linux/pci-ecam.h | 2 +-
> 9 files changed, 252 insertions(+), 57 deletions(-)
> create mode 100644 drivers/pci/host/mcfg-quirks.c
> create mode 100644 drivers/pci/host/mcfg-quirks.h
>
Based on the patchset, tested on HiSilicon D03 board with intel 82599 net card. It worked OK.
Tested-by: Dongdong Liu <liudongdong3@huawei.com>
Thanks
Dongdong
WARNING: multiple messages have this Message-ID (diff)
From: liudongdong3@huawei.com (Dongdong Liu)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH V5 0/5] ECAM quirks handling for ARM64 platforms
Date: Tue, 9 Aug 2016 17:20:42 +0800 [thread overview]
Message-ID: <57A9A06A.30907@huawei.com> (raw)
In-Reply-To: <1470661541-26270-1-git-send-email-tn@semihalf.com>
? 2016/8/8 21:05, Tomasz Nowicki ??:
> Quirk handling relies on an idea of matching MCFG OEM ID, TABLE ID and
> revision (the ones from standard header of MCFG table).
>
> Static array is used to keep quirk entries. Each entry consists of
> mentioned MCFG IDs along with custom pci_ops structure and initialization call.
>
> As an example, the last patch presents quirk handling mechanism usage for
> ThunderX PEM driver.
>
> v4 -> v5
> - rebase against v4.8-rc1
> - rework to exact MCFG OEM ID, TABLE ID, rev match
> - use memcmp instead of strncmp
> - no substring match
> - fix typos and dmesg message
>
> Tomasz Nowicki (5):
> PCI: Embed pci_ecam_ops in pci_config_window structure
> PCI/ACPI: Move ACPI ECAM mapping to generic MCFG driver
> PCI: Check platform specific ECAM quirks
> ARM64/PCI: Start using quirks handling for ACPI based PCI host
> controller
> PCI: thunder-pem: Support quirky configuration space access for ACPI
> based PCI host controller
>
> arch/arm64/kernel/pci.c | 42 +----------------
> drivers/acpi/pci_mcfg.c | 40 ++++++++++++++++
> drivers/pci/ecam.c | 6 +--
> drivers/pci/host/Makefile | 1 +
> drivers/pci/host/mcfg-quirks.c | 93 ++++++++++++++++++++++++++++++++++++
> drivers/pci/host/mcfg-quirks.h | 24 ++++++++++
> drivers/pci/host/pci-thunder-pem.c | 96 ++++++++++++++++++++++++++++++++------
> include/linux/pci-acpi.h | 5 ++
> include/linux/pci-ecam.h | 2 +-
> 9 files changed, 252 insertions(+), 57 deletions(-)
> create mode 100644 drivers/pci/host/mcfg-quirks.c
> create mode 100644 drivers/pci/host/mcfg-quirks.h
>
Based on the patchset, tested on HiSilicon D03 board with intel 82599 net card. It worked OK.
Tested-by: Dongdong Liu <liudongdong3@huawei.com>
Thanks
Dongdong
next prev parent reply other threads:[~2016-08-09 9:21 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-08 13:05 [RFC PATCH V5 0/5] ECAM quirks handling for ARM64 platforms Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-08-08 13:05 ` [RFC PATCH V5 1/5] PCI: Embed pci_ecam_ops in pci_config_window structure Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-09-01 18:23 ` Bjorn Helgaas
2016-09-01 18:23 ` Bjorn Helgaas
2016-09-01 18:23 ` Bjorn Helgaas
2016-09-01 18:23 ` Bjorn Helgaas
2016-09-02 15:38 ` Lorenzo Pieralisi
2016-09-02 15:38 ` Lorenzo Pieralisi
2016-09-05 2:22 ` Bjorn Helgaas
2016-09-05 2:22 ` Bjorn Helgaas
2016-08-08 13:05 ` [RFC PATCH V5 2/5] PCI/ACPI: Move ACPI ECAM mapping to generic MCFG driver Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-09-05 2:22 ` Bjorn Helgaas
2016-09-05 2:22 ` Bjorn Helgaas
2016-09-06 18:04 ` Tomasz Nowicki
2016-09-06 18:04 ` Tomasz Nowicki
2016-08-08 13:05 ` [RFC PATCH V5 3/5] PCI: Check platform specific ECAM quirks Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-08-08 15:34 ` Mark Salter
2016-08-08 15:34 ` Mark Salter
2016-08-09 6:10 ` Tomasz Nowicki
2016-08-09 6:10 ` Tomasz Nowicki
2016-09-05 2:25 ` Bjorn Helgaas
2016-09-05 2:25 ` Bjorn Helgaas
2016-09-06 17:49 ` Tomasz Nowicki
2016-09-06 17:49 ` Tomasz Nowicki
2016-09-06 19:14 ` Arnd Bergmann
2016-09-06 19:14 ` Arnd Bergmann
2016-09-05 2:59 ` Bjorn Helgaas
2016-09-05 2:59 ` Bjorn Helgaas
2016-09-05 2:59 ` Bjorn Helgaas
2016-08-08 13:05 ` [RFC PATCH V5 4/5] ARM64/PCI: Start using quirks handling for ACPI based PCI host controller Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-09-05 2:27 ` Bjorn Helgaas
2016-09-05 2:27 ` Bjorn Helgaas
2016-08-08 13:05 ` [RFC PATCH V5 5/5] PCI: thunder-pem: Support quirky configuration space access " Tomasz Nowicki
2016-08-08 13:05 ` Tomasz Nowicki
2016-08-09 11:04 ` Robert Richter
2016-08-09 11:04 ` Robert Richter
2016-08-09 11:04 ` Robert Richter
2016-08-09 11:04 ` Robert Richter
2016-09-05 2:34 ` Bjorn Helgaas
2016-09-05 2:34 ` Bjorn Helgaas
2016-09-06 18:01 ` Tomasz Nowicki
2016-09-06 18:01 ` Tomasz Nowicki
2016-08-08 15:13 ` [RFC PATCH V5 0/5] ECAM quirks handling for ARM64 platforms Graeme Gregory
2016-08-08 15:13 ` Graeme Gregory
2016-08-08 19:44 ` Mark Salter
2016-08-08 19:44 ` Mark Salter
2016-08-09 9:20 ` Dongdong Liu [this message]
2016-08-09 9:20 ` Dongdong Liu
2016-08-09 9:20 ` Dongdong Liu
2016-08-09 10:29 ` Robert Richter
2016-08-09 10:29 ` Robert Richter
2016-08-09 10:29 ` Robert Richter
2016-08-09 10:29 ` Robert Richter
2016-08-09 14:20 ` Duc Dang
2016-08-09 14:20 ` Duc Dang
2016-08-09 14:20 ` Duc Dang
-- strict thread matches above, loose matches on Subject: below --
2016-08-08 12:56 Tomasz Nowicki
2016-08-08 12:56 ` Tomasz Nowicki
2016-08-08 13:02 ` Tomasz Nowicki
2016-08-08 13:02 ` Tomasz Nowicki
2016-08-30 21:20 ` Duc Dang
2016-08-30 21:20 ` Duc Dang
2016-08-31 12:10 ` Tomasz Nowicki
2016-08-31 12:10 ` Tomasz Nowicki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57A9A06A.30907@huawei.com \
--to=liudongdong3@huawei.com \
--cc=Liviu.Dudau@arm.com \
--cc=Lorenzo.Pieralisi@arm.com \
--cc=andrea.gallo@linaro.org \
--cc=ard.biesheuvel@linaro.org \
--cc=arnd@arndb.de \
--cc=catalin.marinas@arm.com \
--cc=cov@codeaurora.org \
--cc=ddaney@caviumnetworks.com \
--cc=dhdang@apm.com \
--cc=gabriele.paoloni@huawei.com \
--cc=hanjun.guo@linaro.org \
--cc=helgaas@kernel.org \
--cc=jchandra@broadcom.com \
--cc=jcm@redhat.com \
--cc=jeremy.linton@arm.com \
--cc=jhugo@codeaurora.org \
--cc=linaro-acpi@lists.linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=msalter@redhat.com \
--cc=mw@semihalf.com \
--cc=okaya@codeaurora.org \
--cc=rafael@kernel.org \
--cc=robert.richter@caviumnetworks.com \
--cc=tn@semihalf.com \
--cc=wangyijing@huawei.com \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.