Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-18 Thread Boris Brezillon
Hi Sylvain, On Sat, 17 Jan 2015 23:49:00 +0100 Sylvain Rochet sylvain.roc...@finsecur.com wrote: Hi Sergei, On Sun, Jan 18, 2015 at 01:22:38AM +0300, Sergei Shtylyov wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are

Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-18 Thread Sergei Shtylyov
Hello. On 01/18/2015 01:49 AM, Sylvain Rochet wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are named using '-'... Indeed. @@ -187,6 +217,8 @@ static struct platform_driver ehci_atmel_driver = { .probe =

[PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-17 Thread Sylvain Rochet
This patch add suspend/resume support for Atmel EHCI, mostly about disabling and unpreparing clocks so USB PLL is stopped before entering sleep state. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ehci-atmel.c | 32 1 file

Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-17 Thread Sergei Shtylyov
Hello. On 01/18/2015 12:25 AM, Sylvain Rochet wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are named using '-'... This patch add suspend/resume support for Atmel EHCI, mostly about disabling and unpreparing clocks so USB PLL

Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-17 Thread Sylvain Rochet
Hi Sergei, On Sun, Jan 18, 2015 at 01:22:38AM +0300, Sergei Shtylyov wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are named using '-'... Indeed. @@ -187,6 +217,8 @@ static struct platform_driver ehci_atmel_driver = {