From patchwork Sat Oct 7 06:31:06 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Mody, Rasesh" X-Patchwork-Id: 29860 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E0D191B1F5; Sat, 7 Oct 2017 08:31:57 +0200 (CEST) Received: from NAM03-DM3-obe.outbound.protection.outlook.com (mail-dm3nam03on0089.outbound.protection.outlook.com [104.47.41.89]) by dpdk.org (Postfix) with ESMTP id B268E1B19E for ; Sat, 7 Oct 2017 08:31:47 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=FHnV8oMduVlSMTgt+nI6mCw0TxX7cv3wWp2dMhJmgck=; b=VJm9GmXbll1k+ubZVMjJhl+a1w9Eu9fN1MqBWjQzqbjHdXU4H09GbGz4hm4w6cxKsRTCzsUfDYvsfzSbSFbYwwDwgTUDDWuBb5LYEM831hUbQLw+lBQT/RFnTrjmvDhjCBO8ojO13IJE1Ytng2NcLxch8AWK1EJWrzpMJYN9Y48= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Rasesh.Mody@cavium.com; Received: from cavium.com (198.186.0.2) by MWHPR0701MB3833.namprd07.prod.outlook.com (2603:10b6:301:7f::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Sat, 7 Oct 2017 06:31:46 +0000 From: Rasesh Mody To: dev@dpdk.org, ferruh.yigit@intel.com Cc: Rasesh Mody , Dept-EngDPDKDev@cavium.com Date: Fri, 6 Oct 2017 23:31:06 -0700 Message-Id: <1507357872-26475-12-git-send-email-rasesh.mody@cavium.com> X-Mailer: git-send-email 1.7.10.3 In-Reply-To: <1507357872-26475-1-git-send-email-rasesh.mody@cavium.com> References: <1507357872-26475-1-git-send-email-rasesh.mody@cavium.com> MIME-Version: 1.0 X-Originating-IP: [198.186.0.2] X-ClientProxiedBy: SN4PR0501CA0012.namprd05.prod.outlook.com (2603:10b6:803:40::25) To MWHPR0701MB3833.namprd07.prod.outlook.com (2603:10b6:301:7f::26) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0c7afd35-ea2b-4d8c-4595-08d50d4d15a1 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:MWHPR0701MB3833; X-Microsoft-Exchange-Diagnostics: 1; MWHPR0701MB3833; 3:nq+2yMwWer0fh5hzdMGDb8o2NtsdDbTapiouWbXromrZERq4ONeMEao3Z4uXrbVrKqz4F4R+OWnJyoVee7dTbEDYyHpU+9sqsYXmTUWXuhKaGMvrrj5LEu82KzPpFmBHNVrcxm7j+s/K+PMWDnAVSKcNScrqLPBcl/uiFxZM9f6Oig0x/nwGo6lK3KOvQ7KEARq7AFb2J72cnuwrsMTp4X0D+twUTp/EwkvdVl9lkPryowNlErFvn7fFZIPN5o/W; 25:6zlRjOLQWt1w2joPLpoO6Tke3W6pd3OqMRKgebMHILo+BXWtmPVtkpglzwWHMco71zWCzPyt/1RkSaTmXHn5EyOGyPYd0c39+Jpszmkv0LU8hWvbbhcqlg0ETkRzCLwn4ANdCNG5dHhOEsCInkDJeo82vjM6IfypXEtLruPB3yJpsedLHK92C7b05U7mB3bNJuE+iB9o59kjLo7bo8r51mt1rqEL5lHk1X7Q0jFQcRP35QO/yBgCNbiQn08ChVB+RjVATrSDCOBEvaAifD4JDW5eJekz6GrImmjQeh2cw2Ju1ZvcGnD4D6o726T3q/fgK0N3Quxw71Jlj37pSwEMvA==; 31:rIxy+k4Upm9w5t4XJhFD1dRSuK8jYwjE5jxh7jk5eLnRxFV0ReqrKU0zjAiJXMBvoBew4/XJPP6LYyCm/DuEt1OldYk963EWPGh5R04vv4alJp4DXV+E4CBPRA13EJIj8stD8BrYmv/EpzHl64XSDu1VpCwl8iJGoh0nqGEp0i/kSZMWCr/GXRDKDhUEmIWHK/jgo6GFxHBaBa3it92xLlfuet3EfJIAP6tHLGjB6D4= X-MS-TrafficTypeDiagnostic: MWHPR0701MB3833: X-Microsoft-Exchange-Diagnostics: 1; MWHPR0701MB3833; 20:SE7UPV8JYCZaEC0r8L9c7xsN+A6EZzRyVdil5dAw1GLsqZCQ/Y6igyFPL5gEguCKJAEZs5sQGVr1wGbLPX8GDksQyHloQQQsti0I4xXpcvToOZfO1MM7btXjrO/Af9KjCLi5cRfuz4Oev0LMbQfCZHD2FQ5h38bILvsBcdRC1Q432eiV5J2kk6Oo6DDhJBMre+fPQSCKH+g4BHI3Qk2waN3wXvdMvLnkXm1gWUsSqj3aYy/KW0eQ98tuQF0nYJfU7ZRGM/7NmUAcyrNQUGIEeqjNaLPKJjhOp6FTBKjs8HsRgLVPT0fBuUVVJ/SjX0csUiK0YJXt1xCALu2gbg/b0t2J/ugsX5W5AZfDuMHtkzKfzWlkQJuZRK8FoxiXKKKpXFvxbfbYQLr8fgBeDxtUJz0HvqqUxiCRtiCzfPIvCSl/ArUEY3B3ziW8voa+O3uuRxcIXXidQIVG6CI1V0vjEK2qbrYgOnqrNqXB5N71b03QmEzNoT/u3vPkXqtxqr1P; 4:FRAJEY8jtxgCYa7fXc03O/4R6ZlSi6uQHrZ/swGs6Zs8VDTQvhaaSqVMRni6BFBuVa1NSADWUAgrgSwoy3+8TJelKb1ii4IME+8uCoITHzmBWMolwkRkqR7OnCDnFV9xT9PIXkXGh2yPFELPPVSZhNYMxSnWsU5p06tU8d8Tch84hHFsXz11xpnKdkVAFPspIKe+IvKldfkzPLIsC1NxXFSQl8ulKOxah2dqFYidV1YOoZXT7k2TCarWZYAtk+wT X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(10201501046)(3002001)(100000703101)(100105400095)(93006095)(93001095)(6041248)(20161123560025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123558100)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:MWHPR0701MB3833; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:MWHPR0701MB3833; X-Forefront-PRVS: 045315E1EE X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(346002)(376002)(199003)(189002)(2906002)(6116002)(8936002)(3846002)(48376002)(68736007)(478600001)(101416001)(105586002)(50226002)(106356001)(72206003)(5660300001)(50466002)(33646002)(25786009)(4720700003)(21086003)(2950100002)(6666003)(69596002)(36756003)(76176999)(50986999)(81156014)(53936002)(107886003)(4326008)(55016002)(47776003)(66066001)(5003940100001)(81166006)(189998001)(8676002)(305945005)(97736004)(16586007)(316002)(7736002)(16526018)(86362001); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR0701MB3833; H:cavium.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; MWHPR0701MB3833; 23:WNhuTfPjaQu4K/LbFMWrAtbfTpZ2HSLPxAUtjhk?= 6+Rfmk131Rwug27koCDo47qp0/BCu2pu2MU4i6qY6zXlbHifxBPeFQkmRSGvsWy7FnTFBULL8DK7wTGqbLA7uh5Yb60EkIir/lW3fcMU0Q5yTRrHqHyRiUJw+7+tU7Nny96FD+nPO0ikBQfhJ5N5LKz4ydzEi4si7rRtgPkOii074EXGamkQFFnxSQ3+TrerdzNK5aa190kiI+SZLA3++8XV4iS0iLSUw4q4PG0ziomqP1R2y+55fIptniizW3K0BS/AvRgVp7IxfXx/HkJG2ZZUP9F+N5eGep8YmVzF/VXahbRxwal3/VUwb+GVHO+wsczusuPqrjqdHNzU7n/Bae+C6b2L7xkPkjdmnJSiVP3h2awiUWkw7rYmC7oy1xIeFN1cTO85RqLaoNo/q2WJNQCbm34+QZtagAx4fK2eqPXq1/wGQwD/9Pbf/m2ulDirva3grIkq0J0W4Uu9Arihu3Uled5guw1MUgjG6R5odcPT9itJ2hQ/njBs1SjSlcMrXW74BEGrZbwib9CObN/sXJHYL9OJZFw+Z+hiIPIU4qJnaZ7BzslkwBdSqznawKYrpxJkdqbj43ofky1hDQUKpHqM9k1C5jQjYBWZHTy/ZnqnG919vr743fnvaY4Rfc1taDnckCKgxv+5k3Dvk0IRK8rNvxv/TH2Oelofs21pCM/ZVmNSGE4e/WfQbduE0IDsvys2DwsfvhXOXfdGbo2J7urCu2xdNLnWkemjl5pP83v7cdY9TBvY1egOIdOtzRjzhLqKJOVr6BUNIIMEEj9MOxltC6mIJD7RqVhlPWgdB782Q1+p1d3WEPENDiCuId3R979VfYGnNe7Y3EiI25+e5yI+3HhJ/zrwfdEB5nbCZYRR4sNL6wiuAbDrljiaTsHY9zuWbz98DE2r/+ohOaXN7TyfMaIe0W+Hq2g/m0SaqHN2UxAx4qUxMi+HhAD/ffFdTWmLDT+VcmV/SCE0yod5c3y3jbcIOObmSs3+5ULNtNzoLEyFVg+Vkw66c3iPkW6HAu4/AeVkupB6eu9OQWQiJNCaCl0+M4DjNkaRaU5cMrywIeZYBo5ymAXGwzXO2uDoluD4= X-Microsoft-Exchange-Diagnostics: 1; MWHPR0701MB3833; 6:dblxPqJoAGs4gwiU1KGB5q+iURsYqzzTSvNdGqXFn8YTLnj+UoM/s2PeWvBoSUq0TE979TJvelFFCcrT/Zp57VDIXd6nSI9rO/rO0G2dKz2UORoc1lcIglbF4TmwVOfjkJTRSkJY05pyNmkkB7Hd3BXJvH8q0ZSCivcGqpBFgOMBl/HBeRuTzeniku54NYpiWQhg+95qaFVj1w09dMSbPQije0N0hP0/vcRfuNu2tIgcGsnEaLIhOgz/P8k0r2N1ttSMLUwXB0G7+IbSJDtssPDvtd3boLaMDa1ZOlMROPj2bgSPDzzydxcP4ySp6lIQDkiro8rU+7nRkP04lhuoJQ==; 5:i6d9g1JzsTUTG4FHQ9UdYqG/7e6WQh6azcqHI7iKvkj/te4T5dzUWZAe9MQ5RmKyXd/+xsBET5uSA19l7brw61d1cBzzsrMs2BFzZ63YmKQESCAdJT4z5LBTuAU9+nE+1dGScRayjm7PP4VTLUAoGQ==; 24:e4kvW9Tz5h29s/8SEbQlFCpNZ9Oqe5V/VZiRReNlsptZX1RvqYEi5VYl70rco7/lUim8fPHG61kS2M7g8EW1qECx+dy9kc4MgJPLJrq4L54=; 7:k3Af8s3LO+gm81sLscQGicJHKvEpsaVCXgcDTfxuPGmsfjlsfS6vN6v6KIGAaFu9mLH+c5NE5v5pi7NEM52tJ2t+/UHO7DnwBa3RfxPMsqCnarehFv5nXP7rlm6ov6VX+4JaTGMTLFzEzfTfLWDwWSO60niNd8BFSHFqG+eiGR+pk++eP96OJGJ80d1hUVPxDkTXfpGOBUpwiDeTsFYUwTJ7C60DwrhNF5k/0xD5qHg= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Oct 2017 06:31:46.4014 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR0701MB3833 Subject: [dpdk-dev] [PATCH 11/17] net/qede/base: code cleanup X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" - Remove some dead definitions, function declarations and unused variables - Remove an obsolete workaround from ecore_int_igu_enable() - Remove set variables that are not used - Remove needless check in ecore_init_wfq_param() when configuring minimum vport BW. We already check whether total for all vports is greater than the PF's, so no need to check independently the current requested configuration as well. Signed-off-by: Rasesh Mody --- drivers/net/qede/base/ecore_dcbx.c | 9 --------- drivers/net/qede/base/ecore_dcbx_api.h | 1 - drivers/net/qede/base/ecore_dev.c | 7 ------- drivers/net/qede/base/ecore_hw.h | 29 ----------------------------- drivers/net/qede/base/ecore_int.c | 9 --------- drivers/net/qede/base/ecore_mcp.c | 26 ++++++++++++++++++-------- 6 files changed, 18 insertions(+), 63 deletions(-) diff --git a/drivers/net/qede/base/ecore_dcbx.c b/drivers/net/qede/base/ecore_dcbx.c index edb711e..632297a 100644 --- a/drivers/net/qede/base/ecore_dcbx.c +++ b/drivers/net/qede/base/ecore_dcbx.c @@ -163,7 +163,6 @@ u8 ecore_dcbx_get_dscp_value(struct ecore_hwfn *p_hwfn, u8 pri) { enum ecore_pci_personality personality; enum dcbx_protocol_type id; - const char *name; /* @DPDK */ int i; for (i = 0; i < OSAL_ARRAY_SIZE(ecore_dcbx_app_update); i++) { @@ -173,7 +172,6 @@ u8 ecore_dcbx_get_dscp_value(struct ecore_hwfn *p_hwfn, u8 pri) continue; personality = ecore_dcbx_app_update[i].personality; - name = ecore_dcbx_app_update[i].name; ecore_dcbx_set_params(p_data, p_hwfn, enable, prio, tc, type, personality); @@ -882,8 +880,6 @@ enum _ecore_status_t rc = ecore_dcbx_process_mib_info(p_hwfn); if (!rc) { - bool enabled; - /* reconfigure tcs of QM queues according * to negotiation results */ @@ -891,11 +887,6 @@ enum _ecore_status_t /* update storm FW with negotiation results */ ecore_sp_pf_update_dcbx(p_hwfn); - - /* set eagle enigne 1 flow control workaround - * according to negotiation results - */ - enabled = p_hwfn->p_dcbx_info->results.dcbx_enabled; } } diff --git a/drivers/net/qede/base/ecore_dcbx_api.h b/drivers/net/qede/base/ecore_dcbx_api.h index 249fb53..9ff4df4 100644 --- a/drivers/net/qede/base/ecore_dcbx_api.h +++ b/drivers/net/qede/base/ecore_dcbx_api.h @@ -102,7 +102,6 @@ struct ecore_dcbx_params { bool ets_willing; bool ets_enabled; bool ets_cbs; - bool valid; /* Indicate validity of params */ u8 ets_pri_tc_tbl[ECORE_MAX_PFC_PRIORITIES]; u8 ets_tc_bw_tbl[ECORE_MAX_PFC_PRIORITIES]; u8 ets_tc_tsa_tbl[ECORE_MAX_PFC_PRIORITIES]; diff --git a/drivers/net/qede/base/ecore_dev.c b/drivers/net/qede/base/ecore_dev.c index 412af07..75fcfae 100644 --- a/drivers/net/qede/base/ecore_dev.c +++ b/drivers/net/qede/base/ecore_dev.c @@ -5239,13 +5239,6 @@ static enum _ecore_status_t ecore_init_wfq_param(struct ecore_hwfn *p_hwfn, non_requested_count = num_vports - req_count; /* validate possible error cases */ - if (req_rate > min_pf_rate) { - DP_VERBOSE(p_hwfn, ECORE_MSG_LINK, - "Vport [%d] - Requested rate[%d Mbps] is greater than configured PF min rate[%d Mbps]\n", - vport_id, req_rate, min_pf_rate); - return ECORE_INVAL; - } - if (req_rate < min_pf_rate / ECORE_WFQ_UNIT) { DP_VERBOSE(p_hwfn, ECORE_MSG_LINK, "Vport [%d] - Requested rate[%d Mbps] is less than one percent of configured PF min rate[%d Mbps]\n", diff --git a/drivers/net/qede/base/ecore_hw.h b/drivers/net/qede/base/ecore_hw.h index 392351a..0b9814f 100644 --- a/drivers/net/qede/base/ecore_hw.h +++ b/drivers/net/qede/base/ecore_hw.h @@ -249,35 +249,6 @@ void ecore_port_unpretend(struct ecore_hwfn *p_hwfn, */ void ecore_dmae_info_free(struct ecore_hwfn *p_hwfn); -union ecore_qm_pq_params { - struct { - u8 q_idx; - } iscsi; - - struct { - u8 tc; - } core; - - struct { - u8 is_vf; - u8 vf_id; - u8 tc; - } eth; - - struct { - u8 dcqcn; - u8 qpid; /* roce relative */ - } roce; - - struct { - u8 qidx; - } iwarp; -}; - -u16 ecore_get_qm_pq(struct ecore_hwfn *p_hwfn, - enum protocol_type proto, - union ecore_qm_pq_params *params); - enum _ecore_status_t ecore_init_fw_data(struct ecore_dev *p_dev, const u8 *fw_data); diff --git a/drivers/net/qede/base/ecore_int.c b/drivers/net/qede/base/ecore_int.c index 56ad1e7..f9ad513 100644 --- a/drivers/net/qede/base/ecore_int.c +++ b/drivers/net/qede/base/ecore_int.c @@ -1926,15 +1926,6 @@ enum _ecore_status_t enum ecore_int_mode int_mode) { enum _ecore_status_t rc = ECORE_SUCCESS; - u32 tmp; - - /* @@@tmp - Starting with MFW 8.2.1.0 we've started hitting AVS stop - * attentions. Since we're waiting for BRCM answer regarding this - * attention, in the meanwhile we simply mask it. - */ - tmp = ecore_rd(p_hwfn, p_ptt, MISC_REG_AEU_ENABLE4_IGU_OUT_0); - tmp &= ~0x800; - ecore_wr(p_hwfn, p_ptt, MISC_REG_AEU_ENABLE4_IGU_OUT_0, tmp); ecore_int_igu_enable_attn(p_hwfn, p_ptt); diff --git a/drivers/net/qede/base/ecore_mcp.c b/drivers/net/qede/base/ecore_mcp.c index 028fbb3..85cd271 100644 --- a/drivers/net/qede/base/ecore_mcp.c +++ b/drivers/net/qede/base/ecore_mcp.c @@ -2961,11 +2961,16 @@ enum _ecore_status_t ecore_mcp_phy_sfp_read(struct ecore_hwfn *p_hwfn, DRV_MSG_CODE_TRANSCEIVER_READ, nvm_offset, &resp, ¶m, &buf_size, (u32 *)(p_buf + offset)); - if ((resp & FW_MSG_CODE_MASK) == - FW_MSG_CODE_TRANSCEIVER_NOT_PRESENT) { + if (rc != ECORE_SUCCESS) { + DP_NOTICE(p_hwfn, false, + "Failed to send a transceiver read command to the MFW. rc = %d.\n", + rc); + return rc; + } + + if (resp == FW_MSG_CODE_TRANSCEIVER_NOT_PRESENT) return ECORE_NODEV; - } else if ((resp & FW_MSG_CODE_MASK) != - FW_MSG_CODE_TRANSCEIVER_DIAG_OK) + else if (resp != FW_MSG_CODE_TRANSCEIVER_DIAG_OK) return ECORE_UNKNOWN_ERROR; offset += buf_size; @@ -2999,11 +3004,16 @@ enum _ecore_status_t ecore_mcp_phy_sfp_write(struct ecore_hwfn *p_hwfn, DRV_MSG_CODE_TRANSCEIVER_WRITE, nvm_offset, &resp, ¶m, buf_size, (u32 *)&p_buf[buf_idx]); - if ((resp & FW_MSG_CODE_MASK) == - FW_MSG_CODE_TRANSCEIVER_NOT_PRESENT) { + if (rc != ECORE_SUCCESS) { + DP_NOTICE(p_hwfn, false, + "Failed to send a transceiver write command to the MFW. rc = %d.\n", + rc); + return rc; + } + + if (resp == FW_MSG_CODE_TRANSCEIVER_NOT_PRESENT) return ECORE_NODEV; - } else if ((resp & FW_MSG_CODE_MASK) != - FW_MSG_CODE_TRANSCEIVER_DIAG_OK) + else if (resp != FW_MSG_CODE_TRANSCEIVER_DIAG_OK) return ECORE_UNKNOWN_ERROR; buf_idx += buf_size;