From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 75EE845549; Tue, 2 Jul 2024 12:40:31 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7712E4065A; Tue, 2 Jul 2024 12:40:30 +0200 (CEST) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2060.outbound.protection.outlook.com [40.107.20.60]) by mails.dpdk.org (Postfix) with ESMTP id B78E2402C4 for ; Tue, 2 Jul 2024 12:40:29 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FmUF4p8aWfHMP3FsmaMdTL/ja9JsrHnR22qoFAU3tH0ay+MlDXuTGOVnBDXos9lyrpBV6GugT7LvX8Vs8wZJxfDgTKk2Y930Kho4rCbgJ6UcIALLe1Pxo0GvhwtxRNMJfreAZP1bphbbPlOsNHloN8BcTp4HlRTSUXQQWzY5UR8laFxBNXMhVBTocp+ABhyh12T13rl0JrBDfS9GoZlQFC5slmRCxfLCPM4W6IILDr3M/wxUMyATWNyna5dKVtEoxP1Up5yRQPRQQAE9Z1eBFLcch5MuHFPatptVwNAGgNgm7CizkUTtKe86+0xtFDDojt2IHDIhedWsUi6ZlkXUmw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=lqOJMTZFpG3R+/KpFaSwz3X/JHJ6ooeBg4zOWtshUz8=; b=Upe/J3RSVQn08xMI8K8Hmq0oTK1MLYCJUgpLko8OArPr8zz1dKV9++6GzkhHTf4w+e9By9W6FFI8NFSBfMSZA9NezkVfZ3xgDmJv5V1zoRh6ZWDpDW4nVvHjCCa5fk2YALcYscq/QtI3QHZ1heI65SV+Mw2qounjcCt0Skc7lUFO4JikPHT/D2usXUYjNkhNckM1bISUDsl1mnYMb5ZCsf4cUSVnU1Ro3NebB1Zo6tLpvydecfHb2s4uAH7YjtDEcZ36hc5kQlm6qcnoYqkx94JgBY+8OI4+4BpN7ZdTXk1qeINaO8mTMKYBKEHFRb7BMvyd/Vey+0A1s/zvKeFCnw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=lqOJMTZFpG3R+/KpFaSwz3X/JHJ6ooeBg4zOWtshUz8=; b=RbXW8PMN1PnLlcPcmdHxcVFZNAIDHvSna2U4SOfJd+xHKT/VfGm1WYiLk6VFnOHXEOGwtAb3rD+tIW9af0RNk4pf2LELoAmrRIBHjSiGJMTILZM3OJk5QtqBUbM6SlfC+ExIPZnmVvbz3TUeOWGXIaNCQjvl7YTuRgm1onhQ1L0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from PAXPR04MB9328.eurprd04.prod.outlook.com (2603:10a6:102:2b6::15) by PA2PR04MB10423.eurprd04.prod.outlook.com (2603:10a6:102:41c::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7719.29; Tue, 2 Jul 2024 10:40:28 +0000 Received: from PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::367:d59:375:fb87]) by PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::367:d59:375:fb87%4]) with mapi id 15.20.7719.028; Tue, 2 Jul 2024 10:40:28 +0000 From: Hemant Agrawal To: dev@dpdk.org, thomas@monjalon.net, stephen@networkplumber.org, david.marchand@redhat.com Subject: [PATCH v2 1/2] drivers: replace printf with log macros Date: Tue, 2 Jul 2024 16:10:12 +0530 Message-Id: <20240702104013.3813272-1-hemant.agrawal@nxp.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230215102905.22767-1-hemant.agrawal@nxp.com> References: <20230215102905.22767-1-hemant.agrawal@nxp.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: SI2PR01CA0033.apcprd01.prod.exchangelabs.com (2603:1096:4:192::23) To PAXPR04MB9328.eurprd04.prod.outlook.com (2603:10a6:102:2b6::15) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9328:EE_|PA2PR04MB10423:EE_ X-MS-Office365-Filtering-Correlation-Id: 0d304ba8-542e-4cc5-aa69-08dc9a8363bb X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|1800799024|366016|376014|52116014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?v5rqq0PVaBX9FRzfa+VydBMlpOnpBnMP3kzMUhB9M3AOrkRHKGGBuYDni8YJ?= =?us-ascii?Q?cTWdrwkLH53GXU+aEUubyBu91lflqrwHX4MpYxzr/hj6ofgOusmPthoKlZLk?= =?us-ascii?Q?kKd+g2y1aWqQrcfM5XWwygtqifo1dGC1Eyr2DPenNi7kWmd8RB4xlNoVeeCL?= =?us-ascii?Q?rtHPyiClEzSodfw2vnjxmydNDXYPsTH+/djkKcffC8nlmmoFZKs7o2vKSaPe?= =?us-ascii?Q?dXpZeoyLgzxhUtl+Gg5wcG9/CQBqkCMB/1U7pijRkqTRwLOuyVUXDzHhTgl7?= =?us-ascii?Q?/oTslE1Bs5EEf6RilmziIrhwDUtKzNdo9ZjQJturiwRAiSWXLBJTlxFx8cQd?= =?us-ascii?Q?H1r8/72+dKD5sKQHqY7Z/bGHKvrgL4btWpjUjA8VjhX1aJYwC14pYi2nIJlh?= =?us-ascii?Q?q3kx3HtWBy2yw8q73k4T/NeX/H/vtwpViJEsJF/1m/5aaS6RHe4Vn0DHSyA8?= =?us-ascii?Q?yqokVE64/3sV7UIoG+smLnXI/65odnd5cATZsi+py6zXTvR7J9sa+/IEE6NT?= =?us-ascii?Q?WGiMlKyIvz1f8UJWDZiJUDIwy7Sqaz9Ua7KfoVElb8QTQ9ct0JT0YbBC0v7z?= =?us-ascii?Q?Rjnvyk0ke2hH7ovFeBLgcPfbH4CqLionCpD56oROlEpD9wRBzT8C44RrSpnw?= =?us-ascii?Q?IIVlS9O983+R7bsHM0QV/S1Ss2Ohd0n8q947SJGavDrOcx3FOm4eOUBEjAKX?= =?us-ascii?Q?c/HLi5RLS2w9siv5uHIIwdNtzF/v2Pk0iaB4b4803Iu6w1lPuVZmMQ2mplU6?= =?us-ascii?Q?wFmAJ1NlMWWT/oIT2mWRsZ3dR69KFv77q/pf0SF7FQZ1VKJHzgkNfwwuYVGs?= =?us-ascii?Q?T5Y8bKi6lf5/nUWI+FOQQ1YVjq5dUAxZFoJ0Sj7xtRXLCUjnVAM7HWCprbWz?= =?us-ascii?Q?VK4lYzntNjAWaDX97WzkwOm0XV17ztcJWBARtEq5rprjiJtlAVCK4fsNPyts?= =?us-ascii?Q?l19nsvnk2bDBlocygGMEPgfsFI3cmwxhILuafrhPDSH7XCRWWYFt/OKNnHZM?= =?us-ascii?Q?Ubm9YegMPtLhgkSO4iuOvc97maxN+p1l3RscxB6/Bk4pM7n0ft5+wxIrgJ9J?= =?us-ascii?Q?I57ghvDaDXpWH1a4kBvySLFMaHdYfrOpfJGcQWAF6DTFl3ROTE8fkd7T7QXL?= =?us-ascii?Q?NcA4Ad+YXbVz2xTI9IQczfYs+gvfe6M+bwBV05jM4Uz2ygyQFvFU9TnGwlQx?= =?us-ascii?Q?liwfxuY4L8Anab/L+VaceMDIunsGdB/toz34g5Xsjq0DAZb1znhMRT6HPEug?= =?us-ascii?Q?a1dSWxUXDFa460EnfOHn3zBm0uwMnoQ669yhVdOus6z0puonpFiy0GM969F4?= =?us-ascii?Q?Lc0zdcYTWGdeWSqapWrAQikbN0IWqMiLekGOZMcAX1Y3ZaCPYaBz/NsC3u8j?= =?us-ascii?Q?iJ+9Au2XtrLxxb16RTCFfipb+VeqrH08a1CToRDSDO+t5TpGZw=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PAXPR04MB9328.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(1800799024)(366016)(376014)(52116014)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Vv0SVfqHfzM4R2PrruxTtDpAyBxM1DBem9bo2SjgeQRU5+iHP1P2D01ZRnBm?= =?us-ascii?Q?AQiBzvMBDlHlSy0wqpGI/5IKIDJS+xH8l9d3rK/BP95R8V8Jfx/eHnRGafkM?= =?us-ascii?Q?phzUN3MpRy+yTOLxNZaWAjwi9sr3AWWM6TBCV6wlLIF17ciZipN7OjIrTvmS?= =?us-ascii?Q?Fj0VS/4uGF4OCIhcqPxRv5lmIrXlhfIJ+aegpZvurppvw/hrlMjevDrLTwJo?= =?us-ascii?Q?fCm9S5t421s8mYbrUvJPNdY5SaTk8HxSxsQSckLq0vIiNgMef7Uk2bWExKPG?= =?us-ascii?Q?UpQfvu0v9ybKWNUYWSeWM8Gda6zQJBWyFeDViTWU0h1nF2qIlshIb4w7zr86?= =?us-ascii?Q?oJQGLvXb/bLcQQElyQEd9WM2uXNbBVrmAUjCKOqslht65wB/2+yp13qbC6lo?= =?us-ascii?Q?0swQgQXl0zkFPW+m7ZCaVoez04jerVSefCmB6i4FuqLY/VVqujRA6aEWki6i?= =?us-ascii?Q?y3GmgPIvuXWCxE5owEsiRn0XxZ3szn2Sg5XZ1GJNQP32NYVk/Rxzk4WsUYHh?= =?us-ascii?Q?2pqgelNW8XEmR0tzobRR4tGkvhwiDQza2U1QbmgUANAsCCgmQ7+hxx4TpRcY?= =?us-ascii?Q?GAiZ3rXvBKeioITsZr4Zu/n5qXxCTjRlmJLHCbidY2edYY0SQwZM95+UUYce?= =?us-ascii?Q?/SCrZmedqBzHerEPLdlhqZHfl+MH9/hnJLR2VDIn8yCggG3Vg5mVkcc/5BXB?= =?us-ascii?Q?TiQeeXeszq07L+s1jVf9Ygu0fwowD8oYhzNG68mAvuSQruLTCcqK2PFk6MHZ?= =?us-ascii?Q?EMBSA3N67cbgMwnK8up9YCLdoWBo+kqJfzIBZWyV50jhX3aiyJAOkDCQUeCG?= =?us-ascii?Q?mlmN1wWu4VrXvEoKOA9iu4JM1YVFR4B7gRE7EUITjJ5c84+oDagAVci8wooJ?= =?us-ascii?Q?9FGyMMLmAh1IjiOaVQY7Wr2SHsYGwQB93N9zp8tf6Sc8Yq5vlgCZzhCbI2xG?= =?us-ascii?Q?l7FtG2+Lm0Yf6O7fpciWiNNZnlYNlOGeuefcbDsqtmi8Iz9b5rOVqhRtnMG7?= =?us-ascii?Q?D8aMy1BXFT/ZDMYQGyrkZ6fRcgpYtSc/qAcLgxvowngb4A1VjACCtsnL6jJj?= =?us-ascii?Q?NqJ43OrenrVM9fNPorpC0hE3wVq3fBUxen2lEMgwaZ06mlK4rBM63zXngm1h?= =?us-ascii?Q?+elv3ae+7wXxjhg5EMaUw38fCNGiCune/VszYQIrGTuwI55+RgPE6yXseyrk?= =?us-ascii?Q?+q8QiJfjXmufLNKNX45VX2dWngaMWbqWTzMwgrq8eMvCk3SjhV8x31hT1r5K?= =?us-ascii?Q?sdpP2zBdfwGQrlYg0Qh6QLvQudDFNFX3nOrO+LTcP3DruVQXuSlIdYc/RueA?= =?us-ascii?Q?lo1hP/egXl5yYLQ0yf0qnK9C8kQ6u4eKYdsfjYQpcEc+hTJODXcyv/4zgBJG?= =?us-ascii?Q?suaq75go5/h1lL7oFtp87VzwjAVj4ctMPVWNpHKG+UTQN3a+RB/ZHx4LVD7H?= =?us-ascii?Q?OUXUg+2h/kEJle49iPdeW6bXir/dxu0ioBdE1SUnDr3nJ9ijWpFdKlgN1X1O?= =?us-ascii?Q?7GFHgh/6brk91owTbUjnAKH3SmytldWRUDcQKEDHRF6E5vI1/GlONzK85Ma/?= =?us-ascii?Q?2YyD2EYGA1MjBApgRZPW0GgRlNF9/3tQwL0Ftugsr6MZ+Lf1Rlj8iCIGBiTc?= =?us-ascii?Q?yg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0d304ba8-542e-4cc5-aa69-08dc9a8363bb X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9328.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jul 2024 10:40:28.5694 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 4z3F3Xba2eR/iRBAxE4BNA3JQ4BMbE6iwujmPE21Ejhdgif5SGEmlwq0GASqsYNdV9LO3AtSWQPelXR3auRjOg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA2PR04MB10423 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 This patch replaces the printf with related log macros and functions at various places in NXP dpaaX drivers. Signed-off-by: Hemant Agrawal --- drivers/bus/dpaa/base/fman/fman.c | 8 +++---- drivers/bus/dpaa/base/qbman/process.c | 29 +++++++++++++------------- drivers/bus/dpaa/base/qbman/qman.c | 2 +- drivers/bus/fslmc/fslmc_vfio.c | 2 +- drivers/bus/fslmc/qbman/qbman_portal.c | 2 +- drivers/crypto/caam_jr/caam_jr.c | 2 +- drivers/crypto/caam_jr/caam_jr_uio.c | 2 +- drivers/net/dpaa/dpaa_ethdev.c | 6 +++--- drivers/net/dpaa/dpaa_flow.c | 8 +++---- drivers/net/dpaa/dpaa_rxtx.c | 2 +- drivers/net/dpaa2/dpaa2_ethdev.c | 2 +- drivers/net/dpaa2/dpaa2_tm.c | 10 ++++----- 12 files changed, 37 insertions(+), 38 deletions(-) diff --git a/drivers/bus/dpaa/base/fman/fman.c b/drivers/bus/dpaa/base/fman/fman.c index 1814372a40..2fc1e64f36 100644 --- a/drivers/bus/dpaa/base/fman/fman.c +++ b/drivers/bus/dpaa/base/fman/fman.c @@ -425,7 +425,7 @@ fman_if_init(const struct device_node *dpa_node) char_prop = of_get_property(mac_node, "phy-connection-type", NULL); if (!char_prop) { - printf("memac: unknown MII type assuming 1G\n"); + FMAN_ERR(-EINVAL, "memac: unknown MII type assuming 1G\n"); /* Right now forcing memac to 1g in case of error*/ __if->__if.mac_type = fman_mac_1g; } else { @@ -723,10 +723,8 @@ fman_finish(void) /* release the mapping */ _errno = munmap(__if->ccsr_map, __if->regs_size); if (unlikely(_errno < 0)) - fprintf(stderr, "%s:%d:%s(): munmap() = %d (%s)\n", - __FILE__, __LINE__, __func__, - -errno, strerror(errno)); - printf("Tearing down %s\n", __if->node_path); + FMAN_ERR(_errno, "munmap() = (%s)\n", strerror(errno)); + DPAA_BUS_INFO("Tearing down %s\n", __if->node_path); list_del(&__if->__if.node); rte_free(__if); } diff --git a/drivers/bus/dpaa/base/qbman/process.c b/drivers/bus/dpaa/base/qbman/process.c index 3504ec97db..af1e459641 100644 --- a/drivers/bus/dpaa/base/qbman/process.c +++ b/drivers/bus/dpaa/base/qbman/process.c @@ -13,6 +13,7 @@ #include "process.h" #include +#include "rte_dpaa_logs.h" /* As higher-level drivers will be built on top of this (dma_mem, qbman, ...), * it's preferable that the process driver itself not provide any exported API. @@ -99,12 +100,12 @@ void process_release(enum dpaa_id_type id_type, uint32_t base, uint32_t num) int ret = check_fd(); if (ret) { - fprintf(stderr, "Process FD failure\n"); + DPAA_BUS_ERR("Process FD failure\n"); return; } ret = ioctl(fd, DPAA_IOCTL_ID_RELEASE, &id); if (ret) - fprintf(stderr, "Process FD ioctl failure type %d base 0x%x num %d\n", + DPAA_BUS_ERR("Process FD ioctl failure type %d base 0x%x num %d\n", id_type, base, num); } @@ -333,9 +334,9 @@ int dpaa_intr_disable(char *if_name) ret = ioctl(fd, DPAA_IOCTL_DISABLE_LINK_STATUS_INTERRUPT, if_name); if (ret) { if (errno == EINVAL) - printf("Failed to disable interrupt: Not Supported\n"); + DPAA_BUS_ERR("Failed to disable interrupt: Not Supported\n"); else - printf("Failed to disable interrupt\n"); + DPAA_BUS_ERR("Failed to disable interrupt\n"); return ret; } @@ -357,7 +358,7 @@ int dpaa_get_ioctl_version_number(void) if (errno == EINVAL) { version_num = 1; } else { - printf("Failed to get ioctl version number\n"); + DPAA_BUS_ERR("Failed to get ioctl version number\n"); version_num = -1; } } @@ -388,7 +389,7 @@ int dpaa_get_link_status(char *if_name, struct rte_eth_link *link) ret = ioctl(fd, DPAA_IOCTL_GET_LINK_STATUS, &args); if (ret) { - printf("Failed to get link status\n"); + DPAA_BUS_ERR("Failed to get link status\n"); return ret; } @@ -404,9 +405,9 @@ int dpaa_get_link_status(char *if_name, struct rte_eth_link *link) ret = ioctl(fd, DPAA_IOCTL_GET_LINK_STATUS_OLD, &args); if (ret) { if (errno == EINVAL) - printf("Get link status: Not Supported\n"); + DPAA_BUS_ERR("Get link status: Not Supported\n"); else - printf("Failed to get link status\n"); + DPAA_BUS_ERR("Failed to get link status\n"); return ret; } @@ -434,9 +435,9 @@ int dpaa_update_link_status(char *if_name, int link_status) ret = ioctl(fd, DPAA_IOCTL_UPDATE_LINK_STATUS, &args); if (ret) { if (errno == EINVAL) - printf("Failed to set link status: Not Supported\n"); + DPAA_BUS_ERR("Failed to set link status: Not Supported\n"); else - printf("Failed to set link status"); + DPAA_BUS_ERR("Failed to set link status"); return ret; } @@ -462,9 +463,9 @@ int dpaa_update_link_speed(char *if_name, int link_speed, int link_duplex) ret = ioctl(fd, DPAA_IOCTL_UPDATE_LINK_SPEED, &args); if (ret) { if (errno == EINVAL) - printf("Failed to set link speed: Not Supported\n"); + DPAA_BUS_ERR("Failed to set link speed: Not Supported\n"); else - printf("Failed to set link speed\n"); + DPAA_BUS_ERR("Failed to set link speed\n"); return ret; } @@ -484,9 +485,9 @@ int dpaa_restart_link_autoneg(char *if_name) ret = ioctl(fd, DPAA_IOCTL_RESTART_LINK_AUTONEG, if_name); if (ret) { if (errno == EINVAL) - printf("Failed to restart autoneg: Not Supported\n"); + DPAA_BUS_ERR("Failed to restart autoneg: Not Supported\n"); else - printf("Failed to restart autoneg\n"); + DPAA_BUS_ERR("Failed to restart autoneg\n"); return ret; } diff --git a/drivers/bus/dpaa/base/qbman/qman.c b/drivers/bus/dpaa/base/qbman/qman.c index 6d2fbdcf02..b1166fcb1f 100644 --- a/drivers/bus/dpaa/base/qbman/qman.c +++ b/drivers/bus/dpaa/base/qbman/qman.c @@ -817,7 +817,7 @@ qman_ern_poll_free(void) fd = &swapped_msg.ern.fd; if (unlikely(verb & 0x20)) { - printf("HW ERN notification, Nothing to do\n"); + pr_warn("HW ERN notification, Nothing to do\n"); } else { if ((fd->bpid & 0xff) != 0xff) qman_free_mbuf_cb(fd); diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c index 5966776a85..da0669d554 100644 --- a/drivers/bus/fslmc/fslmc_vfio.c +++ b/drivers/bus/fslmc/fslmc_vfio.c @@ -395,7 +395,7 @@ rte_fslmc_vfio_mem_dmamap(uint64_t vaddr, uint64_t iova, uint64_t size) ret = ioctl(group->container->fd, VFIO_IOMMU_MAP_DMA, &dma_map); if (ret) { - printf("Unable to map DMA address (errno = %d)\n", + DPAA2_BUS_ERR("Unable to map DMA address (errno = %d)\n", errno); return ret; } diff --git a/drivers/bus/fslmc/qbman/qbman_portal.c b/drivers/bus/fslmc/qbman/qbman_portal.c index 3a7579c8a7..1f24cdce7e 100644 --- a/drivers/bus/fslmc/qbman/qbman_portal.c +++ b/drivers/bus/fslmc/qbman/qbman_portal.c @@ -1879,7 +1879,7 @@ void qbman_pull_desc_set_rad(struct qbman_pull_desc *d, int rad) else d->pull.verb &= ~(1 << QB_VDQCR_VERB_RAD_SHIFT); } else { - printf("The RAD feature is not valid when RLS = 0\n"); + pr_warn("The RAD feature is not valid when RLS = 0\n"); } } diff --git a/drivers/crypto/caam_jr/caam_jr.c b/drivers/crypto/caam_jr/caam_jr.c index 0fa70c978f..ab06fab387 100644 --- a/drivers/crypto/caam_jr/caam_jr.c +++ b/drivers/crypto/caam_jr/caam_jr.c @@ -587,7 +587,7 @@ hw_poll_job_ring(struct sec_job_ring_t *job_ring, /* todo check if it is false alarm no desc present */ if (!current_desc_addr) { false_alarm++; - printf("false alarm %" PRIu64 "real %" PRIu64 + CAAM_JR_ERR("false alarm %" PRIu64 "real %" PRIu64 " sec_err =0x%x cidx Index =0%d\n", false_alarm, real_poll, sec_error_code, job_ring->cidx); diff --git a/drivers/crypto/caam_jr/caam_jr_uio.c b/drivers/crypto/caam_jr/caam_jr_uio.c index 583ba3b523..586beceb14 100644 --- a/drivers/crypto/caam_jr/caam_jr_uio.c +++ b/drivers/crypto/caam_jr/caam_jr_uio.c @@ -418,7 +418,7 @@ sec_configure(void) d = opendir(SEC_UIO_DEVICE_SYS_ATTR_PATH); if (d == NULL) { - printf("\nError opening directory '%s': %s\n", + CAAM_JR_ERR("Error opening directory '%s': %s\n", SEC_UIO_DEVICE_SYS_ATTR_PATH, strerror(errno)); return -1; } diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c index 935a4f2ada..bcd19f31fb 100644 --- a/drivers/net/dpaa/dpaa_ethdev.c +++ b/drivers/net/dpaa/dpaa_ethdev.c @@ -282,9 +282,9 @@ dpaa_eth_dev_configure(struct rte_eth_dev *dev) dpaa_interrupt_handler, (void *)dev); if (ret == EINVAL) - printf("Failed to enable interrupt: Not Supported\n"); + DPAA_PMD_ERR("Failed to enable interrupt: Not Supported\n"); else - printf("Failed to enable interrupt\n"); + DPAA_PMD_ERR("Failed to enable interrupt\n"); } dev->data->dev_conf.intr_conf.lsc = 0; dev->data->dev_flags &= ~RTE_ETH_DEV_INTR_LSC; @@ -340,7 +340,7 @@ dpaa_eth_dev_configure(struct rte_eth_dev *dev) dpaa_update_link_speed(__fif->node_name, speed, duplex); } else { /* Manual autoneg - custom advertisement speed. */ - printf("Custom Advertisement speeds not supported\n"); + DPAA_PMD_ERR("Custom Advertisement speeds not supported\n"); } } diff --git a/drivers/net/dpaa/dpaa_flow.c b/drivers/net/dpaa/dpaa_flow.c index 690ba6bcb3..5c75f4220e 100644 --- a/drivers/net/dpaa/dpaa_flow.c +++ b/drivers/net/dpaa/dpaa_flow.c @@ -68,7 +68,7 @@ static void fm_prev_cleanup(void) fm_info.fman_handle = fm_open(fman_id); if (!fm_info.fman_handle) { - printf("\n%s- unable to open FMAN", __func__); + DPAA_PMD_ERR("unable to open FMAN"); return; } @@ -78,7 +78,7 @@ static void fm_prev_cleanup(void) /* FM PCD Open */ fm_info.pcd_handle = fm_pcd_open(&fm_pcd_params); if (!fm_info.pcd_handle) { - printf("\n%s- unable to open PCD", __func__); + DPAA_PMD_ERR("unable to open PCD"); return; } @@ -108,11 +108,11 @@ static void fm_prev_cleanup(void) continue; if (dpaa_fm_deconfig(&dpaa_intf, NULL)) - printf("\nDPAA FM deconfig failed\n"); + DPAA_PMD_ERR("DPAA FM deconfig failed\n"); } if (dpaa_fm_term()) - printf("\nDPAA FM term failed\n"); + DPAA_PMD_ERR("DPAA FM term failed\n"); memset(&fm_model, 0, sizeof(struct dpaa_fm_model)); } diff --git a/drivers/net/dpaa/dpaa_rxtx.c b/drivers/net/dpaa/dpaa_rxtx.c index ce4f3d6c85..c2579d65ee 100644 --- a/drivers/net/dpaa/dpaa_rxtx.c +++ b/drivers/net/dpaa/dpaa_rxtx.c @@ -103,7 +103,7 @@ static void dpaa_display_frame_info(const struct qm_fd *fd, for (ii = 0; ii < fd->length20; ii++) { DISPLAY_PRINT("%02x ", ptr[ii]); if (((ii + 1) % 16) == 0) - printf("\n"); + DISPLAY_PRINT("\n"); } DISPLAY_PRINT("\n"); } diff --git a/drivers/net/dpaa2/dpaa2_ethdev.c b/drivers/net/dpaa2/dpaa2_ethdev.c index 97edc00420..89b4454e6b 100644 --- a/drivers/net/dpaa2/dpaa2_ethdev.c +++ b/drivers/net/dpaa2/dpaa2_ethdev.c @@ -2694,7 +2694,7 @@ dpaa2_dev_init(struct rte_eth_dev *eth_dev) priv->max_vlan_filters = attr.vlan_filter_entries; priv->flags = 0; #if defined(RTE_LIBRTE_IEEE1588) - printf("DPDK IEEE1588 is enabled\n"); + DPAA2_PMD_INFO("DPDK IEEE1588 is enabled\n"); priv->flags |= DPAA2_TX_CONF_ENABLE; #endif /* Used with ``fslmc:dpni.1,drv_tx_conf=1`` */ diff --git a/drivers/net/dpaa2/dpaa2_tm.c b/drivers/net/dpaa2/dpaa2_tm.c index 8fe5bfa013..cd6739dcfa 100644 --- a/drivers/net/dpaa2/dpaa2_tm.c +++ b/drivers/net/dpaa2/dpaa2_tm.c @@ -499,7 +499,7 @@ dpaa2_node_add(struct rte_eth_dev *dev, uint32_t node_id, node->channel_id = priv->channel_inuse; priv->channel_inuse++; } else { - printf("error no channel id available\n"); + DPAA2_PMD_ERR("error no channel id available\n"); } } @@ -580,7 +580,7 @@ dpaa2_tm_configure_queue(struct rte_eth_dev *dev, struct dpaa2_tm_node *node) flow_id = 0; if (dpaa2_q == NULL) { - printf("Queue is not configured for node = %d\n", node->id); + DPAA2_PMD_ERR("Queue is not configured for node = %d\n", node->id); return -1; } @@ -590,7 +590,7 @@ dpaa2_tm_configure_queue(struct rte_eth_dev *dev, struct dpaa2_tm_node *node) ((node->parent->channel_id << 8) | tc_id), flow_id, options, &tx_flow_cfg); if (ret) { - printf("Error in setting the tx flow: " + DPAA2_PMD_ERR("Error in setting the tx flow: " "channel id = %d tc_id= %d, param = 0x%x " "flow=%d err=%d\n", node->parent->channel_id, tc_id, ((node->parent->channel_id << 8) | tc_id), flow_id, @@ -605,7 +605,7 @@ dpaa2_tm_configure_queue(struct rte_eth_dev *dev, struct dpaa2_tm_node *node) DPNI_QUEUE_TX, ((node->parent->channel_id << 8) | dpaa2_q->tc_index), dpaa2_q->flow_id, &tx_flow_cfg, &qid); if (ret) { - printf("Error in getting LFQID err=%d", ret); + DPAA2_PMD_ERR("Error in getting LFQID err=%d", ret); return -1; } dpaa2_q->fqid = qid.fqid; @@ -636,7 +636,7 @@ dpaa2_tm_configure_queue(struct rte_eth_dev *dev, struct dpaa2_tm_node *node) ((node->parent->channel_id << 8) | tc_id), &cong_notif_cfg); if (ret) { - printf("Error in setting tx congestion notification: " + DPAA2_PMD_ERR("Error in setting tx congestion notification: " "err=%d", ret); return -ret; } -- 2.25.1