On Sat, Oct 27, 2012 at 06:18:13PM +0100, Ben Hutchings wrote:

On Thu, 2012-10-25 at 17:05 -0700, Greg Kroah-Hartman wrote:

quoted

3.6-stable review patch.  If anyone has any objections, please let me know.

------------------

From: "Alexis R. Cortes" <redacted>

commit 470809741a28c3092279f4e1f3f432e534d46068 upstream.

This minor change adds a new system to which the "Fix Compliance Mode
on SN65LVPE502CP Hardware" patch has to be applied also.

System added:
Vendor: Hewlett-Packard. System Model: Z1

[...]

quoted

--- a/drivers/usb/host/xhci.c
+++ b/drivers/usb/host/xhci.c

@@ -479,7 +479,8 @@ static bool compliance_mode_recovery_tim
 
 	if (strstr(dmi_product_name, "Z420") ||
 			strstr(dmi_product_name, "Z620") ||
-			strstr(dmi_product_name, "Z820"))
+			strstr(dmi_product_name, "Z820") ||
+			strstr(dmi_product_name, "Z1"))

This will also match any future models with extra digits after the '1'.
It might be worth using a slightly stricter match.

Hmm, Ben has a point.  We don't want to have the timer run on a machine
that really doesn't need the quirk, or we impact battery life.

Alex, can you make a new patch that fixes this?

Sarah Sharp