[dpdk-test-report] |WARNING| pw18335 [PATCH v2 15/30] net/ixgbe/base: support FW commands to control some PHYs

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Dec 21 10:52:33 CET 2016


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

_coding style issues_


WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct ixgbe_hw *' should also have an identifier name
#62: FILE: drivers/net/ixgbe/base/ixgbe_common.h:164:
+s32 ixgbe_shutdown_fw_phy(struct ixgbe_hw *);

WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct ixgbe_hw *' should also have an identifier name
#63: FILE: drivers/net/ixgbe/base/ixgbe_common.h:165:
+s32 ixgbe_fw_phy_activity(struct ixgbe_hw *, u16 activity,

WARNING:FUNCTION_ARGUMENTS: function definition argument 'u32' should also have an identifier name
#63: FILE: drivers/net/ixgbe/base/ixgbe_common.h:165:
+s32 ixgbe_fw_phy_activity(struct ixgbe_hw *, u16 activity,

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(ixgbe_fw_map)
#264: FILE: drivers/net/ixgbe/base/ixgbe_x550.c:548:
+	for (i = 0; i < sizeof(ixgbe_fw_map) / sizeof(ixgbe_fw_map[0]); ++i) {

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(ixgbe_fw_map)
#375: FILE: drivers/net/ixgbe/base/ixgbe_x550.c:800:
+	for (i = 0; i < sizeof(ixgbe_fw_map) / sizeof(ixgbe_fw_map[0]); ++i) {

total: 0 errors, 5 warnings, 0 checks, 367 lines checked


More information about the test-report mailing list