|SUCCESS| net/ixgbe: remove unreachable WFQ weight mode check

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/172057

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-10-24

# DPDK Patch Review

## Analysis

This patch removes a dead code branch in `ixgbe_node_param_check()`. The submitter's analysis is correct: earlier in the function, any non-NULL `nonleaf.wfq_weight_mode` is rejected with "WFQ not supported", making the subsequent check unreachable.

I have verified this by examining the code flow:

1. Lines before the patch context check: `if (params->nonleaf.wfq_weight_mode != NULL)` and return `-EINVAL` with "WFQ not supported"
2. The removed branch checks the same pointer again: `else if (params->nonleaf.wfq_weight_mode && !(*params->nonleaf.wfq_weight_mode))`
3. Since the pointer is guaranteed to be NULL at this point (or the function would have already returned), the branch is indeed unreachable

The patch correctly removes dead code. The .mailmap addition is standard practice for new contributors.

## ERRORS

None.

## WARNINGS

None.

## INFO

**Typo in removed error message:**
The removed error message says "WFP should be byte mode" but likely meant "WFQ should be byte mode" (Weighted Fair Queuing, not "WFP"). Since the code is being deleted anyway, this is just a note for historical interest.

Review-Result: CLEAN