From mboxrd@z Thu Jan 1 00:00:00 1970 From: Catalin Marinas Subject: Re: [PATCH v3 3/3] arm64: Use include/asm-generic/io.h Date: Fri, 18 Jul 2014 16:50:03 +0100 Message-ID: <20140718155002.GE4608@arm.com> References: <1405508484-18303-1-git-send-email-thierry.reding@gmail.com> <1405508484-18303-3-git-send-email-thierry.reding@gmail.com> <20140717120441.GI18203@arm.com> <20140717122649.GA1006@ulmo> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from fw-tnat.austin.arm.com ([217.140.110.23]:50102 "EHLO collaborate-mta1.arm.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751376AbaGRPuZ (ORCPT ); Fri, 18 Jul 2014 11:50:25 -0400 Content-Disposition: inline In-Reply-To: <20140717122649.GA1006@ulmo> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Thierry Reding Cc: Russell King , Will Deacon , Arnd Bergmann , Stephen Boyd , "linux-arm-kernel@lists.infradead.org" , "linux-arch@vger.kernel.org" , "linux-kernel@vger.kernel.org" On Thu, Jul 17, 2014 at 01:26:51PM +0100, Thierry Reding wrote: > On Thu, Jul 17, 2014 at 01:04:41PM +0100, Catalin Marinas wrote: > > On Wed, Jul 16, 2014 at 12:01:24PM +0100, Thierry Reding wrote: > > > From: Thierry Reding > > > > > > Include the generic I/O header file so that duplicate implementations > > > can be removed. This will also help to establish consistency across more > > > architectures regarding which accessors they support. > > > > > > Acked-by: Catalin Marinas > > > Signed-off-by: Thierry Reding > > > --- > > > Changes in v3: > > > - add io{read,write}{16,32}be() to match what 32-bit ARM does > > > > It looks fine. Thanks. > > Great. Russell hasn't commented on this yet, anyone know how to get his > attention? Given he dependency on the first patch, they should either go in together or first patch in 3.17 and the rest later (but I would prefer that they all go in at the same time). I don't mind how they are merged (the arm32 tree, arm-soc or arm64). If Russell doesn't have an opinion, I can take the 1st and 3rd patches via the arm64 tree. -- Catalin