[dpdk-test-report] |WARNING| pw23188 [PATCH v2] kni: define HAVE_VF_VLAN_PROTO for SLES12SP3

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Apr 4 15:20:43 CEST 2017


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

_coding style issues_


WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#30: FILE: lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h:713:
+#elif (LINUX_VERSION_CODE >= KERNEL_VERSION(4, 4, 57))

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#41: FILE: lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h:3935:
+#if ((LINUX_VERSION_CODE >= KERNEL_VERSION(4, 9, 0)) \

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#42: FILE: lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h:3936:
+    || (defined(SLE_VERSION_CODE) && \$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#43: FILE: lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h:3937:
+       (SLE_VERSION_CODE >= SLE_VERSION(12, 3, 0))))$

WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#43: FILE: lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h:3937:
+       (SLE_VERSION_CODE >= SLE_VERSION(12, 3, 0))))

total: 0 errors, 5 warnings, 21 lines checked


More information about the test-report mailing list