|SUCCESS| net/iavf: remove unreachable hash action case

[dpdklab at iol.unh.edu]

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

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-06-08

# DPDK Patch Review

## Patch Summary
This patch removes an unreachable `RTE_FLOW_ACTION_TYPE_END` case from a switch statement in `iavf_hash_parse_action()`. The patch correctly identifies that the loop terminates when `RTE_FLOW_ACTION_TYPE_END` is encountered, making the case unreachable.

---

## Review Findings

###  Correctness
No correctness issues found. The patch removes dead code without changing behavior.

###  Code Style
No style violations.

###  Warnings

#### **Warning: Unnecessary empty line after case removal**
**Location:** `drivers/net/intel/iavf/iavf_hash.c:1493`

The patch removes the `RTE_FLOW_ACTION_TYPE_END` case and its `break;`, but leaves a blank line between the removed code and the `default:` case, creating a double blank line.

**Current result after patch:**
```c
			       rss_type, pattern_hint);
		break;


	default:
		rte_flow_error_set(error, EINVAL,
```

**Suggested:**
```c
			       rss_type, pattern_hint);
		break;

	default:
		rte_flow_error_set(error, EINVAL,
```

Remove one blank line to maintain consistent formatting (single blank line between cases).

---

## Summary
- **Errors:** 0
- **Warnings:** 1 (formatting)
- **Info:** 0

The logic change is correct. The only issue is minor formatting inconsistency from the line removal. This is a valid cleanup patch.