[dpdk-dev,1/4] bus/fslmc: fix registering dpaa2 mempool ops

Message ID 1517808117-28562-1-git-send-email-nipun.gupta@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Nipun Gupta Feb. 5, 2018, 5:21 a.m. UTC
  Fixes: 8824b74d8f5f ("bus/fslmc: register platform HW mempool on runtime")

Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
---
Though I have sent these patches yesterday, but they did not reach out.
If by any chance they have entered your mailbox before please ignore
the older patches.

 drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Shreyansh Jain Feb. 5, 2018, 6:32 a.m. UTC | #1
> -----Original Message-----
> From: Nipun Gupta
> Sent: Monday, February 5, 2018 10:52 AM
> To: thomas@monjalon.net; jerin.jacob@caviumnetworks.com
> Cc: dev@dpdk.org; santosh.shukla@caviumnetworks.com; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Shreyansh Jain <shreyansh.jain@nxp.com>;
> Sunil Kumar Kori <sunil.kori@nxp.com>; Nipun Gupta
> <nipun.gupta@nxp.com>
> Subject: [PATCH 1/4] bus/fslmc: fix registering dpaa2 mempool ops
> 
> Fixes: 8824b74d8f5f ("bus/fslmc: register platform HW mempool on
> runtime")
> 
> Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> ---

Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>
  
Thomas Monjalon Feb. 5, 2018, 11:44 p.m. UTC | #2
> > Fixes: 8824b74d8f5f ("bus/fslmc: register platform HW mempool on
> > runtime")
> > 
> > Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> 
> Acked-by: Shreyansh Jain <shreyansh.jain@nxp.com>

Series applied, thanks
  

Patch

diff --git a/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c b/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
index 1539739..f1f14e2 100644
--- a/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
+++ b/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
@@ -77,8 +77,8 @@ 
 	RTE_LOG(DEBUG, PMD, "DPAA2: Added [dpbp.%d]\n", dpbp_id);
 
 	if (!register_once) {
-		if (rte_mbuf_set_platform_mempool_ops(DPAA2_MEMPOOL_OPS_NAME))
-			register_once = 1;
+		rte_mbuf_set_platform_mempool_ops(DPAA2_MEMPOOL_OPS_NAME);
+		register_once = 1;
 	}
 
 	return 0;