Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Sudeep Holla
On 22/02/17 01:14, Rafael J. Wysocki wrote: > On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: [...] >> >> I take this back, you have everything you need in place, nothing needs >> to be done. I just checked again. If I don't register PSCI suspend_ops, >> I still get mem in /sys/po

Re: The failure summary report of GEN2 for linux stable v4.10-rc2

2017-02-22 Thread Hiep Cao Minh
Hello Laurent, I am sorry for the late reply!. I've investigated the problem further here now that I'm back home with access to my Lager board, but have been unable to reproduce it even by testing all sync signal polarities. First of all, the "CMA enabled" kernel configuration file attached t

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Geert Uytterhoeven
Hi Rafael, On Wed, Feb 22, 2017 at 2:14 AM, Rafael J. Wysocki wrote: > On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: >> On 21/02/17 18:27, Sudeep Holla wrote: >> > On 21/02/17 17:51, Sudeep Holla wrote: >> >> On 21/02/17 17:34, Geert Uytterhoeven wrote: >> >>> That's more or less

RE: [PATCH v2 1/3] power: reset: Add Renesas reset driver

2017-02-22 Thread Chris Brandt
Hello Geert and Guenter, On Thursday, February 16, 2017, Guenter Roeck wrote: > FWIW, the watchdog subsystem should support that easily, even with 125 ms > hardware timeout. We added that capability for that very purpose. That > would only fail if the system is stuck with interrupts disabled for m

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Geert Uytterhoeven
Hi Sudeep, On Wed, Feb 22, 2017 at 12:03 PM, Sudeep Holla wrote: > On 22/02/17 01:14, Rafael J. Wysocki wrote: >> On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: > > [...] > >>> I take this back, you have everything you need in place, nothing needs >>> to be done. I just checked aga

Re: [PATCH/RFC 3/6] drivers: firmware: psci: Implement shallow suspend mode

2017-02-22 Thread Geert Uytterhoeven
Hi Sudeep, On Tue, Feb 21, 2017 at 6:22 PM, Sudeep Holla wrote: > On 21/02/17 16:32, Geert Uytterhoeven wrote: >> On Tue, Feb 21, 2017 at 12:14 PM, Sudeep Holla wrote: >>> On 21/02/17 11:07, Pavel Machek wrote: > Enable support for "shallow" suspend mode, also known as "Standby" or > "Po

Re: [PATCH/RFC 4/6] drivers: firmware: psci: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Geert Uytterhoeven
Hi Mark, On Tue, Feb 21, 2017 at 6:48 PM, Mark Rutland wrote: > On Mon, Feb 20, 2017 at 09:33:27PM +0100, Geert Uytterhoeven wrote: >> @@ -440,12 +442,14 @@ static int psci_system_suspend_valid(suspend_state_t >> state) >> static int psci_system_suspend_enter(suspend_state_t state) >> { >>

Re: [PATCH RESEND 1/1] media: platform: Renesas IMR driver

2017-02-22 Thread Rob Herring
On Sat, Feb 11, 2017 at 11:02:01PM +0300, Sergei Shtylyov wrote: > From: Konstantin Kozhevnikov > > The image renderer light extended 4 (IMR-LX4) or the distortion correction > engine is a drawing processor with a simple instruction system capable of > referencing data on an external memory as 2

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Rafael J. Wysocki
On Wed, Feb 22, 2017 at 2:14 PM, Geert Uytterhoeven wrote: > Hi Rafael, > > On Wed, Feb 22, 2017 at 2:14 AM, Rafael J. Wysocki wrote: >> On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: >>> On 21/02/17 18:27, Sudeep Holla wrote: >>> > On 21/02/17 17:51, Sudeep Holla wrote: >>> >> On

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Sudeep Holla
On 22/02/17 13:38, Geert Uytterhoeven wrote: > Hi Sudeep, > > On Wed, Feb 22, 2017 at 12:03 PM, Sudeep Holla wrote: >> On 22/02/17 01:14, Rafael J. Wysocki wrote: >>> On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: >> >> [...] >> I take this back, you have everything you need

Re: [PATCH/RFC 3/6] drivers: firmware: psci: Implement shallow suspend mode

2017-02-22 Thread Sudeep Holla
On 22/02/17 13:47, Geert Uytterhoeven wrote: > Hi Sudeep, > > On Tue, Feb 21, 2017 at 6:22 PM, Sudeep Holla wrote: [...] >> >> IIUC, it's not implemented today. I can't talk about future ;), but your > > Good, so there's no need for the DT property, and drivers/firmware/psci.c > should aways

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Rafael J. Wysocki
On Wed, Feb 22, 2017 at 3:32 PM, Sudeep Holla wrote: > > > On 22/02/17 13:38, Geert Uytterhoeven wrote: >> Hi Sudeep, >> >> On Wed, Feb 22, 2017 at 12:03 PM, Sudeep Holla wrote: >>> On 22/02/17 01:14, Rafael J. Wysocki wrote: On Tuesday, February 21, 2017 06:45:13 PM Sudeep Holla wrote: >>>

Re: [PATCH/RFC 4/6] drivers: firmware: psci: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Rafael J. Wysocki
On Wed, Feb 22, 2017 at 3:05 PM, Geert Uytterhoeven wrote: > Hi Mark, > > On Tue, Feb 21, 2017 at 6:48 PM, Mark Rutland wrote: >> On Mon, Feb 20, 2017 at 09:33:27PM +0100, Geert Uytterhoeven wrote: >>> @@ -440,12 +442,14 @@ static int psci_system_suspend_valid(suspend_state_t >>> state) >>> sta

Re: [PATCH/RFC 0/6] PSCI: Fix non-PMIC wake-up if SYSTEM_SUSPEND cuts power

2017-02-22 Thread Sudeep Holla
On 22/02/17 14:50, Rafael J. Wysocki wrote: > On Wed, Feb 22, 2017 at 3:32 PM, Sudeep Holla wrote: [...] >> >> OK, I thought I had told this before. What do you mean by PSCI system >> suspend can't wakeup from the configured wakeup source. You just said >> above that you can wake up from the s

[PATCH 1/1] scripts: bin2png.sh: Fix POSIX shell compliance

2017-02-22 Thread Kieran Bingham
From: Kieran Bingham The string manipulations of the form ${VAR/search/replace} are not posix compliant. Use string deletion, and append instead. Signed-off-by: Kieran Bingham --- scripts/bin2png.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/bin2png.sh b/sc

RE: [PATCH v2 1/3] power: reset: Add Renesas reset driver

2017-02-22 Thread Chris Brandt
Hello Guenter, On Wednesday, February 22, 2017, Guenter Roeck wrote: > > Looking through the drivers in the watchdog directly, I see default > > timeouts of 20, 30, 60, and 120 seconds. > > > 30 and 60 are pretty common for default timeouts, though I personally > think they are a bit long. If you

Re: [PATCH v2 1/3] power: reset: Add Renesas reset driver

2017-02-22 Thread Guenter Roeck
On Wed, Feb 22, 2017 at 01:35:12PM +, Chris Brandt wrote: > Hello Geert and Guenter, > > On Thursday, February 16, 2017, Guenter Roeck wrote: > > FWIW, the watchdog subsystem should support that easily, even with 125 ms > > hardware timeout. We added that capability for that very purpose. That

Re: [PATCH 1/1] scripts: bin2png.sh: Fix POSIX shell compliance

2017-02-22 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Wednesday 22 Feb 2017 15:52:43 Kieran Bingham wrote: > From: Kieran Bingham > > The string manipulations of the form ${VAR/search/replace} are not posix > compliant. Use string deletion, and append instead. > > Signed-off-by: Kieran Bingham Reviewed-by,

Re: The failure summary report of GEN2 for linux stable v4.10-rc2

2017-02-22 Thread Laurent Pinchart
Hello Hiep, On Wednesday 22 Feb 2017 21:23:58 Hiep Cao Minh wrote: > Hello Laurent, > > I am sorry for the late reply!. No worries. > >>> I've investigated the problem further here now that I'm back home with > >>> access to my Lager board, but have been unable to reproduce it even by > >>> tes

Re: [PATCH RESEND 1/1] media: platform: Renesas IMR driver

2017-02-22 Thread Sergei Shtylyov
Hello! On 02/22/2017 05:25 PM, Rob Herring wrote: From: Konstantin Kozhevnikov The image renderer light extended 4 (IMR-LX4) or the distortion correction engine is a drawing processor with a simple instruction system capable of referencing data on an external memory as 2D texture data and pe

[PATCHv2] Checkpatch fixes for staging/drivers/ks7010.

2017-02-22 Thread Matthew Giassa
Wrapping complex macros with parentheses. Adding extra spaces between macros and integer literals. Indentation changes to remove checkpatch warnings. Swapping out C++-style comments with C-style comments. Signed-off-by: Matthew Giassa --- drivers/staging/ks7010/ks_wlan_ioctl.h | 74 +