Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-11 Thread Doug Anderson
Hi, On Thu, Aug 6, 2020 at 10:33 AM Sibi Sankar wrote: > > On 2020-08-06 22:40, Doug Anderson wrote: > > Hi, > > > > On Thu, Aug 6, 2020 at 7:36 AM Sibi Sankar > > wrote: > >> > >> On 2020-08-06 04:32, Stephen Boyd wrote: > >> > +Sibi who wrote the code > >> > > >> > Quoting Doug Anderson (2020-

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-06 Thread Sibi Sankar
On 2020-08-06 04:32, Stephen Boyd wrote: +Sibi who wrote the code Quoting Doug Anderson (2020-08-05 13:24:06) On Wed, Aug 5, 2020 at 10:36 AM Stephen Boyd wrote: > > Why is the genpd being powered off at all? It looks like the driver is > written in a way that it doesn't expect this to happ

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-06 Thread Sibi Sankar
On 2020-08-06 22:40, Doug Anderson wrote: Hi, On Thu, Aug 6, 2020 at 7:36 AM Sibi Sankar wrote: On 2020-08-06 04:32, Stephen Boyd wrote: > +Sibi who wrote the code > > Quoting Doug Anderson (2020-08-05 13:24:06) >> >> On Wed, Aug 5, 2020 at 10:36 AM Stephen Boyd >> wrote: >> > >> > Why is t

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-06 Thread Doug Anderson
Hi, On Thu, Aug 6, 2020 at 7:36 AM Sibi Sankar wrote: > > On 2020-08-06 04:32, Stephen Boyd wrote: > > +Sibi who wrote the code > > > > Quoting Doug Anderson (2020-08-05 13:24:06) > >> > >> On Wed, Aug 5, 2020 at 10:36 AM Stephen Boyd > >> wrote: > >> > > >> > Why is the genpd being powered off

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-05 Thread Stephen Boyd
+Sibi who wrote the code Quoting Doug Anderson (2020-08-05 13:24:06) > > On Wed, Aug 5, 2020 at 10:36 AM Stephen Boyd wrote: > > > > Why is the genpd being powered off at all? It looks like the driver is > > written in a way that it doesn't expect this to happen. See where > > adsp_pds_disable()

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-05 Thread Doug Anderson
Hi, On Wed, Aug 5, 2020 at 10:36 AM Stephen Boyd wrote: > > Quoting Douglas Anderson (2020-08-05 09:16:10) > > Running suspend/resume tests on a sc7180-based board with a modem I > > found that both system suspend and system resume would hang for 1 > > second. These messages indicate where: > >

Re: [PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-05 Thread Stephen Boyd
Quoting Douglas Anderson (2020-08-05 09:16:10) > Running suspend/resume tests on a sc7180-based board with a modem I > found that both system suspend and system resume would hang for 1 > second. These messages indicate where: > > genpd genpd:0:408.remoteproc: calling genpd_suspend_noirq+0x0

[PATCH 1/2] soc: qcom: aoss: Don't wait for IRQ if we might be in suspend/resume noirq

2020-08-05 Thread Douglas Anderson
Running suspend/resume tests on a sc7180-based board with a modem I found that both system suspend and system resume would hang for 1 second. These messages indicate where: genpd genpd:0:408.remoteproc: calling genpd_suspend_noirq+0x0/0x2c @ 18659, parent: none genpd genpd:0:408.remo