[dpdk-test-report] |WARNING| pw37629 [PATCH v5 08/18] net/axgbe: add transmit and receive queue setup apis

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Apr 9 06:50:56 CEST 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#22: 
Subject: [dpdk-dev] [PATCH] [PATCH v5 08/18] net/axgbe: add transmit and receive queue setup apis

ERROR:TRAILING_WHITESPACE: trailing whitespace
#225: FILE: drivers/net/axgbe/axgbe_ethdev.c:12:
+static void axgbe_dev_close(struct rte_eth_dev *dev); static void $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#263: FILE: drivers/net/axgbe/axgbe_ethdev.c:99:
+/* Clear all resources like TX/RX queues. */ static void $

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#268: FILE: drivers/net/axgbe/axgbe_ethdev.c:104:
+static void
+axgbe_dev_info_get(struct rte_eth_dev *dev,
+		   struct rte_eth_dev_info *dev_info) {

ERROR:CORRUPTED_PATCH: patch seems to be corrupt (line wrapped?)
#276: FILE: drivers/net/axgbe/axgbe_ethdev.c:111:
[Rami Rosen] Shouldn't it be here:

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#429: FILE: drivers/net/axgbe/axgbe_rxtx.c:14:
+static void
+axgbe_rx_queue_release(struct axgbe_rx_queue *rx_queue) {

ERROR:OPEN_BRACE: open brace '{' following function declarations go on the next line
#447: FILE: drivers/net/axgbe/axgbe_rxtx.c:32:
+void axgbe_dev_rx_queue_release(void *rxq) {

ERROR:OPEN_BRACE: open brace '{' following function declarations go on the next line
#530: FILE: drivers/net/axgbe/axgbe_rxtx.c:115:
+static void axgbe_tx_queue_release(struct axgbe_tx_queue *tx_queue) {

ERROR:OPEN_BRACE: open brace '{' following function declarations go on the next line
#547: FILE: drivers/net/axgbe/axgbe_rxtx.c:132:
+void axgbe_dev_tx_queue_release(void *txq) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#551: FILE: drivers/net/axgbe/axgbe_rxtx.c:136:
+int axgbe_dev_tx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
+			     uint16_t nb_desc, unsigned int socket_id,
+			     const struct rte_eth_txconf *tx_conf) {

ERROR:OPEN_BRACE: open brace '{' following function declarations go on the next line
#627: FILE: drivers/net/axgbe/axgbe_rxtx.c:212:
+void axgbe_dev_clear_queues(struct rte_eth_dev *dev) {

total: 10 errors, 1 warnings, 0 checks, 414 lines checked


More information about the test-report mailing list