From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [RESEND PATCH 1/4] asm-generic: io: remove {read,write} string functions Date: Fri, 26 Oct 2012 14:38:44 +0100 Message-ID: <20121026133843.GL20914@mudshark.cambridge.arm.com> References: <1350488704-3711-1-git-send-email-will.deacon@arm.com> <1350488704-3711-2-git-send-email-will.deacon@arm.com> <201210261329.03386.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <201210261329.03386.arnd@arndb.de> Sender: linux-kernel-owner@vger.kernel.org To: Arnd Bergmann Cc: "linux-kernel@vger.kernel.org" , "linux-arch@vger.kernel.org" , Mike Frysinger , Ben Herrenschmidt List-Id: linux-arch.vger.kernel.org On Fri, Oct 26, 2012 at 02:29:03PM +0100, Arnd Bergmann wrote: > On Wednesday 17 October 2012, Will Deacon wrote: > > The {read,write}s{b,w,l} functions are not defined across all > > architectures and therefore shouldn't be used by portable drivers. We > > should encourage driver writers to use the io{read,write}{8,16,32}_rep > > functions instead. > > > > This patch removes the {read,write} string functions for the generic IO > > header as they have no place in a new architecture port. > > > > Cc: Arnd Bergmann > > Cc: Mike Frysinger > > Cc: Ben Herrenschmidt > > Signed-off-by: Will Deacon > > Acked-by: Arnd Bergmann Cheers Arnd! > What should we do with the series? I can take them through the asm-generic > tree if you like, or we can let Catalin take them through the arm64 tree. Can you take patch 2/4 via asm-generic please? I'll respin the rest of the series and try and update some more drivers in the process so that the removal of the generic stream functions doesn't cause breakage for drivers that aren't arch-specific already. Thanks, Will From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:64518 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932956Ab2JZNnw (ORCPT ); Fri, 26 Oct 2012 09:43:52 -0400 Date: Fri, 26 Oct 2012 14:38:44 +0100 From: Will Deacon Subject: Re: [RESEND PATCH 1/4] asm-generic: io: remove {read,write} string functions Message-ID: <20121026133843.GL20914@mudshark.cambridge.arm.com> References: <1350488704-3711-1-git-send-email-will.deacon@arm.com> <1350488704-3711-2-git-send-email-will.deacon@arm.com> <201210261329.03386.arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201210261329.03386.arnd@arndb.de> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arnd Bergmann Cc: "linux-kernel@vger.kernel.org" , "linux-arch@vger.kernel.org" , Mike Frysinger , Ben Herrenschmidt Message-ID: <20121026133844.rcztYkp7tVVmbKShdo2LdmprFESRF7L0SyVEukryoWA@z> On Fri, Oct 26, 2012 at 02:29:03PM +0100, Arnd Bergmann wrote: > On Wednesday 17 October 2012, Will Deacon wrote: > > The {read,write}s{b,w,l} functions are not defined across all > > architectures and therefore shouldn't be used by portable drivers. We > > should encourage driver writers to use the io{read,write}{8,16,32}_rep > > functions instead. > > > > This patch removes the {read,write} string functions for the generic IO > > header as they have no place in a new architecture port. > > > > Cc: Arnd Bergmann > > Cc: Mike Frysinger > > Cc: Ben Herrenschmidt > > Signed-off-by: Will Deacon > > Acked-by: Arnd Bergmann Cheers Arnd! > What should we do with the series? I can take them through the asm-generic > tree if you like, or we can let Catalin take them through the arm64 tree. Can you take patch 2/4 via asm-generic please? I'll respin the rest of the series and try and update some more drivers in the process so that the removal of the generic stream functions doesn't cause breakage for drivers that aren't arch-specific already. Thanks, Will