net: netcp: Fix an error message
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sat, 8 May 2021 05:38:22 +0000 (07:38 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 3 Jun 2021 06:36:23 +0000 (08:36 +0200)
[ Upstream commit ddb6e00f8413e885ff826e32521cff7924661de0 ]

'ret' is known to be 0 here.
The expected error code is stored in 'tx_pipe->dma_queue', so use it
instead.

While at it, switch from %d to %pe which is more user friendly.

Fixes: 84640e27f230 ("net: netcp: Add Keystone NetCP core ethernet driver")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/ti/netcp_core.c

index 437d36289786bbbe8f153910ee797b13379b61e3..452cf6cfd17953a8feaf01aecf404902572f2f1b 100644 (file)
@@ -1364,9 +1364,9 @@ int netcp_txpipe_open(struct netcp_tx_pipe *tx_pipe)
        tx_pipe->dma_queue = knav_queue_open(name, tx_pipe->dma_queue_id,
                                             KNAV_QUEUE_SHARED);
        if (IS_ERR(tx_pipe->dma_queue)) {
+               ret = PTR_ERR(tx_pipe->dma_queue);
                dev_err(dev, "Could not open DMA queue for channel \"%s\": %d\n",
                        name, ret);
-               ret = PTR_ERR(tx_pipe->dma_queue);
                goto err;
        }