linux-kernel - Re: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130821184042.GC3577@xanatos>
Date:	Wed, 21 Aug 2013 11:40:42 -0700
From:	Sarah Sharp <sarah.a.sharp@...ux.intel.com>
To:	Julius Werner <jwerner@...omium.org>
Cc:	linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Vincent Palatin <vpalatin@...omium.org>,
	Benson Leung <bleung@...omium.org>, Felipe Balbi <balbi@...com>
Subject: Re: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for
 platform xHCs

Hi Julius,

Thanks for the patch!  Did you test with a USB analyzer to see if the
device was actually going into USB 2.0 Link PM?  I'd like to confirm we
really aren't breaking anything for DW3 hosts by enabling this.

Sarah Sharp

On Tue, Aug 20, 2013 at 04:21:49PM -0700, Julius Werner wrote:
> The driver methods required for hardware LPM have only been added to the
> PCI version of the XHCI driver, for no apparent reason. They seem to
> work just as well with the platform driver, so let's add them to give
> more devices the chance for additional power savings. Tested on the DWC3
> xHC of an Exynos5420.
> 
> Signed-off-by: Julius Werner <jwerner@...omium.org>
> ---
>  drivers/usb/host/xhci-plat.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/usb/host/xhci-plat.c b/drivers/usb/host/xhci-plat.c
> index 51e22bf..7f46b5d 100644
> --- a/drivers/usb/host/xhci-plat.c
> +++ b/drivers/usb/host/xhci-plat.c
> @@ -80,6 +80,10 @@ static const struct hc_driver xhci_plat_xhci_driver = {
>  	.hub_status_data =	xhci_hub_status_data,
>  	.bus_suspend =		xhci_bus_suspend,
>  	.bus_resume =		xhci_bus_resume,
> +
> +	/* LPM support */
> +	.update_device =	xhci_update_device,
> +	.set_usb2_hw_lpm =	xhci_set_usb2_hardware_lpm,
>  };
>  
>  static int xhci_plat_probe(struct platform_device *pdev)
> -- 
> 1.7.12.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/