[dpdk-test-report] |WARNING| pw34772 [PATCH v4 08/10] drivers/raw: support for rawdev testcases

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Jan 31 15:03:46 CET 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#32: 
cast from pointer to integer of different size [-Werror=pointer-to-int-cast]

ERROR:DIFF_IN_COMMIT_MSG: Avoid using diff content in the commit message - patch(1) might not work
#48: 
--- a/drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#55: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:277:
+       ret = rte_rawdev_set_attr(TEST_DEV_ID, "Test2", (uintptr_t)dummy_value);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#61: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:282:
+                             "Attribute (Test1) not set correctly (%" PRIu64 ")",$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#61: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:282:
+                             "Attribute (Test1) not set correctly (%" PRIu64 ")",$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#68: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:287:
+       RTE_TEST_ASSERT_EQUAL(*((int *)(uintptr_t)ret_value), 200,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#69: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:288:
+                             "Attribute (Test2) not set correctly (%" PRIu64 ")",$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#69: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:288:
+                             "Attribute (Test2) not set correctly (%" PRIu64 ")",$

ERROR:CORRUPTED_PATCH: patch seems to be corrupt (line wrapped?)
#72: FILE: drivers/raw/skeleton_rawdev/skeleton_rawdev_test.c:290:
And in previous patch:

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 5 errors, 5 warnings, 29 lines checked


More information about the test-report mailing list