Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger & fuel-gauge device

2014-09-27 Thread Jonathan Cameron
On 23/09/14 11:53, Adam Thomson wrote: > DA9150 is a combined Charger and Fuel-Gauge IC, with additional > GPIO and GPADC functionality. > > Signed-off-by: Adam Thomson A random suggestion to reduced the massive set of defines... Why not just compute the shift from the mask where needed? Just use

Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger fuel-gauge device

2014-09-27 Thread Jonathan Cameron
On 23/09/14 11:53, Adam Thomson wrote: DA9150 is a combined Charger and Fuel-Gauge IC, with additional GPIO and GPADC functionality. Signed-off-by: Adam Thomson adam.thomson.opensou...@diasemi.com A random suggestion to reduced the massive set of defines... Why not just compute the shift from

[PATCH 1/8] mfd: Add support for DA9150 combined charger & fuel-gauge device

2014-09-23 Thread Adam Thomson
DA9150 is a combined Charger and Fuel-Gauge IC, with additional GPIO and GPADC functionality. Signed-off-by: Adam Thomson --- drivers/mfd/Kconfig | 12 + drivers/mfd/Makefile |2 +- drivers/mfd/da9150-core.c| 413

[PATCH 1/8] mfd: Add support for DA9150 combined charger fuel-gauge device

2014-09-23 Thread Adam Thomson
DA9150 is a combined Charger and Fuel-Gauge IC, with additional GPIO and GPADC functionality. Signed-off-by: Adam Thomson adam.thomson.opensou...@diasemi.com --- drivers/mfd/Kconfig | 12 + drivers/mfd/Makefile |2 +- drivers/mfd/da9150-core.c|

Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger & fuel-gauge device

2014-06-21 Thread Jonathan Cameron
On 16/06/14 14:12, Opensource [Adam Thomson] wrote: On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: Hi Adam, Some general comments inline. It's been a while since I've looked at any particularly similar parts, but it seems to me that a lot of indirection gets added here that if

Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger fuel-gauge device

2014-06-21 Thread Jonathan Cameron
On 16/06/14 14:12, Opensource [Adam Thomson] wrote: On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: Hi Adam, Some general comments inline. It's been a while since I've looked at any particularly similar parts, but it seems to me that a lot of indirection gets added here that if

Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger & fuel-gauge device

2014-06-16 Thread Jonathan Cameron
On 16/06/14 14:12, Opensource [Adam Thomson] wrote: On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: Hi Adam, Some general comments inline. It's been a while since I've looked at any particularly similar parts, but it seems to me that a lot of indirection gets added here that if

RE: [PATCH 1/8] mfd: Add support for DA9150 combined charger & fuel-gauge device

2014-06-16 Thread Opensource [Adam Thomson]
On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: > Hi Adam, > > Some general comments inline. > > It's been a while since I've looked at any particularly similar parts, > but it seems to me that a lot of indirection gets added here that > if anything makes the codes slightly harder to

RE: [PATCH 1/8] mfd: Add support for DA9150 combined charger fuel-gauge device

2014-06-16 Thread Opensource [Adam Thomson]
On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: Hi Adam, Some general comments inline. It's been a while since I've looked at any particularly similar parts, but it seems to me that a lot of indirection gets added here that if anything makes the codes slightly harder to follow...

Re: [PATCH 1/8] mfd: Add support for DA9150 combined charger fuel-gauge device

2014-06-16 Thread Jonathan Cameron
On 16/06/14 14:12, Opensource [Adam Thomson] wrote: On Sun, Jun 15, 2014 at 20:49, Jonathan Cameron wrote: Hi Adam, Some general comments inline. It's been a while since I've looked at any particularly similar parts, but it seems to me that a lot of indirection gets added here that if