Commit 88128729 authored by Ulf Hansson's avatar Ulf Hansson Committed by Rafael J. Wysocki

net: smsc911x: Synchronize the runtime PM status during system suspend

The smsc911c driver puts its device into low power state when entering
system suspend. Although it doesn't update the device's runtime PM status
to RPM_SUSPENDED, which causes problems for a parent device.

In particular, when the runtime PM status of the parent is requested to be
updated to RPM_SUSPENDED, the runtime PM core prevent this, because it's
forbidden to runtime suspend a device, which has an active child.

Fix this by updating the runtime PM status of the smsc911x device to
RPM_SUSPENDED during system suspend. In system resume, let's reverse that
action by runtime resuming the device and thus also the parent.
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Tested-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent b1a60995
...@@ -2584,6 +2584,9 @@ static int smsc911x_suspend(struct device *dev) ...@@ -2584,6 +2584,9 @@ static int smsc911x_suspend(struct device *dev)
PMT_CTRL_PM_MODE_D1_ | PMT_CTRL_WOL_EN_ | PMT_CTRL_PM_MODE_D1_ | PMT_CTRL_WOL_EN_ |
PMT_CTRL_ED_EN_ | PMT_CTRL_PME_EN_); PMT_CTRL_ED_EN_ | PMT_CTRL_PME_EN_);
pm_runtime_disable(dev);
pm_runtime_set_suspended(dev);
return 0; return 0;
} }
...@@ -2593,6 +2596,9 @@ static int smsc911x_resume(struct device *dev) ...@@ -2593,6 +2596,9 @@ static int smsc911x_resume(struct device *dev)
struct smsc911x_data *pdata = netdev_priv(ndev); struct smsc911x_data *pdata = netdev_priv(ndev);
unsigned int to = 100; unsigned int to = 100;
pm_runtime_enable(dev);
pm_runtime_resume(dev);
/* Note 3.11 from the datasheet: /* Note 3.11 from the datasheet:
* "When the LAN9220 is in a power saving state, a write of any * "When the LAN9220 is in a power saving state, a write of any
* data to the BYTE_TEST register will wake-up the device." * data to the BYTE_TEST register will wake-up the device."
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment