From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Date: Thu, 30 Mar 2017 17:19:31 +0100 From: Will Deacon To: Lorenzo Pieralisi Subject: Re: [PATCH v2 05/22] ARM64: implement ioremap_nopost() interface Message-ID: <20170330161931.GA5623@arm.com> References: <20170327094954.7162-1-lorenzo.pieralisi@arm.com> <20170327094954.7162-6-lorenzo.pieralisi@arm.com> MIME-Version: 1.0 In-Reply-To: <20170327094954.7162-6-lorenzo.pieralisi@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Wenrui Li , Gabriele Paoloni , linux-pci@vger.kernel.org, Shawn Lin , Michal Simek , Thierry Reding , Tanmay Inamdar , Joao Pinto , Pratyush Anand , Russell King , Bharat Kumar Gogada , Murali Karicheri , Catalin Marinas , Arnd Bergmann , Jon Mason , Ray Jui , John Garry , Bjorn Helgaas , Mingkai Hu , linux-arm-kernel@lists.infradead.org, "Luis R. Rodriguez" , Thomas Petazzoni , Jingoo Han , linux-kernel@vger.kernel.org, Stanimir Varbanov , Minghuan Lian , Zhou Wang , Roy Zang Content-Type: text/plain; charset="us-ascii" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+bjorn=helgaas.com@lists.infradead.org List-ID: On Mon, Mar 27, 2017 at 10:49:33AM +0100, Lorenzo Pieralisi wrote: > The PCI bus specifications (rev 3.0, 3.2.5 "Transaction Ordering > and Posting") defines rules for PCI configuration space transactions > ordering and posting, that state that configuration writes > are non-posted transactions. > > This rule is reinforced by the ARM v8 architecture reference manual > (issue A.k, Early Write Acknowledgment) that explicitly recommends > that No Early Write Acknowledgment attribute should be used to map > PCI configuration (write) transactions. > > Current ioremap interface on ARM64 implements mapping functions > where the Early Write Acknowledgment hint is enabled, so they > cannot be used to map PCI configuration space in a PCI specs > compliant way. > > Implement an ARM64 specific ioremap_nopost() interface > that allows to map PCI config region with nGnRnE attributes, providing > a remap function that complies with PCI specifications and the ARMv8 > architecture reference manual recommendations. > > Signed-off-by: Lorenzo Pieralisi > Cc: Will Deacon > Cc: Catalin Marinas > --- > arch/arm64/include/asm/io.h | 12 ++++++++++++ > 1 file changed, 12 insertions(+) Acked-by: Will Deacon Let me know if you need this taken via the arm64 tree. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel