Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Rafael J. Wysocki [171108 00:47]: > On Wednesday, November 8, 2017 1:28:24 AM CET Florian Fainelli wrote: > > On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > > > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > > >> * Florian Fainelli

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Rafael J. Wysocki [171108 00:47]: > On Wednesday, November 8, 2017 1:28:24 AM CET Florian Fainelli wrote: > > On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > > > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > > >> * Florian Fainelli [171104 17:21]: > > >>> > > >>> > >

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Rafael J. Wysocki [171108 00:25]: > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > > For runtime PM, restoring the state constantly is unnecessary and not > > good for battery life. The logic can be just: > > > > 1. Device driver runtime PM suspend saves

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Rafael J. Wysocki [171108 00:25]: > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > > For runtime PM, restoring the state constantly is unnecessary and not > > good for battery life. The logic can be just: > > > > 1. Device driver runtime PM suspend saves the state when

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Rafael J. Wysocki
On Wednesday, November 8, 2017 1:28:24 AM CET Florian Fainelli wrote: > On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > >> * Florian Fainelli [171104 17:21]: > >>> > >>> > >>> On 11/04/2017 05:25 AM,

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Rafael J. Wysocki
On Wednesday, November 8, 2017 1:28:24 AM CET Florian Fainelli wrote: > On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > >> * Florian Fainelli [171104 17:21]: > >>> > >>> > >>> On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote:

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Florian Fainelli
On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: >> * Florian Fainelli [171104 17:21]: >>> >>> >>> On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: On Friday, November 3, 2017 6:33:53 PM CET Tony

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Florian Fainelli
On 11/07/2017 04:23 PM, Rafael J. Wysocki wrote: > On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: >> * Florian Fainelli [171104 17:21]: >>> >>> >>> On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > *

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Rafael J. Wysocki
On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > * Florian Fainelli [171104 17:21]: > > > > > > On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > > > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > > >> * Florian Fainelli

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Rafael J. Wysocki
On Tuesday, November 7, 2017 5:00:06 PM CET Tony Lindgren wrote: > * Florian Fainelli [171104 17:21]: > > > > > > On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > > > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > > >> * Florian Fainelli [171103 17:04]: > > >>> On

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Florian Fainelli [171104 17:21]: > > > On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > >> * Florian Fainelli [171103 17:04]: > >>> On 11/03/2017 09:11 AM, Tony Lindgren wrote: >

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Charles Keepax [171104 08:38]: > On Fri, Nov 03, 2017 at 10:33:53AM -0700, Tony Lindgren wrote: > > * Florian Fainelli [171103 17:04]: > > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > > The pinctrl provider is losing its state, hence

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Florian Fainelli [171104 17:21]: > > > On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > >> * Florian Fainelli [171103 17:04]: > >>> On 11/03/2017 09:11 AM, Tony Lindgren wrote: > >>> The pinctrl provider is losing its

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-07 Thread Tony Lindgren
* Charles Keepax [171104 08:38]: > On Fri, Nov 03, 2017 at 10:33:53AM -0700, Tony Lindgren wrote: > > * Florian Fainelli [171103 17:04]: > > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > > The pinctrl provider is losing its state, hence these two patches. > > > > OK > > > > > > Anyways,

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Florian Fainelli
On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: >> * Florian Fainelli [171103 17:04]: >>> On 11/03/2017 09:11 AM, Tony Lindgren wrote: >>> The pinctrl provider is losing its state, hence these two patches.

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Florian Fainelli
On 11/04/2017 05:25 AM, Rafael J. Wysocki wrote: > On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: >> * Florian Fainelli [171103 17:04]: >>> On 11/03/2017 09:11 AM, Tony Lindgren wrote: >>> The pinctrl provider is losing its state, hence these two patches. >> >> OK >>

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Rafael J. Wysocki
On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > * Florian Fainelli [171103 17:04]: > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > The pinctrl provider is losing its state, hence these two patches. > > OK > > > > Anyways, the context lost flag should

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Rafael J. Wysocki
On Friday, November 3, 2017 6:33:53 PM CET Tony Lindgren wrote: > * Florian Fainelli [171103 17:04]: > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > The pinctrl provider is losing its state, hence these two patches. > > OK > > > > Anyways, the context lost flag should be managed in the PM

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Charles Keepax
On Fri, Nov 03, 2017 at 10:33:53AM -0700, Tony Lindgren wrote: > * Florian Fainelli [171103 17:04]: > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > The pinctrl provider is losing its state, hence these two patches. > > OK > > > > Anyways, the context lost flag should

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-04 Thread Charles Keepax
On Fri, Nov 03, 2017 at 10:33:53AM -0700, Tony Lindgren wrote: > * Florian Fainelli [171103 17:04]: > > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > > The pinctrl provider is losing its state, hence these two patches. > > OK > > > > Anyways, the context lost flag should be managed in the PM

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Tony Lindgren
* Florian Fainelli [171103 17:04]: > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > The pinctrl provider is losing its state, hence these two patches. OK > > Anyways, the context lost flag should be managed in the PM core for > > the device, so adding linux-pm and Rafael

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Tony Lindgren
* Florian Fainelli [171103 17:04]: > On 11/03/2017 09:11 AM, Tony Lindgren wrote: > The pinctrl provider is losing its state, hence these two patches. OK > > Anyways, the context lost flag should be managed in the PM core for > > the device, so adding linux-pm and Rafael to Cc. > > I don't

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Florian Fainelli
On 11/03/2017 03:37 AM, Charles Keepax wrote: > On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: >> Hello Linus, >> >> It's me again, so I have been thinking about the problem originally >> reported in: [PATCH fixes v3] pinctrl: Really force states during >> suspend/resume >> >>

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Florian Fainelli
On 11/03/2017 03:37 AM, Charles Keepax wrote: > On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: >> Hello Linus, >> >> It's me again, so I have been thinking about the problem originally >> reported in: [PATCH fixes v3] pinctrl: Really force states during >> suspend/resume >> >>

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Florian Fainelli
On 11/03/2017 09:11 AM, Tony Lindgren wrote: > * Charles Keepax [171103 10:38]: >> On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: >>> Hello Linus, >>> >>> It's me again, so I have been thinking about the problem originally >>> reported in: [PATCH

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Florian Fainelli
On 11/03/2017 09:11 AM, Tony Lindgren wrote: > * Charles Keepax [171103 10:38]: >> On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: >>> Hello Linus, >>> >>> It's me again, so I have been thinking about the problem originally >>> reported in: [PATCH fixes v3] pinctrl: Really force

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Tony Lindgren
* Charles Keepax [171103 10:38]: > On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: > > Hello Linus, > > > > It's me again, so I have been thinking about the problem originally > > reported in: [PATCH fixes v3] pinctrl: Really force states during

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Tony Lindgren
* Charles Keepax [171103 10:38]: > On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: > > Hello Linus, > > > > It's me again, so I have been thinking about the problem originally > > reported in: [PATCH fixes v3] pinctrl: Really force states during > > suspend/resume > > > > and

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Charles Keepax
On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: > Hello Linus, > > It's me again, so I have been thinking about the problem originally > reported in: [PATCH fixes v3] pinctrl: Really force states during > suspend/resume > > and other similar patches a while ago, and this new

Re: [PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-03 Thread Charles Keepax
On Thu, Nov 02, 2017 at 04:15:49PM -0700, Florian Fainelli wrote: > Hello Linus, > > It's me again, so I have been thinking about the problem originally > reported in: [PATCH fixes v3] pinctrl: Really force states during > suspend/resume > > and other similar patches a while ago, and this new

[PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-02 Thread Florian Fainelli
Hello Linus, It's me again, so I have been thinking about the problem originally reported in: [PATCH fixes v3] pinctrl: Really force states during suspend/resume and other similar patches a while ago, and this new version allows a platform using pinctrl-single to specify whether its pins are

[PATCH v2 0/2] pinctrl: Allow indicating loss of state across suspend/resume

2017-11-02 Thread Florian Fainelli
Hello Linus, It's me again, so I have been thinking about the problem originally reported in: [PATCH fixes v3] pinctrl: Really force states during suspend/resume and other similar patches a while ago, and this new version allows a platform using pinctrl-single to specify whether its pins are