From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Duszynski Subject: Re: [PATCH v2 1/2] crypto/mrvl: sync compilation with musdk-17.10 Date: Mon, 8 Jan 2018 15:45:16 +0100 Message-ID: <20180108144516.GA8817@sh.semihalf.local> References: <1512048774-23838-1-git-send-email-tdu@semihalf.com> <1513001223-9881-1-git-send-email-tdu@semihalf.com> <1513001223-9881-2-git-send-email-tdu@semihalf.com> <20180108142752.GA8186@sh.semihalf.local> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: Tomasz Duszynski , "dev@dpdk.org" , "mw@semihalf.com" , "jck@semihalf.com" , "dima@marvell.com" , "nsamsono@marvell.com" , "jianbo.liu@arm.com" To: "De Lara Guarch, Pablo" Return-path: Received: from mail-lf0-f66.google.com (mail-lf0-f66.google.com [209.85.215.66]) by dpdk.org (Postfix) with ESMTP id 598A31B040 for ; Mon, 8 Jan 2018 15:45:18 +0100 (CET) Received: by mail-lf0-f66.google.com with SMTP id e27so12322485lfb.9 for ; Mon, 08 Jan 2018 06:45:18 -0800 (PST) Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Mon, Jan 08, 2018 at 02:41:30PM +0000, De Lara Guarch, Pablo wrote: > Hi Tomasz, > > > -----Original Message----- > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Tomasz Duszynski > > Sent: Monday, January 8, 2018 2:28 PM > > To: dev@dpdk.org > > Cc: mw@semihalf.com; jck@semihalf.com; dima@marvell.com; > > nsamsono@marvell.com; jianbo.liu@arm.com; Tomasz Duszynski > > > > Subject: Re: [dpdk-dev] [PATCH v2 1/2] crypto/mrvl: sync compilation wi= th > > musdk-17.10 > > > > On Mon, Dec 11, 2017 at 03:07:02PM +0100, Tomasz Duszynski wrote: > > > With a new version of the musdk library it's no longer necessary to > > > explicitly define MVCONF_ARCH_DMA_ADDR_T_64BIT. > > > > > > Proper defines are autogenerated by ./configure script based on passed > > > options and available after mv_autogen_comp_flags.h inclusion. > > > > > > Signed-off-by: Tomasz Duszynski > > > Acked-by: Jianbo Liu > > > --- > > > drivers/crypto/mrvl/Makefile | 3 ++- > > > drivers/crypto/mrvl/rte_mrvl_compat.h | 1 + > > > 2 files changed, 3 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/crypto/mrvl/Makefile > > > b/drivers/crypto/mrvl/Makefile index 3532f7c..5515b40 100644 > > > --- a/drivers/crypto/mrvl/Makefile > > > +++ b/drivers/crypto/mrvl/Makefile > > > @@ -47,7 +47,8 @@ LIB =3D librte_pmd_mrvl_crypto.a CFLAGS +=3D -O3 > > > CFLAGS +=3D $(WERROR_FLAGS) CFLAGS +=3D -I$(LIBMUSDK_PATH)/include > > > -CFLAGS +=3D -DMVCONF_ARCH_DMA_ADDR_T_64BIT > > > +CFLAGS +=3D -DMVCONF_TYPES_PUBLIC > > > +CFLAGS +=3D -DMVCONF_DMA_PHYS_ADDR_T_PUBLIC > > > > > > # library version > > > LIBABIVER :=3D 1 > > > diff --git a/drivers/crypto/mrvl/rte_mrvl_compat.h > > > b/drivers/crypto/mrvl/rte_mrvl_compat.h > > > index c29fa10..22cd184 100644 > > > --- a/drivers/crypto/mrvl/rte_mrvl_compat.h > > > +++ b/drivers/crypto/mrvl/rte_mrvl_compat.h > > > @@ -43,6 +43,7 @@ > > > #ifdef container_of > > > #undef container_of > > > #endif > > > +#include "env/mv_autogen_comp_flags.h" > > > #include "drivers/mv_sam.h" > > > #include "drivers/mv_sam_cio.h" > > > #include "drivers/mv_sam_session.h" > > > -- > > > 2.7.4 > > > > > > > This patch has been available here for some time already and it looks i= t has > > not been picked up yet. Should I respin the series? > > Sorry, I missed this. I was waiting for the network driver patches to be = applied, which are now. > > Will merge soon. > > Pablo > No problem, thanks. -- - Tomasz Duszy=C5=84ski