From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id 3A3294C96 for ; Tue, 19 Jun 2018 15:56:54 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Jun 2018 06:56:53 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,243,1526367600"; d="scan'208";a="60453951" Received: from irsmsx108.ger.corp.intel.com ([163.33.3.3]) by orsmga003.jf.intel.com with ESMTP; 19 Jun 2018 06:56:51 -0700 Received: from irsmsx156.ger.corp.intel.com (10.108.20.68) by IRSMSX108.ger.corp.intel.com (163.33.3.3) with Microsoft SMTP Server (TLS) id 14.3.319.2; Tue, 19 Jun 2018 14:56:50 +0100 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.139]) by IRSMSX156.ger.corp.intel.com ([169.254.3.252]) with mapi id 14.03.0319.002; Tue, 19 Jun 2018 14:56:50 +0100 From: "De Lara Guarch, Pablo" To: Jerin Jacob , Anoob Joseph CC: Akhil Goyal , Thomas Monjalon , Nithin Dabilpuram , Ankur Dwivedi , Murthy NSSR , Narayana Prasad , "Ragothaman Jayaraman" , Srisivasubramanian Srinivasan , "dev@dpdk.org" Thread-Topic: [PATCH 03/16] crypto/cpt/base: add hardware initialization API for CPT Thread-Index: AQHT/0iYLUKBFFguAUeLpllHb+wLdaRfDAqAgAieywA= Date: Tue, 19 Jun 2018 13:56:49 +0000 Message-ID: References: <1528476325-15585-1-git-send-email-anoob.joseph@caviumnetworks.com> <1528476325-15585-4-git-send-email-anoob.joseph@caviumnetworks.com> <20180614031336.GC16602@jerin> In-Reply-To: <20180614031336.GC16602@jerin> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiOGQ3OWQ4ZDAtY2U2NC00YjQ2LWJhMjItYWYyNzIzOGQ2OThkIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiK1wvdmJPMFwvWGx4K2ZKMFZtQnVvMVlxYmhXMEMxVUVGZFk0dHp0dSs3RUxnMG55Zitzb1gxQzR2Nlwvd0cyaHJROCJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.200.100 dlp-reaction: no-action x-originating-ip: [163.33.239.181] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH 03/16] crypto/cpt/base: add hardware initialization API for CPT X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jun 2018 13:56:55 -0000 > -----Original Message----- > From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com] > Sent: Thursday, June 14, 2018 4:14 AM > To: Anoob Joseph > Cc: Akhil Goyal ; De Lara Guarch, Pablo > ; Thomas Monjalon > ; Nithin Dabilpuram ; > Ankur Dwivedi ; Murthy NSSR > ; Narayana Prasad > ; Ragothaman Jayaraman > ; Srisivasubramanian Srinivasan > ; dev@dpdk.org > Subject: Re: [PATCH 03/16] crypto/cpt/base: add hardware initialization A= PI for > CPT >=20 > -----Original Message----- > > Date: Fri, 8 Jun 2018 22:15:12 +0530 > > From: Anoob Joseph > > To: Akhil Goyal , Pablo de Lara > > , Thomas Monjalon > > > > Cc: Nithin Dabilpuram , Ankur Dwivedi > > , Jerin Jacob > > , Murthy NSSR > > , Narayana Prasad > > , Ragothaman Jayaraman > > , Srisivasubramanian Srinivasan > > , dev@dpdk.org > > Subject: [PATCH 03/16] crypto/cpt/base: add hardware initialization > > API for CPT > > X-Mailer: git-send-email 2.7.4 > > > > From: Nithin Dabilpuram > > > > Adds hardware device initialization specific api for Cavium CPT device. > > > > Signed-off-by: Ankur Dwivedi > > Signed-off-by: Murthy NSSR > > Signed-off-by: Nithin Dabilpuram > > Signed-off-by: Ragothaman Jayaraman > > > Signed-off-by: Srisivasubramanian Srinivasan > > > > --- > > drivers/crypto/cpt/base/cpt8xxx_device.c | 200 ++++++++++++++++ > > drivers/crypto/cpt/base/cpt8xxx_device.h | 85 +++++++ > > drivers/crypto/cpt/base/cpt_debug.h | 231 +++++++++++++++++++ > > drivers/crypto/cpt/base/cpt_device.c | 383 > +++++++++++++++++++++++++++++++ > > drivers/crypto/cpt/base/cpt_device.h | 162 +++++++++++++ > > drivers/crypto/cpt/base/cpt_vf_mbox.c | 176 ++++++++++++++ > > drivers/crypto/cpt/base/cpt_vf_mbox.h | 60 +++++ > > 7 files changed, 1297 insertions(+) > > create mode 100644 drivers/crypto/cpt/base/cpt8xxx_device.c > > create mode 100644 drivers/crypto/cpt/base/cpt8xxx_device.h > > create mode 100644 drivers/crypto/cpt/base/cpt_debug.h > > create mode 100644 drivers/crypto/cpt/base/cpt_device.c > > create mode 100644 drivers/crypto/cpt/base/cpt_device.h > > create mode 100644 drivers/crypto/cpt/base/cpt_vf_mbox.c > > create mode 100644 drivers/crypto/cpt/base/cpt_vf_mbox.h > > > > +#include > > +#include "cpt_request_mgr.h" > > +#include List the external to DPDK includes first, then internal to DPDK and then in= ternal to this PMD, separating these three blocks with a blank line. > > + > > +#ifdef CPT_DEBUG >=20 > Remove CPT_DEBUG. No harming in compiling these definitions either case. > It will avoid the chance of build breakage. >=20 ... > > +int cptvf_deinit_device(struct cpt_vf *dev) { > > + struct cpt_vf *cptvf =3D (struct cpt_vf *)dev; > > + > > + /* Do misc work one last time */ > > + cptvf_poll_misc(cptvf); > > + > > + /* TODO anything else ?? */ >=20 > Remove unclear TODOs >=20 > > + > > + return 0; > > +} > > + > > +int cptvf_init_device(struct cpt_vf *cptvf, Return type should be in the line above the function name, as per DPDK coding style guidelines. > > + void *pdev, > > + void *reg_base, > > + char *name, > > + uint32_t flags) > > +{ > > + (void) flags; >=20 >=20 > RTE_SET_USED Actually, why don't you just remove the parameter? Thanks, Pablo