|SUCCESS| common/cnxk: disable flow control on pool destroy

[dpdklab at iol.unh.edu]

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

_AI Code Review_

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

# DPDK Patch Review

## ANALYSIS

This patch adds flow control disable logic to `npa_aura_pool_fini()` in the cnxk common driver. The changes clear both `fc_ena` and `ena` fields when tearing down NPA aura and pool contexts.

**Evidence-based review:**
I have read the added lines and the surrounding function context visible in the patch. The changes follow an existing pattern in the code where `ena` is cleared along with its mask. The new lines apply the same pattern to `fc_ena`.

## FINDINGS

### ERRORS

None.

### WARNINGS

None.

### INFO

**Line consistency with existing pattern:**
The added lines at +369-370 and +386-387 follow the exact same pattern used for the `ena` field:
```c
pool_req->pool.fc_ena = 0;
pool_req->pool_mask.fc_ena = ~pool_req->pool_mask.fc_ena;
```
This mirrors the pattern on lines 367-368, which is correct. The mask inversion (`~mask.field`) sets all bits in the mask to enable writing that field.

**Pre-existing code observation (Info only, not introduced by this patch):**
The function `npa_aura_pool_fini()` does not check the return value of `mbox_process()` calls that follow these structure setups. The mailbox requests could fail, but the function has `void` return type and cannot propagate errors. This is pre-existing behavior not changed by this patch.

Review-Result: CLEAN