[patch] USB after second suspend/resume on ThinkPads.

From: Edward Tomasz Napierała <trasz_at_FreeBSD.org>
Date: Mon, 16 Jun 2014 21:21:55 +0200
Hi.  Patch below should fix a problem where USB stops working after
_second_ suspend/resume, which happens on various ThinkPad models.
Please test, and report both success stories and failures.  If nothing
comes up, I'll commit it in a week or so.

(Btw, has anyone encountered the problem on hardware other than ThinkPads?)


Index: sys/dev/acpi_support/acpi_ibm.c
===================================================================
--- sys/dev/acpi_support/acpi_ibm.c	(revision 267417)
+++ sys/dev/acpi_support/acpi_ibm.c	(working copy)
_at__at_ -169,6 +169,9 _at__at_ struct acpi_ibm_softc {
 	int		light_get_supported;
 	int		light_set_supported;
 
+	/* USB power workaround */
+	ACPI_HANDLE	power_handle;
+
 	/* led(4) interface */
 	struct cdev	*led_dev;
 	int		led_busy;
_at__at_ -365,6 +368,7 _at__at_ acpi_ibm_attach(device_t dev)
 {
 	struct acpi_ibm_softc	*sc;
 	devclass_t		ec_devclass;
+	ACPI_STATUS		status;
 
 	ACPI_FUNCTION_TRACE((char *)(uintptr_t) __func__);
 
_at__at_ -448,6 +452,17 _at__at_ acpi_ibm_attach(device_t dev)
 	if (sc->light_set_supported)
 		sc->led_dev = led_create_state(ibm_led, sc, "thinklight", sc->light_val);
 
+	/*
+	 * Obtain a handle to the power resource available on many models.
+	 * This must be turned on manually upon resume. Otherwise the system
+	 * may, for instance, resume from S3 with usb(4) powered down.
+	 */
+	status = AcpiGetHandle(sc->handle, "\\_SB.PCI0.LPC.EC.PUBS", &sc->power_handle);
+	if (ACPI_FAILURE(status)) {
+		device_printf(dev, "Failed to get power handle\n");
+		return (status);
+	}
+
 	return (0);
 }
 
_at__at_ -476,6 +491,7 _at__at_ static int
 acpi_ibm_resume(device_t dev)
 {
 	struct acpi_ibm_softc *sc = device_get_softc(dev);
+	ACPI_STATUS status;
 
 	ACPI_FUNCTION_TRACE((char *)(uintptr_t) __func__);
 
_at__at_ -495,6 +511,15 _at__at_ acpi_ibm_resume(device_t dev)
 
 		acpi_ibm_sysctl_set(sc, i, val);
 	}
+	if (sc->power_handle != NULL) {
+		status = AcpiEvaluateObject(sc->power_handle,
+		    "_ON", NULL, NULL);
+		if (ACPI_FAILURE(status)) {
+			device_printf(dev, "failed to switch %s on - %s\n",
+			    acpi_name(sc->power_handle),
+			    AcpiFormatException(status));
+		}
+	}
 	ACPI_SERIAL_END(ibm);
 
 	return (0);
Received on Mon Jun 16 2014 - 17:22:00 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:50 UTC