mbox series

[net,v3,0/2] net: ethernet: ti: am65-cpsw: Fixes to multi queue RX feature

Message ID 20241101-am65-cpsw-multi-rx-j7-fix-v3-0-338fdd6a55da@kernel.org (mailing list archive)
Headers show
Series net: ethernet: ti: am65-cpsw: Fixes to multi queue RX feature | expand

Message

Roger Quadros Nov. 1, 2024, 10:18 a.m. UTC
On J7 platforms, setting up multiple RX flows was failing
as the RX free descriptor ring 0 is shared among all flows
and we did not allocate enough elements in the RX free descriptor
ring 0 to accommodate for all RX flows. Patch 1 fixes this.

The second patch fixes a warning if there was any error in
am65_cpsw_nuss_init_rx_chns() and am65_cpsw_nuss_cleanup_rx_chns()
was called after that.

Signed-off-by: Roger Quadros <rogerq@kernel.org>
---
Changes in v3:
- Reverse Xmas tree style fixes
- Split out unrelated code fix to a new patch

Changes in v2:
- Dropped unused variables desc-idx and flow
- Link to v1: https://lore.kernel.org/r/20241029-am65-cpsw-multi-rx-j7-fix-v1-1-426ca805918c@kernel.org

---
Roger Quadros (2):
      net: ethernet: ti: am65-cpsw: Fix multi queue Rx on J7
      net: ethernet: ti: am65-cpsw: fix warning in am65_cpsw_nuss_remove_rx_chns()

 drivers/net/ethernet/ti/am65-cpsw-nuss.c | 75 ++++++++++++++------------------
 drivers/net/ethernet/ti/am65-cpsw-nuss.h |  6 ++-
 2 files changed, 37 insertions(+), 44 deletions(-)
---
base-commit: 42f7652d3eb527d03665b09edac47f85fb600924
change-id: 20241023-am65-cpsw-multi-rx-j7-fix-f9a2149be6dd

Best regards,

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 5, 2024, 3:10 p.m. UTC | #1
Hello:

This series was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@redhat.com>:

On Fri, 01 Nov 2024 12:18:49 +0200 you wrote:
> On J7 platforms, setting up multiple RX flows was failing
> as the RX free descriptor ring 0 is shared among all flows
> and we did not allocate enough elements in the RX free descriptor
> ring 0 to accommodate for all RX flows. Patch 1 fixes this.
> 
> The second patch fixes a warning if there was any error in
> am65_cpsw_nuss_init_rx_chns() and am65_cpsw_nuss_cleanup_rx_chns()
> was called after that.
> 
> [...]

Here is the summary with links:
  - [net,v3,1/2] net: ethernet: ti: am65-cpsw: Fix multi queue Rx on J7
    https://git.kernel.org/netdev/net/c/de794169cf17
  - [net,v3,2/2] net: ethernet: ti: am65-cpsw: fix warning in am65_cpsw_nuss_remove_rx_chns()
    https://git.kernel.org/netdev/net/c/ba3b7ac4f714

You are awesome, thank you!