|WARNING| pw114831 [PATCH] net/bnxt: fix null pointer dereference in bnxt_hwrm_port_led_cfg()

[checkpatch at dpdk.org]

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

_coding style issues_


WARNING:TYPO_SPELLING: 'allways' may be misspelled - perhaps 'always'?
#54: 
VFs's "bp->leds" is allways null, check bp->leds is

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around '!bp->leds'
#87: FILE: drivers/net/bnxt/bnxt_hwrm.c:4538:
+	if (BNXT_VF(bp) || (!bp->leds) || (!bp->leds->num_leds))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around '!bp->leds->num_leds'
#87: FILE: drivers/net/bnxt/bnxt_hwrm.c:4538:
+	if (BNXT_VF(bp) || (!bp->leds) || (!bp->leds->num_leds))

WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email name mismatch: 'From: maoyingming <maoyingming at baidu.com>' != 'Signed-off-by: Mao YingMing <maoyingming at baidu.com>'

total: 0 errors, 2 warnings, 2 checks, 8 lines checked