[dpdk-test-report] |WARNING| pw44488 [17/37] net/sfc/base: add API to retrieve sensor limits
checkpatch at dpdk.org
checkpatch at dpdk.org
Mon Sep 10 11:35:15 CEST 2018
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/44488
_coding style issues_
CHECK:CAMELCASE: Avoid CamelCase: <__checkReturn>
#50: FILE: drivers/net/sfc/base/efx.h:829:
+extern __checkReturn efx_rc_t
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#51: FILE: drivers/net/sfc/base/efx.h:830:
+efx_mon_limits_update(
WARNING:FUNCTION_ARGUMENTS: function definition argument 'efx_nic_t *' should also have an identifier name
#66: FILE: drivers/net/sfc/base/efx_impl.h:320:
+ efx_rc_t (*emo_limits_update)(efx_nic_t *,
WARNING:FUNCTION_ARGUMENTS: function definition argument 'efx_mon_stat_limits_t *' should also have an identifier name
#66: FILE: drivers/net/sfc/base/efx_impl.h:320:
+ efx_rc_t (*emo_limits_update)(efx_nic_t *,
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#90: FILE: drivers/net/sfc/base/efx_mon.c:820:
+efx_mon_limits_update(
CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around enp->en_mon
#94: FILE: drivers/net/sfc/base/efx_mon.c:824:
+ efx_mon_t *emp = &(enp->en_mon);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#100: FILE: drivers/net/sfc/base/efx_mon.c:830:
+ return (emop->emo_limits_update(enp, values));
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#115: FILE: drivers/net/sfc/base/mcdi_mon.c:338:
+efx_mcdi_sensor_info_page(
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#119: FILE: drivers/net/sfc/base/mcdi_mon.c:342:
+ __out_ecount((sizeof (*mask_part) * 8) - 1)
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#134: FILE: drivers/net/sfc/base/mcdi_mon.c:357:
+ ((sizeof (*mask_part) * 8) - 1) * sizeof (efx_mon_stat_limits_t));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#134: FILE: drivers/net/sfc/base/mcdi_mon.c:357:
+ ((sizeof (*mask_part) * 8) - 1) * sizeof (efx_mon_stat_limits_t));
CHECK:SPACING: No space is necessary after a cast
#136: FILE: drivers/net/sfc/base/mcdi_mon.c:359:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#136: FILE: drivers/net/sfc/base/mcdi_mon.c:359:
+ (void) memset(payload, 0, sizeof (payload));
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#152: FILE: drivers/net/sfc/base/mcdi_mon.c:375:
+ EFSYS_ASSERT(sizeof (*limit_info) ==
CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#162: FILE: drivers/net/sfc/base/mcdi_mon.c:385:
+ while (mask_copy) {
+
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#192: FILE: drivers/net/sfc/base/mcdi_mon.c:415:
+ return (rc);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#205: FILE: drivers/net/sfc/base/mcdi_mon.c:443:
+ return (rc);
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#209: FILE: drivers/net/sfc/base/mcdi_mon.c:447:
+lowest_set_bit(
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#240: FILE: drivers/net/sfc/base/mcdi_mon.c:478:
+mcdi_mon_limits_update(
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#248: FILE: drivers/net/sfc/base/mcdi_mon.c:486:
+ efx_mon_stat_limits_t limits[sizeof (page_mask) * 8];
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#273: FILE: drivers/net/sfc/base/mcdi_mon.c:511:
+ page_index + (sizeof (page_mask) * 8 * page);
ERROR:RETURN_PARENTHESES: return is not a function, parentheses are not required
#290: FILE: drivers/net/sfc/base/mcdi_mon.c:528:
+ return (rc);
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#304: FILE: drivers/net/sfc/base/mcdi_mon.h:43:
+mcdi_mon_limits_update(
total: 4 errors, 9 warnings, 10 checks, 255 lines checked
More information about the test-report
mailing list