Messages in this thread Patch in this message | | | From | <> | Subject | Re: [PATCH] net/cadence: using pm_runtime_resume_and_get instead of pm_runtime_get_sync | Date | Mon, 11 Apr 2022 08:49:15 +0000 |
| |
On 11.04.2022 04:38, cgel.zte@gmail.com wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > From: Minghao Chi <chi.minghao@zte.com.cn> > > Using pm_runtime_resume_and_get is more appropriate > for simplifing code > > Reported-by: Zeal Robot <zealci@zte.com.cn> > Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>
> --- > drivers/net/ethernet/cadence/macb_main.c | 22 ++++++++-------------- > 1 file changed, 8 insertions(+), 14 deletions(-) > > diff --git a/drivers/net/ethernet/cadence/macb_main.c b/drivers/net/ethernet/cadence/macb_main.c > index 800d5ced5800..5555daee6f13 100644 > --- a/drivers/net/ethernet/cadence/macb_main.c > +++ b/drivers/net/ethernet/cadence/macb_main.c > @@ -337,11 +337,9 @@ static int macb_mdio_read(struct mii_bus *bus, int mii_id, int regnum) > struct macb *bp = bus->priv; > int status; > > - status = pm_runtime_get_sync(&bp->pdev->dev); > - if (status < 0) { > - pm_runtime_put_noidle(&bp->pdev->dev); > + status = pm_runtime_resume_and_get(&bp->pdev->dev); > + if (status < 0) > goto mdio_pm_exit; > - } > > status = macb_mdio_wait_for_idle(bp); > if (status < 0) > @@ -391,11 +389,9 @@ static int macb_mdio_write(struct mii_bus *bus, int mii_id, int regnum, > struct macb *bp = bus->priv; > int status; > > - status = pm_runtime_get_sync(&bp->pdev->dev); > - if (status < 0) { > - pm_runtime_put_noidle(&bp->pdev->dev); > + status = pm_runtime_resume_and_get(&bp->pdev->dev); > + if (status < 0) > goto mdio_pm_exit; > - } > > status = macb_mdio_wait_for_idle(bp); > if (status < 0) > @@ -2745,9 +2741,9 @@ static int macb_open(struct net_device *dev) > > netdev_dbg(bp->dev, "open\n"); > > - err = pm_runtime_get_sync(&bp->pdev->dev); > + err = pm_runtime_resume_and_get(&bp->pdev->dev); > if (err < 0) > - goto pm_exit; > + return err; > > /* RX buffers initialization */ > macb_init_rx_buffer_size(bp, bufsz); > @@ -4134,11 +4130,9 @@ static int at91ether_open(struct net_device *dev) > u32 ctl; > int ret; > > - ret = pm_runtime_get_sync(&lp->pdev->dev); > - if (ret < 0) { > - pm_runtime_put_noidle(&lp->pdev->dev); > + ret = pm_runtime_resume_and_get(&lp->pdev->dev); > + if (ret < 0) > return ret; > - } > > /* Clear internal statistics */ > ctl = macb_readl(lp, NCR); > -- > 2.25.1 >
| |