On 3/24/20 6:59 AM, Hans de Goede wrote:

On devices with an AXP288, we need to wakeup from suspend when a charger is plugged in, so that we can do charger-type detection and so that the axp288-charger driver, which listens for our extcon events, can configure the input-current-limit accordingly.

Cc: stable@vger.kernel.org Signed-off-by: Hans de Goede hdegoede@redhat.com

---

drivers/extcon/extcon-axp288.c | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+)

diff --git a/drivers/extcon/extcon-axp288.c b/drivers/extcon/extcon-axp288.c index a7f216191493..710a3bb66e95 100644 --- a/drivers/extcon/extcon-axp288.c +++ b/drivers/extcon/extcon-axp288.c @@ -443,9 +443,40 @@ static int axp288_extcon_probe(struct platform_device *pdev) /* Start charger cable type detection */ axp288_extcon_enable(info);

• device_init_wakeup(dev, true);
• platform_set_drvdata(pdev, info);
• return 0;

+}

+static int __maybe_unused axp288_extcon_suspend(struct device *dev) +{

• struct axp288_extcon_info *info = dev_get_drvdata(dev);
• if (device_may_wakeup(dev))

• enable_irq_wake(info->irq[VBUS_RISING_IRQ]);
  

• return 0;

} +static int __maybe_unused axp288_extcon_resume(struct device *dev) +{

• struct axp288_extcon_info *info = dev_get_drvdata(dev);
• /*

• * Wakeup when a charger is connected to do charger-type
  

• * connection and generate an extcon event which makes the
  

• * axp288 charger driver set the input current limit.
  

• */
  

• if (device_may_wakeup(dev))

• disable_irq_wake(info->irq[VBUS_RISING_IRQ]);
  

• return 0;

+}

+static SIMPLE_DEV_PM_OPS(axp288_extcon_pm_ops, axp288_extcon_suspend,

• 	 axp288_extcon_resume);
  

static const struct platform_device_id axp288_extcon_table[] = { { .name = "axp288_extcon" }, {}, @@ -457,6 +488,7 @@ static struct platform_driver axp288_extcon_driver = { .id_table = axp288_extcon_table, .driver = { .name = "axp288_extcon",

• .pm = &axp288_extcon_pm_ops,
  

  },

};

Applied it. Thanks.