[v3,32/37] event/dpaa, event/dpaa2: use local logtype

Message ID 20231213163820.894987-33-stephen@networkplumber.org (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series Replace uso of RTE_LOGTYPE_PMD |

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 to use local logtype, but some changes
added PMD log messages.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/event/dpaa/dpaa_eventdev.c            | 2 +-
 drivers/event/dpaa2/dpaa2_eventdev.c          | 4 ++--
 drivers/event/dpaa2/dpaa2_eventdev_logs.h     | 2 +-
 drivers/event/dpaa2/dpaa2_eventdev_selftest.c | 6 +++---
 4 files changed, 7 insertions(+), 7 deletions(-)
  

Comments

Hemant Agrawal Dec. 15, 2023, 5:55 a.m. UTC | #1
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
  

Patch

diff --git a/drivers/event/dpaa/dpaa_eventdev.c b/drivers/event/dpaa/dpaa_eventdev.c
index 46a9b88c73ae..a8e5c3421df1 100644
--- a/drivers/event/dpaa/dpaa_eventdev.c
+++ b/drivers/event/dpaa/dpaa_eventdev.c
@@ -1025,7 +1025,7 @@  dpaa_event_dev_create(const char *name, const char *params, struct rte_vdev_devi
 	eventdev->txa_enqueue = dpaa_eventdev_txa_enqueue;
 	eventdev->txa_enqueue_same_dest	= dpaa_eventdev_txa_enqueue_same_dest;
 
-	RTE_LOG(INFO, PMD, "%s eventdev added", name);
+	DPAA_EVENTDEV_INFO("%s eventdev added", name);
 
 	/* For secondary processes, the primary has done all the work */
 	if (rte_eal_process_type() != RTE_PROC_PRIMARY)
diff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c
index dd4e64395fe5..85c2dbd998dd 100644
--- a/drivers/event/dpaa2/dpaa2_eventdev.c
+++ b/drivers/event/dpaa2/dpaa2_eventdev.c
@@ -1141,7 +1141,7 @@  dpaa2_eventdev_create(const char *name, struct rte_vdev_device *vdev)
 		priv->max_event_queues++;
 	} while (dpcon_dev && dpci_dev);
 
-	RTE_LOG(INFO, PMD, "%s eventdev created\n", name);
+	DPAA2_EVENTDEV_INFO("%s eventdev created", name);
 
 done:
 	event_dev_probing_finish(eventdev);
@@ -1178,7 +1178,7 @@  dpaa2_eventdev_destroy(const char *name)
 	}
 	priv->max_event_queues = 0;
 
-	RTE_LOG(INFO, PMD, "%s eventdev cleaned\n", name);
+	DPAA2_EVENTDEV_INFO("%s eventdev cleaned", name);
 	return 0;
 }
 
diff --git a/drivers/event/dpaa2/dpaa2_eventdev_logs.h b/drivers/event/dpaa2/dpaa2_eventdev_logs.h
index 66c8c772740d..69aebf569fa5 100644
--- a/drivers/event/dpaa2/dpaa2_eventdev_logs.h
+++ b/drivers/event/dpaa2/dpaa2_eventdev_logs.h
@@ -26,7 +26,7 @@  extern int dpaa2_logtype_event;
 
 /* DP Logs, toggled out at compile time if level lower than current level */
 #define DPAA2_EVENTDEV_DP_LOG(level, fmt, args...) \
-	RTE_LOG_DP(level, PMD, fmt, ## args)
+	rte_log_dp(RTE_LOG_ ## level, dpaa2_logtype_event, fmt, ## args)
 
 #define DPAA2_EVENTDEV_DP_DEBUG(fmt, args...) \
 	DPAA2_EVENTDEV_DP_LOG(DEBUG, fmt, ## args)
diff --git a/drivers/event/dpaa2/dpaa2_eventdev_selftest.c b/drivers/event/dpaa2/dpaa2_eventdev_selftest.c
index 427aff4b1111..9d4938efe6aa 100644
--- a/drivers/event/dpaa2/dpaa2_eventdev_selftest.c
+++ b/drivers/event/dpaa2/dpaa2_eventdev_selftest.c
@@ -774,15 +774,15 @@  static void dpaa2_test_run(int (*setup)(void), void (*tdown)(void),
 		int (*test)(void), const char *name)
 {
 	if (setup() < 0) {
-		RTE_LOG(INFO, PMD, "Error setting up test %s", name);
+		DPAA2_EVENTDEV_INFO("Error setting up test %s", name);
 		unsupported++;
 	} else {
 		if (test() < 0) {
 			failed++;
-			RTE_LOG(INFO, PMD, "%s Failed\n", name);
+			DPAA2_EVENTDEV_INFO("%s Failed", name);
 		} else {
 			passed++;
-			RTE_LOG(INFO, PMD, "%s Passed", name);
+			DPAA2_EVENTDEV_INFO("%s Passed", name);
 		}
 	}