From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00063.outbound.protection.outlook.com [40.107.0.63]) by dpdk.org (Postfix) with ESMTP id 4C2954CA6 for ; Tue, 18 Sep 2018 16:21:22 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=6MDOmgp56EX13TAFGFHsHqvdTR40fkegdLQak3w6nW0=; b=ioW/P71In9VSzvQC1eAGWiXSw4p8WNUfZ4Bd+35GZSvuLIM3iPKL+wSuUpbONz0rpq0uJDYZK9TY6p6OGrDczSw4xBb0OQVhH3XuD+cTvgTo1hTrY3wtBWrwQGfrKF2Dbp7fEjY+I/OWcsAU50klOfth82UI7nqzRhqHtnhCCeQ= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=akhil.goyal@nxp.com; Received: from [10.232.134.144] (14.143.30.134) by DB7PR04MB4889.eurprd04.prod.outlook.com (2603:10a6:10:18::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1143.18; Tue, 18 Sep 2018 14:21:20 +0000 To: Gagandeep Singh , dev@dpdk.org, Hemant Agrawal References: <20180913060846.29930-1-g.singh@nxp.com> From: Akhil Goyal Message-ID: <2eb867f9-a148-9e82-2faa-016c778b2b42@nxp.com> Date: Tue, 18 Sep 2018 19:51:08 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20180913060846.29930-1-g.singh@nxp.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-Originating-IP: [14.143.30.134] X-ClientProxiedBy: BM1PR01CA0093.INDPRD01.PROD.OUTLOOK.COM (2603:1096:b00:1::33) To DB7PR04MB4889.eurprd04.prod.outlook.com (2603:10a6:10:18::20) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: e6bcc15b-25b3-4bca-7c79-08d61d7201ad X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:DB7PR04MB4889; X-Microsoft-Exchange-Diagnostics: 1; DB7PR04MB4889; 3:Cu9Q0ebJxfSteZA3NZ7sB2ajctY9oLlbEF/mogNohyiXNA6LTTktIfF+X+J8FxW2N6JvJ/DTM0aelZDP/kJEKN51ATSZOORuXOsfgtku7iPtmx5y5RTRb1q4fsW7hc+RzYoQBvyiouoAp1qXvZhOpBfsgp71vWGC6k2kHYCllYUNx9Mjir6XSMsQQgyNZZjEOevkJjpkpuBCuaX6conI7WTJtui8/OwRpFYuDUPh4cMU1MMTojUT1SYC8OnBOh9d; 25:bS/jKiwPMZPy5k5KvMa7nj+5/Wm+oOjztNRstYjThGL6LjIolh1N8auqzPmzsP6tvDVqZpx8520jlSBQ82HfN4KqW1hmFns3VwkG6ELF+OHJmtSP34gQO48sL0omwFQCVibIzfYXIWOLJ68iLw/FjQF233MxQunVxuU12bku/3uiL2CNb7VLMXOadUuqJIevbOYpSvCpXR7uKyThg/pbcp/qi4QG20Vzt6RfmrUhnEHTEuqn2ikbo4FhF7vjK17KoF/jtoE1MECVjzNjugaAsObl39cPHST74x5oH3lyMgSsdB1vbVw/krHhAANcpSLtLzL2e1qf3Tmixm88cSULjg==; 31:CDPgGEzcfHzFsGw6JIL88SvhCUqPvTpwo9xCl8oKqna4Gf7z1WyDqWmcG+5j3M+bwuPjrEw5plJ89NNfpjb66sGUeWXWtHccF8FC7fElK1YSP8EcUQK/0qnfIdzyorSLDy2v6LsSQNsxwPS6vId+lYmzly9Ol/baNoXNmI/2/qFsVIDbBQztP3e0KyOsZH9+aChzgqI7sJt+S7ETJSakTQTQkTbuJ+O7lIbCLdXkEtc= X-MS-TrafficTypeDiagnostic: DB7PR04MB4889: X-Microsoft-Exchange-Diagnostics: 1; DB7PR04MB4889; 20:AnZra+7xuqgkyaBsoKP3hkhIOi7xMByrHApcdJXIxUmuQHNC+Qgyi6X8HaGpkTTm7MDGoebDF9lRqJGpRRMyjuHphQpU/zORTsRnZ0JpB8B7b2F4EQFjQbzYmQDhDx9KxGrmgw4an+eWRIU00G4sSgVN1zCm2Adqax0aVqZBuxUIQytzz2uGJkQ6utHtpvk0mvOBuuOJIRcGxPmak0o0u3xxrR1sd/Cq26LnHKcAU69GWdqM4ks2dlSyC2eX1lb/hKxcqVfObdRMXbUaR1yy2oclOlEJxbU+Y4iYMrzffMd41LAJbT3cPAV8Jh8PyjQIZmPlYXe+GAG/cfpmWXI1Qqr5zJsVeg7teSRl3xKi7/z4z0FDgBIwirCWjPo4NI2B1EsdKXVuLq5br7j9ntHiu3iXUKOjs8Z5fgRmEzvF5ILYd4/EeN/VEU0od/QbqF9+asj5HedncwuBu0AjI2NgpqFUBQ8rEr+K1x4JyhX+WlMfv7Zm2XUcdpTIl6CjTa2p; 4:b2U4Hz5XtMhI/xXxzFwIFBQoMH082ZVXGhr8lGO0Woy0Jd3qeouuIHnk7Iq6jdVT4mDdX1lP/OwsC0dddXZPsCY2p8rDicV12wALBvOwKPuVmpnCyxSIFjXcVDYtY72KNNAGi7hUHrMXATUK7TorsMzMWW7P7UaRdyOdPR1BwyTRlqlmmiYUDPye0KHUp1HChQQOsqfxALL102LQ8qd3iCCrFSStRFMHmGCJeIeFxLhHmq+7WXseU2thheD9AaDtlMngzMEeXg6XU0cE7EZWs0AV2FpyP/XcFIkFxmuzPViruVjSrVUBofzGiJeMH59g X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(192374486261705); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(93006095)(93001095)(10201501046)(3002001)(6055026)(149027)(150027)(6041310)(20161123562045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123560045)(201708071742011)(7699050); SRVR:DB7PR04MB4889; BCL:0; PCL:0; RULEID:; SRVR:DB7PR04MB4889; X-Forefront-PRVS: 0799B1B2D7 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6049001)(346002)(396003)(376002)(136003)(366004)(39860400002)(189003)(199004)(966005)(6116002)(3846002)(81156014)(110136005)(81166006)(16576012)(68736007)(316002)(53936002)(6636002)(6486002)(229853002)(6306002)(5660300001)(65826007)(6666003)(25786009)(8936002)(77096007)(36756003)(26005)(47776003)(58126008)(3260700006)(5009440100003)(2906002)(64126003)(16526019)(11346002)(66066001)(53546011)(956004)(386003)(31696002)(2616005)(65806001)(305945005)(52146003)(2486003)(7736002)(476003)(105586002)(217873002)(86362001)(14444005)(76176011)(8676002)(65956001)(52116002)(486006)(67846002)(106356001)(230700001)(23676004)(478600001)(97736004)(446003)(50466002)(31686004)(44832011)(6246003)(110426005); DIR:OUT; SFP:1101; SCL:1; SRVR:DB7PR04MB4889; H:[10.232.134.144]; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtEQjdQUjA0TUI0ODg5OzIzOlVoaGl1NXpBMDB0aUdVanhyNTA4bEtIdXo0?= =?utf-8?B?a1RtZnZjOXAxcEh6MUZjV2hsZFNSVW0za2s1OE5JVjZzSnN2TkI0N29nTU41?= =?utf-8?B?RURwOVZkbzlRSXptcEhYQ0ovS0NRaWRMWGk4WksvcmlFL3JIR0YxbDVmN0Fh?= =?utf-8?B?TXlzVGFBWVVDWDBJbncxWVdsdW1FcEovbVZzR3dScFlYaWtXVFB4UkY3S3BZ?= =?utf-8?B?WVgxWWkveTQ5V3BZR2hsRGhxVHQzVXArSHR0SDZZVnF1QzdoL2hDSkNTOWhN?= =?utf-8?B?OEx6eTFFQmZCVmpZc2VZOXJvblJYV3BpTy81Sm52b213Q3BrWkk1VEplSXYw?= =?utf-8?B?NUU1bHBkYUVSdEVQdnRreFFjK3BaMkVHdXgwYzJKeG91WGZaZUYzNzR6U21P?= =?utf-8?B?eFlkcmZKbFF5dlkxODJ4VWJoTEMwdWx6dUJnQ3I4VHh5RnQwYWJyWkRvVyta?= =?utf-8?B?eC9hQ0FBUTd4Q0ZlSk1WR3o4aGxYK2w3UHA4dk1LN0tMWGwyMDdTb2JGQzhl?= =?utf-8?B?VVM5clVHc2p0UlZDY1lXRHNEN09GT25rR3QyZFYrMGFjb294Z3cxYUxtVEI2?= =?utf-8?B?cC9iUDg1cUFuTVZRY0M1V1hWNTh4Ukk3NHBucXczT0ZoalFHQ0V1NHRBa1NS?= =?utf-8?B?OERlbmZma2c0RmYwKzZhbnlhMUtuVmdhelVQM1g1d2gyZzBPWE9GeS9jbWZ6?= =?utf-8?B?NnhHaGFWb0UxaWVlcDBYaHhVYmQrU3d4MDJ0aFNkcEhJd1ZsdzdCYzRFWUhI?= =?utf-8?B?L1Q1Q3lPdUY1cjA3WjFSN21NdU5PMW9yQ0RiRUo0RldYbTJBT3duTlhlbUtV?= =?utf-8?B?NjArRHZKL21xSXhlc04raHZVMTgzUUIxb0NpL2JEbXZVbXVPZUNoMkwxTFZp?= =?utf-8?B?OEJIQitNbkh0UlpZSnpETHdaUjFGajQvVzNKTGJBQ2s1YlR2M2J6RUFhV3Jh?= =?utf-8?B?V1FvaDgzbXRvWjJxcmd3Wlp6alg2d2cxMnFSNklaYm1CaFMybm1saGtXWDcz?= =?utf-8?B?dGtuTVZlUEp4VWtsM1pqb2VYSVcxZ0lvOGZvb1Y3ck5UNVE2UVdNRlFWSklj?= =?utf-8?B?ZFg1Mm1JNXBhbWZ0TWRvZFhET0VLVkRQeitoeUJKeTZVL1ZkUXdYeGNReHZz?= =?utf-8?B?MHVsUWU5aG9UTEMzKzQ5cjhteWlpUUI1Y2dIWmNqUk9IS1JmMHVIYkRLOW5S?= =?utf-8?B?cU9ldzdQVFptYlBuSEZCQW93S0hQSWFKT3pIdWFrekNLS0JFWHR2T3U4ZHdS?= =?utf-8?B?WGRTMERTTnYwSkhmcVA5UHhjd2ZQUFIwWW1IYjNsZEpndWVLclp2VmdkeGhm?= =?utf-8?B?L3d6UFYwSXB2Q3RoUFl2ZE5iRG1xZnVkd3FHL2lnZklIdTU2eVBHMGpRR1Zz?= =?utf-8?B?MnN5OXAvRmR2dHlNS0ErZXdKYlN5allJZ25mdlhrSzNka2FqbEU5MlA4aHdW?= =?utf-8?B?bVNJdGdueW9VOGhsdXp2ak9mbFlaOTFKa09Qa2pqdlhmbDc4QVpvS0dLdXBF?= =?utf-8?B?eFlmRWhETWQrVUIwdTl1WDJWOXhDT1YxQXdha3VnOUxYS1d5UytLYVlOOXR0?= =?utf-8?B?N2hFUndHOStxL0sxMExPUUdaQ0xuSFAyWGdKbk0xM1dJeGVwYUJQQmJzYTUy?= =?utf-8?B?Vnp1cFlvTWdVUzJnOTNvYWU5eWZaM3BORnhieEpmY0VhbitMSHQ0VW04azdR?= =?utf-8?B?Rk1nTWV1MkNhQzhWQ2pVSXN5MEdhSU9nbDFZWDhwSFZNQUp3QXVqWG9rUSti?= =?utf-8?B?N1dkUGxKdzRnQnRyVnR4UkRndlUvb1NuOFRTTUxrZ0FidFVaNHl1L1NqNFA1?= =?utf-8?B?RVZWNEU1VE90TGgxdGVpTml4MGFrbmo0SCtPajE4U1N5UnJ3dVFiRFo4WXMr?= =?utf-8?B?OTg3OC9ZQXVoa08zNlRhSUs5eTB2R0xzTUtKdDg2WjVyNmpXSFRwcWFKWWN2?= =?utf-8?B?ZEd3cVIyOUg0NFFSelBaOEpRenhqOTJNUTFnaC9yc3l0NVRNWDkxalREY2d2?= =?utf-8?B?OThCN1Z6QXNqVDNxWmN0OGZlTUt3YSt0Y3psOWhYb3hKYjJsMGkybEVKVVRw?= =?utf-8?B?U3prTmRhcFNicjZqbDV2c2VXQ1VYd01wK2w4eGtQUmdmZUpZV0s4K01uVytt?= =?utf-8?Q?vbq1r/rCocfdbLiH8VZ8/7gh5NLLzayyPuO8XZ9iMANy?= X-Microsoft-Antispam-Message-Info: 9KUIt73wEJO6/wtlCi2rnpaPJayrXRC+g8ZN23niXMSJAprnZTvU9kMJ/9u3EIG5vzI8PjbBpbU3y6zv1fMf6GylDs832630xHWxj1v69wuncHaCVrj7CdV5YBHSc5Y8tZypwcF0gYkaVbhiYgdvNsJku17YtWZYndOybIjqLcpehAk3EjOHh5RMffGuz95nqZAQmsXAatRLmzuWhaZqXNEhQ6eW29HJJh2bpKgQh5DdILRUqzk0qGm9LoIkytCsoIIvwwf6/xnjc1dROyexL4k4DHi8IooFoy1DGiJmaXStWVbP/kVLxct6RZ/HC44IeTp5X9agPCRDdop3T5AopdolJ00WEQyQxFk85zopYyY= X-Microsoft-Exchange-Diagnostics: 1; DB7PR04MB4889; 6:MVajoyCeGNvusMOMg3JC+5Fk9BR8tiHVqXHZKKxzkWXtFCU91UXvCIQu5Xwte9v4oRvU+6Gj7jXJIEp0C92lXjewNzuUKhF7yHsL5zbpXoMCVFgR0mHijwFO1J/vTec9JOksqeaRrF2ZWFRcsM+QrtLNw0Y4JHqPbr/8veKFleNEpK2YAqcaafsqVItCMnS1QeR8a4yb1f1YWYQ/DMVMt/Nbx3oPrZq9eUyXNLklR8qgo4HJPsafSRCJvK2AFzdXEPC7iE7U6T4kf4nE+BlCYmo2unv/eOlxk1zKP9QpPZkv8kZlTFO5SdayEkRu8IWbmw3wVKdDtoB7L2Be7kvKFiIwUDVS4e3QkopuPb2A9zXrUrwpfCC6LioPDeUKgAWOiM7da8heanmbmDbSRmy/KfTkKC11hSreebx43w0+tLX9Z5RQGNaGONM9Rx4V8N8sYtCFNBF+JJTvj8YxJYW2cg==; 5:wrB2IWGVhPryxMBHT1axGSBx73w37akRdQWdr6dkpqGaE9uu35yqhg8gPG/6au5cBy7eV6ZBRJGSLvJ1cUi46gZ+YWRHjNaS20If6g4ILS1CC3wsVCQcYdSUmu6Ns9UUA1ch+/OFtevgN3zkTMaXpkOXhNC/ESZUhWRtzIKevAs=; 7:fgunjTBwjGBD124P/sWP5J5WISXWZ62CLfzRyeFt4SVi6H/5lIVAJ049+6du9RVOYMX25zAVq68VGX+anN5uu+57o3GgcFp7sPrsmF2qokPQPIqEPWb5GQrYxGqx0DjFZ7w3rUqzDgYMRLsyjIWIhIzT9W6xEBK8wMKu9V7WTh6tJsjeUJWDwpXqHHl0iYNR8cN5ktp/lmmUVC/g20QnBeXLQIFUTrh6vLP+I34oDsVpBvPEUVogWWl0QsO6A1Hc SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Sep 2018 14:21:20.3102 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: e6bcc15b-25b3-4bca-7c79-08d61d7201ad X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR04MB4889 Subject: Re: [dpdk-dev] [PATCH 00/10] Introducing the NXP CAAM job ring driver 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, 18 Sep 2018 14:21:22 -0000 Hi Gagan, On 9/13/2018 11:38 AM, Gagandeep Singh wrote: > The caam_jr PMD provides poll mode crypto driver > support for NXP SEC 4.x+ (CAAM) hardware accelerator. > > This patch has dependancy on below patches: > http://patchwork.dpdk.org/patch/43986/ > http://patchwork.dpdk.org/patch/43964/ > > Hemant Agrawal (10): > doc: add caam jr cryptodev details > crypto/caam_jr: introduce basic driver > crypto/caam_jr: add HW config for job rings > crypto/caam_jr: add device configuration routines > crypto/caam_jr: add queue config functions > crypto/caam_jr: add basic session config routines > crypto/caam_jr: add enqueue and dequeue routines > crypto/caam_jr: add auth cipher and aead session support > crypto/caam_jr: add stats support > crypto/caam_jr: add security offload support > > config/common_base | 8 + > config/common_linuxapp | 1 + > config/defconfig_arm64-dpaa-linuxapp-gcc | 4 + > doc/guides/cryptodevs/caam_jr.rst | 159 ++ > doc/guides/cryptodevs/index.rst | 1 + > drivers/crypto/Makefile | 1 + > drivers/crypto/caam_jr/Makefile | 46 + > drivers/crypto/caam_jr/caam_jr.c | 2485 +++++++++++++++++ > drivers/crypto/caam_jr/caam_jr.h | 257 ++ > drivers/crypto/caam_jr/caam_jr_config.h | 207 ++ > drivers/crypto/caam_jr/caam_jr_desc.h | 289 ++ > drivers/crypto/caam_jr/caam_jr_hw.c | 365 +++ > drivers/crypto/caam_jr/caam_jr_hw_specific.h | 503 ++++ > drivers/crypto/caam_jr/caam_jr_log.h | 42 + > drivers/crypto/caam_jr/caam_jr_pvt.h | 288 ++ > drivers/crypto/caam_jr/caam_jr_uio.c | 491 ++++ > drivers/crypto/caam_jr/meson.build | 14 + > .../caam_jr/rte_pmd_caam_jr_version.map | 4 + > drivers/crypto/meson.build | 2 +- > 19 files changed, 5166 insertions(+), 1 deletion(-) > create mode 100644 doc/guides/cryptodevs/caam_jr.rst > create mode 100644 drivers/crypto/caam_jr/Makefile > create mode 100644 drivers/crypto/caam_jr/caam_jr.c > create mode 100644 drivers/crypto/caam_jr/caam_jr.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_config.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_desc.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_hw.c > create mode 100644 drivers/crypto/caam_jr/caam_jr_hw_specific.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_log.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_pvt.h > create mode 100644 drivers/crypto/caam_jr/caam_jr_uio.c > create mode 100644 drivers/crypto/caam_jr/meson.build > create mode 100644 drivers/crypto/caam_jr/rte_pmd_caam_jr_version.map > I have some generic comments on your patchset: 1. Please add description to the patches. None of them have it. 2. Split of the patches could be better. You can have Hardware specific code in one patch and then you can use that in the next patch. 3. SG related stuff can be a separate patch. 4. there are checkpatch issues which can be resolved. 5. I could see extra spaces at many places. Please check. 6. Function definitions are not as per DPDK coding style. Thanks, Akhil