Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-07 Thread Rafael J. Wysocki
On Tue, Jul 7, 2015 at 10:11 AM, Felipe Balbi wrote: > On Tue, Jul 07, 2015 at 12:40:53AM -0700, Tony Lindgren wrote: >> * Rafael J. Wysocki [150706 15:49]: >> > On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: >> > > on a first call to dev_pm_attach_wake_irq(), if it >> > > fails, it wil

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-07 Thread Felipe Balbi
On Tue, Jul 07, 2015 at 12:40:53AM -0700, Tony Lindgren wrote: > * Rafael J. Wysocki [150706 15:49]: > > On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: > > > on a first call to dev_pm_attach_wake_irq(), if it > > > fails, it will leave dev->power.wakeirq set to a > > > dangling pointer.

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-07 Thread Tony Lindgren
* Rafael J. Wysocki [150706 15:49]: > On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: > > on a first call to dev_pm_attach_wake_irq(), if it > > fails, it will leave dev->power.wakeirq set to a > > dangling pointer. Instead, let's clear it to make > > sure a subsequent call to dev_pm_atta

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-06 Thread Michael Trimarchi
Hi On Mon, Jul 6, 2015 at 8:09 PM, Felipe Balbi wrote: > On Mon, Jul 06, 2015 at 08:06:17PM +0200, Michael Trimarchi wrote: >> Hi >> >> On Jul 6, 2015 8:01 PM, "Felipe Balbi" wrote: >> > >> > on a first call to dev_pm_attach_wake_irq(), if it >> > fails, it will leave dev->power.wakeirq set to a

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-06 Thread Rafael J. Wysocki
On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: > on a first call to dev_pm_attach_wake_irq(), if it > fails, it will leave dev->power.wakeirq set to a > dangling pointer. Instead, let's clear it to make > sure a subsequent call to dev_pm_attach_wake_irq() > has chance to succeed. > > Cc:

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-06 Thread Felipe Balbi
On Mon, Jul 06, 2015 at 08:06:17PM +0200, Michael Trimarchi wrote: > Hi > > On Jul 6, 2015 8:01 PM, "Felipe Balbi" wrote: > > > > on a first call to dev_pm_attach_wake_irq(), if it > > fails, it will leave dev->power.wakeirq set to a > > dangling pointer. Instead, let's clear it to make > > sure

[PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-06 Thread Felipe Balbi
on a first call to dev_pm_attach_wake_irq(), if it fails, it will leave dev->power.wakeirq set to a dangling pointer. Instead, let's clear it to make sure a subsequent call to dev_pm_attach_wake_irq() has chance to succeed. Cc: Tony Lindgren Signed-off-by: Felipe Balbi --- drivers/base/power/wa