summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWilken Gottwalt <wilken.gottwalt@posteo.net>2021-06-03 11:51:02 +0000
committerGuenter Roeck <linux@roeck-us.net>2021-06-09 11:51:30 -0700
commit7656cd2177612aa7c299b083ecff30a4d3e9a587 (patch)
tree59f4269a448be3567698a0b7a001023f155f43d5
parentfaffc5d8576ed827e2e8e4d2a3771dbb52667381 (diff)
downloadlinux-7656cd2177612aa7c299b083ecff30a4d3e9a587.tar.gz
linux-7656cd2177612aa7c299b083ecff30a4d3e9a587.tar.bz2
linux-7656cd2177612aa7c299b083ecff30a4d3e9a587.zip
hwmon: (corsair-psu) fix suspend behavior
During standby some PSUs turn off the microcontroller. A re-init is required during resume or the microcontroller stays unresponsive. Fixes: d115b51e0e56 ("hwmon: add Corsair PSU HID controller driver") Signed-off-by: Wilken Gottwalt <wilken.gottwalt@posteo.net> Link: https://lore.kernel.org/r/YLjCJiVtu5zgTabI@monster.powergraphx.local Signed-off-by: Guenter Roeck <linux@roeck-us.net>
-rw-r--r--drivers/hwmon/corsair-psu.c14
1 files changed, 14 insertions, 0 deletions
diff --git a/drivers/hwmon/corsair-psu.c b/drivers/hwmon/corsair-psu.c
index 02298b86b57b..731d5117f9f1 100644
--- a/drivers/hwmon/corsair-psu.c
+++ b/drivers/hwmon/corsair-psu.c
@@ -771,6 +771,16 @@ static int corsairpsu_raw_event(struct hid_device *hdev, struct hid_report *repo
return 0;
}
+#ifdef CONFIG_PM
+static int corsairpsu_resume(struct hid_device *hdev)
+{
+ struct corsairpsu_data *priv = hid_get_drvdata(hdev);
+
+ /* some PSUs turn off the microcontroller during standby, so a reinit is required */
+ return corsairpsu_init(priv);
+}
+#endif
+
static const struct hid_device_id corsairpsu_idtable[] = {
{ HID_USB_DEVICE(0x1b1c, 0x1c03) }, /* Corsair HX550i */
{ HID_USB_DEVICE(0x1b1c, 0x1c04) }, /* Corsair HX650i */
@@ -793,6 +803,10 @@ static struct hid_driver corsairpsu_driver = {
.probe = corsairpsu_probe,
.remove = corsairpsu_remove,
.raw_event = corsairpsu_raw_event,
+#ifdef CONFIG_PM
+ .resume = corsairpsu_resume,
+ .reset_resume = corsairpsu_resume,
+#endif
};
module_hid_driver(corsairpsu_driver);