Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1306735

Re: /sys/class/power_supply/bq27200-0/capacity changed meaning between 4.1 and 4.4?

Show all headers | View raw

On 01/11/2016 03:42 PM, Pavel Machek wrote:
> Hi!
>
>>> Did /sys/class/power_supply/bq27200-0/capacity change meaning between
>>> 4.1 and 4.4?
>
>> There was a resent overhaul of this driver and a lot of code is
>> vestigial as you have seen. I've been meaning to continue
>> cleaning it up, my next step would probably to bring up regmap
>
> Ok, 1wire is probably reasonable approach. But we have an regression
> between 4.1 and 4.4, and to debug it is probably by reading code.
>
> I can't prove you caused the regression, but you basically rewrote the
> driver, so it should be easier for you to spot the bug.
>

Makes sense.

> n900 has this variant:
>
>        	 bq27200: bq27200@55 {
> 	                 compatible = "ti,bq27200";
>   			 reg = <0x55>;
>   	};
>

Hmmm, not sure if I have this one on hand, I'll see if I can find an
n900 around here (they seem to be pretty popular around here for testing
(had a lot of TI parts)).

> Will you try?
>
> If not, tell me, and I can do it, but it will involve a lot of swaring...
>

I'll give it a look over.

> Thanks,
> 									Pavel
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

/sys/class/power_supply/bq27200-0/capacity changed meaning between  4.1 and 4.4? Pavel Machek <pavel@ucw.cz> - 2016-01-10 00:10 +0100
  Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pavel Machek <pavel@ucw.cz> - 2016-01-10 09:10 +0100
    Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? "Andrew F. Davis" <afd@ti.com> - 2016-01-11 15:50 +0100
  Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? "Andrew F. Davis" <afd@ti.com> - 2016-01-11 15:30 +0100
    Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pavel Machek <pavel@ucw.cz> - 2016-01-11 22:50 +0100
      Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? "Andrew F. Davis" <afd@ti.com> - 2016-01-11 23:00 +0100
        Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? "Andrew F. Davis" <afd@ti.com> - 2016-01-12 16:30 +0100
          Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pavel Machek <pavel@ucw.cz> - 2016-01-12 23:00 +0100
            Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pali Rohár <pali.rohar@gmail.com> - 2016-01-13 09:50 +0100
              Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Sebastian Reichel <sre@kernel.org> - 2016-01-13 11:30 +0100
                Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pali Rohár <pali.rohar@gmail.com> - 2016-01-13 11:40 +0100
                Re: /sys/class/power_supply/bq27200-0/capacity changed meaning  between 4.1 and 4.4? Pavel Machek <pavel@ucw.cz> - 2016-01-13 12:10 +0100

csiph-web