From patchwork Tue Feb 7 10:48:02 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Conor Walsh X-Patchwork-Id: 123223 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 DBBCA41C2D; Tue, 7 Feb 2023 11:48:07 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C77F642D0E; Tue, 7 Feb 2023 11:48:07 +0100 (CET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mails.dpdk.org (Postfix) with ESMTP id 8E33540EF0 for ; Tue, 7 Feb 2023 11:48:06 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1675766886; x=1707302886; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=R4mEukdv163ZiztPs5EgqzkAuYzZd83dItFZ0eXp+vw=; b=Uf/Ro/VUhyz1NfEGG5nW4hdgd8LlwFB3NbdjbZEFD/dV2hgGuTvXBuPq 782/nhCTabPGJ8CjlStJA7CSytEjoFnAxFjQ4C7J2BojQr12ZFQVOqH2p bDGS22cd72osQSS8p0E9p16dYUAS6+zEGL5ziPA/vd/SiijL1PBHn4bdC wMBmM/qjmWzyX07xTOMaA29SREBM3ZdmO0lcnZUKAJkma81EI+rMzly14 7pH8AzO2SdUCc2LIl7q+6tmRaVlY1JSZRh8odK+LlINbvDxgEoTgC/O3e ahbhT/vsjVcSztIb60zLzpfdrE2rinTF6vaVStEn7L6lti+EZnbvbD6fL w==; X-IronPort-AV: E=McAfee;i="6500,9779,10613"; a="394073766" X-IronPort-AV: E=Sophos;i="5.97,278,1669104000"; d="scan'208";a="394073766" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2023 02:48:05 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10613"; a="755593187" X-IronPort-AV: E=Sophos;i="5.97,278,1669104000"; d="scan'208";a="755593187" Received: from silpixa00401160.ir.intel.com ([10.55.129.111]) by FMSMGA003.fm.intel.com with ESMTP; 07 Feb 2023 02:48:04 -0800 From: Conor Walsh To: bruce.richardson@intel.com, thomas@monjalon.net Cc: dev@dpdk.org, Conor Walsh Subject: [PATCH v2] dma/ioat: remove printf within ioat driver Date: Tue, 7 Feb 2023 10:48:02 +0000 Message-Id: <20230207104802.105754-1-conor.walsh@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230203163514.621178-1-conor.walsh@intel.com> References: <20230203163514.621178-1-conor.walsh@intel.com> 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 --- v2: fix whitespace issue --- drivers/dma/ioat/ioat_dmadev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/ioat/ioat_dmadev.c b/drivers/dma/ioat/ioat_dmadev.c index 5906eb45aa..84b3dfdb75 100644 --- a/drivers/dma/ioat/ioat_dmadev.c +++ b/drivers/dma/ioat/ioat_dmadev.c @@ -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);