[dpdk-test-report] |WARNING| [PATCH v2 10/55] net/sfc: import libefx SFN7xxx family support
checkpatch at dpdk.org
checkpatch at dpdk.org
Tue Nov 29 17:33:33 CET 2016
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/17314
_coding style issues_
CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#172: FILE: drivers/net/sfc/base/ef10_ev.c:45:
+static __checkReturn boolean_t
CHECK:SPACING: No space is necessary after a cast
#220: FILE: drivers/net/sfc/base/ef10_ev.c:93:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#220: FILE: drivers/net/sfc/base/ef10_ev.c:93:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#244: FILE: drivers/net/sfc/base/ef10_ev.c:117:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#251: FILE: drivers/net/sfc/base/ef10_ev.c:124:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#283: FILE: drivers/net/sfc/base/ef10_ev.c:156:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#283: FILE: drivers/net/sfc/base/ef10_ev.c:156:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#336: FILE: drivers/net/sfc/base/ef10_ev.c:209:
+ if ((rc = efx_ev_usecs_to_ticks(enp, us, &ticks)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#375: FILE: drivers/net/sfc/base/ef10_ev.c:248:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#388: FILE: drivers/net/sfc/base/ef10_ev.c:261:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#420: FILE: drivers/net/sfc/base/ef10_ev.c:293:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#420: FILE: drivers/net/sfc/base/ef10_ev.c:293:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#463: FILE: drivers/net/sfc/base/ef10_ev.c:336:
+ if ((rc = efx_ev_usecs_to_ticks(enp, us, &ticks)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#505: FILE: drivers/net/sfc/base/ef10_ev.c:378:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#518: FILE: drivers/net/sfc/base/ef10_ev.c:391:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#531: FILE: drivers/net/sfc/base/ef10_ev.c:404:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#531: FILE: drivers/net/sfc/base/ef10_ev.c:404:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#547: FILE: drivers/net/sfc/base/ef10_ev.c:420:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#552: FILE: drivers/net/sfc/base/ef10_ev.c:425:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#562: FILE: drivers/net/sfc/base/ef10_ev.c:435:
+ return (0);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#583: FILE: drivers/net/sfc/base/ef10_ev.c:456:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#662: FILE: drivers/net/sfc/base/ef10_ev.c:535:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#675: FILE: drivers/net/sfc/base/ef10_ev.c:548:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#687: FILE: drivers/net/sfc/base/ef10_ev.c:560:
+ (void) efx_mcdi_fini_evq(eep->ee_enp, eep->ee_index);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#728: FILE: drivers/net/sfc/base/ef10_ev.c:601:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#762: FILE: drivers/net/sfc/base/ef10_ev.c:635:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#767: FILE: drivers/net/sfc/base/ef10_ev.c:640:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#783: FILE: drivers/net/sfc/base/ef10_ev.c:656:
+ (void) efx_mcdi_driver_event(enp, eep->ee_index, event);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#792: FILE: drivers/net/sfc/base/ef10_ev.c:665:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
WARNING:BRACES: braces {} are not necessary for any arm of this statement
#813: FILE: drivers/net/sfc/base/ef10_ev.c:686:
+ if (us == 0) {
[...]
+ } else {
[...]
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#828: FILE: drivers/net/sfc/base/ef10_ev.c:701:
+ if ((rc = efx_ev_usecs_to_ticks(enp, us, &ticks)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#848: FILE: drivers/net/sfc/base/ef10_ev.c:721:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#857: FILE: drivers/net/sfc/base/ef10_ev.c:730:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#887: FILE: drivers/net/sfc/base/ef10_ev.c:760:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#903: FILE: drivers/net/sfc/base/ef10_ev.c:776:
+ return (B_FALSE);
WARNING:BRACES: braces {} are not necessary for any arm of this statement
#972: FILE: drivers/net/sfc/base/ef10_ev.c:845:
+ if (EFX_QWORD_FIELD(*eqp, ESF_DZ_RX_IPCKSUM_ERR)) {
[...]
+ } else {
[...]
WARNING:BRACES: braces {} are not necessary for any arm of this statement
#1010: FILE: drivers/net/sfc/base/ef10_ev.c:883:
+ if (EFX_QWORD_FIELD(*eqp, ESF_DZ_RX_TCPUDP_CKSUM_ERR)) {
[...]
+ } else {
[...]
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1025: FILE: drivers/net/sfc/base/ef10_ev.c:898:
+ return (should_abort);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1044: FILE: drivers/net/sfc/base/ef10_ev.c:917:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1048: FILE: drivers/net/sfc/base/ef10_ev.c:921:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1060: FILE: drivers/net/sfc/base/ef10_ev.c:933:
+ return (should_abort);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1110: FILE: drivers/net/sfc/base/ef10_ev.c:983:
+ return (should_abort);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1132: FILE: drivers/net/sfc/base/ef10_ev.c:1005:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1138: FILE: drivers/net/sfc/base/ef10_ev.c:1011:
+ return (should_abort);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1315: FILE: drivers/net/sfc/base/ef10_ev.c:1188:
+ return (should_abort);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1414: FILE: drivers/net/sfc/base/ef10_filter.c:55:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1416: FILE: drivers/net/sfc/base/ef10_filter.c:57:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1425: FILE: drivers/net/sfc/base/ef10_filter.c:66:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1427: FILE: drivers/net/sfc/base/ef10_filter.c:68:
+ return (B_FALSE);
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'match' - possible side-effects?
#1483: FILE: drivers/net/sfc/base/ef10_filter.c:124:
+#define MATCH_MASK(match) (EFX_MASK32(match) << EFX_LOW_BIT(match))
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1506: FILE: drivers/net/sfc/base/ef10_filter.c:147:
+ EFSYS_KMEM_ALLOC(enp->en_esip, sizeof (ef10_filter_table_t), eftp);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1515: FILE: drivers/net/sfc/base/ef10_filter.c:156:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1520: FILE: drivers/net/sfc/base/ef10_filter.c:161:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1531: FILE: drivers/net/sfc/base/ef10_filter.c:172:
+ EFSYS_KMEM_FREE(enp->en_esip, sizeof (ef10_filter_table_t),
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1549: FILE: drivers/net/sfc/base/ef10_filter.c:190:
+ memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1635: FILE: drivers/net/sfc/base/ef10_filter.c:276:
+ EFX_STATIC_ASSERT(sizeof (spec->efs_rem_host) ==
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1637: FILE: drivers/net/sfc/base/ef10_filter.c:278:
+ EFX_STATIC_ASSERT(sizeof (spec->efs_loc_host) ==
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1663: FILE: drivers/net/sfc/base/ef10_filter.c:304:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1672: FILE: drivers/net/sfc/base/ef10_filter.c:313:
+ return (rc);
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#1674: FILE: drivers/net/sfc/base/ef10_filter.c:315:
+
+}
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1687: FILE: drivers/net/sfc/base/ef10_filter.c:328:
+ memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1724: FILE: drivers/net/sfc/base/ef10_filter.c:365:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1734: FILE: drivers/net/sfc/base/ef10_filter.c:375:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1744: FILE: drivers/net/sfc/base/ef10_filter.c:385:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1746: FILE: drivers/net/sfc/base/ef10_filter.c:387:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1748: FILE: drivers/net/sfc/base/ef10_filter.c:389:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1750: FILE: drivers/net/sfc/base/ef10_filter.c:391:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1752: FILE: drivers/net/sfc/base/ef10_filter.c:393:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1754: FILE: drivers/net/sfc/base/ef10_filter.c:395:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1756: FILE: drivers/net/sfc/base/ef10_filter.c:397:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1758: FILE: drivers/net/sfc/base/ef10_filter.c:399:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1760: FILE: drivers/net/sfc/base/ef10_filter.c:401:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1762: FILE: drivers/net/sfc/base/ef10_filter.c:403:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1764: FILE: drivers/net/sfc/base/ef10_filter.c:405:
+ return (B_FALSE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1766: FILE: drivers/net/sfc/base/ef10_filter.c:407:
+ return (B_TRUE);
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#1768: FILE: drivers/net/sfc/base/ef10_filter.c:409:
+
+}
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1778: FILE: drivers/net/sfc/base/ef10_filter.c:419:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1782: FILE: drivers/net/sfc/base/ef10_filter.c:423:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1784: FILE: drivers/net/sfc/base/ef10_filter.c:425:
+ return (B_FALSE);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1791: FILE: drivers/net/sfc/base/ef10_filter.c:432:
+ EFX_STATIC_ASSERT((sizeof (efx_filter_spec_t) % sizeof (uint32_t))
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1791: FILE: drivers/net/sfc/base/ef10_filter.c:432:
+ EFX_STATIC_ASSERT((sizeof (efx_filter_spec_t) % sizeof (uint32_t))
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1794: FILE: drivers/net/sfc/base/ef10_filter.c:435:
+ sizeof (uint32_t)) == 0);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1802: FILE: drivers/net/sfc/base/ef10_filter.c:443:
+ (sizeof (efx_filter_spec_t) -
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1804: FILE: drivers/net/sfc/base/ef10_filter.c:445:
+ sizeof (uint32_t), 0));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1819: FILE: drivers/net/sfc/base/ef10_filter.c:460:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1826: FILE: drivers/net/sfc/base/ef10_filter.c:467:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1829: FILE: drivers/net/sfc/base/ef10_filter.c:470:
+ return (B_TRUE);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1832: FILE: drivers/net/sfc/base/ef10_filter.c:473:
+ return (B_FALSE);
CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#1850: FILE: drivers/net/sfc/base/ef10_filter.c:491:
+ for (tbl_id = 0; tbl_id < EFX_EF10_FILTER_TBL_ROWS; tbl_id++) {
+
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1889: FILE: drivers/net/sfc/base/ef10_filter.c:530:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#1894: FILE: drivers/net/sfc/base/ef10_filter.c:535:
+ return (rc);
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1947: FILE: drivers/net/sfc/base/ef10_filter.c:588:
+ if (ins_index < 0) {
+ ins_index = i;
+ }
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#2002: FILE: drivers/net/sfc/base/ef10_filter.c:643:
+
+ }
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2005: FILE: drivers/net/sfc/base/ef10_filter.c:646:
+ EFSYS_KMEM_ALLOC(enp->en_esip, sizeof (*spec), saved_spec);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2060: FILE: drivers/net/sfc/base/ef10_filter.c:701:
+ return (0);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2066: FILE: drivers/net/sfc/base/ef10_filter.c:707:
+ EFSYS_KMEM_FREE(enp->en_esip, sizeof (*spec), saved_spec);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2084: FILE: drivers/net/sfc/base/ef10_filter.c:725:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2099: FILE: drivers/net/sfc/base/ef10_filter.c:740:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2104: FILE: drivers/net/sfc/base/ef10_filter.c:745:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#2132: FILE: drivers/net/sfc/base/ef10_filter.c:773:
+ if ((spec = ef10_filter_entry_spec(table, filter_idx)) != NULL) {
WARNING:BRACES: braces {} are not necessary for single statement blocks
#2132: FILE: drivers/net/sfc/base/ef10_filter.c:773:
+ if ((spec = ef10_filter_entry_spec(table, filter_idx)) != NULL) {
+ ef10_filter_set_entry_busy(table, filter_idx);
+ }
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2162: FILE: drivers/net/sfc/base/ef10_filter.c:803:
+ EFSYS_KMEM_FREE(enp->en_esip, sizeof (*spec), spec);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2168: FILE: drivers/net/sfc/base/ef10_filter.c:809:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2176: FILE: drivers/net/sfc/base/ef10_filter.c:817:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2223: FILE: drivers/net/sfc/base/ef10_filter.c:864:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2234: FILE: drivers/net/sfc/base/ef10_filter.c:875:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#2251: FILE: drivers/net/sfc/base/ef10_filter.c:892:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2251: FILE: drivers/net/sfc/base/ef10_filter.c:892:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2281: FILE: drivers/net/sfc/base/ef10_filter.c:922:
+ (*length) * sizeof (uint32_t));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#2282: FILE: drivers/net/sfc/base/ef10_filter.c:923:
+ EFX_STATIC_ASSERT(sizeof (uint32_t) ==
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2307: FILE: drivers/net/sfc/base/ef10_filter.c:948:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2314: FILE: drivers/net/sfc/base/ef10_filter.c:955:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#2325: FILE: drivers/net/sfc/base/ef10_filter.c:966:
+ if ((rc = efx_mcdi_get_parser_disp_info(enp, list, length)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2328: FILE: drivers/net/sfc/base/ef10_filter.c:969:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2333: FILE: drivers/net/sfc/base/ef10_filter.c:974:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2361: FILE: drivers/net/sfc/base/ef10_filter.c:1002:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2365: FILE: drivers/net/sfc/base/ef10_filter.c:1006:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2391: FILE: drivers/net/sfc/base/ef10_filter.c:1032:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2395: FILE: drivers/net/sfc/base/ef10_filter.c:1036:
+ return (rc);
CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#2403: FILE: drivers/net/sfc/base/ef10_filter.c:1044:
+ __in_ecount(6*count) uint8_t const *addrs,
^
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#2450: FILE: drivers/net/sfc/base/ef10_filter.c:1091:
+
+ }
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2478: FILE: drivers/net/sfc/base/ef10_filter.c:1119:
+ return (0);
CHECK:SPACING: No space is necessary after a cast
#2484: FILE: drivers/net/sfc/base/ef10_filter.c:1125:
+ (void) ef10_filter_delete_internal(enp,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2492: FILE: drivers/net/sfc/base/ef10_filter.c:1133:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2522: FILE: drivers/net/sfc/base/ef10_filter.c:1163:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2527: FILE: drivers/net/sfc/base/ef10_filter.c:1168:
+ return (rc);
WARNING:BRACES: braces {} are not necessary for single statement blocks
#2538: FILE: drivers/net/sfc/base/ef10_filter.c:1179:
+ if (ef10_filter_entry_is_auto_old(table, i)) {
+ (void) ef10_filter_delete_internal(enp, i);
+ }
CHECK:SPACING: No space is necessary after a cast
#2539: FILE: drivers/net/sfc/base/ef10_filter.c:1180:
+ (void) ef10_filter_delete_internal(enp, i);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2571: FILE: drivers/net/sfc/base/ef10_filter.c:1212:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2576: FILE: drivers/net/sfc/base/ef10_filter.c:1217:
+ return (rc);
CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#2578: FILE: drivers/net/sfc/base/ef10_filter.c:1219:
+
+}
CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#2595: FILE: drivers/net/sfc/base/ef10_filter.c:1236:
+ __in_ecount(6*count) uint8_t const *addrs,
^
CHECK:SPACING: No space is necessary after a cast
#2614: FILE: drivers/net/sfc/base/ef10_filter.c:1255:
+ (void) ef10_filter_delete_internal(enp,
CHECK:SPACING: No space is necessary after a cast
#2620: FILE: drivers/net/sfc/base/ef10_filter.c:1261:
+ (void) ef10_filter_delete_internal(enp,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2625: FILE: drivers/net/sfc/base/ef10_filter.c:1266:
+ return (0);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#2695: FILE: drivers/net/sfc/base/ef10_filter.c:1336:
+ if ((rc = ef10_filter_get_workarounds(enp)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2768: FILE: drivers/net/sfc/base/ef10_filter.c:1409:
+ return (rc);
WARNING:BRACES: braces {} are not necessary for single statement blocks
#2781: FILE: drivers/net/sfc/base/ef10_filter.c:1422:
+ if (ef10_filter_entry_is_auto_old(table, i)) {
+ ef10_filter_set_entry_not_auto_old(table, i);
+ }
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#2786: FILE: drivers/net/sfc/base/ef10_filter.c:1427:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#3288: FILE: drivers/net/sfc/base/ef10_impl.h:454:
+#define EFX_PIOBUF_HANDLE_INVALID ((efx_piobuf_handle_t) -1)
CHECK:SPACING: spaces preferred around that '*' (ctx:VxV)
#3458: FILE: drivers/net/sfc/base/ef10_impl.h:624:
+ __in_ecount(6*count) uint8_t const *addrs,
^
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3602: FILE: drivers/net/sfc/base/ef10_intr.c:44:
+ return (0);
CHECK:SPACING: No space is necessary after a cast
#3648: FILE: drivers/net/sfc/base/ef10_intr.c:90:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#3648: FILE: drivers/net/sfc/base/ef10_intr.c:90:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3664: FILE: drivers/net/sfc/base/ef10_intr.c:106:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3672: FILE: drivers/net/sfc/base/ef10_intr.c:114:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#3680: FILE: drivers/net/sfc/base/ef10_intr.c:122:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#3692: FILE: drivers/net/sfc/base/ef10_intr.c:134:
+ if ((rc = efx_mcdi_trigger_interrupt(enp, level)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3695: FILE: drivers/net/sfc/base/ef10_intr.c:137:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3702: FILE: drivers/net/sfc/base/ef10_intr.c:144:
+ return (rc);
CHECK:SPACING: spaces preferred around that '*' (ctx:ExV)
#3737: FILE: drivers/net/sfc/base/ef10_intr.c:179:
+ *fatalp = B_FALSE;
^
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#3803: FILE: drivers/net/sfc/base/ef10_mac.c:42:
+ efx_port_t *epp = &(enp->en_port);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#3807: FILE: drivers/net/sfc/base/ef10_mac.c:46:
+ if ((rc = ef10_phy_get_link(enp, &els)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3815: FILE: drivers/net/sfc/base/ef10_mac.c:54:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3822: FILE: drivers/net/sfc/base/ef10_mac.c:61:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#3837: FILE: drivers/net/sfc/base/ef10_mac.c:76:
+ if ((rc = ef10_phy_get_link(enp, &els)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3842: FILE: drivers/net/sfc/base/ef10_mac.c:81:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3847: FILE: drivers/net/sfc/base/ef10_mac.c:86:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#3861: FILE: drivers/net/sfc/base/ef10_mac.c:100:
+ efx_port_t *epp = &(enp->en_port);
CHECK:SPACING: No space is necessary after a cast
#3867: FILE: drivers/net/sfc/base/ef10_mac.c:106:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#3867: FILE: drivers/net/sfc/base/ef10_mac.c:106:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3886: FILE: drivers/net/sfc/base/ef10_mac.c:125:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3891: FILE: drivers/net/sfc/base/ef10_mac.c:130:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#3900: FILE: drivers/net/sfc/base/ef10_mac.c:139:
+ if ((rc = efx_mcdi_vadapter_set_mac(enp)) != 0) {
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#3908: FILE: drivers/net/sfc/base/ef10_mac.c:147:
+ if ((rc = ef10_mac_reconfigure(enp)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3912: FILE: drivers/net/sfc/base/ef10_mac.c:151:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3920: FILE: drivers/net/sfc/base/ef10_mac.c:159:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#3933: FILE: drivers/net/sfc/base/ef10_mac.c:172:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#3933: FILE: drivers/net/sfc/base/ef10_mac.c:172:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3952: FILE: drivers/net/sfc/base/ef10_mac.c:191:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3957: FILE: drivers/net/sfc/base/ef10_mac.c:196:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#3970: FILE: drivers/net/sfc/base/ef10_mac.c:209:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#3970: FILE: drivers/net/sfc/base/ef10_mac.c:209:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#3997: FILE: drivers/net/sfc/base/ef10_mac.c:236:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4004: FILE: drivers/net/sfc/base/ef10_mac.c:243:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#4011: FILE: drivers/net/sfc/base/ef10_mac.c:250:
+ efx_port_t *epp = &(enp->en_port);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#4012: FILE: drivers/net/sfc/base/ef10_mac.c:251:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#4016: FILE: drivers/net/sfc/base/ef10_mac.c:255:
+ if ((rc = efx_mcdi_mtu_set(enp, epp->ep_mac_pdu)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#4024: FILE: drivers/net/sfc/base/ef10_mac.c:263:
+ if ((rc = ef10_mac_reconfigure(enp)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4028: FILE: drivers/net/sfc/base/ef10_mac.c:267:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4035: FILE: drivers/net/sfc/base/ef10_mac.c:274:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#4045: FILE: drivers/net/sfc/base/ef10_mac.c:284:
+ if ((rc = efx_mcdi_mtu_get(enp, pdu)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4048: FILE: drivers/net/sfc/base/ef10_mac.c:287:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4053: FILE: drivers/net/sfc/base/ef10_mac.c:292:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#4060: FILE: drivers/net/sfc/base/ef10_mac.c:299:
+ efx_port_t *epp = &(enp->en_port);
CHECK:SPACING: No space is necessary after a cast
#4066: FILE: drivers/net/sfc/base/ef10_mac.c:305:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4066: FILE: drivers/net/sfc/base/ef10_mac.c:305:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4127: FILE: drivers/net/sfc/base/ef10_mac.c:366:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4132: FILE: drivers/net/sfc/base/ef10_mac.c:371:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#4139: FILE: drivers/net/sfc/base/ef10_mac.c:378:
+ efx_port_t *epp = &(enp->en_port);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#4146: FILE: drivers/net/sfc/base/ef10_mac.c:385:
+ if ((rc = emop->emo_reconfigure(enp)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4149: FILE: drivers/net/sfc/base/ef10_mac.c:388:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4154: FILE: drivers/net/sfc/base/ef10_mac.c:393:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#4163: FILE: drivers/net/sfc/base/ef10_mac.c:402:
+ efx_port_t *epp = &(enp->en_port);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4181: FILE: drivers/net/sfc/base/ef10_mac.c:420:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4188: FILE: drivers/net/sfc/base/ef10_mac.c:427:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#4195: FILE: drivers/net/sfc/base/ef10_mac.c:434:
+ efx_port_t *epp = &(enp->en_port);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#4262: FILE: drivers/net/sfc/base/ef10_mcdi.c:49:
+ efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);
CHECK:SPACING: No space is necessary after a cast
#4297: FILE: drivers/net/sfc/base/ef10_mcdi.c:84:
+ (void) ef10_mcdi_poll_reboot(enp);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4302: FILE: drivers/net/sfc/base/ef10_mcdi.c:89:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4309: FILE: drivers/net/sfc/base/ef10_mcdi.c:96:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#4316: FILE: drivers/net/sfc/base/ef10_mcdi.c:103:
+ efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#4339: FILE: drivers/net/sfc/base/ef10_mcdi.c:126:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4378: FILE: drivers/net/sfc/base/ef10_mcdi.c:165:
+ for (pos = 0; pos < hdr_len; pos += sizeof (efx_dword_t)) {
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4384: FILE: drivers/net/sfc/base/ef10_mcdi.c:171:
+ for (pos = 0; pos < sdu_len; pos += sizeof (efx_dword_t)) {
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4414: FILE: drivers/net/sfc/base/ef10_mcdi.c:201:
+ return (EFX_DWORD_FIELD(hdr, MCDI_HEADER_RESPONSE) ? B_TRUE : B_FALSE);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4429: FILE: drivers/net/sfc/base/ef10_mcdi.c:216:
+ for (pos = 0; pos < length; pos += sizeof (efx_dword_t)) {
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4432: FILE: drivers/net/sfc/base/ef10_mcdi.c:219:
+ MIN(sizeof (data), length - pos));
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#4440: FILE: drivers/net/sfc/base/ef10_mcdi.c:227:
+ efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4475: FILE: drivers/net/sfc/base/ef10_mcdi.c:262:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4480: FILE: drivers/net/sfc/base/ef10_mcdi.c:267:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#4489: FILE: drivers/net/sfc/base/ef10_mcdi.c:276:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4545: FILE: drivers/net/sfc/base/ef10_mcdi.c:332:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4550: FILE: drivers/net/sfc/base/ef10_mcdi.c:337:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4612: FILE: drivers/net/sfc/base/ef10_nic.c:51:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4612: FILE: drivers/net/sfc/base/ef10_nic.c:51:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4633: FILE: drivers/net/sfc/base/ef10_nic.c:72:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4640: FILE: drivers/net/sfc/base/ef10_nic.c:79:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4657: FILE: drivers/net/sfc/base/ef10_nic.c:96:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4657: FILE: drivers/net/sfc/base/ef10_nic.c:96:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4693: FILE: drivers/net/sfc/base/ef10_nic.c:132:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4702: FILE: drivers/net/sfc/base/ef10_nic.c:141:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4743: FILE: drivers/net/sfc/base/ef10_nic.c:182:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4748: FILE: drivers/net/sfc/base/ef10_nic.c:187:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4763: FILE: drivers/net/sfc/base/ef10_nic.c:202:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4763: FILE: drivers/net/sfc/base/ef10_nic.c:202:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4782: FILE: drivers/net/sfc/base/ef10_nic.c:221:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4787: FILE: drivers/net/sfc/base/ef10_nic.c:226:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4800: FILE: drivers/net/sfc/base/ef10_nic.c:239:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4800: FILE: drivers/net/sfc/base/ef10_nic.c:239:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4816: FILE: drivers/net/sfc/base/ef10_nic.c:255:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4821: FILE: drivers/net/sfc/base/ef10_nic.c:260:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4837: FILE: drivers/net/sfc/base/ef10_nic.c:276:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4837: FILE: drivers/net/sfc/base/ef10_nic.c:276:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4870: FILE: drivers/net/sfc/base/ef10_nic.c:309:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4879: FILE: drivers/net/sfc/base/ef10_nic.c:318:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4895: FILE: drivers/net/sfc/base/ef10_nic.c:334:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4895: FILE: drivers/net/sfc/base/ef10_nic.c:334:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4933: FILE: drivers/net/sfc/base/ef10_nic.c:372:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4942: FILE: drivers/net/sfc/base/ef10_nic.c:381:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#4959: FILE: drivers/net/sfc/base/ef10_nic.c:398:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#4959: FILE: drivers/net/sfc/base/ef10_nic.c:398:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#4989: FILE: drivers/net/sfc/base/ef10_nic.c:428:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5000: FILE: drivers/net/sfc/base/ef10_nic.c:439:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5015: FILE: drivers/net/sfc/base/ef10_nic.c:454:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5015: FILE: drivers/net/sfc/base/ef10_nic.c:454:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5041: FILE: drivers/net/sfc/base/ef10_nic.c:480:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5048: FILE: drivers/net/sfc/base/ef10_nic.c:487:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5063: FILE: drivers/net/sfc/base/ef10_nic.c:502:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5063: FILE: drivers/net/sfc/base/ef10_nic.c:502:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5093: FILE: drivers/net/sfc/base/ef10_nic.c:532:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5100: FILE: drivers/net/sfc/base/ef10_nic.c:539:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5123: FILE: drivers/net/sfc/base/ef10_nic.c:562:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5123: FILE: drivers/net/sfc/base/ef10_nic.c:562:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5154: FILE: drivers/net/sfc/base/ef10_nic.c:593:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5163: FILE: drivers/net/sfc/base/ef10_nic.c:602:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5191: FILE: drivers/net/sfc/base/ef10_nic.c:630:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5196: FILE: drivers/net/sfc/base/ef10_nic.c:635:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5215: FILE: drivers/net/sfc/base/ef10_nic.c:654:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5215: FILE: drivers/net/sfc/base/ef10_nic.c:654:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5236: FILE: drivers/net/sfc/base/ef10_nic.c:675:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5245: FILE: drivers/net/sfc/base/ef10_nic.c:684:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5258: FILE: drivers/net/sfc/base/ef10_nic.c:697:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5258: FILE: drivers/net/sfc/base/ef10_nic.c:697:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5274: FILE: drivers/net/sfc/base/ef10_nic.c:713:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5279: FILE: drivers/net/sfc/base/ef10_nic.c:718:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5293: FILE: drivers/net/sfc/base/ef10_nic.c:732:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5293: FILE: drivers/net/sfc/base/ef10_nic.c:732:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5310: FILE: drivers/net/sfc/base/ef10_nic.c:749:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5315: FILE: drivers/net/sfc/base/ef10_nic.c:754:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#5328: FILE: drivers/net/sfc/base/ef10_nic.c:767:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5328: FILE: drivers/net/sfc/base/ef10_nic.c:767:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5344: FILE: drivers/net/sfc/base/ef10_nic.c:783:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5349: FILE: drivers/net/sfc/base/ef10_nic.c:788:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5441: FILE: drivers/net/sfc/base/ef10_nic.c:880:
+ EFSYS_ASSERT3U(blk_per_buf, <=, (8 * sizeof (*map)));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5459: FILE: drivers/net/sfc/base/ef10_nic.c:898:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5466: FILE: drivers/net/sfc/base/ef10_nic.c:905:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5480: FILE: drivers/net/sfc/base/ef10_nic.c:919:
+ (blknum >= (8 * sizeof (*map)))) {
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5492: FILE: drivers/net/sfc/base/ef10_nic.c:931:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5499: FILE: drivers/net/sfc/base/ef10_nic.c:938:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5508: FILE: drivers/net/sfc/base/ef10_nic.c:947:
+ return (efx_mcdi_link_piobuf(enp, vi_index, handle));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5516: FILE: drivers/net/sfc/base/ef10_nic.c:955:
+ return (efx_mcdi_unlink_piobuf(enp, vi_index));
CHECK:SPACING: No space is necessary after a cast
#5529: FILE: drivers/net/sfc/base/ef10_nic.c:968:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#5529: FILE: drivers/net/sfc/base/ef10_nic.c:968:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5553: FILE: drivers/net/sfc/base/ef10_nic.c:992:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5560: FILE: drivers/net/sfc/base/ef10_nic.c:999:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#5567: FILE: drivers/net/sfc/base/ef10_nic.c:1006:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5573: FILE: drivers/net/sfc/base/ef10_nic.c:1012:
+ if ((rc = efx_mcdi_get_capabilities(enp, &flags, &flags2,
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5577: FILE: drivers/net/sfc/base/ef10_nic.c:1016:
+ if ((rc = ef10_mcdi_get_pf_count(enp, &encp->enc_hw_pf_count)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5689: FILE: drivers/net/sfc/base/ef10_nic.c:1128:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5696: FILE: drivers/net/sfc/base/ef10_nic.c:1135:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#5721: FILE: drivers/net/sfc/base/ef10_nic.c:1160:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5725: FILE: drivers/net/sfc/base/ef10_nic.c:1164:
+ if ((rc = efx_mcdi_privilege_mask(enp, encp->enc_pf, encp->enc_vf,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5742: FILE: drivers/net/sfc/base/ef10_nic.c:1181:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5747: FILE: drivers/net/sfc/base/ef10_nic.c:1186:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5837: FILE: drivers/net/sfc/base/ef10_nic.c:1276:
+ if ((rc = efx_mcdi_get_port_modes(enp, &port_modes, ¤t)) != 0) {
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5842: FILE: drivers/net/sfc/base/ef10_nic.c:1281:
+ if ((rc = efx_mcdi_get_port_modes(enp,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5884: FILE: drivers/net/sfc/base/ef10_nic.c:1323:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5889: FILE: drivers/net/sfc/base/ef10_nic.c:1328:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#5898: FILE: drivers/net/sfc/base/ef10_nic.c:1337:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_drv_cfg
#5899: FILE: drivers/net/sfc/base/ef10_nic.c:1338:
+ efx_drv_cfg_t *edcp = &(enp->en_drv_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5906: FILE: drivers/net/sfc/base/ef10_nic.c:1345:
+ if ((rc = efx_mcdi_read_assertion(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5910: FILE: drivers/net/sfc/base/ef10_nic.c:1349:
+ if ((rc = efx_mcdi_exit_assertion_handler(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5914: FILE: drivers/net/sfc/base/ef10_nic.c:1353:
+ if ((rc = efx_mcdi_drv_attach(enp, B_TRUE)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#5917: FILE: drivers/net/sfc/base/ef10_nic.c:1356:
+ if ((rc = enop->eno_board_cfg(enp)) != 0)
CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided
#5928: FILE: drivers/net/sfc/base/ef10_nic.c:1367:
+ edcp->edc_min_vi_count = edcp->edc_max_vi_count =
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5937: FILE: drivers/net/sfc/base/ef10_nic.c:1376:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#5948: FILE: drivers/net/sfc/base/ef10_nic.c:1387:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#5956: FILE: drivers/net/sfc/base/ef10_nic.c:1395:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_drv_cfg
#5957: FILE: drivers/net/sfc/base/ef10_nic.c:1396:
+ efx_drv_cfg_t *edcp = &(enp->en_drv_cfg);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6017: FILE: drivers/net/sfc/base/ef10_nic.c:1456:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6022: FILE: drivers/net/sfc/base/ef10_nic.c:1461:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6036: FILE: drivers/net/sfc/base/ef10_nic.c:1475:
+ if ((rc = efx_mcdi_read_assertion(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6038: FILE: drivers/net/sfc/base/ef10_nic.c:1477:
+ if ((rc = efx_mcdi_exit_assertion_handler(enp)) != 0)
CHECK:SPACING: No space is necessary after a cast
#6041: FILE: drivers/net/sfc/base/ef10_nic.c:1480:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6041: FILE: drivers/net/sfc/base/ef10_nic.c:1480:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6061: FILE: drivers/net/sfc/base/ef10_nic.c:1500:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6070: FILE: drivers/net/sfc/base/ef10_nic.c:1509:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_drv_cfg
#6077: FILE: drivers/net/sfc/base/ef10_nic.c:1516:
+ efx_drv_cfg_t *edcp = &(enp->en_drv_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6089: FILE: drivers/net/sfc/base/ef10_nic.c:1528:
+ if ((rc = efx_mcdi_log_ctrl(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6113: FILE: drivers/net/sfc/base/ef10_nic.c:1552:
+ if ((rc = efx_mcdi_free_vis(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6121: FILE: drivers/net/sfc/base/ef10_nic.c:1560:
+ if ((rc = efx_mcdi_alloc_vis(enp, min_vi_count, max_vi_count,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6215: FILE: drivers/net/sfc/base/ef10_nic.c:1654:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6233: FILE: drivers/net/sfc/base/ef10_nic.c:1672:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6250: FILE: drivers/net/sfc/base/ef10_nic.c:1689:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6288: FILE: drivers/net/sfc/base/ef10_nic.c:1727:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6293: FILE: drivers/net/sfc/base/ef10_nic.c:1732:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#6303: FILE: drivers/net/sfc/base/ef10_nic.c:1742:
+ (void) efx_mcdi_vadaptor_free(enp, enp->en_vport_id);
CHECK:SPACING: No space is necessary after a cast
#6318: FILE: drivers/net/sfc/base/ef10_nic.c:1757:
+ (void) efx_mcdi_free_vis(enp);
CHECK:SPACING: No space is necessary after a cast
#6326: FILE: drivers/net/sfc/base/ef10_nic.c:1765:
+ (void) efx_mcdi_drv_attach(enp, B_FALSE);
CHECK:BRACES: braces {} should be used on all arms of this statement
#6437: FILE: drivers/net/sfc/base/ef10_phy.c:101:
+ if (fcntl == MC_CMD_FCNTL_OFF)
[...]
+ else if (fcntl == MC_CMD_FCNTL_RESPOND)
[...]
+ else if (fcntl == MC_CMD_FCNTL_GENERATE)
[...]
+ else if (fcntl == MC_CMD_FCNTL_BIDIR)
[...]
+ else {
[...]
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#6458: FILE: drivers/net/sfc/base/ef10_phy.c:122:
+ efx_port_t *epp = &(enp->en_port);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6519: FILE: drivers/net/sfc/base/ef10_phy.c:183:
+ return (0);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6522: FILE: drivers/net/sfc/base/ef10_phy.c:186:
+ if ((rc = ef10_phy_verify(enp)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6527: FILE: drivers/net/sfc/base/ef10_phy.c:191:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6532: FILE: drivers/net/sfc/base/ef10_phy.c:196:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#6545: FILE: drivers/net/sfc/base/ef10_phy.c:209:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6545: FILE: drivers/net/sfc/base/ef10_phy.c:209:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6576: FILE: drivers/net/sfc/base/ef10_phy.c:240:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6583: FILE: drivers/net/sfc/base/ef10_phy.c:247:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#6590: FILE: drivers/net/sfc/base/ef10_phy.c:254:
+ efx_port_t *epp = &(enp->en_port);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#6600: FILE: drivers/net/sfc/base/ef10_phy.c:264:
+ if ((rc = efx_mcdi_link_control_supported(enp, &supported)) != 0)
CHECK:SPACING: No space is necessary after a cast
#6605: FILE: drivers/net/sfc/base/ef10_phy.c:269:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6605: FILE: drivers/net/sfc/base/ef10_phy.c:269:
+ (void) memset(payload, 0, sizeof (payload));
CHECK:SPACING: No space is necessary after a cast
#6642: FILE: drivers/net/sfc/base/ef10_phy.c:306:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6642: FILE: drivers/net/sfc/base/ef10_phy.c:306:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6658: FILE: drivers/net/sfc/base/ef10_phy.c:322:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6667: FILE: drivers/net/sfc/base/ef10_phy.c:331:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#6680: FILE: drivers/net/sfc/base/ef10_phy.c:344:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6680: FILE: drivers/net/sfc/base/ef10_phy.c:344:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6707: FILE: drivers/net/sfc/base/ef10_phy.c:371:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6716: FILE: drivers/net/sfc/base/ef10_phy.c:380:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6726: FILE: drivers/net/sfc/base/ef10_phy.c:390:
+ return (ENOTSUP);
WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOTSUP)
#6726: FILE: drivers/net/sfc/base/ef10_phy.c:390:
+ return (ENOTSUP);
CHECK:SPACING: No space is necessary after a cast
#6803: FILE: drivers/net/sfc/base/ef10_rx.c:68:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6803: FILE: drivers/net/sfc/base/ef10_rx.c:68:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6846: FILE: drivers/net/sfc/base/ef10_rx.c:111:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6851: FILE: drivers/net/sfc/base/ef10_rx.c:116:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#6864: FILE: drivers/net/sfc/base/ef10_rx.c:129:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6864: FILE: drivers/net/sfc/base/ef10_rx.c:129:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6880: FILE: drivers/net/sfc/base/ef10_rx.c:145:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6885: FILE: drivers/net/sfc/base/ef10_rx.c:150:
+ return (rc);
CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#6893: FILE: drivers/net/sfc/base/ef10_rx.c:158:
+{
+
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6894: FILE: drivers/net/sfc/base/ef10_rx.c:159:
+ return (0);
CHECK:SPACING: spaces preferred around that '*' (ctx:ExV)
#6933: FILE: drivers/net/sfc/base/ef10_rx.c:198:
+ *lengthp = buffer[8] | (buffer[9] << 8);
^
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#6934: FILE: drivers/net/sfc/base/ef10_rx.c:199:
+ return (0);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around erp->er_mask
#6955: FILE: drivers/net/sfc/base/ef10_rx.c:220:
+ id = added & (erp->er_mask);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#6968: FILE: drivers/net/sfc/base/ef10_rx.c:233:
+ offset = id * sizeof (efx_qword_t);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around erp->er_mask
#6971: FILE: drivers/net/sfc/base/ef10_rx.c:236:
+ id = (id + 1) & (erp->er_mask);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#7013: FILE: drivers/net/sfc/base/ef10_rx.c:278:
+ if ((rc = efx_mcdi_fini_rxq(enp, erp->er_index)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#7016: FILE: drivers/net/sfc/base/ef10_rx.c:281:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#7021: FILE: drivers/net/sfc/base/ef10_rx.c:286:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#7045: FILE: drivers/net/sfc/base/ef10_rx.c:310:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#7086: FILE: drivers/net/sfc/base/ef10_rx.c:351:
+ if ((rc = efx_mcdi_init_rxq(enp, n, eep->ee_index, label, index,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#7095: FILE: drivers/net/sfc/base/ef10_rx.c:360:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#7106: FILE: drivers/net/sfc/base/ef10_rx.c:371:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#7122: FILE: drivers/net/sfc/base/ef10_rx.c:387:
+ EFSYS_KMEM_FREE(enp->en_esip, sizeof (efx_rxq_t), erp);
WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#7140: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:2:
+/**************************************************************************\
+*//*! ile
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#7148: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:10:
+*//*
+\**************************************************************************/
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7232: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:94:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7233: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:95:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7234: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:96:
+ uint16_t type_id;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7238: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:100:
+ * selected through an RFID command and copied by FW to the location at offset 0. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7239: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:101:
+ uint16_t preset;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7240: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:102:
+ uint32_t generation;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7241: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:103:
+ uint32_t total_length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7254: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:116:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7255: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:117:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7256: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:118:
+ uint32_t generation;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7257: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:119:
+ uint32_t checksum;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7271: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:133:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7272: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:134:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7273: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:135:
+ uint16_t type_id;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7274: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:136:
+ uint16_t reserved;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7290: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:152:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7291: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:153:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7292: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:154:
+ uint32_t flags;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7308: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:170:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7309: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:171:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7310: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:172:
+ uint8_t base_address[6];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7311: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:173:
+ uint16_t reserved;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7312: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:174:
+ uint16_t count;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7313: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:175:
+ uint16_t stride;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7328: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:190:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7329: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:191:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7330: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:192:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7336: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:198:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7337: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:199:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7338: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:200:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7353: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:215:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7354: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:216:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7355: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:217:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7361: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:223:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7362: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:224:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7363: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:225:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7378: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:240:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7379: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:241:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7380: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:242:
+ struct {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7381: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:243:
+ uint16_t addr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7382: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:244:
+ uint16_t byte_enables;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7383: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:245:
+ uint32_t value;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7384: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:246:
+ } items[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7391: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:253:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7392: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:254:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7393: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:255:
+ struct {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7394: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:256:
+ uint16_t addr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7395: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:257:
+ uint16_t byte_enables;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7396: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:258:
+ uint32_t value;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7397: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:259:
+ } items[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7415: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:277:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7416: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:278:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7417: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:279:
+ uint32_t subtype;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7418: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:280:
+ uint8_t description[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7432: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:294:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7433: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:295:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7434: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:296:
+ uint16_t version_w;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7435: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:297:
+ uint16_t version_x;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7436: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:298:
+ uint16_t version_y;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7437: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:299:
+ uint16_t version_z;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7445: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:307:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7446: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:308:
+ uint32_t length;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7447: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:309:
+ int16_t max_pf_number; /**< Largest PF RID (lower PFs may be hidden) */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7447: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:309:
+ int16_t max_pf_number; /**< Largest PF RID (lower PFs may be hidden) */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7448: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:310:
+ uint16_t pf_aper; /**< BIU aperture for PF BAR2 */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7449: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:311:
+ uint16_t vf_aper; /**< BIU aperture for VF BAR0 */$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7450: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:312:
+ uint16_t int_aper; /**< BIU aperture for PF BAR4 and VF BAR2 */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7450: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:312:
+ uint16_t int_aper; /**< BIU aperture for PF BAR4 and VF BAR2 */$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7458: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:320:
+ * tidy things up later */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7463: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:325:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7464: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:326:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7465: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:327:
+ uint8_t vfs_total;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7466: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:328:
+ uint8_t port_allocation;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7467: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:329:
+ uint16_t vectors_per_pf;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7468: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:330:
+ uint16_t vectors_per_vf;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7469: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:331:
+ uint8_t pf_bar0_aperture;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7470: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:332:
+ uint8_t pf_bar2_aperture;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7471: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:333:
+ uint8_t vf_bar0_aperture;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7472: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:334:
+ uint8_t vf_base;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7473: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:335:
+ uint16_t supp_pagesz;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7474: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:336:
+ uint16_t msix_vec_base;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7480: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:342:
+ * settings. This is a temporary measure. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7485: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:347:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7486: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:348:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7488: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:350:
+ uint64_t tx0_tags; /* Bitmap */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7489: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:351:
+ uint64_t tx1_tags; /* Bitmap */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7490: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:352:
+ uint64_t dl_tags; /* Bitmap */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7491: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:353:
+ uint32_t flags;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7498: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:360:
+ uint32_t dut_features; /* All 1s -> leave alone */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7499: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:361:
+ int8_t with_rmon; /* 0 -> off, 1 -> on, -1 -> leave alone */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7501: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:363:
+ int8_t clk_mode; /* 0 -> off, 1 -> on, -1 -> leave alone */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7503: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:365:
+ int8_t rx_dc_size; /* -1 -> leave alone */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7504: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:366:
+ int8_t tx_dc_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7505: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:367:
+ int16_t num_q_allocs;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7516: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:378:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7517: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:379:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7518: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:380:
+ uint32_t ports_per_core;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7519: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:381:
+ uint32_t max_port_speed;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7536: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:398:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7537: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:399:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7538: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:400:
+ uint32_t firmware_variant;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#7551: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:413:
+ MC_CMD_FW_PACKED_STREAM_HASH_MODE_1$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7551: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:413:
+ MC_CMD_FW_PACKED_STREAM_HASH_MODE_1$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7559: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:421:
+ * determined. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7564: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:426:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7565: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:427:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7566: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:428:
+ uint16_t flags; /* Boards with high 0v9 settings may need active cooling */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7568: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:430:
+ uint16_t target_voltage; /* In millivolts */$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7570: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:432:
+ * contain it) they need setting as well. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7571: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:433:
+ uint16_t warn_low; /* In millivolts */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7572: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:434:
+ uint16_t warn_high; /* In millivolts */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7573: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:435:
+ uint16_t panic_low; /* In millivolts */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7574: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:436:
+ uint16_t panic_high; /* In millivolts */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7584: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:446:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7585: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:447:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7586: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:448:
+ uint16_t clk_sys; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7587: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:449:
+ uint16_t clk_dpcpu; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7588: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:450:
+ uint16_t clk_icore; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7589: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:451:
+ uint16_t clk_pcs; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7595: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:457:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7596: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:458:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7597: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:459:
+ uint16_t clk_sys; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7598: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:460:
+ uint16_t clk_mc; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7599: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:461:
+ uint16_t clk_rmon; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7600: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:462:
+ uint16_t clk_vswitch; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7601: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:463:
+ uint16_t clk_dpcpu; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7602: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:464:
+ uint16_t clk_pcs; /* MHz */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7616: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:478:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7617: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:479:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7618: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:480:
+ uint8_t base_address[6];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7619: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:481:
+ uint16_t reserved1;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7620: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:482:
+ uint16_t count;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7621: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:483:
+ uint16_t reserved2;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7628: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:490:
+ * analogue test bus */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7630: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:492:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7631: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:493:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7632: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:494:
+ uint16_t millivolts;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7633: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:495:
+ uint16_t reserved;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7639: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:501:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7640: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:502:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7641: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:503:
+ uint8_t quad_tx_imp2k[4];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7642: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:504:
+ uint8_t quad_tx_imp50[4];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7643: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:505:
+ uint8_t lane_amp[16];$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7651: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:513:
+ * the old tag will be left in the firmware indefinitely). */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7656: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:518:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7657: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:519:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7658: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:520:
+ uint16_t visible_pfs; /**< Bitmap of visible PFs */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7659: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:521:
+ uint16_t pf_aper; /**< BIU aperture for PF BAR2 */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7660: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:522:
+ uint16_t vf_aper; /**< BIU aperture for VF BAR0 */$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7661: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:523:
+ uint16_t int_aper; /**< BIU aperture for PF BAR4 and VF BAR2 */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7661: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:523:
+ uint16_t int_aper; /**< BIU aperture for PF BAR4 and VF BAR2 */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7677: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:539:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7678: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:540:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7679: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:541:
+ uint32_t port_mode;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7683: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:545:
+#define TLV_PORT_MODE_10G_10G (2) /* 2x10G, dual SFP/10G-KR or single QSFP */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7684: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:546:
+#define TLV_PORT_MODE_40G_40G (3) /* 40G + 40G, dual QSFP/40G-KR (Greenport, Medford) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7685: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:547:
+#define TLV_PORT_MODE_10G_10G_10G_10G (4) /* 2x10G + 2x10G, quad SFP/10G-KR or dual QSFP (Greenport) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7686: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:548:
+#define TLV_PORT_MODE_10G_10G_10G_10G_Q1 (4) /* 4x10G, single QSFP, cage 0 (Medford) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7687: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:549:
+#define TLV_PORT_MODE_10G_10G_10G_10G_Q (5) /* 4x10G, single QSFP, cage 0 (Medford) OBSOLETE DO NOT USE */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7688: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:550:
+#define TLV_PORT_MODE_40G_10G_10G (6) /* 1x40G + 2x10G, dual QSFP (Greenport, Medford) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7689: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:551:
+#define TLV_PORT_MODE_10G_10G_40G (7) /* 2x10G + 1x40G, dual QSFP (Greenport, Medford) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7690: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:552:
+#define TLV_PORT_MODE_10G_10G_10G_10G_Q2 (8) /* 4x10G, single QSFP, cage 1 (Medford) */
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7691: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:553:
+#define TLV_PORT_MODE_10G_10G_10G_10G_Q1_Q2 (9) /* 2x10G + 2x10G, dual QSFP (Medford) */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7700: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:562:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7701: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:563:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7702: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:564:
+ uint32_t vswitch_type;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7703: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:565:
+#define TLV_VSWITCH_TYPE_DEFAULT (0xffffffff) /* Firmware default; equivalent to no TLV present for a given port */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7717: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:579:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7718: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:580:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7719: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:581:
+ uint32_t vlan_tag;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7720: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:582:
+#define TLV_VPORT_NO_VLAN_TAG (0xFFFFFFFF) /* Default in the absence of TLV for a given PF */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7728: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:590:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7729: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:591:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7730: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:592:
+ int16_t offset_millivolts;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7731: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:593:
+ uint16_t reserved;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7734: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:596:
+/* A privilege mask given on reset to all non-admin PCIe functions (that is other than first-PF-per-port).
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7735: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:597:
+ * The meaning of particular bits is defined in mcdi_ef10.yml under MC_CMD_PRIVILEGE_MASK, see also bug 44583.
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7736: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:598:
+ * TLV_TAG_PRIVILEGE_MASK_ADD specifies bits that should be added (ORed) to firmware default while
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7737: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:599:
+ * TLV_TAG_PRIVILEGE_MASK_REM specifies bits that should be removed (ANDed) from firmware default:
WARNING:LONG_LINE_COMMENT: line over 90 characters
#7738: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:600:
+ * Initial_privilege_mask = (firmware_default_mask | privilege_mask_add) & ~privilege_mask_rem */
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7738: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:600:
+ * Initial_privilege_mask = (firmware_default_mask | privilege_mask_add) & ~privilege_mask_rem */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7743: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:605:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7744: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:606:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7745: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:607:
+ uint32_t privilege_mask;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7751: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:613:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7752: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:614:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7753: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:615:
+ uint32_t privilege_mask_add;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7759: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:621:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7760: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:622:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7761: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:623:
+ uint32_t privilege_mask_rem;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7765: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:627:
+ * This tag takes precedence over TLV_TAG_PRIVILEGE_MASK_REM. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7770: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:632:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7771: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:633:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7772: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:634:
+ uint32_t privilege_mask_add;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7776: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:638:
+ * This tag takes precedence over TLV_TAG_PRIVILEGE_MASK_REM. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7781: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:643:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7782: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:644:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7783: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:645:
+ uint32_t privilege_mask_add;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#7787: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:649:
+ * This tag only takes effect if TLV_TAG_VSWITCH_TYPE is missing or set to DEFAULT. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7792: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:654:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7793: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:655:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7794: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:656:
+ uint32_t pfiov;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7822: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:684:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7823: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:685:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7824: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:686:
+ uint32_t mode;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7834: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:696:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7835: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:697:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7836: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:698:
+ uint32_t rate_mbps;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7857: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:719:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7858: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:720:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7859: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:721:
+ uint32_t mode;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7875: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:737:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7876: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:738:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7877: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:739:
+ uint8_t rx_desc_cache_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7878: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:740:
+ uint8_t tx_desc_cache_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7879: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:741:
+ uint16_t vi_count;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7893: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:755:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7894: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:756:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7895: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:757:
+ uint32_t max_events;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7897: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:759:
+ uint32_t timeout_ns;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7904: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:766:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7905: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:767:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7906: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:768:
+ uint16_t gen; /* Target PCIe generation: 1, 2, 3 */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7907: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:769:
+ uint16_t width; /* Number of lanes */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7918: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:780:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7919: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:781:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7920: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:782:
+ uint32_t max_events;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7922: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:784:
+ uint32_t timeout_ns;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7923: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:785:
+ uint32_t qempty_timeout_ns; /* Medford only */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7932: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:794:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7933: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:795:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7934: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:796:
+ uint8_t data[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7953: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:815:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7954: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:816:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7955: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:817:
+ uint32_t mode;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7956: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:818:
+ uint32_t ip;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7957: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:819:
+ uint32_t netmask;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7958: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:820:
+ uint32_t gateway;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7959: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:821:
+ uint32_t port;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7960: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:822:
+ uint32_t bind_retry; /* DEPRECATED */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7961: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:823:
+ uint32_t bind_bkout; /* DEPRECATED */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7977: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:839:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7978: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:840:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7979: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:841:
+ uint32_t num_tsacs;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7980: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:842:
+ uint32_t ip[TLV_MAX_TSACS];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7981: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:843:
+ uint32_t port[TLV_MAX_TSACS];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7997: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:859:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7998: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:860:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#7999: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:861:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8014: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:876:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8015: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:877:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8016: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:878:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8032: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:894:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8033: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:895:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8034: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:896:
+ uint8_t bytes[];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8045: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:907:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8046: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:908:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8047: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:909:
+ uint8_t mode;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8057: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:919:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8058: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:920:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8059: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:921:
+ uint8_t mode;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#8060: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:922:
+#define TLV_BIU_VI_WINDOW_MODE_8K 0 /* 8k per VI, CTPIO not mapped, medford/hunt compatible */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8071: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:933:
+ uint32_t tag;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8072: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:934:
+ uint32_t length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#8073: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:935:
+ uint8_t mode;$
WARNING:LONG_LINE_COMMENT: line over 90 characters
#8076: FILE: drivers/net/sfc/base/ef10_tlv_layout.h:938:
+#define TLV_FASTPD_MODE_FAST_SUPPORTED 2 /* Supported packet types to the FastPD; everything else to the SoftPD */
CHECK:SPACING: No space is necessary after a cast
#8152: FILE: drivers/net/sfc/base/ef10_tx.c:67:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8152: FILE: drivers/net/sfc/base/ef10_tx.c:67:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8197: FILE: drivers/net/sfc/base/ef10_tx.c:112:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8204: FILE: drivers/net/sfc/base/ef10_tx.c:119:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#8217: FILE: drivers/net/sfc/base/ef10_tx.c:132:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8217: FILE: drivers/net/sfc/base/ef10_tx.c:132:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8233: FILE: drivers/net/sfc/base/ef10_tx.c:148:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8238: FILE: drivers/net/sfc/base/ef10_tx.c:153:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8246: FILE: drivers/net/sfc/base/ef10_tx.c:161:
+ return (0);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#8274: FILE: drivers/net/sfc/base/ef10_tx.c:189:
+ if ((rc = efx_mcdi_init_txq(enp, n, eep->ee_index, label, index, flags,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8298: FILE: drivers/net/sfc/base/ef10_tx.c:213:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8303: FILE: drivers/net/sfc/base/ef10_tx.c:218:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#8329: FILE: drivers/net/sfc/base/ef10_tx.c:244:
+ if ((rc = ef10_nic_pio_alloc(enp,
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#8340: FILE: drivers/net/sfc/base/ef10_tx.c:255:
+ if ((rc = ef10_nic_pio_link(enp, etp->et_index, handle)) != 0) {
WARNING:BRACES: braces {} are not necessary for single statement blocks
#8340: FILE: drivers/net/sfc/base/ef10_tx.c:255:
+ if ((rc = ef10_nic_pio_link(enp, etp->et_index, handle)) != 0) {
+ goto fail3;
+ }
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8357: FILE: drivers/net/sfc/base/ef10_tx.c:272:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8368: FILE: drivers/net/sfc/base/ef10_tx.c:283:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8402: FILE: drivers/net/sfc/base/ef10_tx.c:317:
+ EFSYS_ASSERT(length % sizeof (efx_qword_t) == 0);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8423: FILE: drivers/net/sfc/base/ef10_tx.c:338:
+ write_offset += sizeof (efx_qword_t);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8426: FILE: drivers/net/sfc/base/ef10_tx.c:341:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8433: FILE: drivers/net/sfc/base/ef10_tx.c:348:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8461: FILE: drivers/net/sfc/base/ef10_tx.c:376:
+ offset = id * sizeof (efx_qword_t);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8479: FILE: drivers/net/sfc/base/ef10_tx.c:394:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8486: FILE: drivers/net/sfc/base/ef10_tx.c:401:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8519: FILE: drivers/net/sfc/base/ef10_tx.c:434:
+ offset = id * sizeof (efx_qword_t);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8538: FILE: drivers/net/sfc/base/ef10_tx.c:453:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8543: FILE: drivers/net/sfc/base/ef10_tx.c:458:
+ return (rc);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8566: FILE: drivers/net/sfc/base/ef10_tx.c:481:
+ offset = id * sizeof (efx_qword_t);
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#8604: FILE: drivers/net/sfc/base/ef10_tx.c:519:
+ offset = id * sizeof (efx_desc_t);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8615: FILE: drivers/net/sfc/base/ef10_tx.c:530:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8620: FILE: drivers/net/sfc/base/ef10_tx.c:535:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8733: FILE: drivers/net/sfc/base/ef10_tx.c:648:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8738: FILE: drivers/net/sfc/base/ef10_tx.c:653:
+ return (rc);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#8748: FILE: drivers/net/sfc/base/ef10_tx.c:663:
+ if ((rc = efx_mcdi_fini_txq(enp, etp->et_index)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8751: FILE: drivers/net/sfc/base/ef10_tx.c:666:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#8756: FILE: drivers/net/sfc/base/ef10_tx.c:671:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#9094: FILE: drivers/net/sfc/base/efx_mcdi.c:1596:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#9094: FILE: drivers/net/sfc/base/efx_mcdi.c:1596:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9117: FILE: drivers/net/sfc/base/efx_mcdi.c:1619:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9124: FILE: drivers/net/sfc/base/efx_mcdi.c:1626:
+ return (rc);
CHECK:SPACING: No space is necessary after a cast
#9139: FILE: drivers/net/sfc/base/efx_mcdi.c:1641:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#9139: FILE: drivers/net/sfc/base/efx_mcdi.c:1641:
+ (void) memset(payload, 0, sizeof (payload));
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9164: FILE: drivers/net/sfc/base/efx_mcdi.c:1666:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9171: FILE: drivers/net/sfc/base/efx_mcdi.c:1673:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9194: FILE: drivers/net/sfc/base/efx_nic.c:64:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9199: FILE: drivers/net/sfc/base/efx_nic.c:69:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9204: FILE: drivers/net/sfc/base/efx_nic.c:74:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#9925: FILE: drivers/net/sfc/base/efx_sram.c:64:
+ return (0);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10141: FILE: drivers/net/sfc/base/hunt_nic.c:54:
+ if ((rc = efx_mcdi_get_port_modes(enp, &port_modes, NULL)) != 0) {
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10152: FILE: drivers/net/sfc/base/hunt_nic.c:65:
+ if ((rc = efx_nic_calculate_pcie_link_bandwidth(8,
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10165: FILE: drivers/net/sfc/base/hunt_nic.c:78:
+ if ((rc = ef10_nic_get_port_mode_bandwidth(max_port_mode,
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#10173: FILE: drivers/net/sfc/base/hunt_nic.c:86:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#10180: FILE: drivers/net/sfc/base/hunt_nic.c:93:
+ return (rc);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mcdi.em_emip
#10187: FILE: drivers/net/sfc/base/hunt_nic.c:100:
+ efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_nic_cfg
#10188: FILE: drivers/net/sfc/base/hunt_nic.c:101:
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_port
#10192: FILE: drivers/net/sfc/base/hunt_nic.c:105:
+ efx_port_t *epp = &(enp->en_port);
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10203: FILE: drivers/net/sfc/base/hunt_nic.c:116:
+ if ((rc = efx_mcdi_get_port_assignment(enp, &port)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10212: FILE: drivers/net/sfc/base/hunt_nic.c:125:
+ if ((rc = ef10_external_port_mapping(enp, port,
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10222: FILE: drivers/net/sfc/base/hunt_nic.c:135:
+ if ((rc = efx_mcdi_get_function_info(enp, &pf, &vf)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10262: FILE: drivers/net/sfc/base/hunt_nic.c:175:
+ if ((rc = efx_mcdi_get_phy_cfg(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10266: FILE: drivers/net/sfc/base/hunt_nic.c:179:
+ if ((rc = ef10_phy_get_link(enp, &els)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10359: FILE: drivers/net/sfc/base/hunt_nic.c:272:
+ if ((rc = efx_mcdi_get_clock(enp, &sysclk, &dpcpu_clk)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10378: FILE: drivers/net/sfc/base/hunt_nic.c:291:
+ if ((rc = ef10_get_datapath_caps(enp)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10416: FILE: drivers/net/sfc/base/hunt_nic.c:329:
+ if ((rc = ef10_get_privilege_mask(enp, &mask)) != 0)
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10421: FILE: drivers/net/sfc/base/hunt_nic.c:334:
+ if ((rc = efx_mcdi_get_vector_cfg(enp, &base, &nvec, NULL)) != 0) {
ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#10438: FILE: drivers/net/sfc/base/hunt_nic.c:351:
+ if ((rc = hunt_nic_get_required_pcie_bandwidth(enp, &bandwidth)) != 0)
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#10445: FILE: drivers/net/sfc/base/hunt_nic.c:358:
+ return (0);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#10478: FILE: drivers/net/sfc/base/hunt_nic.c:391:
+ return (rc);
total: 277 errors, 342 warnings, 94 checks, 10196 lines checked
More information about the test-report
mailing list