[PATCH] dma/ioat: remove printf within ioat driver

Conor Walsh conor.walsh at intel.com
Fri Feb 3 17:35:14 CET 2023


rte_log should be used instead of printf within the drivers this patch
changes a printf within the ioat dma driver's start function to an rte log.
The printfs within ioat's dump function will be retained.

Signed-off-by: Conor Walsh <conor.walsh at intel.com>
---
 drivers/dma/ioat/ioat_dmadev.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/dma/ioat/ioat_dmadev.c b/drivers/dma/ioat/ioat_dmadev.c
index 5906eb45aa..3fe50ea4a4 100644
--- a/drivers/dma/ioat/ioat_dmadev.c
+++ b/drivers/dma/ioat/ioat_dmadev.c
@@ -146,9 +146,9 @@ ioat_dev_start(struct rte_dma_dev *dev)
 	/* Prime the status register to be set to the last element. */
 	ioat->status = ioat->ring_addr + ((ioat->qcfg.nb_desc - 1) * DESC_SZ);
 
-	printf("IOAT.status: %s [0x%"PRIx64"]\n",
-			chansts_readable[ioat->status & IOAT_CHANSTS_STATUS],
-			ioat->status);
+	IOAT_PMD_DEBUG("channel status - %s [0x%"PRIx64"]\n",
+		       chansts_readable[ioat->status & IOAT_CHANSTS_STATUS],
+		       ioat->status);
 
 	if ((ioat->regs->chansts & IOAT_CHANSTS_STATUS) == IOAT_CHANSTS_HALTED) {
 		IOAT_PMD_WARN("Device HALTED on start, attempting to recover\n");
-- 
2.25.1



More information about the dev mailing list