mbox series

[0/2] Add irq PM wakeup

Message ID 20210831120613.1514899-1-Basavaraj.Natikar@amd.com
Headers show
Series Add irq PM wakeup | expand

Message

Basavaraj Natikar Aug. 31, 2021, 12:06 p.m. UTC 
Changes include to handle IRQ wakeup mode wake the system from sleep
states like "suspend to RAM".

Basavaraj Natikar (2):
  pinctrl: amd: Add irq field data
  pinctrl: amd: Handle wake-up interrupt

 drivers/pinctrl/pinctrl-amd.c | 19 ++++++++++++++-----
 drivers/pinctrl/pinctrl-amd.h |  1 +
 2 files changed, 15 insertions(+), 5 deletions(-)

Comments

Shyam Sundar S K Sept. 7, 2021, 7:42 a.m. UTC | #1 
On 8/31/2021 5:36 PM, Basavaraj Natikar wrote:
> Changes include to handle IRQ wakeup mode wake the system from sleep

> states like "suspend to RAM".

> 

> Basavaraj Natikar (2):

>   pinctrl: amd: Add irq field data

>   pinctrl: amd: Handle wake-up interrupt

> 

>  drivers/pinctrl/pinctrl-amd.c | 19 ++++++++++++++-----

>  drivers/pinctrl/pinctrl-amd.h |  1 +

>  2 files changed, 15 insertions(+), 5 deletions(-)

> 


Series Looks good to me.

Acked-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
Mario Limonciello Sept. 7, 2021, 6:34 p.m. UTC | #2 
On 9/7/2021 02:42, Shyam Sundar S K wrote:
> 

> 

> On 8/31/2021 5:36 PM, Basavaraj Natikar wrote:

>> Changes include to handle IRQ wakeup mode wake the system from sleep

>> states like "suspend to RAM".

>>

>> Basavaraj Natikar (2):

>>    pinctrl: amd: Add irq field data

>>    pinctrl: amd: Handle wake-up interrupt

>>

>>   drivers/pinctrl/pinctrl-amd.c | 19 ++++++++++++++-----

>>   drivers/pinctrl/pinctrl-amd.h |  1 +

>>   2 files changed, 15 insertions(+), 5 deletions(-)

>>

> 

> Series Looks good to me.

> 

> Acked-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>

> 


I've confirmed this helps wake-ups on some problematic systems as well 
myself.

Tested-by: Mario Limonciello <mario.limonciello@amd.com>
Mario Limonciello Sept. 15, 2021, 7:39 p.m. UTC | #3 
[Public]

> -----Original Message-----

> From: Limonciello, Mario

> Sent: Tuesday, September 7, 2021 13:35

> To: S-k, Shyam-sundar <Shyam-sundar.S-k@amd.com>; Natikar, Basavaraj

> <Basavaraj.Natikar@amd.com>; linus.walleij@linaro.org; linux-

> gpio@vger.kernel.org

> Subject: Re: [PATCH 0/2] Add irq PM wakeup

> 

> On 9/7/2021 02:42, Shyam Sundar S K wrote:

> >

> >

> > On 8/31/2021 5:36 PM, Basavaraj Natikar wrote:

> >> Changes include to handle IRQ wakeup mode wake the system from sleep

> >> states like "suspend to RAM".

> >>

> >> Basavaraj Natikar (2):

> >>    pinctrl: amd: Add irq field data

> >>    pinctrl: amd: Handle wake-up interrupt

> >>

> >>   drivers/pinctrl/pinctrl-amd.c | 19 ++++++++++++++-----

> >>   drivers/pinctrl/pinctrl-amd.h |  1 +

> >>   2 files changed, 15 insertions(+), 5 deletions(-)

> >>

> >

> > Series Looks good to me.

> >

> > Acked-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>

> >

> 

> I've confirmed this helps wake-ups on some problematic systems as well

> myself.

> 

> Tested-by: Mario Limonciello <mario.limonciello@amd.com>


Linus Walleij, Bartosz,

Ping on this one? 

Thanks,
Linus Walleij Sept. 16, 2021, 10:49 p.m. UTC | #4 
On Tue, Aug 31, 2021 at 2:06 PM Basavaraj Natikar
<Basavaraj.Natikar@amd.com> wrote:

> Changes include to handle IRQ wakeup mode wake the system from sleep

> states like "suspend to RAM".

>

> Basavaraj Natikar (2):

>   pinctrl: amd: Add irq field data

>   pinctrl: amd: Handle wake-up interrupt


Patches applied for fixes.

Thanks! And sorry for taking so long.

Yours,
Linus Walleij