[dpdk-test-report] |WARNING| pw80533 [PATCH 31/36] net/sfc: support per-queue Rx prefix for EF100

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Oct 13 15:55:57 CEST 2020


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

_coding style issues_


WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#184: FILE: drivers/net/sfc/sfc_ef100_rx.c:732:
+		return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#193: FILE: drivers/net/sfc/sfc_ef100_rx.c:741:
+		return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#253: FILE: drivers/net/sfc/sfc_ef10_essb_rx.c:664:
+		return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#257: FILE: drivers/net/sfc/sfc_ef10_essb_rx.c:668:
+		return ENOTSUP;

WARNING:TYPO_SPELLING: 'accesssors' may be misspelled - perhaps 'accessors'?
#271: FILE: drivers/net/sfc/sfc_ef10_rx.c:227:
+ * Below Rx pseudo-header (aka Rx prefix) accesssors rely on the

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#277: FILE: drivers/net/sfc/sfc_ef10_rx.c:233:
+		[EFX_RX_PREFIX_FIELD_RSS_HASH]	=
+		    { 0, sizeof(uint32_t) * CHAR_BIT, B_FALSE },

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#279: FILE: drivers/net/sfc/sfc_ef10_rx.c:235:
+		[EFX_RX_PREFIX_FIELD_LENGTH]	=
+		    { 8 * CHAR_BIT, sizeof(uint16_t) * CHAR_BIT, B_FALSE },

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#301: FILE: drivers/net/sfc/sfc_ef10_rx.c:726:
+		return ENOTSUP;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#329: FILE: drivers/net/sfc/sfc_rx.c:545:
+		return ENOTSUP;

total: 2 errors, 7 warnings, 0 checks, 221 lines checked


More information about the test-report mailing list