[dpdk-test-report] |WARNING| pw88167 [PATCH 05/11] net/bnxt: update HWRM structures

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Feb 24 16:34:40 CET 2021


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/88167

_coding style issues_


WARNING:TYPO_SPELLING: 'hsi' may be misspelled - perhaps 'his'?
#59: 
Brought in the latest hsi_struct_def_dpdk.h.

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#629: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:11974:
+	 * If set to 0xFF... (All Fs), then the the configuration is

WARNING:REPEATED_WORD: Possible repeated word: 'block'
#697: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:12674:
+	 * block block. When this bit is ‘0’, this flag has no effect.

WARNING:REPEATED_WORD: Possible repeated word: 'block'
#706: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:12688:
+	 * block block. When this bit is ‘0’, this flag has no effect.

WARNING:TYPO_SPELLING: 'TIM' may be misspelled - perhaps 'TIME'?
#1215: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:15704:
+	#define HWRM_FUNC_BACKING_STORE_QCFG_OUTPUT_ENABLES_TIM \

WARNING:TYPO_SPELLING: 'chooose' may be misspelled - perhaps 'choose'?
#1930: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:18044:
+	 * The user must chooose a value that is not conflicting with

WARNING:TYPO_SPELLING: 'chooose' may be misspelled - perhaps 'choose'?
#2189: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:18303:
+	 * The user must chooose a value that is not conflicting with

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#3252: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:29818:
+	 * conflict with the the CoS to VNIC steering rules in this mode.

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#5188: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:41315:
+	 * When set to 1, indicates the the FW supports the Centralized

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#5196: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:41322:
+	 * When set to 1, indicates the the FW supports the Detached

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#5488: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:43611:
+	 * When set to 1, indicates the the FW supports the Centralized

WARNING:REPEATED_WORD: Possible repeated word: 'the'
#5496: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:43618:
+	 * When set to 1, indicates the the FW supports the Detached

WARNING:LONG_LINE_COMMENT: line length of 134 exceeds 100 columns
#5688: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:46991:
+	/* The Directory Entry Type (valid values are defined in the bnxnvm_directory_type enum defined in the file bnxnvm_defs.h). */

WARNING:LONG_LINE_COMMENT: line length of 118 exceeds 100 columns
#5700: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47003:
+	 * Length of data to write, in bytes. May be less than or equal to the allocated size for the directory entry.

WARNING:LONG_LINE_COMMENT: line length of 122 exceeds 100 columns
#5701: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47004:
+	 * The data length stored in the directory entry will be updated to reflect this value once the write is complete.

WARNING:LONG_LINE_COMMENT: line length of 164 exceeds 100 columns
#5711: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47017:
+	 * The requested length of the allocated NVM for the item, in bytes. This value may be greater than or equal to the specified data length (dir_data_length).

WARNING:LONG_LINE_COMMENT: line length of 123 exceeds 100 columns
#5719: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47019:
+	 * The response will contain the actual allocated item length, which may be greater than the requested item length.

WARNING:LONG_LINE_COMMENT: line length of 153 exceeds 100 columns
#5720: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47020:
+	 * The purpose for allocating more than the required number of bytes for an item's data is to pre-allocate extra storage (padding) to accommodate

WARNING:LONG_LINE_COMMENT: line length of 137 exceeds 100 columns
#5721: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47021:
+	 * the potential future growth of an item (e.g. upgraded firmware with a size increase, log growth, expanded configuration data).

WARNING:LONG_LINE_COMMENT: line length of 160 exceeds 100 columns
#5734: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47038:
+	 * Length of the allocated NVM for the item, in bytes. The value may be greater than or equal to the specified data length or the requested item length.

WARNING:LONG_LINE_COMMENT: line length of 115 exceeds 100 columns
#5735: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47039:
+	 * The actual item length used when creating a new directory entry will be a multiple of an NVM block size.

WARNING:LONG_LINE_COMMENT: line length of 133 exceeds 100 columns
#5747: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47383:
+	/* Available size that can be used, in bytes.  Available size is the NVRAM size take away the used size and reserved size. */

WARNING:LONG_LINE_COMMENT: line length of 101 exceeds 100 columns
#5759: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47525:
+	/* The Directory Entry Extension flags (see BNX_DIR_EXT_* for extension flag definitions). */

WARNING:LONG_LINE_COMMENT: line length of 104 exceeds 100 columns
#5777: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:47694:
+	 * If set to 1, then unspecified images, images not in the package file, will be safely deleted.

WARNING:LONG_LINE_COMMENT: line length of 111 exceeds 100 columns
#5797: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:48080:
+	/* When this bit is 1, flush internal cache after this write operation (see hwrm_nvm_flush command.) */

WARNING:LONG_LINE_COMMENT: line length of 107 exceeds 100 columns
#5809: FILE: drivers/net/bnxt/hsi_struct_def_dpdk.h:48382:
+	/* AP processor complex (in multi-host environment). Use host_idx to control which core is reset */

total: 0 errors, 26 warnings, 0 checks, 5529 lines checked


More information about the test-report mailing list