diff mbox

[dpdk-dev,v2] aesni_mb: fix control issues in aesni pmd

Message ID 1465292084-189336-1-git-send-email-deepak.k.jain@intel.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Deepak Kumar JAIN June 7, 2016, 9:34 a.m. UTC 
  Fix wrong indentation for return value

Coverity ID: 126585
Fixes: 924e84f87306 ("aesni_mb: add driver for multi buffer based crypto")

Signed-off-by: Deepak Kumar Jain <deepak.k.jain@intel.com>
---
v2: Corrected PMD in commit message

 drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

De Lara Guarch, Pablo June 7, 2016, 11:09 a.m. UTC | #1 
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jain, Deepak K
> Sent: Tuesday, June 07, 2016 10:35 AM
> To: dev@dpdk.org
> Cc: Doherty, Declan; Jain, Deepak K
> Subject: [dpdk-dev] [PATCH v2] aesni_mb: fix control issues in aesni pmd
> 
> Fix wrong indentation for return value
> 
> Coverity ID: 126585
> Fixes: 924e84f87306 ("aesni_mb: add driver for multi buffer based crypto")
> 
> Signed-off-by: Deepak Kumar Jain <deepak.k.jain@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Thomas Monjalon June 7, 2016, 2:33 p.m. UTC | #2 
> > Fix wrong indentation for return value
> > 
> > Coverity ID: 126585
> > Fixes: 924e84f87306 ("aesni_mb: add driver for multi buffer based crypto")
> > 
> > Signed-off-by: Deepak Kumar Jain <deepak.k.jain@intel.com>
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied, thanks
diff mbox

Patch

diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
index 3415ac1..9c42f88 100644
--- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
+++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c
@@ -560,7 +560,7 @@  aesni_mb_pmd_enqueue_burst(void *queue_pair, struct rte_crypto_op **ops,
 		goto flush_jobs;
 	else
 		qp->stats.enqueued_count += processed_jobs;
-		return i;
+	return i;
 
 flush_jobs:
 	/*