From: Horia Ioan Geanta Neag Subject: Re: [PATCH v2 4/8] powerpc: add io{read,write}64 accessors Date: Mon, 9 May 2016 08:20:13 +0000 Message-ID: References: <1462462435-27403-1-git-send-email-horia.geanta@nxp.com> <1462462573-27779-1-git-send-email-horia.geanta@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-2 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: "linux-kernel@vger.kernel.org" , "Cristian Stoica" , Scott Wood , "linux-crypto@vger.kernel.org" , "Tudor-Dan Ambarus" , "linuxppc-dev@lists.ozlabs.org" , "David S. Miller" , Alexandru Porosanu To: Herbert Xu , Benjamin Herrenschmidt , Paul Mackerras , "Michael Ellerman" Return-path: Received: from mail-am1on0097.outbound.protection.outlook.com ([157.56.112.97]:7872 "EHLO emea01-am1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750934AbcEIIxQ convert rfc822-to-8bit (ORCPT ); Mon, 9 May 2016 04:53:16 -0400 Content-Language: en-US Sender: linux-crypto-owner@vger.kernel.org List-ID: On 5/5/2016 6:37 PM, Horia Geant=E3 wrote: > This will allow device drivers to consistently use io{read,write}XX > also for 64-bit accesses. >=20 > Signed-off-by: Horia Geant=E3 It would be great if PPC maintainers could Ack this patch. As stated in the cover letter: https://lkml.org/lkml/2016/5/5/340 I'd like to go with the whole patch set via cryptodev-2.6 tree. Thanks, Horia > --- > arch/powerpc/kernel/iomap.c | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) >=20 > diff --git a/arch/powerpc/kernel/iomap.c b/arch/powerpc/kernel/iomap.= c > index 12e48d56f771..3963f0b68d52 100644 > --- a/arch/powerpc/kernel/iomap.c > +++ b/arch/powerpc/kernel/iomap.c > @@ -38,6 +38,18 @@ EXPORT_SYMBOL(ioread16); > EXPORT_SYMBOL(ioread16be); > EXPORT_SYMBOL(ioread32); > EXPORT_SYMBOL(ioread32be); > +#ifdef __powerpc64__ > +u64 ioread64(void __iomem *addr) > +{ > + return readq(addr); > +} > +u64 ioread64be(void __iomem *addr) > +{ > + return readq_be(addr); > +} > +EXPORT_SYMBOL(ioread64); > +EXPORT_SYMBOL(ioread64be); > +#endif /* __powerpc64__ */ > =20 > void iowrite8(u8 val, void __iomem *addr) > { > @@ -64,6 +76,18 @@ EXPORT_SYMBOL(iowrite16); > EXPORT_SYMBOL(iowrite16be); > EXPORT_SYMBOL(iowrite32); > EXPORT_SYMBOL(iowrite32be); > +#ifdef __powerpc64__ > +void iowrite64(u64 val, void __iomem *addr) > +{ > + writeq(val, addr); > +} > +void iowrite64be(u64 val, void __iomem *addr) > +{ > + writeq_be(val, addr); > +} > +EXPORT_SYMBOL(iowrite64); > +EXPORT_SYMBOL(iowrite64be); > +#endif /* __powerpc64__ */ > =20 > /* > * These are the "repeat read/write" functions. Note the >=20