The ifpga rawdev driver uses printf() directly for runtime messages.
DPDK drivers should use the logging framework instead of printing
to standard output.

Replace the remaining direct printf() calls in ifpga_rawdev.c with
the driver logging macros.

Signed-off-by: Weijun Pan <[email protected]>
Acked-by: Bruce Richardson <[email protected]>

---
v3:
- change AER register-value log from INFO to DEBUG
---
 drivers/raw/ifpga/ifpga_rawdev.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/raw/ifpga/ifpga_rawdev.c b/drivers/raw/ifpga/ifpga_rawdev.c
index 5b9b596435..c6a1b193f7 100644
--- a/drivers/raw/ifpga/ifpga_rawdev.c
+++ b/drivers/raw/ifpga/ifpga_rawdev.c
@@ -434,7 +434,7 @@ static int set_surprise_link_check_aer(
        uint32_t aer_new0, aer_new1;
 
        if (!ifpga_rdev || !ifpga_rdev->rawdev) {
-               printf("\n device does not exist\n");
+               IFPGA_RAWDEV_PMD_ERR("device does not exist");
                return -EFAULT;
        }
 
@@ -491,7 +491,7 @@ static int set_surprise_link_check_aer(
                if (fd != -1)
                        close(fd);
 
-               printf(">>>>>>Set AER %x,%x %x,%x\n",
+               IFPGA_RAWDEV_PMD_DEBUG(">>>>>>Set AER %x,%x %x,%x",
                        ifpga_rdev->aer_old[0], ifpga_rdev->aer_old[1],
                        aer_new0, aer_new1);
 
@@ -527,7 +527,7 @@ ifpga_rawdev_gsd_handle(__rte_unused void *param)
                }
 
                if (gsd_enable)
-                       printf(">>>>>>Pls Shutdown APP\n");
+                       IFPGA_RAWDEV_PMD_WARN(">>>>>>Pls Shutdown APP");
 
                rte_delay_us(100 * MS);
        }
-- 
2.34.1

Reply via email to