From patchwork Mon Oct 7 12:41:10 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jerin Jacob X-Patchwork-Id: 145303 X-Patchwork-Delegate: david.marchand@redhat.com 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 95C4F45AD8; Mon, 7 Oct 2024 14:41:27 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 66B5F402BB; Mon, 7 Oct 2024 14:41:27 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id BF35E4026C for ; Mon, 7 Oct 2024 14:41:25 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 497Amk3c016278; Mon, 7 Oct 2024 05:41:25 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=pfpt0220; bh=D 8d4R53li5SFJyS2Ae8AaaBG5wsAQWjeXVwEqFPFB1E=; b=hUZckcqr8FBmtZaiU IrtCrQ1FCYgxqSfVA65V6GbeJQU8r8pYDr/O9xoU67vSkvkhQoDC6vSrLblO4Ku9 HZl5FPQU8vzVl89a0WJgq8g/JJ5IpEPJdLYI8HVRFI+dmQMxhRPNH8f+xbbP+DwT If28nq2PNLgfq2U0gTUgAWIBrybpH6boH8+Ilb6nm8FMMw8jBamkJGzJ0P37HpZn 3HksiDld//MmoCXrEszteMARS2SJuu74QFoSBcyvAeKq7I1CNWrd5pPW2S4zB67a pcRptUi3dnAYBl00Ux/ldG7/1eQnykKiE4IVDaebAONPV0xg3Qm4koCFYHM7bVs9 Eo5Bw== Received: from dc5-exch05.marvell.com ([199.233.59.128]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 4234yg3bu9-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 07 Oct 2024 05:41:24 -0700 (PDT) Received: from DC5-EXCH05.marvell.com (10.69.176.209) by DC5-EXCH05.marvell.com (10.69.176.209) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 7 Oct 2024 05:41:23 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH05.marvell.com (10.69.176.209) with Microsoft SMTP Server id 15.2.1544.4 via Frontend Transport; Mon, 7 Oct 2024 05:41:23 -0700 Received: from jerin-lab.marvell.com (jerin-lab.marvell.com [10.28.34.14]) by maili.marvell.com (Postfix) with ESMTP id EA0595E6872; Mon, 7 Oct 2024 05:41:20 -0700 (PDT) From: To: Srikanth Yalavarthi , Liron Himi , Chengwen Feng , David Marchand CC: , , Jerin Jacob Subject: [dpdk-dev] [PATCH v2] drivers: fix build issues with redundant newline changes Date: Mon, 7 Oct 2024 18:11:10 +0530 Message-ID: <20241007124110.1020548-1-jerinj@marvell.com> X-Mailer: git-send-email 2.46.1 In-Reply-To: <20241007081301.2999359-1-jerinj@marvell.com> References: <20241007081301.2999359-1-jerinj@marvell.com> MIME-Version: 1.0 X-Proofpoint-GUID: OZm-l_u0C9DHwbfbCzs8HWUrQM2vW7Nl X-Proofpoint-ORIG-GUID: OZm-l_u0C9DHwbfbCzs8HWUrQM2vW7Nl X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.60.29 definitions=2024-09-06_09,2024-09-06_01,2024-09-02_01 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 From: Jerin Jacob Certain section of the code in net/mvpp2 and ml/cnxk is enabled only when all depended library dependences where meet. Fixing those build errors. Fixes: f665790a5dba ("drivers: remove redundant newline from logs") Signed-off-by: Jerin Jacob Reviewed-by: David Marchand --- v2: - Add https://patches.dpdk.org/project/dpdk/patch/20241007081301.2999359-1-jerinj@marvell.com/ review comments (David) drivers/ml/cnxk/mvtvm_ml_dev.c | 8 ++--- drivers/ml/cnxk/mvtvm_ml_model.c | 4 +-- drivers/ml/cnxk/mvtvm_ml_ops.c | 6 ++-- drivers/net/mvpp2/mrvl_ethdev.c | 14 ++++---- drivers/net/mvpp2/mrvl_qos.c | 56 ++++++++++++++++---------------- 5 files changed, 44 insertions(+), 44 deletions(-) diff --git a/drivers/ml/cnxk/mvtvm_ml_dev.c b/drivers/ml/cnxk/mvtvm_ml_dev.c index c9b9444eff..74959a12fc 100644 --- a/drivers/ml/cnxk/mvtvm_ml_dev.c +++ b/drivers/ml/cnxk/mvtvm_ml_dev.c @@ -63,7 +63,7 @@ mvtvm_mldev_parse_devargs(const char *args, struct mvtvm_ml_dev *mvtvm_mldev) kvlist = rte_kvargs_parse(args, valid_args); if (kvlist == NULL) { - plt_err("Error parsing %s devargs\n", "MLDEV_NAME_MVTVM_PMD"); + plt_err("Error parsing %s devargs", "MLDEV_NAME_MVTVM_PMD"); return -EINVAL; } @@ -71,7 +71,7 @@ mvtvm_mldev_parse_devargs(const char *args, struct mvtvm_ml_dev *mvtvm_mldev) ret = rte_kvargs_process(kvlist, MVTVM_ML_DEV_MAX_QPS, &parse_uint_arg, &mvtvm_mldev->max_nb_qpairs); if (ret < 0) { - plt_err("Error processing arguments, key = %s\n", MVTVM_ML_DEV_MAX_QPS); + plt_err("Error processing arguments, key = %s", MVTVM_ML_DEV_MAX_QPS); ret = -EINVAL; goto exit; } @@ -82,7 +82,7 @@ mvtvm_mldev_parse_devargs(const char *args, struct mvtvm_ml_dev *mvtvm_mldev) ret = rte_kvargs_process(kvlist, MVTVM_ML_DEV_CACHE_MODEL_DATA, &parse_integer_arg, &mvtvm_mldev->cache_model_data); if (ret < 0) { - plt_err("Error processing arguments, key = %s\n", + plt_err("Error processing arguments, key = %s", MVTVM_ML_DEV_CACHE_MODEL_DATA); ret = -EINVAL; goto exit; @@ -99,7 +99,7 @@ mvtvm_mldev_parse_devargs(const char *args, struct mvtvm_ml_dev *mvtvm_mldev) mvtvm_mldev->cache_model_data = CN10K_ML_DEV_CACHE_MODEL_DATA_DEFAULT; } else { if ((mvtvm_mldev->cache_model_data < 0) || (mvtvm_mldev->cache_model_data > 1)) { - plt_err("Invalid argument, %s = %d\n", MVTVM_ML_DEV_CACHE_MODEL_DATA, + plt_err("Invalid argument, %s = %d", MVTVM_ML_DEV_CACHE_MODEL_DATA, mvtvm_mldev->cache_model_data); ret = -EINVAL; goto exit; diff --git a/drivers/ml/cnxk/mvtvm_ml_model.c b/drivers/ml/cnxk/mvtvm_ml_model.c index e3234ae442..3ada6f42db 100644 --- a/drivers/ml/cnxk/mvtvm_ml_model.c +++ b/drivers/ml/cnxk/mvtvm_ml_model.c @@ -50,7 +50,7 @@ mvtvm_ml_model_type_get(struct rte_ml_model_params *params) /* Check if all objects are available */ for (i = 0; i < ML_MVTVM_MODEL_OBJECT_MAX; i++) { if (!object_found[i]) { - plt_err("Object %s not found in archive!\n", mvtvm_object_list[i]); + plt_err("Object %s not found in archive!", mvtvm_object_list[i]); return ML_CNXK_MODEL_TYPE_INVALID; } } @@ -100,7 +100,7 @@ mvtvm_ml_model_blob_parse(struct rte_ml_model_params *params, struct mvtvm_ml_mo /* Check if all objects are parsed */ for (i = 0; i < ML_MVTVM_MODEL_OBJECT_MAX; i++) { if (!object_found[i]) { - plt_err("Object %s not found in archive!\n", mvtvm_object_list[i]); + plt_err("Object %s not found in archive!", mvtvm_object_list[i]); goto error; } } diff --git a/drivers/ml/cnxk/mvtvm_ml_ops.c b/drivers/ml/cnxk/mvtvm_ml_ops.c index e825c3fb23..ddde104f5b 100644 --- a/drivers/ml/cnxk/mvtvm_ml_ops.c +++ b/drivers/ml/cnxk/mvtvm_ml_ops.c @@ -123,7 +123,7 @@ mvtvm_ml_dev_configure(struct cnxk_ml_dev *cnxk_mldev, const struct rte_ml_dev_c /* Configure TVMDP library */ ret = tvmdp_configure(cnxk_mldev->mldev->data->nb_models, rte_get_tsc_cycles); if (ret != 0) - plt_err("TVMDP configuration failed, error = %d\n", ret); + plt_err("TVMDP configuration failed, error = %d", ret); return ret; } @@ -138,7 +138,7 @@ mvtvm_ml_dev_close(struct cnxk_ml_dev *cnxk_mldev) /* Close TVMDP library configuration */ ret = tvmdp_close(); if (ret != 0) - plt_err("TVMDP close failed, error = %d\n", ret); + plt_err("TVMDP close failed, error = %d", ret); return ret; } @@ -296,7 +296,7 @@ mvtvm_ml_model_load(struct cnxk_ml_dev *cnxk_mldev, struct rte_ml_model_params * /* Get model metadata - stage 2 */ ret = tvmdp_model_metadata_get_stage2(model->model_id, &model->mvtvm.metadata); if (ret != 0) { - plt_err("TVMDP: Failed to get metadata, model_id = %u, error = %d\n", + plt_err("TVMDP: Failed to get metadata, model_id = %u, error = %d", model->model_id, ret); goto error; } diff --git a/drivers/net/mvpp2/mrvl_ethdev.c b/drivers/net/mvpp2/mrvl_ethdev.c index 36be79b9a5..4200c11d46 100644 --- a/drivers/net/mvpp2/mrvl_ethdev.c +++ b/drivers/net/mvpp2/mrvl_ethdev.c @@ -215,7 +215,7 @@ mrvl_pp2_fixup_init(void) dummy_pool_id[i] = mrvl_reserve_bit(&used_bpools[i], PP2_BPOOL_NUM_POOLS); if (dummy_pool_id[i] < 0) { - MRVL_LOG(ERR, "Can't find free pool\n"); + MRVL_LOG(ERR, "Can't find free pool"); return -1; } @@ -227,7 +227,7 @@ mrvl_pp2_fixup_init(void) bpool_params.dummy_short_pool = 1; err = pp2_bpool_init(&bpool_params, &dummy_pool[i]); if (err != 0 || !dummy_pool[i]) { - MRVL_LOG(ERR, "BPool init failed!\n"); + MRVL_LOG(ERR, "BPool init failed!"); used_bpools[i] &= ~(1 << dummy_pool_id[i]); return -1; } @@ -491,7 +491,7 @@ mrvl_dev_configure(struct rte_eth_dev *dev) } if (dev->data->dev_conf.rxmode.mtu > priv->max_mtu) { - MRVL_LOG(ERR, "MTU %u is larger than max_mtu %u\n", + MRVL_LOG(ERR, "MTU %u is larger than max_mtu %u", dev->data->dev_conf.rxmode.mtu, priv->max_mtu); return -EINVAL; @@ -770,7 +770,7 @@ static int mrvl_populate_vlan_table(struct rte_eth_dev *dev, int on) vbit >>= 1; ret = mrvl_vlan_filter_set(dev, vlan, on); if (ret) { - MRVL_LOG(ERR, "Failed to setup VLAN filter\n"); + MRVL_LOG(ERR, "Failed to setup VLAN filter"); return ret; } } @@ -1875,7 +1875,7 @@ static int mrvl_vlan_offload_set(struct rte_eth_dev *dev, int mask) int ret; if (mask & RTE_ETH_VLAN_STRIP_MASK) { - MRVL_LOG(ERR, "VLAN stripping is not supported\n"); + MRVL_LOG(ERR, "VLAN stripping is not supported"); return -ENOTSUP; } @@ -1890,7 +1890,7 @@ static int mrvl_vlan_offload_set(struct rte_eth_dev *dev, int mask) } if (mask & RTE_ETH_VLAN_EXTEND_MASK) { - MRVL_LOG(ERR, "Extend VLAN not supported\n"); + MRVL_LOG(ERR, "Extend VLAN not supported"); return -ENOTSUP; } @@ -1941,7 +1941,7 @@ mrvl_fill_bpool(struct mrvl_rxq *rxq, int num) != cookie_addr_high) { MRVL_LOG(ERR, "mbuf virtual addr high is out of range " - "0x%x instead of 0x%x\n", + "0x%x instead of 0x%x", (uint32_t)((uint64_t)mbufs[i] >> 32), (uint32_t)(cookie_addr_high >> 32)); goto out; diff --git a/drivers/net/mvpp2/mrvl_qos.c b/drivers/net/mvpp2/mrvl_qos.c index f43217bc58..e298da2724 100644 --- a/drivers/net/mvpp2/mrvl_qos.c +++ b/drivers/net/mvpp2/mrvl_qos.c @@ -500,7 +500,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, entry = rte_cfgfile_get_entry(file, sec_name, MRVL_TOK_PARSER_UDF_PROTO); if (!entry) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set", udf, MRVL_TOK_PARSER_UDF_PROTO); return -1; } @@ -509,7 +509,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, entry_field = rte_cfgfile_get_entry(file, sec_name, MRVL_TOK_PARSER_UDF_FIELD); if (!entry_field) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set", udf, MRVL_TOK_PARSER_UDF_FIELD); return -1; } @@ -523,7 +523,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, field_size = 2; } else { MRVL_LOG(ERR, "UDF[%d]: mismatch between '%s' proto " - "and '%s' field\n", udf, + "and '%s' field", udf, MRVL_TOK_PARSER_UDF_PROTO_ETH, entry_field); return -1; @@ -537,45 +537,45 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, field_size = 2; } else { MRVL_LOG(ERR, "UDF[%d]: mismatch between '%s' proto " - "and '%s' field\n", udf, + "and '%s' field", udf, MRVL_TOK_PARSER_UDF_PROTO_UDP, entry_field); return -1; } } else { - MRVL_LOG(ERR, "UDF[%d]: Unsupported '%s' proto\n", udf, entry); + MRVL_LOG(ERR, "UDF[%d]: Unsupported '%s' proto", udf, entry); return -1; } snprintf(malloc_name, sizeof(malloc_name), "mrvl_udf_%d_key", udf); udf_params->match_key = rte_zmalloc(malloc_name, field_size, 0); if (udf_params->match_key == NULL) { - MRVL_LOG(ERR, "Cannot allocate udf %d key\n", udf); + MRVL_LOG(ERR, "Cannot allocate udf %d key", udf); return -1; } snprintf(malloc_name, sizeof(malloc_name), "mrvl_udf_%d_mask", udf); udf_params->match_mask = rte_zmalloc(malloc_name, field_size, 0); if (udf_params->match_mask == NULL) { - MRVL_LOG(ERR, "Cannot allocate udf %d mask\n", udf); + MRVL_LOG(ERR, "Cannot allocate udf %d mask", udf); return -1; } /* Read 'key' field */ entry = rte_cfgfile_get_entry(file, sec_name, MRVL_TOK_PARSER_UDF_KEY); if (!entry) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set", udf, MRVL_TOK_PARSER_UDF_KEY); return -1; } if (strncmp(entry, "0x", 2) != 0) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must start with '0x'\n", + MRVL_LOG(ERR, "UDF[%d]: '%s' field must start with '0x'", udf, MRVL_TOK_PARSER_UDF_KEY); return -EINVAL; } if (strlen(entry) != FIELD_LEN_IN_CHARS(field_size)) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field's len must be %d\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field's len must be %d", udf, MRVL_TOK_PARSER_UDF_KEY, FIELD_LEN_IN_CHARS(field_size)); return -EINVAL; @@ -588,7 +588,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, if (get_val_securely8(tmp_arr, 16, &udf_params->match_key[i]) < 0) { MRVL_LOG(ERR, "UDF[%d]: '%s' field's value is not in " - "hex format\n", udf, MRVL_TOK_PARSER_UDF_KEY); + "hex format", udf, MRVL_TOK_PARSER_UDF_KEY); return -EINVAL; } entry += 2; @@ -597,18 +597,18 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, /* Read 'mask' field */ entry = rte_cfgfile_get_entry(file, sec_name, MRVL_TOK_PARSER_UDF_MASK); if (!entry) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set", udf, MRVL_TOK_PARSER_UDF_MASK); return -1; } if (strncmp(entry, "0x", 2) != 0) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must start with '0x'\n", + MRVL_LOG(ERR, "UDF[%d]: '%s' field must start with '0x'", udf, MRVL_TOK_PARSER_UDF_MASK); return -EINVAL; } if (strlen(entry) != FIELD_LEN_IN_CHARS(field_size)) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field's len must be %d\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field's len must be %d", udf, MRVL_TOK_PARSER_UDF_MASK, FIELD_LEN_IN_CHARS(field_size)); return -EINVAL; @@ -621,7 +621,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, if (get_val_securely8(tmp_arr, 16, &udf_params->match_mask[i]) < 0) { MRVL_LOG(ERR, "UDF[%d]: '%s' field's value is not in " - "hex format\n", udf, MRVL_TOK_PARSER_UDF_MASK); + "hex format", udf, MRVL_TOK_PARSER_UDF_MASK); return -EINVAL; } entry += 2; @@ -631,7 +631,7 @@ parse_udf(struct rte_cfgfile *file, const char *sec_name, int udf, entry = rte_cfgfile_get_entry(file, sec_name, MRVL_TOK_PARSER_UDF_OFFSET); if (!entry) { - MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set\n", udf, + MRVL_LOG(ERR, "UDF[%d]: '%s' field must be set", udf, MRVL_TOK_PARSER_UDF_OFFSET); return -1; } @@ -663,7 +663,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) char sec_name[32]; if (file == NULL) { - MRVL_LOG(ERR, "Cannot load configuration %s\n", path); + MRVL_LOG(ERR, "Cannot load configuration %s", path); return -1; } @@ -672,7 +672,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) */ *cfg = rte_zmalloc("mrvl_cfg", sizeof(struct mrvl_cfg), 0); if (*cfg == NULL) { - MRVL_LOG(ERR, "Cannot allocate configuration %s\n", path); + MRVL_LOG(ERR, "Cannot allocate configuration %s", path); return -1; } @@ -681,7 +681,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) sizeof(MRVL_TOK_PARSER_UDF) - 1); if (n && n > PP2_MAX_UDFS_SUPPORTED) { - MRVL_LOG(ERR, "found %d udf sections, but only %d are supported\n", + MRVL_LOG(ERR, "found %d udf sections, but only %d are supported", n, PP2_MAX_UDFS_SUPPORTED); return -1; } @@ -693,14 +693,14 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) /* udf sections must be sequential. */ if (rte_cfgfile_num_sections(file, sec_name, strlen(sec_name)) <= 0) { - MRVL_LOG(ERR, "udf sections must be sequential (0 - %d)\n", + MRVL_LOG(ERR, "udf sections must be sequential (0 - %d)", PP2_MAX_UDFS_SUPPORTED - 1); return -1; } ret = parse_udf(file, sec_name, i, *cfg); if (ret) { - MRVL_LOG(ERR, "Error in parsing %s!\n", sec_name); + MRVL_LOG(ERR, "Error in parsing %s!", sec_name); return -1; } } @@ -763,7 +763,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) PP2_PPIO_HDR_ETH_EXT_DSA; } else { MRVL_LOG(ERR, - "Error in parsing %s value (%s)!\n", + "Error in parsing %s value (%s)!", MRVL_TOK_START_HDR, entry); return -1; } @@ -823,7 +823,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) PP2_CLS_QOS_TBL_VLAN_PRI; } else { MRVL_LOG(ERR, - "Error in parsing %s value (%s)!\n", + "Error in parsing %s value (%s)!", MRVL_TOK_MAPPING_PRIORITY, entry); return -1; } @@ -851,7 +851,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) ret = get_outq_cfg(file, n, i, *cfg); if (ret < 0) { MRVL_LOG(ERR, - "Error %d parsing port %d outq %d!\n", + "Error %d parsing port %d outq %d!", ret, n, i); return -1; } @@ -861,7 +861,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) ret = parse_tc_cfg(file, n, i, *cfg); if (ret < 0) { MRVL_LOG(ERR, - "Error %d parsing port %d tc %d!\n", + "Error %d parsing port %d tc %d!", ret, n, i); return -1; } @@ -889,7 +889,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) if (entry) { if (get_val_securely(entry, &val) < 0) { MRVL_LOG(ERR, - "Error in parsing %s value (%s)!\n", + "Error in parsing %s value (%s)!", MRVL_TOK_FWD_BAD_FRAMES, entry); return -1; } @@ -904,7 +904,7 @@ mrvl_get_cfg(const char *key __rte_unused, const char *path, void *extra_args) if (entry) { if (get_val_securely(entry, &val) < 0) { MRVL_LOG(ERR, - "Error in parsing %s value (%s)!\n", + "Error in parsing %s value (%s)!", MRVL_TOK_FILL_BPOOL_BUFFS, entry); return -1; } @@ -972,7 +972,7 @@ setup_policer(struct mrvl_priv *priv, struct pp2_cls_plcr_params *params) * * Lets use 0th then. */ - sprintf(match, "policer-%d:%d\n", priv->pp_id, 0); + sprintf(match, "policer-%d:%d", priv->pp_id, 0); params->match = match; ret = pp2_cls_plcr_init(params, &priv->default_policer);