[v2] dma/ioat: remove printf within ioat driver
Checks
Commit Message
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@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---
v2: fix whitespace issue
---
drivers/dma/ioat/ioat_dmadev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
07/02/2023 11:48, Conor Walsh:
> 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@intel.com>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Applied, thanks.
@@ -146,7 +146,7 @@ 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",
+ IOAT_PMD_DEBUG("channel status - %s [0x%"PRIx64"]\n",
chansts_readable[ioat->status & IOAT_CHANSTS_STATUS],
ioat->status);