> Subject: Re: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support for > OCTEONTX2 CPT engine > > On Fri, Sep 04, 2020 at 04:36:29PM +0000, Sunil Kovvuri Goutham wrote: > > > > > > > -----Original Message----- > > > From: Herbert Xu <herb...@gondor.apana.org.au> > > > Sent: Friday, September 4, 2020 7:48 PM > > > To: Srujana Challa <scha...@marvell.com> > > > Cc: da...@davemloft.net; netdev@vger.kernel.org; linux- > > > cry...@vger.kernel.org; Suheil Chandran <schand...@marvell.com>; > > > Narayana Prasad Raju Athreya <pathr...@marvell.com>; Sunil Kovvuri > > > Goutham <sgout...@marvell.com>; Linu Cherian <lcher...@marvell.com>; > > > Geethasowjanya Akula <gak...@marvell.com>; Jerin Jacob Kollanukkaran > > > <jer...@marvell.com>; Ard Biesheuvel <a...@kernel.org> > > > Subject: Re: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support for > > > OCTEONTX2 CPT engine > > > > > > On Fri, Sep 04, 2020 at 02:14:34PM +0000, Srujana Challa wrote: > > > > > > > > Since LMT store is our platform specific, it cannot be generalized to > > > > all > > > ARM64. > > > > > > I'm not asking you to generalise it to all of ARM64. I'm asking you to > > > move > > > this into a header file under arch/arm64 that can then be shared by both > > > your > > > crypto driver and your network driver so you don't duplicate this > > > everywhere. > > > > > > > For ARM64 , except erratas other platform or machine dependent stuff are not > allowed inside arch/arm64. > > Also an earlier attempt by us to add few APIs addressing 128bit operations > > were > not allowed by ARM folks > > as they don't work in a generic way and are SOC specific. > > https://urldefense.proofpoint.com/v2/url?u=http- > 3A__lkml.iu.edu_hypermail_linux_kernel_1801.3_02211.html&d=DwIBAg&c=nKj > Wec2b6R0mOyPaz7xtfQ&r=Fj4OoD5hcKFpANhTWdwQzjT1Jpf7veC5263T47JVpnc > &m=_Crtymahn2yLpOWm4M2GAuzClE9EQVX2165DSnCx79E&s=FfyU3YaCxgJRXa > 2EzyZrTvnWFYh8K2KmifpZAQN0gJk&e= > > Maybe put it in include/linux/soc/ ? > Okay, I will move this block of code into a header file under include/linux/soc. Thanks. > Andrew
