[dpdk-test-report] |WARNING| pw53934 [PATCH 7/8] raw/ioat: add statistics functions

checkpatch at dpdk.org checkpatch at dpdk.org
Thu May 30 23:27:23 CEST 2019


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

_coding style issues_


WARNING:STATIC_CONST_CHAR_ARRAY: static const char * array should probably be static const char * const
#121: FILE: drivers/raw/ioat/ioat_rawdev.c:110:
+static const char *xstat_names[] = {

ERROR:SPACING: space required before the open brace '{'
#134: FILE: drivers/raw/ioat/ioat_rawdev.c:123:
+		switch (ids[i]){

ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#135: FILE: drivers/raw/ioat/ioat_rawdev.c:124:
+		case 0: values[i] = ioat->enqueue_failed; break;

ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#136: FILE: drivers/raw/ioat/ioat_rawdev.c:125:
+		case 1: values[i] = ioat->enqueued; break;

ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#137: FILE: drivers/raw/ioat/ioat_rawdev.c:126:
+		case 2: values[i] = ioat->started; break;

ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#138: FILE: drivers/raw/ioat/ioat_rawdev.c:127:
+		case 3: values[i] = ioat->completed; break;

ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#139: FILE: drivers/raw/ioat/ioat_rawdev.c:128:
+		default: values[i] = 0; break;

total: 6 errors, 1 warnings, 141 lines checked


More information about the test-report mailing list