From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 00/20] PCI: fix config and I/O Address space memory mappings Date: Wed, 1 Mar 2017 17:18:27 +0100 Message-ID: References: <20170227151436.18698-1-lorenzo.pieralisi@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170227151436.18698-1-lorenzo.pieralisi@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Lorenzo Pieralisi Cc: Pratyush Anand , Gabriele Paoloni , Catalin Marinas , Shawn Lin , Will Deacon , Michal Simek , Thierry Reding , Tanmay Inamdar , linux-arch , Joao Pinto , Jonathan Corbet , Wenrui Li , Russell King , Jon Mason , Murali Karicheri , linux-pci , Bharat Kumar Gogada , Ray Jui , John Garry , Bjorn Helgaas , Mingkai Hu , Linux ARM , Thomas List-Id: linux-arch.vger.kernel.org On Mon, Feb 27, 2017 at 4:14 PM, Lorenzo Pieralisi wrote: > This patch series[1] addresses both issues in one go: > > - It updates the pci_remap_iospace() function to use a page mapping > that guarantees non-posted write transactions for I/O space addresses > - It adds a kernel API to remap PCI config space resources, so that > architecture can override it with a mapping implementation that > guarantees PCI specifications compliancy wrt non-posted write > configuration transactions > - It updates all PCI host controller implementations (and the generic > ECAM layer) to use the newly introduced mapping interface > > Tested on Juno ECAM based interface (DT/ACPI). This looks all good to me, nice work! Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ot0-f194.google.com ([74.125.82.194]:36478 "EHLO mail-ot0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752486AbdCAQ04 (ORCPT ); Wed, 1 Mar 2017 11:26:56 -0500 MIME-Version: 1.0 In-Reply-To: <20170227151436.18698-1-lorenzo.pieralisi@arm.com> References: <20170227151436.18698-1-lorenzo.pieralisi@arm.com> From: Arnd Bergmann Date: Wed, 1 Mar 2017 17:18:27 +0100 Message-ID: Subject: Re: [PATCH 00/20] PCI: fix config and I/O Address space memory mappings Content-Type: text/plain; charset=UTF-8 Sender: linux-arch-owner@vger.kernel.org List-ID: To: Lorenzo Pieralisi Cc: linux-pci , Linux ARM , Linux Kernel Mailing List , linux-arch , Pratyush Anand , Jonathan Corbet , Will Deacon , Jingoo Han , Bjorn Helgaas , Mingkai Hu , Tanmay Inamdar , Murali Karicheri , Russell King , Bharat Kumar Gogada , Ray Jui , Wenrui Li , Shawn Lin , Minghuan Lian , Catalin Marinas , Jon Mason , Gabriele Paoloni , Thomas Petazzoni , Joao Pinto , Thierry Reding , Michal Simek , Stanimir Varbanov , Zhou Wang , Roy Zang , John Garry Message-ID: <20170301161827.QooO624mrFDQWh5BiLMEo-JqiJNKtlPPL5FAb3z_2VY@z> On Mon, Feb 27, 2017 at 4:14 PM, Lorenzo Pieralisi wrote: > This patch series[1] addresses both issues in one go: > > - It updates the pci_remap_iospace() function to use a page mapping > that guarantees non-posted write transactions for I/O space addresses > - It adds a kernel API to remap PCI config space resources, so that > architecture can override it with a mapping implementation that > guarantees PCI specifications compliancy wrt non-posted write > configuration transactions > - It updates all PCI host controller implementations (and the generic > ECAM layer) to use the newly introduced mapping interface > > Tested on Juno ECAM based interface (DT/ACPI). This looks all good to me, nice work! Arnd