linux-kernel - [PATCH] Input: xpad - Turn off wireless controllers on shutdown

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Thu, 24 Mar 2022 10:53:08 -0400
From:   Stephen Douthit <stephen.douthit@...il.com>
To:     Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:     Stephen Douthit <stephen.douthit@...il.com>,
        linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: [PATCH] Input: xpad - Turn off wireless controllers on shutdown

We already do this on suspend, makes sense for shutdown as well

Signed-off-by: Stephen Douthit <stephen.douthit@...il.com>
---
 drivers/input/joystick/xpad.c | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/drivers/input/joystick/xpad.c b/drivers/input/joystick/xpad.c
index 18190b529bca..8be79f91ba56 100644
--- a/drivers/input/joystick/xpad.c
+++ b/drivers/input/joystick/xpad.c
@@ -1738,6 +1738,16 @@ static int xpad_init_input(struct usb_xpad *xpad)
 	return error;
 }
 
+static void xpad360w_shutdown(struct device *dev)
+{
+	struct usb_interface *intf = to_usb_interface(dev);
+	struct usb_xpad *xpad = usb_get_intfdata(intf);
+
+	xpad360w_stop_input(xpad);
+	if (xpad->pad_present)
+		xpad360w_poweroff_controller(xpad);
+}
+
 static int xpad_probe(struct usb_interface *intf, const struct usb_device_id *id)
 {
 	struct usb_device *udev = interface_to_usbdev(intf);
@@ -1863,6 +1873,12 @@ static int xpad_probe(struct usb_interface *intf, const struct usb_device_id *id
 		 * here in this driver and in usb core.
 		 */
 		udev->quirks |= USB_QUIRK_RESET_RESUME;
+
+		/*
+		 * Populate shutdown callback so wireless controllers turn off
+		 * when the machine turns off
+		 */
+		intf->dev.driver->shutdown = xpad360w_shutdown;
 	} else {
 		error = xpad_init_input(xpad);
 		if (error)
-- 
2.35.1