* [PATCH] drivers: net: xgene: Fix the order
@ 2020-01-26 10:44 Christophe JAILLET
2020-01-27 10:18 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Christophe JAILLET @ 2020-01-26 10:44 UTC (permalink / raw)
To: isubramanian, keyur, quan, tinamdar, kdinh, davem
Cc: netdev, linux-kernel, kernel-janitors, Christophe JAILLET
'alloc_etherdev_mqs()' expects first 'tx', then 'rx'. The semantic here
looks reversed.
Reorder the arguments passed to 'alloc_etherdev_mqs()' in order to keep
the correct semantic.
In fact, this is a no-op because both XGENE_NUM_[RT]X_RING are 8.
Fixes: 107dec2749fe ("drivers: net: xgene: Add support for multiple queues")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
index d8612131c55e..cc8031ae9aa3 100644
--- a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
+++ b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
@@ -2020,7 +2020,7 @@ static int xgene_enet_probe(struct platform_device *pdev)
int ret;
ndev = alloc_etherdev_mqs(sizeof(struct xgene_enet_pdata),
- XGENE_NUM_RX_RING, XGENE_NUM_TX_RING);
+ XGENE_NUM_TX_RING, XGENE_NUM_RX_RING);
if (!ndev)
return -ENOMEM;
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drivers: net: xgene: Fix the order
2020-01-26 10:44 [PATCH] drivers: net: xgene: Fix the order Christophe JAILLET
@ 2020-01-27 10:18 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2020-01-27 10:18 UTC (permalink / raw)
To: christophe.jaillet
Cc: isubramanian, keyur, quan, tinamdar, kdinh, netdev, linux-kernel,
kernel-janitors
From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Date: Sun, 26 Jan 2020 11:44:29 +0100
> 'alloc_etherdev_mqs()' expects first 'tx', then 'rx'. The semantic here
> looks reversed.
>
> Reorder the arguments passed to 'alloc_etherdev_mqs()' in order to keep
> the correct semantic.
>
> In fact, this is a no-op because both XGENE_NUM_[RT]X_RING are 8.
>
> Fixes: 107dec2749fe ("drivers: net: xgene: Add support for multiple queues")
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Applied, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-01-27 10:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-26 10:44 [PATCH] drivers: net: xgene: Fix the order Christophe JAILLET
2020-01-27 10:18 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).