| From 8f6eecba689b423e79a3f55ed1ece340e8cf0023 Mon Sep 17 00:00:00 2001 |
| From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| Date: Sun, 26 Jan 2020 11:44:29 +0100 |
| Subject: [PATCH] drivers: net: xgene: Fix the order of the arguments of |
| 'alloc_etherdev_mqs()' |
| |
| commit 5a44c71ccda60a50073c5d7fe3f694cdfa3ab0c2 upstream. |
| |
| '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> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c |
| index 10b1c053e70a..a8ba616fa3ad 100644 |
| --- a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c |
| +++ b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c |
| @@ -2022,7 +2022,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.7.4 |
| |