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

checkpatch at dpdk.org checkpatch at dpdk.org
Sun Dec 4 07:36:58 CET 2016


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

_coding style issues_


WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct ixgbe_hw *' should also have an identifier name
#63: 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
#64: 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
#64: 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)
#265: 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)
#376: 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