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

Groups > linux.kernel > #1306728

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

Show all headers | View raw

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.

n900 has this variant:

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

Will you try?

If not, tell me, and I can do it, but it will involve a lot of swaring...

Thanks,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

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