diff mbox series

[v3,34/37] crypto/caam_jr: replace logtype PMD

Message ID	20231213163820.894987-35-stephen@networkplumber.org (mailing list archive)
State	Superseded, archived
Delegated to:	Thomas Monjalon
Headers	From: Stephen Hemminger <stephen@networkplumber.org> To: dev@dpdk.org Cc: Stephen Hemminger <stephen@networkplumber.org>, Gagandeep Singh <g.singh@nxp.com>, Hemant Agrawal <hemant.agrawal@nxp.com> Subject: [PATCH v3 34/37] crypto/caam_jr: replace logtype PMD Date: Wed, 13 Dec 2023 08:36:08 -0800 Message-ID: <20231213163820.894987-35-stephen@networkplumber.org> In-Reply-To: <20231213163820.894987-1-stephen@networkplumber.org> References: <20231213014408.612051-1-stephen@networkplumber.org> <20231213163820.894987-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	Replace uso of RTE_LOGTYPE_PMD \| [v3,00/37] Replace uso of RTE_LOGTYPE_PMD [v3,01/37] log: fix doc comment for RTE_LOG_DP() [v3,02/37] log: add rte_log_dp() [v3,03/37] net/atlantic: replace RTE_LOG_DP with rte_log_dp [v3,04/37] net/avp: replace RTE_LOG_DP with rte_log_dp [v3,05/37] net/bnxt: replace RTE_LOG_DP with rte_log_dp [v3,06/37] net/dpaa: replace RTE_LOG_DP with rte_log_dp [v3,07/37] net/dpaa2: replace RTE_LOG_DP with rte_log_dp [v3,08/37] net/enetc, net/enetfec: replace RTE_LOG_DP with rte_log_dp [v3,09/37] net/igc: replace RTE_LOG_DP with rte_log_dp [v3,10/37] net/mana: replace RTE_LOG_DP with rte_log_dp [v3,11/37] net/mvpp2: do not use PMD logtype [v3,12/37] net/octeon_ep: replace RTE_LOG_DP with rte_log_dp [v3,13/37] net/pfe: replace RTE_LOG_DP with rte_log_dp [v3,14/37] net/qede: replace RTE_LOG_DP with rte_log_dp [v3,15/37] net/virtio: replace RTE_LOG_DP with rte_log_dp [v3,16/37] net/vmxnet3: do not use PMD logtype [v3,17/37] common/cnxk: replace RTE_LOG_DP with rte_log_dp [v3,18/37] common/cpt: replace RTE_LOG_DP with rte_log_dp [v3,19/37] common/sfc_efx: remove use of PMD logtype [v3,20/37] common/dpaax: do not use PMD logtype [v3,21/37] basband/la12xx: replace RTE_LOG_DP with rte_log_dp [v3,22/37] bus/cdx: replace RTE_LOG_DP with rte_log_dp [v3,23/37] bus/fslmc: replace RTE_LOG_DP with rte_log_dp [v3,24/37] dma/dpaa, dma/dpaa2: replace RTE_LOG_DP with rte_log_dp [v3,25/37] mempool/dpaa, mempool/dpaa2: do not use logtype PMD [v3,26/37] vdpa/ifc: replace use of RTE_LOGTYPE_PMD [v3,27/37] raw/cnxk_bphy: replace PMD logtype with dynamic type [v3,28/37] raw/cnxk_gpio: replace PMD logtype with dynamic type [v3,29/37] raw/dpaa2_cmdif: replace PMD logtype with rte_log_dp [v3,30/37] raw/ifpga: replace PMD logtype with dynamic type [v3,31/37] event/dlb2: replace logtype PMD with rte_log_dp [v3,32/37] event/dpaa, event/dpaa2: use local logtype [v3,33/37] event/skeleton: replace logtype PMD with dynamic type [v3,34/37] crypto/caam_jr: replace logtype PMD [v3,35/37] crypto/ccp: do not use PMD logtype [v3,36/37] crypto/scheduler: replace use of logtype PMD [v3,37/37] crypto/dpaa_sec, crypto/dpaa2_sec: replace use of PMD logtype

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Stephen Hemminger Dec. 13, 2023, 4:36 p.m. UTC

  The driver has macros for logging, they just weren't used
everywhere.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/crypto/caam_jr/caam_jr.c     | 8 +++-----
 drivers/crypto/caam_jr/caam_jr_log.h | 2 +-
 2 files changed, 4 insertions(+), 6 deletions(-)

Comments

Hemant Agrawal Dec. 15, 2023, 5:54 a.m. UTC | #1

Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

diff mbox series

Patch

diff --git a/drivers/crypto/caam_jr/caam_jr.c b/drivers/crypto/caam_jr/caam_jr.c
index b55258689b49..9c713d4e8133 100644
--- a/drivers/crypto/caam_jr/caam_jr.c
+++ b/drivers/crypto/caam_jr/caam_jr.c
@@ -2346,7 +2346,7 @@  caam_jr_dev_init(const char *name,
 
 	rte_cryptodev_pmd_probing_finish(dev);
 
-	RTE_LOG(INFO, PMD, "%s cryptodev init\n", dev->data->name);
+	CAAM_JR_INFO("%s cryptodev init\n", dev->data->name);
 
 	return 0;
 
@@ -2386,8 +2386,7 @@  cryptodev_caam_jr_probe(struct rte_vdev_device *vdev)
 
 	ret = of_init();
 	if (ret) {
-		RTE_LOG(ERR, PMD,
-		"of_init failed\n");
+		CAAM_JR_ERR("of_init failed\n");
 		return -EINVAL;
 	}
 	/* if sec device version is not configured */
@@ -2407,8 +2406,7 @@  cryptodev_caam_jr_probe(struct rte_vdev_device *vdev)
 	}
 #ifdef RTE_LIBRTE_PMD_CAAM_JR_BE
 	if (rta_get_sec_era() > RTA_SEC_ERA_8) {
-		RTE_LOG(ERR, PMD,
-		"CAAM is compiled in BE mode for device with sec era > 8???\n");
+		CAAM_JR_ERR("CAAM is compiled in BE mode for device with sec era > 8???\n");
 		return -EINVAL;
 	}
 #endif
diff --git a/drivers/crypto/caam_jr/caam_jr_log.h b/drivers/crypto/caam_jr/caam_jr_log.h
index 106ff07a70fe..2dded646dbdc 100644
--- a/drivers/crypto/caam_jr/caam_jr_log.h
+++ b/drivers/crypto/caam_jr/caam_jr_log.h
@@ -28,7 +28,7 @@  extern int caam_jr_logtype;
 
 /* DP Logs, toggled out at compile time if level lower than current level */
 #define CAAM_JR_DP_LOG(level, fmt, args...) \
-	RTE_LOG_DP(level, PMD, fmt "\n", ## args)
+	rte_log_dp(RTE_LOG_ ## level, caam_jr_logtype, fmt "\n", ## args)
 
 #define CAAM_JR_DP_DEBUG(fmt, args...) \
 	CAAM_JR_DP_LOG(DEBUG, fmt, ## args)