Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Ulf, On Thu, Dec 14, 2017 at 3:13 PM, Ulf Hansson wrote: > On 14 December 2017 at 11:52, Geert Uytterhoeven wrote: >> On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: >>> On 11 December 2017 at 11:48, Geert

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On 14 December 2017 at 11:52, Geert Uytterhoeven wrote: > Hi Ulf, > > On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: >> On 11 December 2017 at 11:48, Geert Uytterhoeven >> wrote: >>> On Mon, Dec 11, 2017 at 11:24 AM, Ulf

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Ulf, On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: > On 11 December 2017 at 11:48, Geert Uytterhoeven wrote: >> On Mon, Dec 11, 2017 at 11:24 AM, Ulf Hansson wrote: >>> On 10 December 2017 at 11:16, Geert

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On 12 December 2017 at 09:16, Geert Uytterhoeven wrote: > Hi Ulf, > > On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: >> That together with an option of allowing "consumed resource-devices" >> (irqchip) to be included in the wakeup path. I am

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Ulf, On Mon, Dec 11, 2017 at 9:59 PM, Ulf Hansson wrote: > That together with an option of allowing "consumed resource-devices" > (irqchip) to be included in the wakeup path. I am thinking, perhaps > another driver PM flag (DPM_FLAG_WAKEUP_PATH), that the PM core looks

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On 11 December 2017 at 11:48, Geert Uytterhoeven wrote: > Hi Ulf, > > On Mon, Dec 11, 2017 at 11:24 AM, Ulf Hansson wrote: >> On 10 December 2017 at 11:16, Geert Uytterhoeven >> wrote: >>> On Sun, Dec 10, 2017 at 3:30 AM,

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Ulf, On Mon, Dec 11, 2017 at 11:24 AM, Ulf Hansson wrote: > On 10 December 2017 at 11:16, Geert Uytterhoeven wrote: >> On Sun, Dec 10, 2017 at 3:30 AM, Rafael J. Wysocki >> wrote: >>> On Monday, November 13, 2017 4:46:42 PM

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On 10 December 2017 at 11:16, Geert Uytterhoeven wrote: > Hi Rafael, Ulf, > > On Sun, Dec 10, 2017 at 3:30 AM, Rafael J. Wysocki wrote: >> On Monday, November 13, 2017 4:46:42 PM CET Ulf Hansson wrote: >>> For some bus types and PM domains, it's not

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Rafael, Ulf, On Sun, Dec 10, 2017 at 3:30 AM, Rafael J. Wysocki wrote: > On Monday, November 13, 2017 4:46:42 PM CET Ulf Hansson wrote: >> For some bus types and PM domains, it's not sufficient to only check the >> return value from device_may_wakeup(), to fully understand

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On 10 December 2017 at 03:30, Rafael J. Wysocki wrote: > On Monday, November 13, 2017 4:46:42 PM CET Ulf Hansson wrote: >> For some bus types and PM domains, it's not sufficient to only check the >> return value from device_may_wakeup(), to fully understand how to configure >>

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

On Monday, November 13, 2017 4:46:42 PM CET Ulf Hansson wrote: > For some bus types and PM domains, it's not sufficient to only check the > return value from device_may_wakeup(), to fully understand how to configure > wakeup settings for the device during system suspend. > > In particular,

Re: [PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

Hi Ulf, you have left some old naming WAKEUP_POWERED in your patch On 13 November 2017 at 16:46, Ulf Hansson wrote: > For some bus types and PM domains, it's not sufficient to only check the > return value from device_may_wakeup(), to fully understand how to configure >

[PATCH v2 2/3] PM / core: Add IN_BAND_WAKEUP driver flag

For some bus types and PM domains, it's not sufficient to only check the return value from device_may_wakeup(), to fully understand how to configure wakeup settings for the device during system suspend. In particular, sometimes the device may need to remain in its power state, in case the driver