From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [PATCH v3 01/17] asm-generic: io: implement relaxed accessor macros as conditional wrappers Date: Thu, 25 Sep 2014 12:44:17 +0100 Message-ID: <20140925114417.GJ20043@arm.com> References: <1411579056-16966-1-git-send-email-will.deacon@arm.com> <4501051.vdxhyhkTWv@wuerfel> <20140925103846.GI20043@arm.com> <14780144.KRcmhp8YbE@wuerfel> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <14780144.KRcmhp8YbE@wuerfel> Sender: linux-kernel-owner@vger.kernel.org To: Arnd Bergmann Cc: "linux-arch@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "benh@kernel.crashing.org" , "chris@zankel.net" , "cmetcalf@tilera.com" , "davem@davemloft.net" , "deller@gmx.de" , "dhowells@redhat.com" , "geert@linux-m68k.org" , "heiko.carstens@de.ibm.com" , "hpa@zytor.com" , "jcmvbkbc@gmail.com" , "jesper.nilsson@axis.com" , "mingo@redhat.com" , "monstr@monstr.eu" , "paulmck@linux.vnet.ibm.com" , "rdunlap@infradead.org" , "sam@ravnborg.org" , "schwidefsky@de.ibm.com" List-Id: linux-arch.vger.kernel.org On Thu, Sep 25, 2014 at 11:43:41AM +0100, Arnd Bergmann wrote: > On Thursday 25 September 2014 11:38:46 Will Deacon wrote: > > On Thu, Sep 25, 2014 at 11:32:00AM +0100, Arnd Bergmann wrote: > > > On Wednesday 24 September 2014 18:17:20 Will Deacon wrote: > > > > {read,write}{b,w,l,q}_relaxed are implemented by some architectures in > > > > order to permit memory-mapped I/O accesses with weaker barrier semantics > > > > than the non-relaxed variants. > > > > > > > > This patch adds wrappers to asm-generic so that drivers can rely on the > > > > relaxed accessors being available, even if they don't always provide > > > > weaker ordering guarantees. Since some architectures both include > > > > asm-generic/io.h and define some relaxed accessors, the definitions here > > > > are conditional for the time being. > > > > > > > > Cc: Arnd Bergmann > > > > Signed-off-by: Will Deacon > > > > --- > > > > include/asm-generic/io.h | 26 ++++++++++++++++++++++++++ > > > > 1 file changed, 26 insertions(+) > > > > > > > > > > Acked-by: Arnd Bergmann > > > > Thanks, Arnd. Do you have other patches queued for asm-generic, or shall I > > take this patch (on its own) via the arm64 tree? > > I don't have other patches queued up yet, but Thierry has another patch > for asm-generic/io.h that I want to merge, so this time I'd prefer to > take it through my tree in case there are conflicts. Ok, suits me. I'll drop this from my series when it hits mainline. Will From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:57646 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750864AbaIYLpS (ORCPT ); Thu, 25 Sep 2014 07:45:18 -0400 Date: Thu, 25 Sep 2014 12:44:17 +0100 From: Will Deacon Subject: Re: [PATCH v3 01/17] asm-generic: io: implement relaxed accessor macros as conditional wrappers Message-ID: <20140925114417.GJ20043@arm.com> References: <1411579056-16966-1-git-send-email-will.deacon@arm.com> <4501051.vdxhyhkTWv@wuerfel> <20140925103846.GI20043@arm.com> <14780144.KRcmhp8YbE@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <14780144.KRcmhp8YbE@wuerfel> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arnd Bergmann Cc: "linux-arch@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "benh@kernel.crashing.org" , "chris@zankel.net" , "cmetcalf@tilera.com" , "davem@davemloft.net" , "deller@gmx.de" , "dhowells@redhat.com" , "geert@linux-m68k.org" , "heiko.carstens@de.ibm.com" , "hpa@zytor.com" , "jcmvbkbc@gmail.com" , "jesper.nilsson@axis.com" , "mingo@redhat.com" , "monstr@monstr.eu" , "paulmck@linux.vnet.ibm.com" , "rdunlap@infradead.org" , "sam@ravnborg.org" , "schwidefsky@de.ibm.com" , "starvik@axis.com" , "takata@linux-m32r.org" , "tglx@linutronix.de" , "tony.luck@intel.com" , "daniel.thompson@linaro.org" , "broonie@linaro.org" , "linux@arm.linux.org.uk" Message-ID: <20140925114417.Et3J-3msuex7oiAlJ6Qd0Y4IPFkhVSKAJVqScc2TnKk@z> On Thu, Sep 25, 2014 at 11:43:41AM +0100, Arnd Bergmann wrote: > On Thursday 25 September 2014 11:38:46 Will Deacon wrote: > > On Thu, Sep 25, 2014 at 11:32:00AM +0100, Arnd Bergmann wrote: > > > On Wednesday 24 September 2014 18:17:20 Will Deacon wrote: > > > > {read,write}{b,w,l,q}_relaxed are implemented by some architectures in > > > > order to permit memory-mapped I/O accesses with weaker barrier semantics > > > > than the non-relaxed variants. > > > > > > > > This patch adds wrappers to asm-generic so that drivers can rely on the > > > > relaxed accessors being available, even if they don't always provide > > > > weaker ordering guarantees. Since some architectures both include > > > > asm-generic/io.h and define some relaxed accessors, the definitions here > > > > are conditional for the time being. > > > > > > > > Cc: Arnd Bergmann > > > > Signed-off-by: Will Deacon > > > > --- > > > > include/asm-generic/io.h | 26 ++++++++++++++++++++++++++ > > > > 1 file changed, 26 insertions(+) > > > > > > > > > > Acked-by: Arnd Bergmann > > > > Thanks, Arnd. Do you have other patches queued for asm-generic, or shall I > > take this patch (on its own) via the arm64 tree? > > I don't have other patches queued up yet, but Thierry has another patch > for asm-generic/io.h that I want to merge, so this time I'd prefer to > take it through my tree in case there are conflicts. Ok, suits me. I'll drop this from my series when it hits mainline. Will