Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421 + bq27500

Hi Liam, > Am 08.09.2017 um 22:28 schrieb H. Nikolaus Schaller : > > Hi Liam, > >> Am 08.09.2017 um 22:19 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller >> wrote: >>> Hi

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421 + bq27500

Hi Liam, > Am 08.09.2017 um 22:28 schrieb H. Nikolaus Schaller : > > Hi Liam, > >> Am 08.09.2017 um 22:19 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller >> wrote: >>> Hi Liam, >>> I finally continues testing on OpenPandora. >>> Am

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

> Am 08.09.2017 um 22:57 schrieb Liam Breck : > > Howdy, > > On Fri, Sep 8, 2017 at 1:28 PM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> >>> Am 08.09.2017 um 22:19 schrieb Liam Breck : >>> >>> Hi Nikolaus, >>> >>> On Fri,

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

> Am 08.09.2017 um 22:57 schrieb Liam Breck : > > Howdy, > > On Fri, Sep 8, 2017 at 1:28 PM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> >>> Am 08.09.2017 um 22:19 schrieb Liam Breck : >>> >>> Hi Nikolaus, >>> >>> On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller >>> wrote: Hi

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Howdy, On Fri, Sep 8, 2017 at 1:28 PM, H. Nikolaus Schaller wrote: > Hi Liam, > >> Am 08.09.2017 um 22:19 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller >> wrote: >>> Hi Liam,

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Howdy, On Fri, Sep 8, 2017 at 1:28 PM, H. Nikolaus Schaller wrote: > Hi Liam, > >> Am 08.09.2017 um 22:19 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller >> wrote: >>> Hi Liam, >>> I finally continues testing on OpenPandora. >>> Am

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 08.09.2017 um 22:19 schrieb Liam Breck : > > Hi Nikolaus, > > On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> I finally continues testing on OpenPandora. >> >>> Am 31.08.2017 um 22:19 schrieb Liam Breck

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 08.09.2017 um 22:19 schrieb Liam Breck : > > Hi Nikolaus, > > On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> I finally continues testing on OpenPandora. >> >>> Am 31.08.2017 um 22:19 schrieb Liam Breck : >>> >>> Hi, >>> >>> This may be a fix

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller wrote: > Hi Liam, > I finally continues testing on OpenPandora. > >> Am 31.08.2017 um 22:19 schrieb Liam Breck : >> >> Hi, >> >> This may be a fix that allows >0 input from DT, but

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, On Fri, Sep 8, 2017 at 10:38 AM, H. Nikolaus Schaller wrote: > Hi Liam, > I finally continues testing on OpenPandora. > >> Am 31.08.2017 um 22:19 schrieb Liam Breck : >> >> Hi, >> >> This may be a fix that allows >0 input from DT, but won't try to >> program the register since the

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

> Am 08.09.2017 um 19:40 schrieb Pali Rohár : > > On Friday 08 September 2017 19:38:37 H. Nikolaus Schaller wrote: >> and here is the result: >> >>> root@letux:~# dmesg|fgrep bq27 >>> [ 10.391235] bq27xxx_battery_setup >>> [ 10.391265] bq27xxx_battery_setup:

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

> Am 08.09.2017 um 19:40 schrieb Pali Rohár : > > On Friday 08 September 2017 19:38:37 H. Nikolaus Schaller wrote: >> and here is the result: >> >>> root@letux:~# dmesg|fgrep bq27 >>> [ 10.391235] bq27xxx_battery_setup >>> [ 10.391265] bq27xxx_battery_setup: dm_regs=bf0520e0 >>> [

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

On Friday 08 September 2017 19:38:37 H. Nikolaus Schaller wrote: > and here is the result: > > > root@letux:~# dmesg|fgrep bq27 > > [ 10.391235] bq27xxx_battery_setup > > [ 10.391265] bq27xxx_battery_setup: dm_regs=bf0520e0 > > [ 10.393798] (NULL device *): hwmon: 'bq27500-1-0' is not a

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

On Friday 08 September 2017 19:38:37 H. Nikolaus Schaller wrote: > and here is the result: > > > root@letux:~# dmesg|fgrep bq27 > > [ 10.391235] bq27xxx_battery_setup > > [ 10.391265] bq27xxx_battery_setup: dm_regs=bf0520e0 > > [ 10.393798] (NULL device *): hwmon: 'bq27500-1-0' is not a

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, I finally continues testing on OpenPandora. > Am 31.08.2017 um 22:19 schrieb Liam Breck : > > Hi, > > This may be a fix that allows >0 input from DT, but won't try to > program the register since the first 3 fields aren't compatible: > > ... bq27500_dm_regs[] =

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, I finally continues testing on OpenPandora. > Am 31.08.2017 um 22:19 schrieb Liam Breck : > > Hi, > > This may be a fix that allows >0 input from DT, but won't try to > program the register since the first 3 fields aren't compatible: > > ... bq27500_dm_regs[] = { > ... >

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi, This may be a fix that allows >0 input from DT, but won't try to program the register since the first 3 fields aren't compatible: ... bq27500_dm_regs[] = { ... [bq27xxx_dm_design_energy] = { 0, 0, 0,0, 65535 }, /* missing on chip */ NB: align columns with other rows Pls CC me on

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi, This may be a fix that allows >0 input from DT, but won't try to program the register since the first 3 fields aren't compatible: ... bq27500_dm_regs[] = { ... [bq27xxx_dm_design_energy] = { 0, 0, 0,0, 65535 }, /* missing on chip */ NB: align columns with other rows Pls CC me on

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 31.08.2017 um 10:58 schrieb Liam Breck : > > On Wed, Aug 30, 2017 at 11:58 PM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> >>> Am 30.08.2017 um 13:24 schrieb Liam Breck : >>> >>> Hi Nikolaus, >>> >>> On Wed,

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 31.08.2017 um 10:58 schrieb Liam Breck : > > On Wed, Aug 30, 2017 at 11:58 PM, H. Nikolaus Schaller > wrote: >> Hi Liam, >> >>> Am 30.08.2017 um 13:24 schrieb Liam Breck : >>> >>> Hi Nikolaus, >>> >>> On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller >>> wrote: Hi

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

On Wed, Aug 30, 2017 at 11:58 PM, H. Nikolaus Schaller wrote: > Hi Liam, > >> Am 30.08.2017 um 13:24 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller >> wrote: >>> Hi Liam and

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

On Wed, Aug 30, 2017 at 11:58 PM, H. Nikolaus Schaller wrote: > Hi Liam, > >> Am 30.08.2017 um 13:24 schrieb Liam Breck : >> >> Hi Nikolaus, >> >> On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller >> wrote: >>> Hi Liam and Sebastian, >>> Am 29.08.2017 um 22:20 schrieb Liam Breck :

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 30.08.2017 um 13:24 schrieb Liam Breck : > > Hi Nikolaus, > > On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller > wrote: >> Hi Liam and Sebastian, >> >>> Am 29.08.2017 um 22:20 schrieb Liam Breck : >>> >>>

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam, > Am 30.08.2017 um 13:24 schrieb Liam Breck : > > Hi Nikolaus, > > On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller > wrote: >> Hi Liam and Sebastian, >> >>> Am 29.08.2017 um 22:20 schrieb Liam Breck : >>> >>> Hi Nikolaus, thanks for the patch... >>> >>> On Tue, Aug 29, 2017

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller wrote: > Hi Liam and Sebastian, > >> Am 29.08.2017 um 22:20 schrieb Liam Breck : >> >> Hi Nikolaus, thanks for the patch... >> >> On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, On Wed, Aug 30, 2017 at 2:30 AM, H. Nikolaus Schaller wrote: > Hi Liam and Sebastian, > >> Am 29.08.2017 um 22:20 schrieb Liam Breck : >> >> Hi Nikolaus, thanks for the patch... >> >> On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller >> wrote: >>> Tested on Pyra prototype

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam and Sebastian, > Am 29.08.2017 um 22:20 schrieb Liam Breck : > > Hi Nikolaus, thanks for the patch... > > On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller > wrote: >> Tested on Pyra prototype with bq27421. >> >> Signed-off-by: H.

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Liam and Sebastian, > Am 29.08.2017 um 22:20 schrieb Liam Breck : > > Hi Nikolaus, thanks for the patch... > > On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller > wrote: >> Tested on Pyra prototype with bq27421. >> >> Signed-off-by: H. Nikolaus Schaller >> --- >>

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi, On Tue, Aug 29, 2017 at 01:20:27PM -0700, Liam Breck wrote: > Hi Nikolaus, thanks for the patch... > > On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller > wrote: > > Tested on Pyra prototype with bq27421. > > > > Signed-off-by: H. Nikolaus Schaller

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi, On Tue, Aug 29, 2017 at 01:20:27PM -0700, Liam Breck wrote: > Hi Nikolaus, thanks for the patch... > > On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller > wrote: > > Tested on Pyra prototype with bq27421. > > > > Signed-off-by: H. Nikolaus Schaller > > --- > >

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, thanks for the patch... On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller wrote: > Tested on Pyra prototype with bq27421. > > Signed-off-by: H. Nikolaus Schaller > --- > drivers/power/supply/bq27xxx_battery.c | 2 +- > 1 file changed, 1

Re: [PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Hi Nikolaus, thanks for the patch... On Tue, Aug 29, 2017 at 1:10 PM, H. Nikolaus Schaller wrote: > Tested on Pyra prototype with bq27421. > > Signed-off-by: H. Nikolaus Schaller > --- > drivers/power/supply/bq27xxx_battery.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff

[PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Tested on Pyra prototype with bq27421. Signed-off-by: H. Nikolaus Schaller --- drivers/power/supply/bq27xxx_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c index

[PATCH resend v2] power: supply: bq27xxx: enable writing capacity values for bq27421

Tested on Pyra prototype with bq27421. Signed-off-by: H. Nikolaus Schaller --- drivers/power/supply/bq27xxx_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/supply/bq27xxx_battery.c b/drivers/power/supply/bq27xxx_battery.c index