Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-27 Thread Laxman Dewangan
On Saturday 27 July 2013 02:15 AM, Tomasz Figa wrote: On Friday 26 of July 2013 09:40:15 Stephen Warren wrote: (CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) On 07/25/2013 06:29 AM, Laxman Dewangan wrote: Palmas series device

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-27 Thread Laxman Dewangan
On Friday 26 July 2013 10:12 PM, Stephen Warren wrote: On 07/26/2013 10:35 AM, Mark Brown wrote: On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: Given there's only 1 battery, I think "back-" is redundant in the property names. Since that shortens the names a bit, I'd suggest

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-27 Thread Laxman Dewangan
On Friday 26 July 2013 10:12 PM, Stephen Warren wrote: On 07/26/2013 10:35 AM, Mark Brown wrote: On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: Given there's only 1 battery, I think back- is redundant in the property names. Since that shortens the names a bit, I'd suggest

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-27 Thread Laxman Dewangan
On Saturday 27 July 2013 02:15 AM, Tomasz Figa wrote: On Friday 26 of July 2013 09:40:15 Stephen Warren wrote: (CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) On 07/25/2013 06:29 AM, Laxman Dewangan wrote: Palmas series device

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Tomasz Figa
On Friday 26 of July 2013 09:40:15 Stephen Warren wrote: > (CC'ing the new DT binding maintainers and mailing list on this reply, > hence quoting the whole of the DT binding) > > On 07/25/2013 06:29 AM, Laxman Dewangan wrote: > > Palmas series device like TPS65913, TPS80036 supports the backup >

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Stephen Warren
On 07/26/2013 10:35 AM, Mark Brown wrote: > On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: > >> Given there's only 1 battery, I think "back-" is redundant in >> the property names. Since that shortens the names a bit, I'd >> suggest spelling everything out in full, perhaps: > >

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Mark Brown
On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: > Given there's only 1 battery, I think "back-" is redundant in the > property names. Since that shortens the names a bit, I'd suggest > spelling everything out in full, perhaps: Isn't there an integrated charger on the PMIC? If

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Mark Rutland
On Fri, Jul 26, 2013 at 04:40:15PM +0100, Stephen Warren wrote: > (CC'ing the new DT binding maintainers and mailing list on this reply, > hence quoting the whole of the DT binding) Thanks Stephen. > > On 07/25/2013 06:29 AM, Laxman Dewangan wrote: > > Palmas series device like TPS65913,

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Stephen Warren
(CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) On 07/25/2013 06:29 AM, Laxman Dewangan wrote: > Palmas series device like TPS65913, TPS80036 supports the backup battery > for powering the RTC when no other energy source is

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Stephen Warren
(CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) On 07/25/2013 06:29 AM, Laxman Dewangan wrote: Palmas series device like TPS65913, TPS80036 supports the backup battery for powering the RTC when no other energy source is

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Mark Rutland
On Fri, Jul 26, 2013 at 04:40:15PM +0100, Stephen Warren wrote: (CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) Thanks Stephen. On 07/25/2013 06:29 AM, Laxman Dewangan wrote: Palmas series device like TPS65913, TPS80036

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Mark Brown
On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: Given there's only 1 battery, I think back- is redundant in the property names. Since that shortens the names a bit, I'd suggest spelling everything out in full, perhaps: Isn't there an integrated charger on the PMIC? If there

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Stephen Warren
On 07/26/2013 10:35 AM, Mark Brown wrote: On Fri, Jul 26, 2013 at 09:40:15AM -0600, Stephen Warren wrote: Given there's only 1 battery, I think back- is redundant in the property names. Since that shortens the names a bit, I'd suggest spelling everything out in full, perhaps: Isn't there

Re: [PATCH] drivers/rtc/rtc-palmas.c: support for backup battery charging

2013-07-26 Thread Tomasz Figa
On Friday 26 of July 2013 09:40:15 Stephen Warren wrote: (CC'ing the new DT binding maintainers and mailing list on this reply, hence quoting the whole of the DT binding) On 07/25/2013 06:29 AM, Laxman Dewangan wrote: Palmas series device like TPS65913, TPS80036 supports the backup