Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Lee Jones
On Mon, 21 Mar 2016, Dan Carpenter wrote: > Gar, the truth is I'm never going to remember to capitalize patches for > you. No one else has that rule. Plus the last two times we had this It's an awesome rule. Everyone should have that rule. ;) > discussion the patches I was fixing were not

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Lee Jones
On Mon, 21 Mar 2016, Dan Carpenter wrote: > Gar, the truth is I'm never going to remember to capitalize patches for > you. No one else has that rule. Plus the last two times we had this It's an awesome rule. Everyone should have that rule. ;) > discussion the patches I was fixing were not

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Dan Carpenter
Gar, the truth is I'm never going to remember to capitalize patches for you. No one else has that rule. Plus the last two times we had this discussion the patches I was fixing were not capitalized either. git log -p --author="Dan Carpenter" drivers/mfd | grep Fixes | head -n 2 regards, dan

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Dan Carpenter
Gar, the truth is I'm never going to remember to capitalize patches for you. No one else has that rule. Plus the last two times we had this discussion the patches I was fixing were not capitalized either. git log -p --author="Dan Carpenter" drivers/mfd | grep Fixes | head -n 2 regards, dan

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Lee Jones
On Fri, 11 Mar 2016, Dan Carpenter wrote: > Instead to being true/false, the "handled" is true/uninitialized. > Presumably this doesn't cause that many problems in real life because > normally we handle the IRQ. > > Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') > Signed-off-by: Dan

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-21 Thread Lee Jones
On Fri, 11 Mar 2016, Dan Carpenter wrote: > Instead to being true/false, the "handled" is true/uninitialized. > Presumably this doesn't cause that many problems in real life because > normally we handle the IRQ. > > Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') > Signed-off-by: Dan

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-13 Thread Kim, Milo
On 3/11/2016 5:11 PM, Dan Carpenter wrote: Instead to being true/false, the "handled" is true/uninitialized. Presumably this doesn't cause that many problems in real life because normally we handle the IRQ. Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') Signed-off-by: Dan Carpenter

Re: [patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-13 Thread Kim, Milo
On 3/11/2016 5:11 PM, Dan Carpenter wrote: Instead to being true/false, the "handled" is true/uninitialized. Presumably this doesn't cause that many problems in real life because normally we handle the IRQ. Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') Signed-off-by: Dan Carpenter

[patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-11 Thread Dan Carpenter
Instead to being true/false, the "handled" is true/uninitialized. Presumably this doesn't cause that many problems in real life because normally we handle the IRQ. Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') Signed-off-by: Dan Carpenter diff --git

[patch] mfd: lp8788-irq: uninitialized variable in irq handler

2016-03-11 Thread Dan Carpenter
Instead to being true/false, the "handled" is true/uninitialized. Presumably this doesn't cause that many problems in real life because normally we handle the IRQ. Fixes: eea6b7cc53aa ('mfd: Add lp8788 mfd driver') Signed-off-by: Dan Carpenter diff --git a/drivers/mfd/lp8788-irq.c