Re: [ 26/85] usb: host: xhci: New system added for Compliance ModePatch on SN65LVPE502CP

[Ben Hutchings]

On Thu, 2012-10-25 at 17:05 -0700, Greg Kroah-Hartman wrote:
> 3.6-stable review patch.  If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: "Alexis R. Cortes" <alexis.cortes@xxxxxx>
> 
> 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
[...]
> --- 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.

Ben.

>  		return true;
>  
>  	return false;


-- 
Ben Hutchings
Never attribute to conspiracy what can adequately be explained by stupidity.

Attachment: signature.asc
Description: This is a digitally signed message part