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 AF6CA42600; Tue, 19 Sep 2023 11:57:51 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8F0384161A; Tue, 19 Sep 2023 11:56:08 +0200 (CEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2125.outbound.protection.outlook.com [40.107.220.125]) by mails.dpdk.org (Postfix) with ESMTP id F14A8409FA for ; Tue, 19 Sep 2023 11:56:02 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KG5fqAAXw5KNI97UO5FibyM6AMrQfV/l1flVL403rY+xK/LkpsILzga2+8PdTifBGjVF+tWHMde6Vxnrc6CDYxfYkl+yl/eRsyaD+qxw5x8iVTf5OMCsJqPr30bKRoyWTEXeZNCny9BAYlHhwncxY9xMsl3RxiD6z0SsfDrjVqtHjwOJS1xshj3Ax2cS4reSOTOBqnoSf3/F3xMJ34i9RZx7BkD5wrnjCko2/QMcs8O+t3hMgL2QQVwLo/MqXIrDp8uL96Aoa5hn7vW+Oe4d7MW2MIwP/CjpQERNpHNvHWFVEKbGE6IV/TFlYjBYcTfhSlqfEjroerceW2RlD9sl0w== 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=dsPa2RY9T6W+NFYbDSUKnX5RTp0rHV2t6z2yYGsh1vs=; b=ca91A/H1d99/+aTzbIiiF6NNMEzZYctBJdM4hF2WofzGcIAvloxiTUzW9XscwxhSouh34erLfXNom0o5Va5FCj+5WeRMzbRbxcWF+4AnLTTPCJdr5VyzjALwyXPvcwTdqFFXdd2wkseiuDGdfy8mriQXMrAe3XqxpTTQIYsC51buvM1eVjMy2xogvojDzVe7pUseDY2zULc5ldPyihSAD2G6TL9+3T26YLZE32TazsCpO5PDEKvzxdaKzomuJhZ0PZREo7WDLNT+dUcxrUVt6rax9UAm4JPAYvkNbFdCMkHQHR37vP+WeC8fySoRCfJD27jjju3SGOTTdvS4cB/zvA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dsPa2RY9T6W+NFYbDSUKnX5RTp0rHV2t6z2yYGsh1vs=; b=fHR6YbJPFL1thYmhQQ9V+7VTvkqvTecrtbPhZIgh+6dMOCMN7LA03r+01pRkz5bBq+9MEPeZdaDSwp2kjpFR6l1cTJ9X5Rv0W8W7kzUmQhJToy6qkllAlJYuw1TGT9QRmT2SXQl7+0r8mSaMBzIdTr1kuvpfJqXPJu33F5H6FFI= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by PH7PR13MB5868.namprd13.prod.outlook.com (2603:10b6:510:15a::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6792.26; Tue, 19 Sep 2023 09:55:26 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7a1c:2887:348a:84bd]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7a1c:2887:348a:84bd%6]) with mapi id 15.20.6792.026; Tue, 19 Sep 2023 09:55:26 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, Chaoyong He Subject: [PATCH v5 03/26] net/nfp: unify the type of integer variable Date: Tue, 19 Sep 2023 17:54:17 +0800 Message-Id: <20230919095440.45445-4-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230919095440.45445-1-chaoyong.he@corigine.com> References: <20230918024612.1600536-1-chaoyong.he@corigine.com> <20230919095440.45445-1-chaoyong.he@corigine.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: BY5PR04CA0014.namprd04.prod.outlook.com (2603:10b6:a03:1d0::24) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH7PR13MB5868:EE_ X-MS-Office365-Filtering-Correlation-Id: ff6d5660-7836-4bcc-bff4-08dbb8f68c62 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ZbGCsjznj3m9hL9Q6Bng0B5mtPmM8Au9Kd02hkK66CUNcaQVYUlxsd0mVEYLYXJBrAkik3spATwsUBePf641xjZFnxS/Fm5uVbYQTLXd+9Vk09TS2grvFSKfW01ZYoFzS7YLink8l89E7ynQ77oxkVPDI0fSauQEpQ8EnPPgXt1Az7irJY61PXMUjG5ChshyaENhr8Eq8JERcXrAhM2oPD3yWR/83EVfTsgnffVHEK9+TICfB4CMIGdJTBNxcihdmMwaS/791i/rnC2YT5l3j/RAEtJaRGdnQyQNflihi4KajBBQUbeMbz76VImIYDuYNU6G5GqP6BVKjUDFkWZuTbGIPIk9a5RcCWSFMCqnEa3NdD0PUAtrDuujo2my27jYMRin/sVMeIComzwbVsqNJNocDo7YgoDlq5dNI8yvjzCjBh78KfwEnGiSUdmrHCY/mUVI4gycdXo+0bilx25EOVSqOGWryZ5i5mXTJ1R60g7XRJ/XCUV1C+S5eciuIZ0lFZaTXm992RFHZsaoMgE3aeX1krRRqpZGiOEFkwjHKsO/XJ2fntb4XFWT03doHNhSIbiDZUVLBb1fqJxWyMe7t3wY5Xhpf8DhXSBB2Wl0bTy8qWde4qYmgcgfvWCB1zwmQFVh++dngZQ1/UmoMckuwrdohjl5NFMWgjeEX1Vu9xsv4jzv6dYPNFRGHbw1m3EO X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(136003)(39830400003)(366004)(346002)(376002)(396003)(1800799009)(186009)(451199024)(26005)(8936002)(2616005)(1076003)(8676002)(4326008)(107886003)(83380400001)(2906002)(30864003)(36756003)(44832011)(86362001)(52116002)(6506007)(6486002)(478600001)(5660300002)(6666004)(6916009)(316002)(6512007)(38100700002)(66946007)(66476007)(41300700001)(66556008)(38350700002)(309714004); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?3tuvPWFYCsKdqucHpDOmRN80V3hOXjlTyLKiKHkMmHjt3L3X5DRROJ0lZl3k?= =?us-ascii?Q?wPOWHoo8UGAoHQ9gyTkkTWRM8RDdjS4OrkE2tALeNs1SduLEkFJ4mOJPB0uK?= =?us-ascii?Q?Z+PSHswBSanlW6mv7fASUD+n4KkRJGCBhtry+cVKcTSmIDLJQ3bqPEUaUIBj?= =?us-ascii?Q?ri2pHomk6nfgYOQpMt3wduOLM7LIkw/X006qzXD83XN8bZnzYqLflSiZafXY?= =?us-ascii?Q?Jn+3zd9z+HbooLTChAWf7S6bcMh/jM4T7/xJ/5b5ATFGFdzboCApCNtgGrIN?= =?us-ascii?Q?h8H6tRrHAjn5y83L9AcWb+S2b/Whz7Mjjoin5wslv46EDMI9EhoPIBhJDJEn?= =?us-ascii?Q?WlXMXXBEF1hAIQW4PY6+tvNVdKAgHsCwwkXVVgPVhjJvDQOv4bVjn8Jl1lV2?= =?us-ascii?Q?v3PHSE0GxKxWPuXfaS91W1269eA3N5GO4AA52MADP2DdkY6GYBl0IRsGkjyo?= =?us-ascii?Q?nUAjAVZvMuF23V/V0nE0a6CUli5khVNDkx76zVZ8iYNMJHV3CQ3Zw+tELZS4?= =?us-ascii?Q?d9p5N/uxqqxfGp1LAwgP7y2QG9iOgjTT2PhTrRlzhvz4d1qDWri8gD94568I?= =?us-ascii?Q?Al39TfGbwSxbdo6SqfgpfMwHLVMsBPiGhTeY8A/+WXjpKBjH34CZHjkW27YH?= =?us-ascii?Q?WFTth6z/OcAGvgI2l4krKdhhsQ743FD9WutFogFb0h9xvYYs3pLu8I/6zyCc?= =?us-ascii?Q?cEa9YOlAcBtgekWlnihm8dHnoaRyLIx3c2k6HlkuBvVr43qC6M6qCz1rqdOF?= =?us-ascii?Q?Ot8qsV8DXrZFHWce4yodZW73rTx6pUuTa6jLALGxA/Hh9UQBfSURuvF/7L8o?= =?us-ascii?Q?Wd4jF2Cf0N7FeI94RYU8P9PH8NHCGsWF71s7JIETqDfQ6U37fuyw8ZdCUMMl?= =?us-ascii?Q?npHRtsfh3Kq0lTemel79ntihLkzGisVdEVDrES6tD3rgXiXVNf57ds2RYonU?= =?us-ascii?Q?Ompb2XBjIjsAjyVhhLQzQdhpk2ilIzQ9nvJjVexqW+wD/5EVTb54VTd+c3p4?= =?us-ascii?Q?zkKm3qYxBm7MBJiYVSQId8bbSUI0n3Xv0No12X6DrxH9SIHqSQCA9ntXVFuS?= =?us-ascii?Q?4+FI+4YnpQL/eyI0qR7cYhx/jUbay6dBL2pum0IOVHUbUAGASlu+yCtSZNOZ?= =?us-ascii?Q?R2Yjm3PgEUblzeAN7Hkj0/SKnzUgghH1S4c8bn5AhxVu4zGrp3CP/nL6ey1n?= =?us-ascii?Q?SmUzEIPOpm8h/czP/2G5VU2YG+ZAUkNAoKNBJ02OoS7PiliTebA2+0PsM0Ma?= =?us-ascii?Q?E1c0ytKna8vgFeee4drrGiknSwuMMcrDD5G3bv9dM5yYhampgBwm5ciqoMAC?= =?us-ascii?Q?w5IQkLbflcqIhOUg/S+ZyN8A9r88/xS8cRjNf3JBMGyf3o6F+tHyov2qgx2B?= =?us-ascii?Q?2lIkyYw/Q27TyoT2yG8HB+1aUd/owL+hgZKgTB/qT3/x/UyFfKUcmZGsT2BY?= =?us-ascii?Q?mCs5Lb9j6AkP61VxWlKWwUTLsGac7sS9ip/6hbjhTf4tVVE0Zely74WMceAO?= =?us-ascii?Q?qQMmzHvI46ApBwGFH8jQLIC847nFB4rIN4ZtFqk3I0fIYt+jECVs7m413dTl?= =?us-ascii?Q?BFAdnyK1RrQutmGYwgxADz+fFczXLYx/SwKlUqUBVtfim55K7n03/vPkKg65?= =?us-ascii?Q?Vg=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: ff6d5660-7836-4bcc-bff4-08dbb8f68c62 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Sep 2023 09:55:26.4455 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: DwxywQLsHOSwCTp4ka64xMlf2vjQ5Uz7kicsXBf+hN4z47fCmMxBvevdAroW9NlCuJ/irSyV19iQMTFXyv6gNNU/UjxKMkNdYXxfOtXmaCs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR13MB5868 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 Unify the type of integer variable to the DPDK prefer style. Also change the return type of 'nfp_eth_speed2rate()' to the corresponding enum, which make it more readable. Signed-off-by: Chaoyong He --- drivers/net/nfp/nfpcore/nfp_cpp.h | 53 ++++++++++--------- drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c | 20 ++++---- drivers/net/nfp/nfpcore/nfp_cppcore.c | 40 +++++++-------- drivers/net/nfp/nfpcore/nfp_crc.c | 2 +- drivers/net/nfp/nfpcore/nfp_mutex.c | 15 +++--- drivers/net/nfp/nfpcore/nfp_nffw.c | 6 +-- drivers/net/nfp/nfpcore/nfp_nsp.c | 28 +++++----- drivers/net/nfp/nfpcore/nfp_nsp.h | 59 +++++++++++----------- drivers/net/nfp/nfpcore/nfp_nsp_cmds.c | 2 +- drivers/net/nfp/nfpcore/nfp_nsp_eth.c | 52 ++++++++++--------- drivers/net/nfp/nfpcore/nfp_resource.c | 2 +- drivers/net/nfp/nfpcore/nfp_rtsym.c | 2 +- drivers/net/nfp/nfpcore/nfp_rtsym.h | 2 +- 13 files changed, 142 insertions(+), 141 deletions(-) diff --git a/drivers/net/nfp/nfpcore/nfp_cpp.h b/drivers/net/nfp/nfpcore/nfp_cpp.h index 54bef3cb6b..782272a3f4 100644 --- a/drivers/net/nfp/nfpcore/nfp_cpp.h +++ b/drivers/net/nfp/nfpcore/nfp_cpp.h @@ -71,8 +71,8 @@ struct nfp_cpp_operations { */ int (*area_init)(struct nfp_cpp_area *area, uint32_t dest, - unsigned long long address, - unsigned long size); + uint64_t address, + size_t size); /* * Clean up a NFP CPP area before it is freed * NOTE: This is _not_ serialized @@ -103,16 +103,16 @@ struct nfp_cpp_operations { */ int (*area_read)(struct nfp_cpp_area *area, void *kernel_vaddr, - unsigned long offset, - unsigned int length); + uint32_t offset, + size_t length); /* * Perform a write to a NFP CPP area * Serialized */ int (*area_write)(struct nfp_cpp_area *area, const void *kernel_vaddr, - unsigned long offset, - unsigned int length); + uint32_t offset, + size_t length); }; /* @@ -398,7 +398,7 @@ int nfp_cpp_serial(struct nfp_cpp *cpp, const uint8_t **serial); * @return NFP CPP handle, or NULL on failure. */ struct nfp_cpp_area *nfp_cpp_area_alloc(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, unsigned long size); + uint64_t address, size_t size); /* * Allocate a NFP CPP area handle, as an offset into a CPP ID, by a named owner @@ -411,8 +411,8 @@ struct nfp_cpp_area *nfp_cpp_area_alloc(struct nfp_cpp *cpp, uint32_t cpp_id, * @return NFP CPP handle, or NULL on failure. */ struct nfp_cpp_area *nfp_cpp_area_alloc_with_name(struct nfp_cpp *cpp, - uint32_t cpp_id, const char *name, unsigned long long address, - unsigned long size); + uint32_t cpp_id, const char *name, uint64_t address, + uint32_t size); /* * Free an allocated NFP CPP area handle @@ -446,7 +446,7 @@ void nfp_cpp_area_release(struct nfp_cpp_area *area); * @return NFP CPP handle, or NULL on failure. */ struct nfp_cpp_area *nfp_cpp_area_alloc_acquire(struct nfp_cpp *cpp, - uint32_t cpp_id, unsigned long long address, unsigned long size); + uint32_t cpp_id, uint64_t address, size_t size); /* * Release the resources, then free the NFP CPP area handle @@ -455,7 +455,8 @@ struct nfp_cpp_area *nfp_cpp_area_alloc_acquire(struct nfp_cpp *cpp, void nfp_cpp_area_release_free(struct nfp_cpp_area *area); uint8_t *nfp_cpp_map_area(struct nfp_cpp *cpp, uint32_t cpp_id, - uint64_t addr, unsigned long size, struct nfp_cpp_area **area); + uint64_t addr, uint32_t size, struct nfp_cpp_area **area); + /* * Return an IO pointer to the beginning of the NFP CPP area handle. The area * must be acquired with 'nfp_cpp_area_acquire()' before calling this operation. @@ -478,7 +479,7 @@ void *nfp_cpp_area_mapped(struct nfp_cpp_area *area); * @return bytes read on success, negative value on failure. * */ -int nfp_cpp_area_read(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_read(struct nfp_cpp_area *area, uint32_t offset, void *buffer, size_t length); /* @@ -492,7 +493,7 @@ int nfp_cpp_area_read(struct nfp_cpp_area *area, unsigned long offset, * * @return bytes written on success, negative value on failure. */ -int nfp_cpp_area_write(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_write(struct nfp_cpp_area *area, uint32_t offset, const void *buffer, size_t length); /* @@ -547,7 +548,7 @@ const char *nfp_cpp_area_name(struct nfp_cpp_area *cpp_area); * @return bytes read on success, -1 on failure. */ int nfp_cpp_read(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, void *kernel_vaddr, size_t length); + uint64_t address, void *kernel_vaddr, size_t length); /* * Write a block of data to a NFP CPP ID @@ -561,9 +562,7 @@ int nfp_cpp_read(struct nfp_cpp *cpp, uint32_t cpp_id, * @return bytes written on success, -1 on failure. */ int nfp_cpp_write(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, const void *kernel_vaddr, - size_t length); - + uint64_t address, const void *kernel_vaddr, size_t length); /* @@ -593,7 +592,7 @@ int nfp_cpp_area_fill(struct nfp_cpp_area *area, unsigned long offset, * * @return 0 on success, or -1 on error. */ -int nfp_cpp_area_readl(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_readl(struct nfp_cpp_area *area, uint32_t offset, uint32_t *value); /* @@ -610,7 +609,7 @@ int nfp_cpp_area_readl(struct nfp_cpp_area *area, unsigned long offset, * * @return 0 on success, or -1 on error. */ -int nfp_cpp_area_writel(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_writel(struct nfp_cpp_area *area, uint32_t offset, uint32_t value); /* @@ -627,7 +626,7 @@ int nfp_cpp_area_writel(struct nfp_cpp_area *area, unsigned long offset, * * @return 0 on success, or -1 on error. */ -int nfp_cpp_area_readq(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_readq(struct nfp_cpp_area *area, uint32_t offset, uint64_t *value); /* @@ -644,7 +643,7 @@ int nfp_cpp_area_readq(struct nfp_cpp_area *area, unsigned long offset, * * @return 0 on success, or -1 on error. */ -int nfp_cpp_area_writeq(struct nfp_cpp_area *area, unsigned long offset, +int nfp_cpp_area_writeq(struct nfp_cpp_area *area, uint32_t offset, uint64_t value); /* @@ -707,7 +706,7 @@ int nfp_xpb_waitlm(struct nfp_cpp *cpp, uint32_t xpb_tgt, uint32_t mask, * @return 0 on success, or -1 on failure. */ int nfp_cpp_readl(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, uint32_t *value); + uint64_t address, uint32_t *value); /* * Write a 32-bit value to a NFP CPP ID @@ -721,7 +720,7 @@ int nfp_cpp_readl(struct nfp_cpp *cpp, uint32_t cpp_id, * */ int nfp_cpp_writel(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, uint32_t value); + uint64_t address, uint32_t value); /* * Read a 64-bit work from a NFP CPP ID @@ -734,7 +733,7 @@ int nfp_cpp_writel(struct nfp_cpp *cpp, uint32_t cpp_id, * @return 0 on success, or -1 on failure. */ int nfp_cpp_readq(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, uint64_t *value); + uint64_t address, uint64_t *value); /* * Write a 64-bit value to a NFP CPP ID @@ -747,7 +746,7 @@ int nfp_cpp_readq(struct nfp_cpp *cpp, uint32_t cpp_id, * @return 0 on success, or -1 on failure. */ int nfp_cpp_writeq(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, uint64_t value); + uint64_t address, uint64_t value); /* * Initialize a mutex location @@ -768,7 +767,7 @@ int nfp_cpp_writeq(struct nfp_cpp *cpp, uint32_t cpp_id, * @return 0 on success, negative value on failure. */ int nfp_cpp_mutex_init(struct nfp_cpp *cpp, int target, - unsigned long long address, uint32_t key_id); + uint64_t address, uint32_t key_id); /* * Create a mutex handle from an address controlled by a MU Atomic engine @@ -788,7 +787,7 @@ int nfp_cpp_mutex_init(struct nfp_cpp *cpp, int target, * failure. */ struct nfp_cpp_mutex *nfp_cpp_mutex_alloc(struct nfp_cpp *cpp, int target, - unsigned long long address, uint32_t key_id); + uint64_t address, uint32_t key_id); /* * Get the NFP CPP handle the mutex was created with diff --git a/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c b/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c index 884cc84eaa..881f21bfd0 100644 --- a/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c +++ b/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c @@ -101,7 +101,7 @@ struct nfp_bar { uint64_t base; /* CPP address base */ uint64_t mask; /* Bit mask of the bar */ uint32_t bitsize; /* Bit size of the bar */ - int index; + uint32_t index; int lock; char *csr; @@ -400,8 +400,8 @@ struct nfp6000_area_priv { static int nfp6000_area_init(struct nfp_cpp_area *area, uint32_t dest, - unsigned long long address, - unsigned long size) + uint64_t address, + size_t size) { struct nfp_pcie_user *nfp = nfp_cpp_priv(nfp_cpp_area_cpp(area)); struct nfp6000_area_priv *priv = nfp_cpp_area_priv(area); @@ -501,8 +501,8 @@ nfp6000_area_iomem(struct nfp_cpp_area *area) static int nfp6000_area_read(struct nfp_cpp_area *area, void *kernel_vaddr, - unsigned long offset, - unsigned int length) + uint32_t offset, + size_t length) { uint64_t *wrptr64 = kernel_vaddr; const volatile uint64_t *rdptr64; @@ -510,7 +510,7 @@ nfp6000_area_read(struct nfp_cpp_area *area, uint32_t *wrptr32 = kernel_vaddr; const volatile uint32_t *rdptr32; int width; - unsigned int n; + size_t n; bool is_64; priv = nfp_cpp_area_priv(area); @@ -571,8 +571,8 @@ nfp6000_area_read(struct nfp_cpp_area *area, static int nfp6000_area_write(struct nfp_cpp_area *area, const void *kernel_vaddr, - unsigned long offset, - unsigned int length) + uint32_t offset, + size_t length) { const uint64_t *rdptr64 = kernel_vaddr; uint64_t *wrptr64; @@ -580,7 +580,7 @@ nfp6000_area_write(struct nfp_cpp_area *area, struct nfp6000_area_priv *priv; uint32_t *wrptr32; int width; - unsigned int n; + size_t n; bool is_64; priv = nfp_cpp_area_priv(area); @@ -752,7 +752,7 @@ static int nfp6000_set_barsz(struct rte_pci_device *dev, struct nfp_pcie_user *desc) { - unsigned long tmp; + uint64_t tmp; int i = 0; tmp = dev->mem_resource[0].len; diff --git a/drivers/net/nfp/nfpcore/nfp_cppcore.c b/drivers/net/nfp/nfpcore/nfp_cppcore.c index f764208a9a..627d4eaa5d 100644 --- a/drivers/net/nfp/nfpcore/nfp_cppcore.c +++ b/drivers/net/nfp/nfpcore/nfp_cppcore.c @@ -169,8 +169,8 @@ struct nfp_cpp_area * nfp_cpp_area_alloc_with_name(struct nfp_cpp *cpp, uint32_t dest, const char *name, - unsigned long long address, - unsigned long size) + uint64_t address, + uint32_t size) { struct nfp_cpp_area *area; uint64_t tmp64 = (uint64_t)address; @@ -184,7 +184,7 @@ nfp_cpp_area_alloc_with_name(struct nfp_cpp *cpp, if (err < 0) return NULL; - address = (unsigned long long)tmp64; + address = tmp64; if (name == NULL) name = ""; @@ -213,8 +213,8 @@ nfp_cpp_area_alloc_with_name(struct nfp_cpp *cpp, struct nfp_cpp_area * nfp_cpp_area_alloc(struct nfp_cpp *cpp, uint32_t dest, - unsigned long long address, - unsigned long size) + uint64_t address, + size_t size) { return nfp_cpp_area_alloc_with_name(cpp, dest, NULL, address, size); } @@ -237,8 +237,8 @@ nfp_cpp_area_alloc(struct nfp_cpp *cpp, struct nfp_cpp_area * nfp_cpp_area_alloc_acquire(struct nfp_cpp *cpp, uint32_t destination, - unsigned long long address, - unsigned long size) + uint64_t address, + size_t size) { struct nfp_cpp_area *area; @@ -352,7 +352,7 @@ nfp_cpp_area_iomem(struct nfp_cpp_area *area) */ int nfp_cpp_area_read(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, void *kernel_vaddr, size_t length) { @@ -378,7 +378,7 @@ nfp_cpp_area_read(struct nfp_cpp_area *area, */ int nfp_cpp_area_write(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, const void *kernel_vaddr, size_t length) { @@ -461,7 +461,7 @@ nfp_xpb_to_cpp(struct nfp_cpp *cpp, int nfp_cpp_area_readl(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, uint32_t *value) { int sz; @@ -475,7 +475,7 @@ nfp_cpp_area_readl(struct nfp_cpp_area *area, int nfp_cpp_area_writel(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, uint32_t value) { int sz; @@ -487,7 +487,7 @@ nfp_cpp_area_writel(struct nfp_cpp_area *area, int nfp_cpp_area_readq(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, uint64_t *value) { int sz; @@ -501,7 +501,7 @@ nfp_cpp_area_readq(struct nfp_cpp_area *area, int nfp_cpp_area_writeq(struct nfp_cpp_area *area, - unsigned long offset, + uint32_t offset, uint64_t value) { int sz; @@ -515,7 +515,7 @@ nfp_cpp_area_writeq(struct nfp_cpp_area *area, int nfp_cpp_readl(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, + uint64_t address, uint32_t *value) { int sz; @@ -530,7 +530,7 @@ nfp_cpp_readl(struct nfp_cpp *cpp, int nfp_cpp_writel(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, + uint64_t address, uint32_t value) { int sz; @@ -544,7 +544,7 @@ nfp_cpp_writel(struct nfp_cpp *cpp, int nfp_cpp_readq(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, + uint64_t address, uint64_t *value) { int sz; @@ -559,7 +559,7 @@ nfp_cpp_readq(struct nfp_cpp *cpp, int nfp_cpp_writeq(struct nfp_cpp *cpp, uint32_t cpp_id, - unsigned long long address, + uint64_t address, uint64_t value) { int sz; @@ -758,7 +758,7 @@ nfp_xpb_waitlm(struct nfp_cpp *cpp, int nfp_cpp_read(struct nfp_cpp *cpp, uint32_t destination, - unsigned long long address, + uint64_t address, void *kernel_vaddr, size_t length) { @@ -788,7 +788,7 @@ nfp_cpp_read(struct nfp_cpp *cpp, int nfp_cpp_write(struct nfp_cpp *cpp, uint32_t destination, - unsigned long long address, + uint64_t address, const void *kernel_vaddr, size_t length) { @@ -902,7 +902,7 @@ uint8_t * nfp_cpp_map_area(struct nfp_cpp *cpp, uint32_t cpp_id, uint64_t addr, - unsigned long size, + uint32_t size, struct nfp_cpp_area **area) { uint8_t *res; diff --git a/drivers/net/nfp/nfpcore/nfp_crc.c b/drivers/net/nfp/nfpcore/nfp_crc.c index 2cd8866253..68f27f8c68 100644 --- a/drivers/net/nfp/nfpcore/nfp_crc.c +++ b/drivers/net/nfp/nfpcore/nfp_crc.c @@ -14,7 +14,7 @@ nfp_crc32_be_generic(uint32_t crc, size_t len, uint32_t polynomial) { - int i; + uint32_t i; while (len--) { crc ^= *p++ << 24; for (i = 0; i < 8; i++) diff --git a/drivers/net/nfp/nfpcore/nfp_mutex.c b/drivers/net/nfp/nfpcore/nfp_mutex.c index 047e755416..7ac06e897a 100644 --- a/drivers/net/nfp/nfpcore/nfp_mutex.c +++ b/drivers/net/nfp/nfpcore/nfp_mutex.c @@ -28,16 +28,16 @@ struct nfp_cpp_mutex { struct nfp_cpp *cpp; uint8_t target; uint16_t depth; - unsigned long long address; + uint64_t address; uint32_t key; - unsigned int usage; + uint32_t usage; struct nfp_cpp_mutex *prev, *next; }; static int _nfp_cpp_mutex_validate(uint32_t model, int *target, - unsigned long long address) + uint64_t address) { /* Address must be 64-bit aligned */ if ((address & 7) != 0) @@ -76,7 +76,7 @@ _nfp_cpp_mutex_validate(uint32_t model, int nfp_cpp_mutex_init(struct nfp_cpp *cpp, int target, - unsigned long long address, + uint64_t address, uint32_t key) { uint32_t model = nfp_cpp_model(cpp); @@ -119,7 +119,7 @@ nfp_cpp_mutex_init(struct nfp_cpp *cpp, struct nfp_cpp_mutex * nfp_cpp_mutex_alloc(struct nfp_cpp *cpp, int target, - unsigned long long address, + uint64_t address, uint32_t key) { uint32_t model = nfp_cpp_model(cpp); @@ -269,9 +269,10 @@ nfp_cpp_mutex_lock(struct nfp_cpp_mutex *mutex) if (err < 0 && err != -EBUSY) return err; if (time(NULL) >= warn_at) { - PMD_DRV_LOG(ERR, "Warning: waiting for NFP mutex usage:%u depth:%hd] target:%d addr:%llx key:%08x]", + PMD_DRV_LOG(ERR, "Warning: waiting for NFP mutex usage:%u depth:%hd] " + "target:%d key:%08x] addr:%" PRIx64, mutex->usage, mutex->depth, mutex->target, - mutex->address, mutex->key); + mutex->key, mutex->address); warn_at = time(NULL) + 60; } sched_yield(); diff --git a/drivers/net/nfp/nfpcore/nfp_nffw.c b/drivers/net/nfp/nfpcore/nfp_nffw.c index fa65956baa..2412dc0e00 100644 --- a/drivers/net/nfp/nfpcore/nfp_nffw.c +++ b/drivers/net/nfp/nfpcore/nfp_nffw.c @@ -68,7 +68,7 @@ nffw_fwinfo_mip_offset_get(const struct nffw_fwinfo *fi) static int nfp_mip_mu_locality_lsb(struct nfp_cpp *cpp) { - unsigned int mode, addr40; + uint32_t mode, addr40; uint32_t xpbaddr, imbcppat; int err; @@ -84,7 +84,7 @@ nfp_mip_mu_locality_lsb(struct nfp_cpp *cpp) return nfp_cppat_mu_locality_lsb(mode, addr40); } -static unsigned int +static uint32_t nffw_res_fwinfos(struct nfp_nffw_info_data *fwinf, struct nffw_fwinfo **arr) { @@ -184,7 +184,7 @@ static struct nffw_fwinfo * nfp_nffw_info_fwid_first(struct nfp_nffw_info *state) { struct nffw_fwinfo *fwinfo; - unsigned int cnt, i; + uint32_t cnt, i; cnt = nffw_res_fwinfos(&state->fwinf, &fwinfo); if (cnt == 0) diff --git a/drivers/net/nfp/nfpcore/nfp_nsp.c b/drivers/net/nfp/nfpcore/nfp_nsp.c index 4f476f6f2b..dfe396c8ac 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp.c +++ b/drivers/net/nfp/nfpcore/nfp_nsp.c @@ -34,7 +34,7 @@ nfp_nsp_config_entries(struct nfp_nsp *state) return state->entries; } -unsigned int +uint32_t nfp_nsp_config_idx(struct nfp_nsp *state) { return state->idx; @@ -43,7 +43,7 @@ nfp_nsp_config_idx(struct nfp_nsp *state) void nfp_nsp_config_set_state(struct nfp_nsp *state, void *entries, - unsigned int idx) + uint32_t idx) { state->entries = entries; state->idx = idx; @@ -59,12 +59,12 @@ nfp_nsp_config_clear_state(struct nfp_nsp *state) static void nfp_nsp_print_extended_error(uint32_t ret_val) { - int i; + uint32_t i; if (ret_val == 0) return; - for (i = 0; i < (int)RTE_DIM(nsp_errors); i++) + for (i = 0; i < RTE_DIM(nsp_errors); i++) if (ret_val == (uint32_t)nsp_errors[i].code) PMD_DRV_LOG(ERR, "err msg: %s", nsp_errors[i].msg); } @@ -171,7 +171,7 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, uint64_t val) { struct timespec wait; - int count; + uint32_t count; int err; wait.tv_sec = 0; @@ -297,7 +297,7 @@ nfp_nsp_command_buf(struct nfp_nsp *nsp, unsigned int out_size) { struct nfp_cpp *cpp = nsp->cpp; - unsigned int max_size; + size_t max_size; uint64_t reg, cpp_buf; int ret, err; uint32_t cpp_id; @@ -316,7 +316,7 @@ nfp_nsp_command_buf(struct nfp_nsp *nsp, max_size = RTE_MAX(in_size, out_size); if (FIELD_GET(NSP_DFLT_BUFFER_SIZE_MB, reg) * SZ_1M < max_size) { - PMD_DRV_LOG(ERR, "NSP: default buffer too small for command 0x%04x (%llu < %u)", + PMD_DRV_LOG(ERR, "NSP: default buffer too small for command 0x%04x (%llu < %zu)", code, FIELD_GET(NSP_DFLT_BUFFER_SIZE_MB, reg) * SZ_1M, max_size); return -EINVAL; } @@ -361,7 +361,7 @@ int nfp_nsp_wait(struct nfp_nsp *state) { struct timespec wait; - int count; + uint32_t count; int err; wait.tv_sec = 0; @@ -401,7 +401,7 @@ nfp_nsp_mac_reinit(struct nfp_nsp *state) int nfp_nsp_load_fw(struct nfp_nsp *state, void *buf, - unsigned int size) + size_t size) { return nfp_nsp_command_buf(state, SPCODE_FW_LOAD, size, buf, size, NULL, 0); @@ -410,7 +410,7 @@ nfp_nsp_load_fw(struct nfp_nsp *state, int nfp_nsp_read_eth_table(struct nfp_nsp *state, void *buf, - unsigned int size) + size_t size) { return nfp_nsp_command_buf(state, SPCODE_ETH_RESCAN, size, NULL, 0, buf, size); @@ -419,7 +419,7 @@ nfp_nsp_read_eth_table(struct nfp_nsp *state, int nfp_nsp_write_eth_table(struct nfp_nsp *state, const void *buf, - unsigned int size) + size_t size) { return nfp_nsp_command_buf(state, SPCODE_ETH_CONTROL, size, buf, size, NULL, 0); @@ -428,7 +428,7 @@ nfp_nsp_write_eth_table(struct nfp_nsp *state, int nfp_nsp_read_identify(struct nfp_nsp *state, void *buf, - unsigned int size) + size_t size) { return nfp_nsp_command_buf(state, SPCODE_NSP_IDENTIFY, size, NULL, 0, buf, size); @@ -436,9 +436,9 @@ nfp_nsp_read_identify(struct nfp_nsp *state, int nfp_nsp_read_sensors(struct nfp_nsp *state, - unsigned int sensor_mask, + uint32_t sensor_mask, void *buf, - unsigned int size) + size_t size) { return nfp_nsp_command_buf(state, SPCODE_NSP_SENSORS, sensor_mask, NULL, 0, buf, size); diff --git a/drivers/net/nfp/nfpcore/nfp_nsp.h b/drivers/net/nfp/nfpcore/nfp_nsp.h index 1e2deaabb4..edb56e26ca 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp.h +++ b/drivers/net/nfp/nfpcore/nfp_nsp.h @@ -110,11 +110,11 @@ uint16_t nfp_nsp_get_abi_ver_major(struct nfp_nsp *state); uint16_t nfp_nsp_get_abi_ver_minor(struct nfp_nsp *state); int nfp_nsp_wait(struct nfp_nsp *state); int nfp_nsp_device_soft_reset(struct nfp_nsp *state); -int nfp_nsp_load_fw(struct nfp_nsp *state, void *buf, unsigned int size); +int nfp_nsp_load_fw(struct nfp_nsp *state, void *buf, size_t size); int nfp_nsp_mac_reinit(struct nfp_nsp *state); -int nfp_nsp_read_identify(struct nfp_nsp *state, void *buf, unsigned int size); -int nfp_nsp_read_sensors(struct nfp_nsp *state, unsigned int sensor_mask, - void *buf, unsigned int size); +int nfp_nsp_read_identify(struct nfp_nsp *state, void *buf, size_t size); +int nfp_nsp_read_sensors(struct nfp_nsp *state, uint32_t sensor_mask, + void *buf, size_t size); static inline int nfp_nsp_has_mac_reinit(struct nfp_nsp *state) @@ -188,17 +188,17 @@ enum nfp_eth_fec { * @fec_modes_supported: bitmap of FEC modes supported */ struct nfp_eth_table { - unsigned int count; - unsigned int max_index; + uint32_t count; + uint32_t max_index; struct nfp_eth_table_port { - unsigned int eth_index; - unsigned int index; - unsigned int nbi; - unsigned int base; - unsigned int lanes; - unsigned int speed; - - unsigned int interface; + uint32_t eth_index; + uint32_t index; + uint32_t nbi; + uint32_t base; + uint32_t lanes; + uint32_t speed; + + uint32_t interface; enum nfp_eth_media media; enum nfp_eth_fec fec; @@ -218,51 +218,50 @@ struct nfp_eth_table { /* Computed fields */ uint8_t port_type; - unsigned int port_lanes; + uint32_t port_lanes; int is_split; - unsigned int fec_modes_supported; + uint32_t fec_modes_supported; } ports[]; }; struct nfp_eth_table *nfp_eth_read_ports(struct nfp_cpp *cpp); -int nfp_eth_set_mod_enable(struct nfp_cpp *cpp, unsigned int idx, int enable); -int nfp_eth_set_configured(struct nfp_cpp *cpp, unsigned int idx, - int configed); -int nfp_eth_set_fec(struct nfp_cpp *cpp, unsigned int idx, enum nfp_eth_fec mode); +int nfp_eth_set_mod_enable(struct nfp_cpp *cpp, uint32_t idx, int enable); +int nfp_eth_set_configured(struct nfp_cpp *cpp, uint32_t idx, int configed); +int nfp_eth_set_fec(struct nfp_cpp *cpp, uint32_t idx, enum nfp_eth_fec mode); -int nfp_nsp_read_eth_table(struct nfp_nsp *state, void *buf, unsigned int size); +int nfp_nsp_read_eth_table(struct nfp_nsp *state, void *buf, size_t size); int nfp_nsp_write_eth_table(struct nfp_nsp *state, const void *buf, - unsigned int size); + size_t size); void nfp_nsp_config_set_state(struct nfp_nsp *state, void *entries, - unsigned int idx); + uint32_t idx); void nfp_nsp_config_clear_state(struct nfp_nsp *state); void nfp_nsp_config_set_modified(struct nfp_nsp *state, int modified); void *nfp_nsp_config_entries(struct nfp_nsp *state); int nfp_nsp_config_modified(struct nfp_nsp *state); -unsigned int nfp_nsp_config_idx(struct nfp_nsp *state); +uint32_t nfp_nsp_config_idx(struct nfp_nsp *state); static inline int nfp_eth_can_support_fec(struct nfp_eth_table_port *eth_port) { - return !!eth_port->fec_modes_supported; + return eth_port->fec_modes_supported != 0; } -static inline unsigned int +static inline uint32_t nfp_eth_supported_fec_modes(struct nfp_eth_table_port *eth_port) { return eth_port->fec_modes_supported; } -struct nfp_nsp *nfp_eth_config_start(struct nfp_cpp *cpp, unsigned int idx); +struct nfp_nsp *nfp_eth_config_start(struct nfp_cpp *cpp, uint32_t idx); int nfp_eth_config_commit_end(struct nfp_nsp *nsp); void nfp_eth_config_cleanup_end(struct nfp_nsp *nsp); int __nfp_eth_set_aneg(struct nfp_nsp *nsp, enum nfp_eth_aneg mode); -int __nfp_eth_set_speed(struct nfp_nsp *nsp, unsigned int speed); -int __nfp_eth_set_split(struct nfp_nsp *nsp, unsigned int lanes); +int __nfp_eth_set_speed(struct nfp_nsp *nsp, uint32_t speed); +int __nfp_eth_set_split(struct nfp_nsp *nsp, uint32_t lanes); /** * struct nfp_nsp_identify - NSP static information @@ -298,6 +297,6 @@ enum nfp_nsp_sensor_id { }; int nfp_hwmon_read_sensor(struct nfp_cpp *cpp, enum nfp_nsp_sensor_id id, - long *val); + uint32_t *val); #endif diff --git a/drivers/net/nfp/nfpcore/nfp_nsp_cmds.c b/drivers/net/nfp/nfpcore/nfp_nsp_cmds.c index 28dba27124..31677b66e6 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp_cmds.c +++ b/drivers/net/nfp/nfpcore/nfp_nsp_cmds.c @@ -75,7 +75,7 @@ struct nfp_sensors { int nfp_hwmon_read_sensor(struct nfp_cpp *cpp, enum nfp_nsp_sensor_id id, - long *val) + uint32_t *val) { struct nfp_sensors s; struct nfp_nsp *nsp; diff --git a/drivers/net/nfp/nfpcore/nfp_nsp_eth.c b/drivers/net/nfp/nfpcore/nfp_nsp_eth.c index 51bd57033e..5492840aa1 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp_eth.c +++ b/drivers/net/nfp/nfpcore/nfp_nsp_eth.c @@ -124,7 +124,7 @@ union eth_table_entry { static const struct { enum nfp_eth_rate rate; - unsigned int speed; + uint32_t speed; } nsp_eth_rate_tbl[] = { { RATE_INVALID, 0, }, { RATE_10M, SPEED_10, }, @@ -134,24 +134,24 @@ static const struct { { RATE_25G, SPEED_25000, }, }; -static unsigned int +static uint32_t nfp_eth_rate2speed(enum nfp_eth_rate rate) { - int i; + uint32_t i; - for (i = 0; i < (int)RTE_DIM(nsp_eth_rate_tbl); i++) + for (i = 0; i < RTE_DIM(nsp_eth_rate_tbl); i++) if (nsp_eth_rate_tbl[i].rate == rate) return nsp_eth_rate_tbl[i].speed; return 0; } -static unsigned int -nfp_eth_speed2rate(unsigned int speed) +static enum nfp_eth_rate +nfp_eth_speed2rate(uint32_t speed) { - int i; + uint32_t i; - for (i = 0; i < (int)RTE_DIM(nsp_eth_rate_tbl); i++) + for (i = 0; i < RTE_DIM(nsp_eth_rate_tbl); i++) if (nsp_eth_rate_tbl[i].speed == speed) return nsp_eth_rate_tbl[i].rate; @@ -162,7 +162,7 @@ static void nfp_eth_copy_mac_reverse(uint8_t *dst, const uint8_t *src) { - int i; + uint32_t i; for (i = 0; i < RTE_ETHER_ADDR_LEN; i++) dst[RTE_ETHER_ADDR_LEN - i - 1] = src[i]; @@ -171,11 +171,11 @@ nfp_eth_copy_mac_reverse(uint8_t *dst, static void nfp_eth_port_translate(struct nfp_nsp *nsp, const union eth_table_entry *src, - unsigned int index, + uint32_t index, struct nfp_eth_table_port *dst) { - unsigned int rate; - unsigned int fec; + uint32_t rate; + uint32_t fec; uint64_t port, state; port = rte_le_to_cpu_64(src->port); @@ -224,7 +224,7 @@ nfp_eth_port_translate(struct nfp_nsp *nsp, static void nfp_eth_calc_port_geometry(struct nfp_eth_table *table) { - unsigned int i, j; + uint32_t i, j; for (i = 0; i < table->count; i++) { table->max_index = RTE_MAX(table->max_index, @@ -269,7 +269,9 @@ __nfp_eth_read_ports(struct nfp_nsp *nsp) union eth_table_entry *entries; struct nfp_eth_table *table; uint32_t table_sz; - int i, j, ret, cnt = 0; + uint32_t i; + uint32_t j; + int ret, cnt = 0; const struct rte_ether_addr *mac; entries = malloc(NSP_ETH_TABLE_SIZE); @@ -319,7 +321,7 @@ __nfp_eth_read_ports(struct nfp_nsp *nsp) } nfp_eth_calc_port_geometry(table); - for (i = 0; i < (int)table->count; i++) + for (i = 0; i < table->count; i++) nfp_eth_calc_port_type(&table->ports[i]); free(entries); @@ -358,7 +360,7 @@ nfp_eth_read_ports(struct nfp_cpp *cpp) struct nfp_nsp * nfp_eth_config_start(struct nfp_cpp *cpp, - unsigned int idx) + uint32_t idx) { union eth_table_entry *entries; struct nfp_nsp *nsp; @@ -452,7 +454,7 @@ nfp_eth_config_commit_end(struct nfp_nsp *nsp) */ int nfp_eth_set_mod_enable(struct nfp_cpp *cpp, - unsigned int idx, + uint32_t idx, int enable) { union eth_table_entry *entries; @@ -494,7 +496,7 @@ nfp_eth_set_mod_enable(struct nfp_cpp *cpp, */ int nfp_eth_set_configured(struct nfp_cpp *cpp, - unsigned int idx, + uint32_t idx, int configed) { union eth_table_entry *entries; @@ -532,14 +534,14 @@ nfp_eth_set_configured(struct nfp_cpp *cpp, static int nfp_eth_set_bit_config(struct nfp_nsp *nsp, - unsigned int raw_idx, + uint32_t raw_idx, const uint64_t mask, - const unsigned int shift, - unsigned int val, + const uint32_t shift, + uint32_t val, const uint64_t ctrl_bit) { union eth_table_entry *entries = nfp_nsp_config_entries(nsp); - unsigned int idx = nfp_nsp_config_idx(nsp); + uint32_t idx = nfp_nsp_config_idx(nsp); uint64_t reg; /* @@ -623,7 +625,7 @@ __nfp_eth_set_fec(struct nfp_nsp *nsp, */ int nfp_eth_set_fec(struct nfp_cpp *cpp, - unsigned int idx, + uint32_t idx, enum nfp_eth_fec mode) { struct nfp_nsp *nsp; @@ -656,7 +658,7 @@ nfp_eth_set_fec(struct nfp_cpp *cpp, */ int __nfp_eth_set_speed(struct nfp_nsp *nsp, - unsigned int speed) + uint32_t speed) { enum nfp_eth_rate rate; @@ -682,7 +684,7 @@ __nfp_eth_set_speed(struct nfp_nsp *nsp, */ int __nfp_eth_set_split(struct nfp_nsp *nsp, - unsigned int lanes) + uint32_t lanes) { return NFP_ETH_SET_BIT_CONFIG(nsp, NSP_ETH_RAW_PORT, NSP_ETH_PORT_LANES, lanes, NSP_ETH_CTRL_SET_LANES); diff --git a/drivers/net/nfp/nfpcore/nfp_resource.c b/drivers/net/nfp/nfpcore/nfp_resource.c index 57089c770f..9dd4832779 100644 --- a/drivers/net/nfp/nfpcore/nfp_resource.c +++ b/drivers/net/nfp/nfpcore/nfp_resource.c @@ -156,7 +156,7 @@ nfp_resource_acquire(struct nfp_cpp *cpp, struct nfp_resource *res; int err; struct timespec wait; - int count; + uint16_t count; res = malloc(sizeof(*res)); if (res == NULL) diff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.c b/drivers/net/nfp/nfpcore/nfp_rtsym.c index aa3b7a483e..243d3c9ce5 100644 --- a/drivers/net/nfp/nfpcore/nfp_rtsym.c +++ b/drivers/net/nfp/nfpcore/nfp_rtsym.c @@ -382,7 +382,7 @@ nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, uint8_t * nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, - unsigned int min_size, + uint32_t min_size, struct nfp_cpp_area **area) { int ret; diff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.h b/drivers/net/nfp/nfpcore/nfp_rtsym.h index 30768f1ccf..e7295258b3 100644 --- a/drivers/net/nfp/nfpcore/nfp_rtsym.h +++ b/drivers/net/nfp/nfpcore/nfp_rtsym.h @@ -56,5 +56,5 @@ const struct nfp_rtsym *nfp_rtsym_lookup(struct nfp_rtsym_table *rtbl, uint64_t nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name, int *error); uint8_t *nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, - unsigned int min_size, struct nfp_cpp_area **area); + uint32_t min_size, struct nfp_cpp_area **area); #endif -- 2.39.1