|WARNING| [3/3] ml/cnxk: remove unnecessary NULL check

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: WARNING
https://dpdk.org/patch/166973

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-07-28

# DPDK Patch Review

## PATCH 1/3: net/sxe2: remove unnecessary checks

### Errors

None.

### Warnings

None.

### Info

None.

---

## PATCH 2/3: net/gve: remove unnecessary check for NULL

### Errors

**Line 209: Meaningless brace addition**

The patch adds braces around a single-statement `else` branch (`rte_pktmbuf_free(ctx->mbuf_head)`) where the `if` branch has only a single statement as well.
DPDK style requires no braces for single-statement branches:

```c
/* Original (correct) */
else if (ctx->mbuf_head != NULL)
	rte_pktmbuf_free(ctx->mbuf_head);

/* Patched (wrong) - adds unnecessary braces */
else {
	rte_pktmbuf_free(ctx->mbuf_head);
}
```

Removing the NULL check is correct (rte_pktmbuf_free handles NULL), but the braces should not be added.

**Suggested fix:**

```c
if (gve_rx(rxq, rxd, rx_id)) {
	if (!ctx->drop_pkt)
		rx_pkts[nb_rx++] = ctx->mbuf_head;
	else
		rte_pktmbuf_free(ctx->mbuf_head);
	rxq->nb_avail += ctx->total_frags;
	gve_rx_ctx_clear(ctx);
}
```

### Warnings

None.

### Info

None.

---

## PATCH 3/3: ml/cnxk: remove unnecessary NULL check

### Errors

None.

### Warnings

None.

### Info

None.

---

## Summary

- **PATCH 1/3**: Clean removal of unnecessary NULL checks before `rte_free()`. No issues.
- **PATCH 2/3**: Correct NULL check removal but **introduces incorrect brace style** (Error).
- **PATCH 3/3**: Clean removal of unnecessary NULL checks before `rte_free()`. No issues.