* [PATCH v2] net: macb: undo operations in case of failure
@ 2020-06-18 8:37 Claudiu Beznea
2020-06-18 15:53 ` Nicolas Ferre
2020-06-20 2:59 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Claudiu Beznea @ 2020-06-18 8:37 UTC (permalink / raw)
To: nicolas.ferre, davem, kuba, linux
Cc: antoine.tenart, netdev, linux-kernel, Claudiu Beznea
Undo previously done operation in case macb_phylink_connect()
fails. Since macb_reset_hw() is the 1st undo operation the
napi_exit label was renamed to reset_hw.
Fixes: 7897b071ac3b ("net: macb: convert to phylink")
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
---
Changes in v2:
- corrected fixes SHA1
drivers/net/ethernet/cadence/macb_main.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/cadence/macb_main.c b/drivers/net/ethernet/cadence/macb_main.c
index 67933079aeea..257c4920cb88 100644
--- a/drivers/net/ethernet/cadence/macb_main.c
+++ b/drivers/net/ethernet/cadence/macb_main.c
@@ -2558,7 +2558,7 @@ static int macb_open(struct net_device *dev)
err = macb_phylink_connect(bp);
if (err)
- goto napi_exit;
+ goto reset_hw;
netif_tx_start_all_queues(dev);
@@ -2567,9 +2567,11 @@ static int macb_open(struct net_device *dev)
return 0;
-napi_exit:
+reset_hw:
+ macb_reset_hw(bp);
for (q = 0, queue = bp->queues; q < bp->num_queues; ++q, ++queue)
napi_disable(&queue->napi);
+ macb_free_consistent(bp);
pm_exit:
pm_runtime_put_sync(&bp->pdev->dev);
return err;
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] net: macb: undo operations in case of failure
2020-06-18 8:37 [PATCH v2] net: macb: undo operations in case of failure Claudiu Beznea
@ 2020-06-18 15:53 ` Nicolas Ferre
2020-06-20 2:59 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Nicolas Ferre @ 2020-06-18 15:53 UTC (permalink / raw)
To: Claudiu Beznea, davem, kuba, linux; +Cc: antoine.tenart, netdev, linux-kernel
On 18/06/2020 at 10:37, Claudiu Beznea wrote:
> Undo previously done operation in case macb_phylink_connect()
> fails. Since macb_reset_hw() is the 1st undo operation the
> napi_exit label was renamed to reset_hw.
>
> Fixes: 7897b071ac3b ("net: macb: convert to phylink")
> Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Thanks Claudiu.
Regards,
Nicolas
> ---
>
> Changes in v2:
> - corrected fixes SHA1
>
> drivers/net/ethernet/cadence/macb_main.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ethernet/cadence/macb_main.c b/drivers/net/ethernet/cadence/macb_main.c
> index 67933079aeea..257c4920cb88 100644
> --- a/drivers/net/ethernet/cadence/macb_main.c
> +++ b/drivers/net/ethernet/cadence/macb_main.c
> @@ -2558,7 +2558,7 @@ static int macb_open(struct net_device *dev)
>
> err = macb_phylink_connect(bp);
> if (err)
> - goto napi_exit;
> + goto reset_hw;
>
> netif_tx_start_all_queues(dev);
>
> @@ -2567,9 +2567,11 @@ static int macb_open(struct net_device *dev)
>
> return 0;
>
> -napi_exit:
> +reset_hw:
> + macb_reset_hw(bp);
> for (q = 0, queue = bp->queues; q < bp->num_queues; ++q, ++queue)
> napi_disable(&queue->napi);
> + macb_free_consistent(bp);
> pm_exit:
> pm_runtime_put_sync(&bp->pdev->dev);
> return err;
>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] net: macb: undo operations in case of failure
2020-06-18 8:37 [PATCH v2] net: macb: undo operations in case of failure Claudiu Beznea
2020-06-18 15:53 ` Nicolas Ferre
@ 2020-06-20 2:59 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2020-06-20 2:59 UTC (permalink / raw)
To: claudiu.beznea
Cc: nicolas.ferre, kuba, linux, antoine.tenart, netdev, linux-kernel
From: Claudiu Beznea <claudiu.beznea@microchip.com>
Date: Thu, 18 Jun 2020 11:37:40 +0300
> Undo previously done operation in case macb_phylink_connect()
> fails. Since macb_reset_hw() is the 1st undo operation the
> napi_exit label was renamed to reset_hw.
>
> Fixes: 7897b071ac3b ("net: macb: convert to phylink")
> Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Applied and queued up for -stable, thank you.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-20 2:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-18 8:37 [PATCH v2] net: macb: undo operations in case of failure Claudiu Beznea
2020-06-18 15:53 ` Nicolas Ferre
2020-06-20 2:59 ` 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).