Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-09 Thread Nishanth Menon
On 05/08/2014 08:03 PM, Tony Lindgren wrote: > * Nishanth Menon [140506 17:25]: >> Subject: [PATCH] power: twl4030_charger: detect battery presence prior to >> enabling charger >> >> TWL4030's Battery Charger seems to be designed for non-hotpluggable >> batteries. >> >> If battery is not present

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-09 Thread Nishanth Menon
On 05/08/2014 08:03 PM, Tony Lindgren wrote: * Nishanth Menon n...@ti.com [140506 17:25]: Subject: [PATCH] power: twl4030_charger: detect battery presence prior to enabling charger TWL4030's Battery Charger seems to be designed for non-hotpluggable batteries. If battery is not present in

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-08 Thread Tony Lindgren
* Nishanth Menon [140506 17:25]: > Subject: [PATCH] power: twl4030_charger: detect battery presence prior to > enabling charger > > TWL4030's Battery Charger seems to be designed for non-hotpluggable > batteries. > > If battery is not present in the system, BATSTS is always set with the >

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-08 Thread Tony Lindgren
* Nishanth Menon n...@ti.com [140506 17:25]: Subject: [PATCH] power: twl4030_charger: detect battery presence prior to enabling charger TWL4030's Battery Charger seems to be designed for non-hotpluggable batteries. If battery is not present in the system, BATSTS is always set with the

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-06 Thread Nishanth Menon
On 16:00-20140425, Felipe Balbi wrote: > On Fri, Apr 25, 2014 at 03:58:10PM -0500, Nishanth Menon wrote: > > On 04/16/2014 11:35 AM, Tony Lindgren wrote: > > > * Felipe Balbi [140416 08:18]: > > >> TRM says we *must* write 1 to each bit we're handling > > >> in order to clear the IRQ status and

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-05-06 Thread Nishanth Menon
On 16:00-20140425, Felipe Balbi wrote: On Fri, Apr 25, 2014 at 03:58:10PM -0500, Nishanth Menon wrote: On 04/16/2014 11:35 AM, Tony Lindgren wrote: * Felipe Balbi ba...@ti.com [140416 08:18]: TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-25 Thread Felipe Balbi
On Fri, Apr 25, 2014 at 03:58:10PM -0500, Nishanth Menon wrote: > On 04/16/2014 11:35 AM, Tony Lindgren wrote: > > * Felipe Balbi [140416 08:18]: > >> TRM says we *must* write 1 to each bit we're handling > >> in order to clear the IRQ status and bring IRQ line > >> low. This patch implements

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-25 Thread Nishanth Menon
On 04/16/2014 11:35 AM, Tony Lindgren wrote: > * Felipe Balbi [140416 08:18]: >> TRM says we *must* write 1 to each bit we're handling >> in order to clear the IRQ status and bring IRQ line >> low. This patch implements that. >> >> Signed-off-by: Felipe Balbi >> --- >> >> Russell, I don't have

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-25 Thread Nishanth Menon
On 04/16/2014 11:35 AM, Tony Lindgren wrote: * Felipe Balbi ba...@ti.com [140416 08:18]: TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and bring IRQ line low. This patch implements that. Signed-off-by: Felipe Balbi ba...@ti.com --- Russell, I don't

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-25 Thread Felipe Balbi
On Fri, Apr 25, 2014 at 03:58:10PM -0500, Nishanth Menon wrote: On 04/16/2014 11:35 AM, Tony Lindgren wrote: * Felipe Balbi ba...@ti.com [140416 08:18]: TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and bring IRQ line low. This patch implements

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-16 Thread Tony Lindgren
* Felipe Balbi [140416 08:18]: > TRM says we *must* write 1 to each bit we're handling > in order to clear the IRQ status and bring IRQ line > low. This patch implements that. > > Signed-off-by: Felipe Balbi > --- > > Russell, I don't have HW to test, but this should > solve the problem you

[PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-16 Thread Felipe Balbi
TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and bring IRQ line low. This patch implements that. Signed-off-by: Felipe Balbi --- Russell, I don't have HW to test, but this should solve the problem you saw when not using battery with Zoom board. Let me

[PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-16 Thread Felipe Balbi
TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and bring IRQ line low. This patch implements that. Signed-off-by: Felipe Balbi ba...@ti.com --- Russell, I don't have HW to test, but this should solve the problem you saw when not using battery with Zoom

Re: [PATCH] power: twl4030_charger: clear IRQs after handling them

2014-04-16 Thread Tony Lindgren
* Felipe Balbi ba...@ti.com [140416 08:18]: TRM says we *must* write 1 to each bit we're handling in order to clear the IRQ status and bring IRQ line low. This patch implements that. Signed-off-by: Felipe Balbi ba...@ti.com --- Russell, I don't have HW to test, but this should solve the