|WARNING| pw140465 [PATCH v3 27/30] net/ixgbe/base: add NVM init and populate

checkpatch at dpdk.org checkpatch at dpdk.org
Thu May 30 13:20:52 CEST 2024


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

_coding style issues_


WARNING:TYPO_SPELLING: 'aci' may be misspelled - perhaps ''?
#109: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2552:
+	status = ixgbe_read_ee_aci_E610(hw, offset, &value);

WARNING:TYPO_SPELLING: 'aci' may be misspelled - perhaps ''?
#142: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2585:
+	status = ixgbe_read_ee_aci_E610(hw, offset, &value);

WARNING:TYPO_SPELLING: 'ACI' may be misspelled - perhaps ''?
#164: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2607:
+	u32 min_size = 0, max_size = IXGBE_ACI_NVM_MAX_OFFSET + 1;

WARNING:TYPO_SPELLING: 'ACI' may be misspelled - perhaps ''?
#177: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2620:
+		if (status == IXGBE_ERR_ACI_ERROR &&

WARNING:TYPO_SPELLING: 'aci' may be misspelled - perhaps ''?
#178: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2621:
+		    hw->aci.last_status == IXGBE_ACI_RC_EINVAL) {

WARNING:TYPO_SPELLING: 'ACI' may be misspelled - perhaps ''?
#178: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2621:
+		    hw->aci.last_status == IXGBE_ACI_RC_EINVAL) {

WARNING:TYPO_SPELLING: 'aci' may be misspelled - perhaps ''?
#215: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2658:
+	status = ixgbe_read_ee_aci_E610(hw, E610_SR_NVM_CTRL_WORD, &ctrl_word);

WARNING:BRACES: braces {} are not necessary for single statement blocks
#216: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2659:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#243: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2686:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#249: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2692:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#255: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2698:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#261: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2704:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#267: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2710:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#273: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2716:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#315: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2758:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#320: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2763:
+	if (status) {
+		return status;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#326: FILE: drivers/net/ixgbe/base/ixgbe_e610.c:2769:
+	if (status) {
+		return status;
+	}

total: 0 errors, 17 warnings, 0 checks, 259 lines checked


More information about the test-report mailing list