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 D581F42CFB; Mon, 19 Jun 2023 17:58:11 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 60C8040E0F; Mon, 19 Jun 2023 17:58:11 +0200 (CEST) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2054.outbound.protection.outlook.com [40.107.237.54]) by mails.dpdk.org (Postfix) with ESMTP id A528B40DF8 for ; Mon, 19 Jun 2023 17:58:09 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RuBZR4GxUNJcaNmxM7y/rBAoNp1ec+iYTx5QsOHy04B95feqC0OuKt5heA5+0dWExAY+P2AUxzGayTiLl7NUFFBj3GZeUUBso/CVJO+axaS+R5TJgJFRnPd9zl7nOEh3jbPPzHc6BxiNY1yWjCZHg6Pwy9caxS0RilTB/xvNOAn485DuyQwjqgu8KneK1R6KlC0UgmEVwCICtBgIS19eatBshHOaKRPpyPlJQbR39wNusgCzZGo4MkOfNF0UasB9vX5gmWRuQzN/O4umrzjaayBzqt1CixMQtE6RCX0OBtmZj2ZH+G2fgRschTiE5mWo1sxDxrqVQWhyPIZGbxvneg== 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=Vru/b37Z15SCIMlaUzrgCqAWEcBaLi1ZEbPUyM5slJQ=; b=kPi49gJnqdwySDEMxWn7MEBwanj7Q3VVym/8JRC3L022iTLce6owXQnfCjnjL/yXV0vGxvd4Kp1qRm5hkfvdsrin83XjKrVG/ti5a9eL885ETDH4YCblnDm8GQtjyjp+lFcLmxvqSRS8EVVfFeDVJkfBaCQdobjyt4n/qj+KBAorNCLT+6OhAOF3E0xz3Icigz7Pk3J8axYXcYBI2aNj1HYDoNYrgCQkS31pixM9VFxSc5vtlXE6IchC/pOKX00lzHHfHu37YaZja6NDCbPLMKT1ZLA56ewgrMjk7wU8vInIh5MxYwC+gl23SCZieADfBFzkQJgm03eXh6P4R0MmHg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Vru/b37Z15SCIMlaUzrgCqAWEcBaLi1ZEbPUyM5slJQ=; b=wa6ZAbvh7qGHlbPvUqOo1yE+zf3YVuvrj2opkx5T9Q2aT+Y22ZGPsjmYemKFxp4PG+yrMRcaoCXZt2oJOtr2p5JzyMI8bGybGyyN1aMMmc7xzMwGhnaZmOFT/svurZ81XLxhhHLYTCys+lHeb79kQq5huqd9wF2KDBBsxn5BUm8= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by MN2PR12MB4143.namprd12.prod.outlook.com (2603:10b6:208:1d0::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6500.36; Mon, 19 Jun 2023 15:58:07 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::16e3:326c:5c2a:be42]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::16e3:326c:5c2a:be42%3]) with mapi id 15.20.6500.036; Mon, 19 Jun 2023 15:58:07 +0000 Message-ID: <1170a8ae-f3f6-88b1-c48c-ce4bb74710bc@amd.com> Date: Mon, 19 Jun 2023 16:58:02 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Content-Language: en-US To: Ivan Malov , dev@dpdk.org Cc: Andrew Rybchenko , Denis Pryazhennikov , Andy Moreton References: <20230601195538.8265-1-ivan.malov@arknetworks.am> <20230607130245.8048-1-ivan.malov@arknetworks.am> <20230607130245.8048-4-ivan.malov@arknetworks.am> From: Ferruh Yigit Subject: Re: [PATCH v4 03/34] common/sfc_efx/base: add API to list HW tables In-Reply-To: <20230607130245.8048-4-ivan.malov@arknetworks.am> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO4P265CA0076.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:2bd::11) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|MN2PR12MB4143:EE_ X-MS-Office365-Filtering-Correlation-Id: 9be84a6a-3d18-40b9-17ed-08db70ddf939 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: w0pYEEjCndJJYSqn23vHHUzl1ltU5uVuFWUDXoWmZ65FV9dFcjExMYhAFMMkFiax5I7dBbFOPewkM38+LmnwzQceeg+QLFZhVEdmpeod4F3xEgxzse5wMorIgtlfcE8te4EtUGiRWtWw1sR1hkTpDRm73+fUHOmt4wUKNxAIAJ0vWSu9/1yXbkPA6f22vhqKwmdTuhD96iczoVbWTpsg0QzLZR+hXBgueI6r4E8/RaJG5QeIRkn/hGAFdD8oGP/gHhFfcvp4oIg4NgOjxA+fOFVMAxdqiPuaKEdj1NXPhfxAxwdJ3ubYFgR4VKj3Ln56yx7m2bjvH7r0lV1o+dp8GDXBa/4dGmGKFtlhqfK07Toe1MUPBNIin/rfNE93vVOqmV0MpQKDyWWhNpaCEkRJbYPYDJKlzEEPZiQxU+rTrflW9TNiVp3vdQiauhlplmJmpvBYG6YUKT/6MOOygUzOkbDxtux2C7fFLa0BV08ShsrMznmMg/y9guAovswj3ZnrKSu6wdnvl0TYRILFVugX7i0pU7mdqQb9qDampyUl1LTZUYWjIYyWSsBEQAd0VDjlTWlTO8chwRd0MKXamBDrvkRlE8jQK+MfpUjeB6N25CuotdUdBiCa6A6gREdtjfF4LexPhQji59FEk7JIqGjW/w== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(376002)(136003)(39860400002)(366004)(346002)(396003)(451199021)(31696002)(478600001)(2906002)(54906003)(6666004)(6486002)(2616005)(86362001)(36756003)(26005)(186003)(6506007)(6512007)(53546011)(8936002)(8676002)(66476007)(66556008)(66946007)(5660300002)(38100700002)(4326008)(44832011)(316002)(83380400001)(31686004)(41300700001)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T2pnUjNXTUI0akJCZEFWNHlCTFYvaTdaWWZuWW5aV3FKc2JMZUhaaFhCd2ky?= =?utf-8?B?anBMUm94M3l4aWJGRGM4emlwUjhvZ2lMdHRSLzQ5bmREeE44SC9VMVJRTEl2?= =?utf-8?B?dWlQYjNrbTdIeitiTHc3TmFQL3pSd0EwdHZnR1NjRTBQNnZ2aExYTTR6bmFn?= =?utf-8?B?ajh0NGFpZExvYTJsZk5LbUlWVVA5ZjV3eHJteFErY2E3VTFDVTVnZjlCQWpE?= =?utf-8?B?RCtHSUdwVWI3bnFzNDRpdCt6YjAxeFdIU0w1UkRibWhCemRsZVRvV3VtVmFM?= =?utf-8?B?YnNDZGIxZTBxV3Q1L2tsNWhKSGF1K2hxRCtRRGJDU3N5aG5jZHpZaDdSWlEw?= =?utf-8?B?bldLQThrRUVJeHQ0ZDkwQjJjRElWV1ZickdlYyt3N21WVDBLR2ZlV0VXakQr?= =?utf-8?B?aHJkMjNOeDJsaVYrSVRBd3ZNOHlpcG56KzI2TStiTVY4aHVZQUtWc1JXa2tB?= =?utf-8?B?cjFqSENUcUkvN0p2VWcyZ1ZHY2FtMEsvRHBDdkl6bDVyWFQyZ2I2VFJDdU1B?= =?utf-8?B?UXYrTURjTDRZajJtelBuQml4SHBGSitma0FJUVZ6cTNqbHdya0NraFMvR2VV?= =?utf-8?B?MFJkNnR0ZlRhQnZzdExnZ0hWRDY4aCtHa2VkV1NCU3NvOUo5TTZNQ09ZQkhs?= =?utf-8?B?aTlPa2xNaXhFV0MvNWZXa0JsWUNlblJidnc3WmdiNC8zcHNQSk1aa1pIc0tv?= =?utf-8?B?ajJCZnZOeE5Na1huTXNPUCt1aGZGUWNpb2V1NGhyWnBIKy8zQ3hiVlFWSXNK?= =?utf-8?B?L0h4REp0RTlTYjRCem5KTDV1ZWoyUUVVbEF5ais4emozbnJLUWFaSnVKblRV?= =?utf-8?B?TlBwQXRtS1pKSTVQejNidTlORkIyWXg3cHFvTVI3RmZINE50QVZxZWRJcTJw?= =?utf-8?B?YUY1RHJyOWdHNG1wdlM2YmZSMVdja0VNSUNzQTNNa3ZDUGppanR1VVRBejNE?= =?utf-8?B?ck5uOEhFN1dJN1ExOW5BMy9VeW44VUdoMlhhOXkrZHNWV3c2QTUydlZSb3N6?= =?utf-8?B?c09Vb0tVRytQM2krWFFwVVBTQ04vZE5SVlVzOGhTbDFjTXpPVE1HOEFnSHVa?= =?utf-8?B?RThJSXNTK0ZPOXhHTU5IZm5aN0RORTdNVk1JbGo3VzN4dWJwSnFqZS9Cd09K?= =?utf-8?B?YWtzRS8xb1AwNmFSQmZLbUxyTVMvVW1McWUvSlJvdVFIYWYydUpXcE85Vkg1?= =?utf-8?B?S2lCMUFPMW9EcnE4bVd2MzNtejhOZHc3ZDNqc0RGemZDQjlnbVh5MDVrY3dT?= =?utf-8?B?WVJqMFI0T2JZQ3lEcVZwV0hYUlJyNG05TzBnS3RRd3BjcTVndUV2RTFOWkg3?= =?utf-8?B?ZGEzd2xGN1RpVzRrdFR4OFgybUxlZ3YwNFl6NzhBUGFydTVJWklDQjhyZmZ6?= =?utf-8?B?VWRLRnpEZ3RmN3FaMnZheEhCOWZzZlNBNG5sQmpTaThNdDdHTTd0MGlib2Ni?= =?utf-8?B?RUwxMk5xeWhUSWduaGNEOGRCVWQwdzJtYkpUQ3d4Zmh0Y1lURWFYUDQ3eUFJ?= =?utf-8?B?YmszY2RFMC9NQzJUNGdpZ3hITDBxRlJLUDVzQXdqSjV0V0ZzZVRRdFVYcjgr?= =?utf-8?B?bTZ4TjVETWNEaThyTzNqaVRqRjVZNVJFemZxY09GcVRVOHdWMGNCalN1MitD?= =?utf-8?B?KzlKaWpSZUt3eXN5T0ZFV0l2WTZtcUJSZzRad0JyWnRrUEIranFzdTlDN25y?= =?utf-8?B?Sk1sN2hKVEovSHJSRnhXRzgxVW9JRFBxMkJpK205K3dSbTd3MHM4aGhNQ0J2?= =?utf-8?B?YXB3SGQyN21hYUtMcHFPMEhST2o2dWNHMG4vWnZnWURKbTExbWdUOVF4VnVM?= =?utf-8?B?RkNFbm50V1Z1eW1oL2YrYy9ZcFRUVGt5dDRFcEJ4WWdReWNDZllKTmxyVEE4?= =?utf-8?B?ME4zUHplYjFpSng1dERmVndJeUFxU2hPUUVDL1hvUVYwVFU5L3g3ZkoxMHA2?= =?utf-8?B?T0p1dGNZbHp6dThXc2ZpaU8rMllBaTl5VVloWG52TjRLdjVaTE4vRHp3YmIw?= =?utf-8?B?YlgrOXBGSG14b1RIQVU3WDZScjluR2QrZDNtbkNxVWsyd0JWVmYyWkd6cmZN?= =?utf-8?B?cG5GRjJDelBpU1ZJajRTSm42RGpVSkpGWUt5aFZrL0swU1d6SjJJY0ZOYkcw?= =?utf-8?Q?XvRsOUqoy3BTqFL0xMgklUIbb?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9be84a6a-3d18-40b9-17ed-08db70ddf939 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jun 2023 15:58:07.6602 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: jmecmV8waqbAU5cDrLNJ0gsH/ukDd7yYqFe9hZgl4g90O5k1BSa2KuyAPYUI3JZ9 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4143 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 On 6/7/2023 2:02 PM, Ivan Malov wrote: > From: Denis Pryazhennikov > > New MCDI Table Access API allows management of > the HW tables' content. > This part of API helps to list all supported tables. > In the near future, only the CT table is planned > to be used, so only one identifier for this table > was added to the efx. > New table IDs will be added as needed. > > Signed-off-by: Denis Pryazhennikov > Reviewed-by: Andy Moreton > This patch adds a function to the base code, but it is disconnected from the context. In the future if someone looks this function in git log, there is no easy way to see why this function added and where/how it is used at time it is added etc.. So, instead of making commit per function, can you please split commits based on functionality/logic? Please combine the commit that new function and commit where new function is used to single commit, making a commit per feature? If you are concerned about checkpatch warnings related to the component (like common/sfc_efx/base), please ignore it for the case when a feature is distributed into multiple components, and feel free to use most appropriate component name, I assume it will be driver component (net/sfc) most of the times. There is apply errors on CI, which prevents CI checks, can you please rebase set on top of latest head? Btw, we call 'API' to end-user facing functions, that user directly call, for this context better to call it 'function', but after patches merged probably you won't need it at all. Thanks, Ferruh