Messages in this thread | | | Subject | Re: [PATCH v2 5/7] net: macb: fix call to pm_runtime in the suspend/resume functions | From | Florian Fainelli <> | Date | Tue, 21 Apr 2020 10:21:18 -0700 |
| |
On 4/21/2020 3:41 AM, nicolas.ferre@microchip.com wrote: > From: Nicolas Ferre <nicolas.ferre@microchip.com> > > The calls to pm_runtime_force_suspend/resume() functions are only > relevant if the device is not configured to act as a WoL wakeup source. > Add the device_may_wakeup() test before calling them. > > Fixes: 3e2a5e153906 ("net: macb: add wake-on-lan support via magic packet") > Cc: Claudiu Beznea <claudiu.beznea@microchip.com> > Cc: Harini Katakam <harini.katakam@xilinx.com> > Cc: Sergio Prado <sergio.prado@e-labworks.com> > Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com> > --- > Changes in v2: > - new in v2 serries > > 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 72b8983a763a..8cf8e21fbb07 100644 > --- a/drivers/net/ethernet/cadence/macb_main.c > +++ b/drivers/net/ethernet/cadence/macb_main.c > @@ -4564,7 +4564,8 @@ static int __maybe_unused macb_suspend(struct device *dev) > > if (bp->ptp_info) > bp->ptp_info->ptp_remove(netdev); > - pm_runtime_force_suspend(dev); > + if (!(device_may_wakeup(dev))) > + pm_runtime_force_suspend(dev);
Only if you need to respin, the parenthesis around device_may_wakeup() are not required:
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com> -- Florian
| |