drivers: net: xgene: Fix the order of the arguments of 'alloc_etherdev_mqs()'
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 26 Jan 2020 10:44:29 +0000 (11:44 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 5 Mar 2020 15:43:49 +0000 (16:43 +0100)
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: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/net/ethernet/apm/xgene/xgene_enet_main.c

index d8612131c55e0c2de0fa0c2f0bb9df39a17ccd5d..cc8031ae9aa3fe78e57367d05f08c7786b658a66 100644 (file)
@@ -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;