From patchwork Fri Feb 3 16:35:14 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Conor Walsh X-Patchwork-Id: 123039 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id DA3B841BBE; Fri, 3 Feb 2023 17:35:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 70C604067B; Fri, 3 Feb 2023 17:35:21 +0100 (CET) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mails.dpdk.org (Postfix) with ESMTP id 615064021E for ; Fri, 3 Feb 2023 17:35:20 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1675442120; x=1706978120; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=7t+y5SCSlfhPPOdUcq2YKFwFF5VeTLgFsRds3mlMttY=; b=Lkq41Jun7e/IEyZ7XwzO9vDz4+JFBoIL2Xakh4K0mPVqjSMZN58WVKwq skdDEvanTIGaEWz5A22uWPkzAd1jK6GB635ZtHQLca3AeHiMgRlT0Jv59 PP/ZmUP5rhv9d24E2eSp9pZCGZ5JYqd3WP5mjlTwYF2ezM5S6Jh+rYOhY q+LXjlGdbR5srtMi7pxw/NsUO8gy/Le7WYmVFwmqogBp7mqwNt+OWQHh4 M75stMhmldv8xoNsRPuRyNVObnFYhEI3C17oUXrHnaAJ9NImPS77RVjoL 8ssfePCjeD/zDx4Fl8le8FL6eT4/+n9Zv7ks8i/lDBpvlK5HqWSugaFyN w==; X-IronPort-AV: E=McAfee;i="6500,9779,10610"; a="309123879" X-IronPort-AV: E=Sophos;i="5.97,271,1669104000"; d="scan'208";a="309123879" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Feb 2023 08:35:18 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10610"; a="774390196" X-IronPort-AV: E=Sophos;i="5.97,270,1669104000"; d="scan'208";a="774390196" Received: from silpixa00401160.ir.intel.com ([10.55.129.111]) by fmsmga002.fm.intel.com with ESMTP; 03 Feb 2023 08:35:17 -0800 From: Conor Walsh To: bruce.richardson@intel.com, thomas@monjalon.net Cc: dev@dpdk.org, Conor Walsh Subject: [PATCH] dma/ioat: remove printf within ioat driver Date: Fri, 3 Feb 2023 16:35:14 +0000 Message-Id: <20230203163514.621178-1-conor.walsh@intel.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 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 Acked-by: Bruce Richardson --- 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");