Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-27 Thread Andy Shevchenko
On Thu, Feb 23, 2017 at 12:30 PM, Linus Walleij wrote: > On Wed, Feb 8, 2017 at 10:46 PM, Andy Shevchenko > wrote: > >> Btw, I have got similar issue and thinking about those states they are >> quite orthogonal to the pin states. Wouldn't be

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-27 Thread Andy Shevchenko
On Thu, Feb 23, 2017 at 12:30 PM, Linus Walleij wrote: > On Wed, Feb 8, 2017 at 10:46 PM, Andy Shevchenko > wrote: > >> Btw, I have got similar issue and thinking about those states they are >> quite orthogonal to the pin states. Wouldn't be better to actually >> differentiate PM related states

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-27 Thread Florian Fainelli
On 02/23/2017 02:23 AM, Linus Walleij wrote: > Hi Florian, a big sorry for not looking into this earlier, I've been > having a mess in my inbox... and too much fun with the Gemini > hehe :) > > On Wed, Feb 8, 2017 at 2:17 AM, Florian Fainelli wrote: > >> In case a platform

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-27 Thread Florian Fainelli
On 02/23/2017 02:23 AM, Linus Walleij wrote: > Hi Florian, a big sorry for not looking into this earlier, I've been > having a mess in my inbox... and too much fun with the Gemini > hehe :) > > On Wed, Feb 8, 2017 at 2:17 AM, Florian Fainelli wrote: > >> In case a platform only defaults a

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-23 Thread Linus Walleij
On Wed, Feb 8, 2017 at 10:46 PM, Andy Shevchenko wrote: > Btw, I have got similar issue and thinking about those states they are > quite orthogonal to the pin states. Wouldn't be better to actually > differentiate PM related states and pin states? I don't fully

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-23 Thread Linus Walleij
On Wed, Feb 8, 2017 at 10:46 PM, Andy Shevchenko wrote: > Btw, I have got similar issue and thinking about those states they are > quite orthogonal to the pin states. Wouldn't be better to actually > differentiate PM related states and pin states? I don't fully understand what you mean here,

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-23 Thread Linus Walleij
Hi Florian, a big sorry for not looking into this earlier, I've been having a mess in my inbox... and too much fun with the Gemini hehe :) On Wed, Feb 8, 2017 at 2:17 AM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a >

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-23 Thread Linus Walleij
Hi Florian, a big sorry for not looking into this earlier, I've been having a mess in my inbox... and too much fun with the Gemini hehe :) On Wed, Feb 8, 2017 at 2:17 AM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a > "sleep" set of pins, and

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-14 Thread Florian Fainelli
On 02/07/2017 05:17 PM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a > "sleep" set of pins, and this particular platform suspends and resumes > in a way that the pin states are not preserved by the hardware, when we > resume, we would call

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-14 Thread Florian Fainelli
On 02/07/2017 05:17 PM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a > "sleep" set of pins, and this particular platform suspends and resumes > in a way that the pin states are not preserved by the hardware, when we > resume, we would call

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-08 Thread Florian Fainelli
On 02/08/2017 01:46 PM, Andy Shevchenko wrote: > On Wed, Feb 8, 2017 at 3:17 AM, Florian Fainelli wrote: >> In case a platform only defaults a "default" set of pins, but not a >> "sleep" set of pins, and this particular platform suspends and resumes >> in a way that the pin

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-08 Thread Florian Fainelli
On 02/08/2017 01:46 PM, Andy Shevchenko wrote: > On Wed, Feb 8, 2017 at 3:17 AM, Florian Fainelli wrote: >> In case a platform only defaults a "default" set of pins, but not a >> "sleep" set of pins, and this particular platform suspends and resumes >> in a way that the pin states are not

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-08 Thread Andy Shevchenko
On Wed, Feb 8, 2017 at 3:17 AM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a > "sleep" set of pins, and this particular platform suspends and resumes > in a way that the pin states are not preserved by the hardware, when we >

Re: [PATCH] pinctrl: Really force states during suspend/resume

2017-02-08 Thread Andy Shevchenko
On Wed, Feb 8, 2017 at 3:17 AM, Florian Fainelli wrote: > In case a platform only defaults a "default" set of pins, but not a > "sleep" set of pins, and this particular platform suspends and resumes > in a way that the pin states are not preserved by the hardware, when we > resume, we would call

[PATCH] pinctrl: Really force states during suspend/resume

2017-02-07 Thread Florian Fainelli
In case a platform only defaults a "default" set of pins, but not a "sleep" set of pins, and this particular platform suspends and resumes in a way that the pin states are not preserved by the hardware, when we resume, we would call pinctrl_single_resume() -> pinctrl_force_default() ->

[PATCH] pinctrl: Really force states during suspend/resume

2017-02-07 Thread Florian Fainelli
In case a platform only defaults a "default" set of pins, but not a "sleep" set of pins, and this particular platform suspends and resumes in a way that the pin states are not preserved by the hardware, when we resume, we would call pinctrl_single_resume() -> pinctrl_force_default() ->