linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
@ 2016-01-21 18:42 Pavel Machek
  2016-01-21 19:15 ` Sebastian Reichel
  2016-01-21 19:34 ` Pali Rohár
  0 siblings, 2 replies; 6+ messages in thread
From: Pavel Machek @ 2016-01-21 18:42 UTC (permalink / raw)
  To: pali.rohar, sre, sre, kernel list, linux-arm-kernel, linux-omap,
	tony, khilman, aaro.koskinen, ivo.g.dimitrov.75, patrikbachan,
	serge

Hi!

In 4.4, n900 battery charging works. If indicator is turned off, it
stays off, and if I select 1.8A charge, it sticks. (There may be
problems after system suspend and resume. But normal operation seems
to be pretty reliable).

In 4.5-rc0 (7fdec82af6a9e190e53d07a1463d2a9ac49a8750) battery charging
works for few hours. But then indicator (yellow light on 3color LED)
turns back on, and if I ask for 1.8A charge, it goes back to .1A,
discharging battery.

Any ideas?

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
  2016-01-21 18:42 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on Pavel Machek
@ 2016-01-21 19:15 ` Sebastian Reichel
  2016-01-21 19:41   ` Pavel Machek
  2016-01-21 19:34 ` Pali Rohár
  1 sibling, 1 reply; 6+ messages in thread
From: Sebastian Reichel @ 2016-01-21 19:15 UTC (permalink / raw)
  To: Pavel Machek
  Cc: pali.rohar, kernel list, linux-arm-kernel, linux-omap, tony,
	khilman, aaro.koskinen, ivo.g.dimitrov.75, patrikbachan, serge

[-- Attachment #1: Type: text/plain, Size: 844 bytes --]

Hi,

On Thu, Jan 21, 2016 at 07:42:34PM +0100, Pavel Machek wrote:
> In 4.4, n900 battery charging works. If indicator is turned off, it
> stays off, and if I select 1.8A charge, it sticks. (There may be
> problems after system suspend and resume. But normal operation seems
> to be pretty reliable).
> 
> In 4.5-rc0 (7fdec82af6a9e190e53d07a1463d2a9ac49a8750) battery charging
> works for few hours. But then indicator (yellow light on 3color LED)
> turns back on, and if I ask for 1.8A charge, it goes back to .1A,
> discharging battery.
> 
> Any ideas?

Maybe a side-effect of isp1704 driver actually doing what was
intended, which is is 88025632515a ("power: isp1704_charger: Fix
isp1704_write()").

P.S.: No need to Cc me two times and when you update your
script/contacts, please use sre@kernel.org :)

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
  2016-01-21 18:42 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on Pavel Machek
  2016-01-21 19:15 ` Sebastian Reichel
@ 2016-01-21 19:34 ` Pali Rohár
  2016-01-23 12:04   ` Pavel Machek
  1 sibling, 1 reply; 6+ messages in thread
From: Pali Rohár @ 2016-01-21 19:34 UTC (permalink / raw)
  To: Pavel Machek, kernel list
  Cc: linux-arm-kernel, linux-omap, tony, khilman, aaro.koskinen,
	ivo.g.dimitrov.75, patrikbachan, serge, sre

[-- Attachment #1: Type: Text/Plain, Size: 914 bytes --]

On Thursday 21 January 2016 19:42:34 Pavel Machek wrote:
> Hi!
> 
> In 4.4, n900 battery charging works. If indicator is turned off, it
> stays off, and if I select 1.8A charge, it sticks. (There may be
> problems after system suspend and resume. But normal operation seems
> to be pretty reliable).

Hm... right, bq2415x driver does not have support for suspend/resume. 
Before suspend registers should be stored and after resume restored, 
plus check charger auto detection.

> In 4.5-rc0 (7fdec82af6a9e190e53d07a1463d2a9ac49a8750) battery
> charging works for few hours. But then indicator (yellow light on
> 3color LED) turns back on, and if I ask for 1.8A charge, it goes
> back to .1A, discharging battery.
> 
> Any ideas?

Look into bq24150a sysfs node what changes. Also if there is something 
in dmesg.

> Best regards,
> 									Pavel


-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
  2016-01-21 19:15 ` Sebastian Reichel
@ 2016-01-21 19:41   ` Pavel Machek
  2016-01-21 21:14     ` Sebastian Reichel
  0 siblings, 1 reply; 6+ messages in thread
From: Pavel Machek @ 2016-01-21 19:41 UTC (permalink / raw)
  To: Sebastian Reichel
  Cc: pali.rohar, kernel list, linux-arm-kernel, linux-omap, tony,
	khilman, aaro.koskinen, ivo.g.dimitrov.75, patrikbachan, serge

On Thu 2016-01-21 20:15:25, Sebastian Reichel wrote:
> Hi,
> 
> On Thu, Jan 21, 2016 at 07:42:34PM +0100, Pavel Machek wrote:
> > In 4.4, n900 battery charging works. If indicator is turned off, it
> > stays off, and if I select 1.8A charge, it sticks. (There may be
> > problems after system suspend and resume. But normal operation seems
> > to be pretty reliable).
> > 
> > In 4.5-rc0 (7fdec82af6a9e190e53d07a1463d2a9ac49a8750) battery charging
> > works for few hours. But then indicator (yellow light on 3color LED)
> > turns back on, and if I ask for 1.8A charge, it goes back to .1A,
> > discharging battery.
> > 
> > Any ideas?
> 
> Maybe a side-effect of isp1704 driver actually doing what was
> intended, which is is 88025632515a ("power: isp1704_charger: Fix
> isp1704_write()").

Hmm. It looks like 7fdec82af6a does not yet contain 88025632515a. Let
me try to add that fix, instead...

> P.S.: No need to Cc me two times and when you update your
> script/contacts, please use sre@kernel.org :)

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
  2016-01-21 19:41   ` Pavel Machek
@ 2016-01-21 21:14     ` Sebastian Reichel
  0 siblings, 0 replies; 6+ messages in thread
From: Sebastian Reichel @ 2016-01-21 21:14 UTC (permalink / raw)
  To: Pavel Machek
  Cc: pali.rohar, kernel list, linux-arm-kernel, linux-omap, tony,
	khilman, aaro.koskinen, ivo.g.dimitrov.75, patrikbachan, serge

[-- Attachment #1: Type: text/plain, Size: 675 bytes --]

Hi,

On Thu, Jan 21, 2016 at 08:41:26PM +0100, Pavel Machek wrote:
> Hmm. It looks like 7fdec82af6a does not yet contain 88025632515a. Let
> me try to add that fix, instead...

Right, your checkout does not contain the power supply
pull request. I think it's unlikely, that the patch will
fix your issue.

Since there are no other power supply changes between 4.4 and your
checkout, the next most likely candidate is the usb tree (which has
been merged before the commit you mentioned).

> > P.S.: No need to Cc me two times and when you update your
> > script/contacts, please use sre@kernel.org :)
> 
> Address book updated :-)

Thanks.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on
  2016-01-21 19:34 ` Pali Rohár
@ 2016-01-23 12:04   ` Pavel Machek
  0 siblings, 0 replies; 6+ messages in thread
From: Pavel Machek @ 2016-01-23 12:04 UTC (permalink / raw)
  To: Pali Rohár
  Cc: kernel list, linux-arm-kernel, linux-omap, tony, khilman,
	aaro.koskinen, ivo.g.dimitrov.75, patrikbachan, serge, sre

On Thu 2016-01-21 20:34:07, Pali Rohár wrote:
> On Thursday 21 January 2016 19:42:34 Pavel Machek wrote:
> > Hi!
> > 
> > In 4.4, n900 battery charging works. If indicator is turned off, it
> > stays off, and if I select 1.8A charge, it sticks. (There may be
> > problems after system suspend and resume. But normal operation seems
> > to be pretty reliable).
> 
> Hm... right, bq2415x driver does not have support for suspend/resume. 
> Before suspend registers should be stored and after resume restored, 
> plus check charger auto detection.
> 
> > In 4.5-rc0 (7fdec82af6a9e190e53d07a1463d2a9ac49a8750) battery
> > charging works for few hours. But then indicator (yellow light on
> > 3color LED) turns back on, and if I ask for 1.8A charge, it goes
> > back to .1A, discharging battery.
> > 
> > Any ideas?
> 
> Look into bq24150a sysfs node what changes. Also if there is something 
> in dmesg.

Hmm, seems like I can't reproduce that any more. I now noticed that if
I suspend machine it auto-resumes quickly, so maybe suspend was
responsbile. I'll try running 4.5-rc0 for a while...

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-01-23 12:04 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-21 18:42 4.5-rc0: n900 battery charging reverts to 100mA, indicator turns back on Pavel Machek
2016-01-21 19:15 ` Sebastian Reichel
2016-01-21 19:41   ` Pavel Machek
2016-01-21 21:14     ` Sebastian Reichel
2016-01-21 19:34 ` Pali Rohár
2016-01-23 12:04   ` Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).