|WARNING| pw165257 [PATCH v8 1/1] net/mana: add device reset support

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#462: FILE: drivers/net/mana/mana.c:955:
+ * Join the reset thread if it is active. Uses CAS on
                                                ^^^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#470: FILE: drivers/net/mana/mana.c:963:
+	if (rte_atomic_compare_exchange_strong_explicit(

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#484: FILE: drivers/net/mana/mana.c:977:
+ * Clear per-queue burst_state so the data path CAS can succeed again.
                                                 ^^^

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#918: FILE: drivers/net/mana/mana.c:1572:
+	 * Use CAS to avoid double-join if another path joined first.
 	       ^^^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#925: FILE: drivers/net/mana/mana.c:1579:
+		if (rte_atomic_compare_exchange_strong_explicit(

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1040: FILE: drivers/net/mana/mana.c:1694:
+	 * data path CAS can succeed again.
 	             ^^^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#1197: FILE: drivers/net/mana/mana.c:1844:
+					pthread_mutex_unlock(

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1539: FILE: drivers/net/mana/mana.h:357:
+ *   Bit 0: in-burst (set by data path CAS 0→1, cleared on exit).
                                        ^^^

CHECK:SPACING: No space is necessary after a cast
#1551: FILE: drivers/net/mana/mana.h:393:
+	RTE_ATOMIC(enum mana_device_state) dev_state;

CHECK:SPACING: No space is necessary after a cast
#1556: FILE: drivers/net/mana/mana.h:398:
+	RTE_ATOMIC(bool) reset_thread_active;

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1572: FILE: drivers/net/mana/mana.h:466:
+	 * Data path CAS 0→1 to enter; fails if blocked bit is set.
 	             ^^^

CHECK:SPACING: No space is necessary after a cast
#1574: FILE: drivers/net/mana/mana.h:468:
+	RTE_ATOMIC(uint32_t) burst_state;

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1587: FILE: drivers/net/mana/mana.h:509:
+	 * Data path CAS 0→1 to enter; fails if blocked bit is set.
 	             ^^^

CHECK:SPACING: No space is necessary after a cast
#1589: FILE: drivers/net/mana/mana.h:511:
+	RTE_ATOMIC(uint32_t) burst_state;

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1835: FILE: drivers/net/mana/rx.c:464:
+	/* Single atomic CAS: enter burst only if device is active (0→1).
 	                 ^^^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#1838: FILE: drivers/net/mana/rx.c:467:
+	if (unlikely(!rte_atomic_compare_exchange_strong_explicit(

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1897: FILE: drivers/net/mana/tx.c:210:
+	/* Single atomic CAS: enter burst only if device is active (0→1).
 	                 ^^^

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#1900: FILE: drivers/net/mana/tx.c:213:
+	if (unlikely(!rte_atomic_compare_exchange_strong_explicit(

WARNING:TYPO_SPELLING: 'CAS' may be misspelled - perhaps 'CASE'?
#1904: FILE: drivers/net/mana/tx.c:217:
+		if (!expected) /* CAS succeeded but db_page NULL — undo */
 		                  ^^^

total: 0 errors, 10 warnings, 9 checks, 1735 lines checked