[dpdk-test-report] [PatchWork]|ERROR| pw13307-13326 thunderx/nicvf/base: add hardware API for ThunderX nicvf inbuilt NIC

[sys_stv at intel.com]

Test-Label: Intel Niantic on Fedora
Test-Status: ERROR

Patchwork ID: 13307-13326
http://www.dpdk.org/dev/patchwork/patch/13326/
Submitter: Jerin Jacob <jerin.jacob at caviumnetworks.com>
Date: Tue, 7 Jun 2016 22:10:13 +0530
DPDK git baseline: ca173a909538a2f1082cd0dcb4d778a97dab69c3

Check patch error:
13307: 
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#53: FILE: drivers/net/thunderx/base/nicvf_hw.c:2:
+/*
+ *   BSD LICENSE

ERROR: return is not a function, parentheses are not required
#320: FILE: drivers/net/thunderx/base/nicvf_hw.c:269:
+return (*(const uint32_t *)a - *(const uint32_t *)b);

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#394: FILE: drivers/net/thunderx/base/nicvf_hw.c:343:
+/*
+ * Handle poll mode driver interested "mbox" and "queue-set error" interrupts.

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#967: FILE: drivers/net/thunderx/base/nicvf_hw.h:2:
+/*
+ *   BSD LICENSE

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#1213: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:2:
+/*
+ *   BSD LICENSE

WARNING: do not add new typedefs
#1750: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:539:
+typedef uint64_t nicvf_phys_addr_t;

WARNING: do not add new typedefs
#1761: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:550:
+typedef union {

WARNING: do not add new typedefs
#1806: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:595:
+typedef union {

WARNING: do not add new typedefs
#1831: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:620:
+typedef union {

WARNING: do not add new typedefs
#1848: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:637:
+typedef union {

WARNING: do not add new typedefs
#1865: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:654:
+typedef union {

WARNING: do not add new typedefs
#1882: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:671:
+typedef union {

WARNING: do not add new typedefs
#1899: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:688:
+typedef union {

ERROR: spaces prohibited around that ':' (ctx:VxW)
#2336: FILE: drivers/net/thunderx/base/nicvf_hw_defs.h:1125:
+uint64_t ena: 1;
            ^

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2435: FILE: drivers/net/thunderx/base/nicvf_mbox.c:2:
+/*
+ *   BSD LICENSE

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2628: FILE: drivers/net/thunderx/base/nicvf_mbox.c:195:
+/*
+ * Checks if VF is able to communicate with PF

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#2857: FILE: drivers/net/thunderx/base/nicvf_mbox.h:2:
+/*
+ *   BSD LICENSE

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#3095: FILE: drivers/net/thunderx/base/nicvf_plat.h:2:
+/*
+ *   BSD LICENSE

ERROR: space prohibited before open square bracket '['
#3176: FILE: drivers/net/thunderx/base/nicvf_plat.h:83:
+    : [val] "r" (val), [addr] "r" (addr));

ERROR: space prohibited before open square bracket '['
#3186: FILE: drivers/net/thunderx/base/nicvf_plat.h:93:
+    : [val] "=r" (val)

ERROR: space prohibited before open square bracket '['
#3187: FILE: drivers/net/thunderx/base/nicvf_plat.h:94:
+    : [addr] "r" (addr));

ERROR: space prohibited before open square bracket '['
#3194: FILE: drivers/net/thunderx/base/nicvf_plat.h:101:
+: [x1]"=r"(reg1), [x0]"=r"(reg2)
ERROR: space prohibited before open square bracket '['
#3195: FILE: drivers/net/thunderx/base/nicvf_plat.h:102:
+: [p1]"r"(addr)
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#3205: FILE: drivers/net/thunderx/base/nicvf_plat.h:112:
+*(volatile uint64_t *)addr = val;

WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#3211: FILE: drivers/net/thunderx/base/nicvf_plat.h:118:
+return*(volatile uint64_t *)addr;

total: 7 errors, 18 warnings, 3144 lines checked

/home/patchWorkOrg/patches/dpdk-dev-v3-01-20-thunderx-nicvf-base-add-hardware-API-for-ThunderX-nicvf-inbuilt-NIC.patch has style problems, please review.

If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.

13318: 
WARNING: networking block comments don't use an empty /* line, use /* Comment...
#83: FILE: drivers/net/thunderx/nicvf_rxtx.c:2:
+/*
+ *   BSD LICENSE

ERROR: return is not a function, parentheses are not required
#221: FILE: drivers/net/thunderx/nicvf_rxtx.c:140:
+return ((sq->head - sq->tail - 1) & sq->qlen_mask);

WARNING: networking block comments don't use an empty /* line, use /* Comment...
#345: FILE: drivers/net/thunderx/nicvf_rxtx.h:2:
+/*
+ *   BSD LICENSE

ERROR: code indent should use tabs where possible
#422: FILE: drivers/net/thunderx/nicvf_rxtx.h:79:
+        entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$

WARNING: please, no spaces at the start of a line
#422: FILE: drivers/net/thunderx/nicvf_rxtx.h:79:
+        entry->buff[0] = (uint64_t)SQ_DESC_TYPE_GATHER << 60 |$

ERROR: code indent should use tabs where possible
#425: FILE: drivers/net/thunderx/nicvf_rxtx.h:82:
+        entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$

WARNING: please, no spaces at the start of a line
#425: FILE: drivers/net/thunderx/nicvf_rxtx.h:82:
+        entry->buff[1] = rte_mbuf_data_dma_addr(pkt);$

total: 3 errors, 4 warnings, 378 lines checked

NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
      scripts/cleanfile
  
  /home/patchWorkOrg/patches/dpdk-dev-v3-12-20-thunderx-nicvf-add-single-and-multi-segment-tx-functions.patch has style problems, please review.
  
  If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.
  
  
  Compilation:
  OS: fedora
  Nic: niantic
  GCC: gcc_x86-64, 4.8.3
  ICC:16.0.2
  i686-native-linuxapp-icc: compile pass
  x86_64-native-linuxapp-gcc-combined: compile pass
  i686-native-linuxapp-gcc: compile pass
  x86_64-native-linuxapp-gcc: compile pass
  x86_64-native-linuxapp-icc: compile pass
  x86_64-native-linuxapp-gcc-debug: compile pass
  x86_64-native-linuxapp-gcc-shared: compile pass
  x86_64-native-linuxapp-clang: compile pass
  
  
  
  DPDK STV team