Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Srujana Challa <schalla@marvell.com>
To:     Andrew Lunn <andrew@lunn.ch>,
        Sunil Kovvuri Goutham <sgoutham@marvell.com>
CC:     Herbert Xu <herbert@gondor.apana.org.au>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-crypto@vger.kernel.org" <linux-crypto@vger.kernel.org>,
        "Suheil Chandran" <schandran@marvell.com>,
        Narayana Prasad Raju Athreya <pathreya@marvell.com>,
        Linu Cherian <lcherian@marvell.com>,
        "Geethasowjanya Akula" <gakula@marvell.com>,
        Jerin Jacob Kollanukkaran <jerinj@marvell.com>,
        Ard Biesheuvel <ardb@kernel.org>
Subject: RE: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support for
 OCTEONTX2 CPT engine
Thread-Topic: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support for
 OCTEONTX2 CPT engine
Thread-Index: AQHWbMTfW+P0JoWQZkyWfmrU0nLSuak1PvSAgCNkG6CAAAgsAIAABVvQgAACMACAACbEgIAAIhwAgAW6ZAA=
Date:   Tue, 8 Sep 2020 14:20:52 +0000
Message-ID: <BYAPR18MB2791E9E6B435E27E8143836BA0290@BYAPR18MB2791.namprd18.prod.outlook.com>
References: <1596809360-12597-1-git-send-email-schalla@marvell.com>
 <1596809360-12597-3-git-send-email-schalla@marvell.com>
 <20200813005407.GB24593@gondor.apana.org.au>
 <BYAPR18MB2791C6451CDE93CA053E0208A02D0@BYAPR18MB2791.namprd18.prod.outlook.com>
 <20200904135044.GA2836@gondor.apana.org.au>
 <BYAPR18MB2791A52DA3BF1D7BE5574F99A02D0@BYAPR18MB2791.namprd18.prod.outlook.com>
 <20200904141744.GA3092@gondor.apana.org.au>
 <BY5PR18MB32984DAF0FDED5D9CD1BEB45C62D0@BY5PR18MB3298.namprd18.prod.outlook.com>
 <20200904183834.GS3112546@lunn.ch>
In-Reply-To: <20200904183834.GS3112546@lunn.ch>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 8c6bf6bc-0310-41bd-dd7c-08d8540264e8
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Sep 2020 14:20:52.9817
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: d86z1ZiMDt0zjkMlHypyUuhDPRXpynH4JC1c5f5LrLZrdhXXNWuuzdfaYNnajfHt5g9+i0wpgIgtLR13RFk82g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR18MB2856
X-OriginatorOrg: marvell.com
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.235,18.0.687
 definitions=2020-09-08_07:2020-09-08,2020-09-08 signatures=0
Sender: linux-crypto-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-crypto.vger.kernel.org>
X-Mailing-List: linux-crypto@vger.kernel.org

> Subject: Re: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support for
> OCTEONTX2 CPT engine
>=20
> On Fri, Sep 04, 2020 at 04:36:29PM +0000, Sunil Kovvuri Goutham wrote:
> >
> >
> > > -----Original Message-----
> > > From: Herbert Xu <herbert@gondor.apana.org.au>
> > > Sent: Friday, September 4, 2020 7:48 PM
> > > To: Srujana Challa <schalla@marvell.com>
> > > Cc: davem@davemloft.net; netdev@vger.kernel.org; linux-
> > > crypto@vger.kernel.org; Suheil Chandran <schandran@marvell.com>;
> > > Narayana Prasad Raju Athreya <pathreya@marvell.com>; Sunil Kovvuri
> > > Goutham <sgoutham@marvell.com>; Linu Cherian <lcherian@marvell.com>;
> > > Geethasowjanya Akula <gakula@marvell.com>; Jerin Jacob Kollanukkaran
> > > <jerinj@marvell.com>; Ard Biesheuvel <ardb@kernel.org>
> > > Subject: Re: [EXT] Re: [PATCH v2 2/3] drivers: crypto: add support fo=
r
> > > 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 b=
oth your
> > > crypto driver and your network driver so you don't duplicate this
> > > everywhere.
> > >
> >
> > For ARM64 , except erratas other platform or machine dependent stuff ar=
e not
> allowed inside arch/arm64.
> > Also an earlier attempt by us to add few APIs addressing 128bit operati=
ons 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=3Dhttp-
> 3A__lkml.iu.edu_hypermail_linux_kernel_1801.3_02211.html&d=3DDwIBAg&c=3Dn=
Kj
> Wec2b6R0mOyPaz7xtfQ&r=3DFj4OoD5hcKFpANhTWdwQzjT1Jpf7veC5263T47JVpnc
> &m=3D_Crtymahn2yLpOWm4M2GAuzClE9EQVX2165DSnCx79E&s=3DFfyU3YaCxgJRXa
> 2EzyZrTvnWFYh8K2KmifpZAQN0gJk&e=3D
>=20
> Maybe put it in include/linux/soc/ ?
>
 Okay, I will move this block of code into a header file under include/linu=
x/soc.
Thanks.
 >       Andrew