* [dpdk-test-report] |WARNING| pw21069 [PATCH 3/6] prgdev: add bus probe and remove functions
[not found] <1488427438-13646-4-git-send-email-jing.d.chen@intel.com>
@ 2017-03-02 11:08 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-03-02 11:08 UTC (permalink / raw)
To: test-report; +Cc: Chen Jing D(Mark)
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/21069
_coding style issues_
WARNING:TYPO_SPELLING: 'occured' may be misspelled - perhaps 'occurred'?
#49:
hotplug occured.
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#178: FILE: lib/librte_prgdev/rte_prgdev.c:111:
+ (unsigned) pci_dev->id.vendor_id,
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#179: FILE: lib/librte_prgdev/rte_prgdev.c:112:
+ (unsigned) pci_dev->id.device_id);
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#232: FILE: lib/librte_prgdev/rte_prgdev.c:165:
+ unsigned i;
WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#244: FILE: lib/librte_prgdev/rte_prgdev.c:177:
+ unsigned i;
WARNING:TYPO_SPELLING: 'unkown' may be misspelled - perhaps 'unknown'?
#323: FILE: lib/librte_prgdev/rte_prgdev.h:70:
+ RTE_PRG_STAT_UNKNOWN = 0, /** Device in a unkown state */
WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#363: FILE: lib/librte_prgdev/rte_prgdev.h:110:
+/*
+* prg_dev_init routine
WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#370: FILE: lib/librte_prgdev/rte_prgdev.h:117:
+/*
+* prg_dev_uninit routine
WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#446: FILE: lib/librte_prgdev/rte_prgdev.h:193:
+ /* A array to store hardware, firmware, running image info.Each device
+ * can define and interpret the info. For example, a device can define
WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#488: FILE: lib/librte_prgdev/rte_prgdev.h:235:
+#define RTE_PRG_VALID_DEVID_OR_ERR_RET(dev_id, retval) do { \
+ if (!rte_prgdev_is_valid_dev(dev_id)) { \
+ PRG_LOG_DEBUG("Invalid dev_id=%d", dev_id); \
+ return retval; \
+ } \
+} while (0)
WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#495: FILE: lib/librte_prgdev/rte_prgdev.h:242:
+#define RTE_PRG_FUNC_PTR_OR_ERR_RET(func, retval) do { \
+ if ((func) == NULL) { \
+ PRG_LOG_DEBUG("Function not supported
"); \
+ return retval; \
+ } \
+} while (0)
WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#502: FILE: lib/librte_prgdev/rte_prgdev.h:249:
+#define RTE_PRG_PRIMARY_PROC_OR_ERR_RET(retval) do { \
+ if (rte_eal_process_type() != RTE_PROC_PRIMARY) { \
+ PRG_LOG_DEBUG("prgdev doesn't support secondary process"); \
+ return retval; \
+ } \
+} while (0)
total: 0 errors, 12 warnings, 473 lines checked
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2017-03-02 11:08 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1488427438-13646-4-git-send-email-jing.d.chen@intel.com>
2017-03-02 11:08 ` [dpdk-test-report] |WARNING| pw21069 [PATCH 3/6] prgdev: add bus probe and remove functions checkpatch
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).