From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kumar, Ravi1" Subject: Re: [PATCH v3 01/19] crypto/ccp: add AMD ccp skeleton PMD Date: Thu, 11 Jan 2018 06:36:41 +0000 Message-ID: References: <1515145198-97367-1-git-send-email-Ravi1.kumar@amd.com> <1515577379-18453-1-git-send-email-Ravi1.kumar@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "Shippen, Greg" To: "De Lara Guarch, Pablo" , "dev@dpdk.org" Return-path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0079.outbound.protection.outlook.com [104.47.42.79]) by dpdk.org (Postfix) with ESMTP id CE47F1B16B for ; Thu, 11 Jan 2018 07:36:44 +0100 (CET) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- >> From: Kumar, Ravi1 [mailto:Ravi1.Kumar@amd.com] >> Sent: Wednesday, January 10, 2018 10:30 AM >> To: De Lara Guarch, Pablo ;=20 >> dev@dpdk.org >> Cc: Shippen, Greg >> Subject: RE: [PATCH v3 01/19] crypto/ccp: add AMD ccp skeleton PMD >>=20 >> >Hi Ravi, >> > >> >> -----Original Message----- >> >> From: Ravi Kumar [mailto:Ravi1.kumar@amd.com] >> >> Sent: Wednesday, January 10, 2018 9:43 AM >> >> To: dev@dpdk.org >> >> Cc: De Lara Guarch, Pablo >> >> Subject: [PATCH v3 01/19] crypto/ccp: add AMD ccp skeleton PMD >> >> >> >> Signed-off-by: Ravi Kumar >> >> --- >> > >> >... >> > >> >> --- /dev/null >> >> +++ b/drivers/crypto/ccp/Makefile >> >> @@ -0,0 +1,55 @@ >> >> +# >> >> +# Copyright(c) 2018 Advanced Micro Devices, Inc. >> >> +# All rights reserved. >> >> +# >> > >> >As Hemant commented, you need to change this full license with SPDX >> tags: >> > >> >http://dpdk.org/ml/archives/dev/2018-January/085510.html >> > >> >Could you submit a v4 with these changes today? >> > >> >Thanks, >> >Pablo >>=20 >> Hi Pablo, >>=20 >> Our legal team is still working on the license. We want to get the=20 >> code reviewed in parallel. >> I will give you and update later if we can upload the v4 patch with=20 >> SPDX tags today. >>=20 > >Ok, I have looked at the overall patchset and it looks ok to me. >The only thing that I would change is the title of patch 7/19: >I would change it to "crypto/ccp: support sessionless operations". > >Regards, >Pablo Hi Pablo, Thanks for going through our code. Good to know the code is fine.=20 The new changes in licensing policy has introduced some delays as we work t= hrough the implications and get agreement from the third party. We are working on it and will upload the v4 with SPDX license tags as soon = as possible.=20 Regards, Ravi > >> Regards, >> Ravi