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 B2F17430EF; Thu, 24 Aug 2023 13:11:19 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D757443260; Thu, 24 Aug 2023 13:10:57 +0200 (CEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2100.outbound.protection.outlook.com [40.107.220.100]) by mails.dpdk.org (Postfix) with ESMTP id 3BA1D42D12 for ; Thu, 24 Aug 2023 13:10:54 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WuA2ieWlXldYh4bIGCpgyd971KpVp3iscgtd5dt+TClciHsOjGWWEQQldlt4g47m7KDJ7I/5w1mlWrCMqcV+UsymHfIUcIWP7P0N8Yr/Hknm9tGznWprgfPcSNVRT4R7bexiJgTq7Tg+bib0gkS45YT61L2eskFWwKBgCUTPfPm7kwAbd1G9ziRDzXeZ+V8Pp3WliO/6zo1B4dhHsKS9xengJCqOu5ccWeVtJlmhNgsURzcSWs58mIHsiA6zH1cVMaXi2JfSUb4m71pigofB+QbxEfO/2eRpIzbw/FiUNog+023JstPdCkXa4y0Rs91G0+rX/Khk8eUTmazweymu+w== 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=UQeo+IU30n5NjVzCoogDeDTyKYT6dme0H3LUHUQ5Jb4=; b=diUIBSEivIizzXxIB8l2nlkSETD+IZit0ryyKU5OyF09ht/3vb6KtdtRQw3YZI1Nz8rtr3NoUp0U2Z10hYZQ12hh95Y3zo1Zy5kxKhbOfXbjUjehy7bm8A+wSijEMkiGjHy8hp5isJTkXnWujaCxM5c2PjdkDQlA4oHCTo/ry/IIla17mP/VqzAYOyTaXz9WWvX9hzA2bstk3pxzplT8jJAtOZhJTBZnkGo0eZnHSmOECUNdIU0hZcca+zPeSHaQxyPgLiH5MAptjDYq1KaH3wdFVl/GTx6ktZ6uVqWSEZJKESZU15xc7ieRUqmqIqshs0yW+ULRzGl1hlYILKC6Sg== 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=UQeo+IU30n5NjVzCoogDeDTyKYT6dme0H3LUHUQ5Jb4=; b=KWZZZb1V1dvgisSjNlrx8p0Y/YhIPem+u5YGkYSzXB15Zc+IyX/33eD1ZxkBCtN+m0Y1Yk5sH6huRJJczpMGyLojnXVG4yYGfhltFqnS4p4fZl8eb2V8qwmTr6ItcGBceM79cBSVnAH8jJvMByQGnarSqmgVX8PlAO7l3Zqy2Xw= 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 PH0PR13MB6134.namprd13.prod.outlook.com (2603:10b6:510:295::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6678.26; Thu, 24 Aug 2023 11:10:51 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::51fe:5846:af8b:bace]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::51fe:5846:af8b:bace%3]) with mapi id 15.20.6699.020; Thu, 24 Aug 2023 11:10:51 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He Subject: [PATCH 03/27] net/nfp: unify the type of integer variable Date: Thu, 24 Aug 2023 19:09:32 +0800 Message-Id: <20230824110956.1943559-4-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230824110956.1943559-1-chaoyong.he@corigine.com> References: <20230824110956.1943559-1-chaoyong.he@corigine.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SJ0PR05CA0005.namprd05.prod.outlook.com (2603:10b6:a03:33b::10) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH0PR13MB6134:EE_ X-MS-Office365-Filtering-Correlation-Id: a48d09c5-7bb2-4d9f-c990-08dba492c6f4 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: pNcH5j9c8+jyy7HMMu/mU4XffZ5fajR9FM80PKsDHhSYHwMeM7j3RuKUhnx6c3M3PpMXSztkA8q9D5cUckpqz8ZbWvIBxdDkk/5zozqA17nCVzERINihqGUVda2bZnm+2Qa+s5dtAfhxuH+2iSLsSJrsMNkrLWxsihkwuRzRBqBhXahwQ6GmpIBqhP4yzfgTtL5pQchUQpOt5HWwMI9Jm0qV73mGu4mtuB8uJt9WP00e4mbTk1qK7TwEWPqFtyDsQ8CyLd+olJTZW+sR5sLt3cKizLRW/ikzKf75zrUyZqk4PDE0orpDRvl59+FOIeGHbUk5xa4kCLmScfiL91mki5Gst2I0u/x5YtY6BXZ+pPeCm4aM6Eeec9uaXu3pOFa7QL+aMFQZf9L9St+N4Jrlpg4KI2g2sbiSOvuFhgFpRzQD7+WC9leWT4RRUDcrvZQST0j5H6PVgKNXy/dZsResS4Px+bmQoHkS5fNMr41r47FjV6bSUKiQ+oEB/ibpnXXcwXvFVFdNFaHjkzE3fxY4meywxuTyi9I+9ivPis16bb+x30X8eukA0BsYxnvxD7tLw3EVravh9gTktrMtjA4j29E4qNeGOwfeLE+FiJSRfUubaiVCOJhZyTdvV6VD/AbJ+xuBpiTvcIeFeLyslzJmE4g+gU/CEaUKWEBUpp/izuNoLSzidJuuYeqmISHC4OKgIWqwmsIUzf8yiwoRCwBZuryZx0ZYMy4AqjjEbd8ozC8= 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)(346002)(396003)(366004)(136003)(376002)(39830400003)(1800799009)(451199024)(186009)(316002)(66556008)(6916009)(66946007)(66476007)(12101799020)(41300700001)(44832011)(5660300002)(8936002)(8676002)(38100700002)(66574015)(38350700002)(4326008)(30864003)(2906002)(83380400001)(26005)(86362001)(478600001)(107886003)(52116002)(36756003)(6506007)(6666004)(6512007)(6486002)(2616005)(1076003)(309714004); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WXRURDhwYU9JUUh2alJURndORmFwaXV0cUMxNk16dkpBN2ZsbmpFY0VPajB3?= =?utf-8?B?aVVrS0lsZktvSTBCUUdZRHpzNThzV0VYd1FSWk5YSTMrZVNQRk55ZEhFMHkw?= =?utf-8?B?OXZLejVKYmhrTkp3ZUYveWlQc2dFVk9zQXRoc3hWMTNBTmV5b3doMVVCVFk1?= =?utf-8?B?dU9ONHVBUVZqYVNsNHZiY09CWThaRGsyTXRiRW9sbG9hNUdIWHJxenVRQnZG?= =?utf-8?B?WDlmeEZab3lJMGVsdTN3bFNmNXpjQ1BEMUhuVlpacjlBQmN6NXpMUDhlRmUr?= =?utf-8?B?OGRSc2ZUTkE0RWd5M21oV2lNM01YSkxnV0dIQVZjcG8wZHppNDRLdCt2OTd3?= =?utf-8?B?SHBVK0Y5OG51dVlCRGVoZVkvdjJtNFcvdXVuYXhXRGJBT1dxV0d1M0NZbFp6?= =?utf-8?B?QzcxUC9iaG15VmpTZEprV1RSSlZZbE1IY2V5blNQaVJpK3RNQzZhdTI4OW1F?= =?utf-8?B?MFRpUjdvVDNGNDY2KzdqRVZsdjQzempsN1RvTlNyaGZLbFJiTVlDT3Y4WGJB?= =?utf-8?B?dkpKemNYbTF6L0dEVU5tWFo0cVBQUXpjS1JQT0FRTlM3QVJQK3dlek1ZcVJl?= =?utf-8?B?cWdEWHppU0gvKzFUV2V6SnFZZ3dlWndxem9SYk9zMnpaVWFIUFlOcm9maFpj?= =?utf-8?B?R25yMjBzSk9Lblo3bjBpMkJNMFlvWStjek1IanpjN2QxYU5JWndUSXJPYWZW?= =?utf-8?B?aUVHUWppN2k2MlN2bFc3cm1FSVN6R2RTeUJwWk4zTUNyTXRlUXlXcnBwaHRq?= =?utf-8?B?YTZFbTVyTTkrWDhUbVdrb0lSTDlZS1VGbjdTV05YYjQ0QTcvTWkrNWhydXZx?= =?utf-8?B?TjJSQnlKMVBNOU1sU1FiUFBrcXVDeEZZMEJMdXFnYTYvbHdxQUNKNlFDTytz?= =?utf-8?B?c1lZcWQ5bDMydkpoaENVYzl4UHNXN3lJdVdMWnVzRW54VG5wTWJlZlpuTlBv?= =?utf-8?B?TUtuUU1hbXh6aFpmeWFSSHBxRW02bE4zaHVRYmRNVU5acEl6S25QbGh6NzYz?= =?utf-8?B?WWcxbGR0L2lzTXIvYXFrWjRUWnBCbmQ2allSOFVtUkR0TkJldlh1OEp4MHRT?= =?utf-8?B?MlpJU080WCszT0tKbTVITWRNNjNRNXcraWNvbzFadTlxY045S3lBc3o5cTE5?= =?utf-8?B?T0hXNGNKY3UrUWhVMWlDcWordjFuUXJraERETGtxSk1OT1MxUS9aVlhJSUN3?= =?utf-8?B?ckg1d1ZsVG84QWhoKytGNGZ2dTU3TytNb0tsYVd1NkNzYko3NG56TG1DWUNH?= =?utf-8?B?MkVKQzNjYWlReUtJcS9GTzBwUUlYUG9CS3dLbkFlRS95WXlBR0NJQk5kcTU0?= =?utf-8?B?YWRSa3N4RDd0MFduMzN3b2JRbXF2TkdRTVB6dExtREZkQ05HcmxmYi9lZ2lK?= =?utf-8?B?MjJ3NndPbkkrSk9OV1V1YzVvQXQrZXA5Rk1ZWDl1TDd1OXNRU3hCUW1xZWg2?= =?utf-8?B?Z3oxd0t1MHlzM0Q0RWlHRnZNaTZhTUFWVFVLbTlxYXYvWTNNc3Q5MEQ5UFI1?= =?utf-8?B?cTRNVTNseWd5NldZcVFqV2pjcUdjYnZIWTRrcDY5akFsUExCM2FSRU5UWElH?= =?utf-8?B?L1ErRzdxdGFLc0tQNnZUV2xBMFRtRURGdFR2VVFBOWI0dlFLTUhrT05yK08x?= =?utf-8?B?am1YUWVrWnk1V0VVd3BDWWpsRjErZTZqcW5JS0lkMnJ3U3FNL3ZaQVhsMW5K?= =?utf-8?B?YTVOaUVPR3ZyK3BzRzJZQmJFaGUvekRQY2p1NzE0aGh1NGdGRVlFcFhmYURN?= =?utf-8?B?cGpScWhMVUxnQWVJemUvUHRpOW5sTVYwRUhaQ2FIY0hFR1ZyQTZhWmZZWXlY?= =?utf-8?B?TWFyK1pmYVJnWHFiL2NEMVVUdFVXSmNxY1ZjSVROdGNSSlRmeFk2a0dBelph?= =?utf-8?B?MXZhRU9yZVIxYXhmdTNBVy8yN2YxQTBuYmIzN3ZkYzBnMUlkSlovWldsQzNI?= =?utf-8?B?NWFLcnZNWE9ZYVljWUdHblZHSldZY3B5TTIvamk1OXRRelh6am9kNTRiaVFE?= =?utf-8?B?WmQ4R0ZJOUUydldNeWd4T2hzdHNabS9NN2U1RjErR1pqcHRQOU1iOE82WTBh?= =?utf-8?B?aUZQQmRHbHhQWnlyQ0NVTGNIRUhpemhLTm0rbTRkdzZUM3BndnB1Nlk3RjNn?= =?utf-8?B?KzhoaDNwQWl5WEtHaURGcGQxbGZteGdSR0J0WWpOMmZ1ZElLTGtILzZqbjZs?= =?utf-8?B?aFE9PQ==?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: a48d09c5-7bb2-4d9f-c990-08dba492c6f4 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2023 11:10:51.6639 (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: q4EkUQeZnmFsVazwlZ6CxgmQMbeHH210WKBh0TQa6VOq1OMrgtC8ddoD/lZbKoigC4FQvcWTulUY61Au1lCjgiC3hlzaOvv3ydrSrVOy9Tg= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR13MB6134 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 Reviewed-by: Niklas Söderlund --- 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_mutex.c | 12 ++--- 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 +- 12 files changed, 139 insertions(+), 139 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 25f7700b08..7b32721d60 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_mutex.c b/drivers/net/nfp/nfpcore/nfp_mutex.c index 047e755416..697361da4d 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,7 +269,7 @@ 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 addr:%lx key:%08x]", mutex->usage, mutex->depth, mutex->target, mutex->address, mutex->key); warn_at = time(NULL) + 60; diff --git a/drivers/net/nfp/nfpcore/nfp_nffw.c b/drivers/net/nfp/nfpcore/nfp_nffw.c index 8bdc69766e..7ff468373b 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) { /* @@ -183,7 +183,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..87eed3d951 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 < %lu)", 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 3eeefc74af..9213023690 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; @@ -161,7 +161,7 @@ nfp_eth_speed2rate(unsigned int speed) 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]; @@ -170,11 +170,11 @@ nfp_eth_copy_mac_reverse(uint8_t *dst, const uint8_t *src) 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); @@ -223,7 +223,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, @@ -268,7 +268,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); @@ -318,7 +320,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); @@ -357,7 +359,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; @@ -451,7 +453,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; @@ -493,7 +495,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; @@ -531,14 +533,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; /* @@ -622,7 +624,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; @@ -655,7 +657,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; @@ -681,7 +683,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