[dpdk-test-report] |WARNING| pw64174 [PATCH v2 7/7] net/qede: remove 'typedef int bool'

[checkpatch at dpdk.org]

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

_coding style issues_


CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'p_iov->b_pre_fp_hsi == true'
#98: FILE: drivers/net/qede/base/ecore_vf.c:449:
+	if (((p_iov->b_pre_fp_hsi == true) &
 	    ETH_HSI_VER_MINOR) &&
 	    (resp->pfdev_info.minor_fp_hsi < ETH_HSI_VER_MINOR))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'resp->pfdev_info.minor_fp_hsi < ETH_HSI_VER_MINOR'
#98: FILE: drivers/net/qede/base/ecore_vf.c:449:
+	if (((p_iov->b_pre_fp_hsi == true) &
 	    ETH_HSI_VER_MINOR) &&
 	    (resp->pfdev_info.minor_fp_hsi < ETH_HSI_VER_MINOR))

CHECK:BOOL_COMPARISON: Using comparison to true is error prone
#98: FILE: drivers/net/qede/base/ecore_vf.c:449:
+	if (((p_iov->b_pre_fp_hsi == true) &

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'qdev->enable_tx_switching == false'
#111: FILE: drivers/net/qede/qede_ethdev.c:453:
+	if ((qdev->enable_tx_switching == false) && (flg == true)) {

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'flg == true'
#111: FILE: drivers/net/qede/qede_ethdev.c:453:
+	if ((qdev->enable_tx_switching == false) && (flg == true)) {

CHECK:BOOL_COMPARISON: Using comparison to false is error prone
#111: FILE: drivers/net/qede/qede_ethdev.c:453:
+	if ((qdev->enable_tx_switching == false) && (flg == true)) {

total: 0 errors, 0 warnings, 6 checks, 33 lines checked