[dpdk-test-report] |WARNING| pw50000 [1/2] kni: fix build for 5.0 for dev_open()
checkpatch at dpdk.org
checkpatch at dpdk.org
Tue Jan 22 16:45:44 CET 2019
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/50000
_coding style issues_
ERROR:SPACING: space required after that ',' (ctx:VxV)
#56: FILE: kernel/linux/kni/ethtool/igb/kcompat.h:3943:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#56: FILE: kernel/linux/kni/ethtool/igb/kcompat.h:3943:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
^
ERROR:SPACING: space prohibited after that open parenthesis '('
#56: FILE: kernel/linux/kni/ethtool/igb/kcompat.h:3943:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
ERROR:SPACING: space prohibited before that close parenthesis ')'
#56: FILE: kernel/linux/kni/ethtool/igb/kcompat.h:3943:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#56: FILE: kernel/linux/kni/ethtool/igb/kcompat.h:3943:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
ERROR:SPACING: space required after that ',' (ctx:VxV)
#71: FILE: kernel/linux/kni/ethtool/ixgbe/kcompat.h:3128:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#71: FILE: kernel/linux/kni/ethtool/ixgbe/kcompat.h:3128:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
^
ERROR:SPACING: space prohibited after that open parenthesis '('
#71: FILE: kernel/linux/kni/ethtool/ixgbe/kcompat.h:3128:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
ERROR:SPACING: space prohibited before that close parenthesis ')'
#71: FILE: kernel/linux/kni/ethtool/ixgbe/kcompat.h:3128:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
WARNING:CONSTANT_COMPARISON: Comparisons should place the constant on the right side of the test
#71: FILE: kernel/linux/kni/ethtool/ixgbe/kcompat.h:3128:
+#if ( LINUX_VERSION_CODE >= KERNEL_VERSION(5,0,0) )
total: 8 errors, 2 warnings, 20 lines checked
More information about the test-report
mailing list