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 E4658A0564; Fri, 12 Mar 2021 01:27:00 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5F2114067E; Fri, 12 Mar 2021 01:27:00 +0100 (CET) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mails.dpdk.org (Postfix) with ESMTP id A78304014D for ; Fri, 12 Mar 2021 01:26:58 +0100 (CET) IronPort-SDR: aYldlNMgYcJ1n7z9tcmvUsF5nj3yrhMxqiRPmXbtpq+qE6SZZ4PWAk2z2hMTVLYsG151GtZgk7 fxWC0v2gPBVg== X-IronPort-AV: E=McAfee;i="6000,8403,9920"; a="168678544" X-IronPort-AV: E=Sophos;i="5.81,241,1610438400"; d="scan'208";a="168678544" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Mar 2021 16:26:53 -0800 IronPort-SDR: 5kBSjo9KYjfbLrXx3pzYS0QH9+T1N1AJUM2pWBuhQZFlqiguBi1b1AVOFAKsA5WZNRRZ5gDC/f KJs8a3LQ7Xtg== X-IronPort-AV: E=Sophos;i="5.81,241,1610438400"; d="scan'208";a="404252215" Received: from bcarpe2x-mobl4.ger.corp.intel.com (HELO [10.252.22.85]) ([10.252.22.85]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Mar 2021 16:26:52 -0800 To: Ajit Khaparde Cc: dpdk-dev , Kalesh Anakkur Purayil References: <20210310212637.72205-1-ajit.khaparde@broadcom.com> <20210310212637.72205-7-ajit.khaparde@broadcom.com> From: Ferruh Yigit X-User: ferruhy Message-ID: <38bd6a53-8a74-9336-47af-c1d68d06c8ac@intel.com> Date: Fri, 12 Mar 2021 00:26:49 +0000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 06/12] net/bnxt: update HWRM structures 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 Sender: "dev" On 3/12/2021 12:17 AM, Ajit Khaparde wrote: > On Thu, Mar 11, 2021 at 4:08 PM Ferruh Yigit wrote: >> >> On 3/10/2021 9:26 PM, Ajit Khaparde wrote: >>> From: Kalesh AP >>> >>> Patch brings in the latest hsi_struct_def_dpdk.h. >>> HWRM API is now updated to version 1.10.2.15. >>> >>> Signed-off-by: Kalesh AP >>> Reviewed-by: Ajit Khaparde >>> --- >>> doc/guides/rel_notes/release_21_05.rst | 4 + >>> drivers/net/bnxt/hsi_struct_def_dpdk.h | 3689 +++++++++++++++++++----- >>> 2 files changed, 2972 insertions(+), 721 deletions(-) >>> >>> diff --git a/doc/guides/rel_notes/release_21_05.rst b/doc/guides/rel_notes/release_21_05.rst >>> index 9fc5e0d6ca..d5cdb2cd2c 100644 >>> --- a/doc/guides/rel_notes/release_21_05.rst >>> +++ b/doc/guides/rel_notes/release_21_05.rst >>> @@ -80,6 +80,10 @@ New Features >>> >>> * Added support for preferred busy polling. >>> >>> +* **Updated Broadcom bnxt driver.** >>> + >>> + * Updated HWRM structures to 1.10.2.15 version. >>> + >> >> Can you please chage the order in the release notes, the order is: >> ethdev library, pysical PMDs (alphabetical order for vendor), vitual PMDs, apps >> (like testpms) >> Acording this order, in current release notes the PMD should be first item in >> the section. > Ok. Will do. > >> >>> @@ -45297,8 +47545,8 @@ struct hwrm_nvm_mod_dir_entry_input { >>> */ >>> uint16_t dir_ext; >>> /* >>> - * Directory Entry Attribute flags (see BNX_DIR_ATTR_* for attribute flag >>> - * definitions). >>> + * Directory Entry Attribute flags (see BNX_DIR_ATTR_* for attribute >>> + * flag definitions). >>> */ >>> uint16_t dir_attr; >>> /* >>> @@ -45474,8 +47722,7 @@ struct hwrm_nvm_install_update_input { >>> #define HWRM_NVM_INSTALL_UPDATE_INPUT_FLAGS_REMOVE_UNUSED_PKG \ >>> UINT32_C(0x2) >>> /* >>> - * If set to 1, FW will defragment the NVM if defragmentation is required >>> - * for the update. >>> + * If set to 1, FW will defragment the NVM if defragmentation is required for the update. >>> * Allow additional time for this command to complete if this bit is set to 1. >>> */ >>> #define HWRM_NVM_INSTALL_UPDATE_INPUT_FLAGS_ALLOWED_TO_DEFRAG \ >>> >> >> These changes are only because of word wrapping, there are many samples in the >> patch. >> I guess the original code has long lines, as the first version, and when they >> are converted to shorter lines, it has been done differently from previous time >> causing these diffs. >> Can you keep the previous version wrapping to prevent the unnecesarry diffs? > Ideally I wanted to get this from the auto-generator script. Agree that is better for long term. > But that was going to take time, so I edited the file manually. > And yes, I modified a few extra lines which checkpatch was not > complaining about. I think we can tolerate a few checkpatch warnings for base code changes if the line length is reasonable and if it prevents modifying exiting line. First version was converting wrapped lines back to long lines and they were really long, so that was hard to tolerate. > I will send another version to minimize the changes. > Thank you.