From: Pavel Machek <pavel@ucw.cz> To: Tony Lindgren <tony@atomide.com> Cc: kernel list <linux-kernel@vger.kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, linux-omap@vger.kernel.org, sre@kernel.org, nekit1000@gmail.com, mpartap@gmx.net, merlijn@wizzup.org Subject: Re: Motorola Droid 4 progress, power consumption Date: Fri, 4 May 2018 22:20:44 +0200 [thread overview] Message-ID: <20180504202044.GC22519@amd> (raw) In-Reply-To: <20180503142240.GQ98604@atomide.com> [-- Attachment #1: Type: text/plain, Size: 2186 bytes --] Hi! > > user@devuan:/sys/class/power_supply/battery$ date; cat current_now > > current_avg > > Thu May 3 09:33:07 CEST 2018 > > -56000 > > -72308 > > user@devuan:/sys/class/power_supply/battery$ date; cat current_now > > current_avg > > Thu May 3 09:33:08 CEST 2018 > > -37000 > > -211362 > > I recall it just being calculated based on the time between > the reads. Hard to say what it does, -ENODOCS.. Maybe the mc13783 > PMIC docs have something similar. Hmm. current_avg is world readable; if it does changes to the hardware, that's somehow bad. But I guess I can sample charge_counter every minute or so and get what I need? > > > Hmm oh and the POWER_SUPPLY_CHARGE_COUNTER value should be monitored > > > by your libbattery and it's low value and high value should be saved > > > to a file. Low should be saved when we get the battery low interrupt > > > and battery state changes to POWER_SUPPLY_CAPACITY_LEVEL_CRITICAL. > > > High value should be saved on POWER_SUPPLY_CAPACITY_LEVEL_FULL. > > > > > > Then when you know the high value and low value, you can calculate > > > the remaining capacity based on the current value and > > > POWER_SUPPLY_POWER_AVG. > > > > I'll look into that... but probably will do experiments with python, > > first. > > Sure. The reason why I think we should not do it in the kernel in > addition to the earlier reasoning is that we don't have a place > to store the min and max values across reboot. I think there's > some flash on the battery over 1w bus, but it probably just > contains battery serial number or something and is not intended > for writing much to it. I guess the min and max values should be > really keyd based on this battery id in case the battery gets > changed. I agree this should not be done in kernel as it requires persistent storage. Battery is not user-replacable (and this will recalibrate itself on charge/discharge cycle) so.. I don't think keying is strictly neccessary for D4. Best regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 181 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: pavel@ucw.cz (Pavel Machek) To: linux-arm-kernel@lists.infradead.org Subject: Motorola Droid 4 progress, power consumption Date: Fri, 4 May 2018 22:20:44 +0200 [thread overview] Message-ID: <20180504202044.GC22519@amd> (raw) In-Reply-To: <20180503142240.GQ98604@atomide.com> Hi! > > user at devuan:/sys/class/power_supply/battery$ date; cat current_now > > current_avg > > Thu May 3 09:33:07 CEST 2018 > > -56000 > > -72308 > > user at devuan:/sys/class/power_supply/battery$ date; cat current_now > > current_avg > > Thu May 3 09:33:08 CEST 2018 > > -37000 > > -211362 > > I recall it just being calculated based on the time between > the reads. Hard to say what it does, -ENODOCS.. Maybe the mc13783 > PMIC docs have something similar. Hmm. current_avg is world readable; if it does changes to the hardware, that's somehow bad. But I guess I can sample charge_counter every minute or so and get what I need? > > > Hmm oh and the POWER_SUPPLY_CHARGE_COUNTER value should be monitored > > > by your libbattery and it's low value and high value should be saved > > > to a file. Low should be saved when we get the battery low interrupt > > > and battery state changes to POWER_SUPPLY_CAPACITY_LEVEL_CRITICAL. > > > High value should be saved on POWER_SUPPLY_CAPACITY_LEVEL_FULL. > > > > > > Then when you know the high value and low value, you can calculate > > > the remaining capacity based on the current value and > > > POWER_SUPPLY_POWER_AVG. > > > > I'll look into that... but probably will do experiments with python, > > first. > > Sure. The reason why I think we should not do it in the kernel in > addition to the earlier reasoning is that we don't have a place > to store the min and max values across reboot. I think there's > some flash on the battery over 1w bus, but it probably just > contains battery serial number or something and is not intended > for writing much to it. I guess the min and max values should be > really keyd based on this battery id in case the battery gets > changed. I agree this should not be done in kernel as it requires persistent storage. Battery is not user-replacable (and this will recalibrate itself on charge/discharge cycle) so.. I don't think keying is strictly neccessary for D4. Best regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 181 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180504/1468df3d/attachment.sig>
next prev parent reply other threads:[~2018-05-04 20:20 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-05-01 18:31 Motorola Droid 4 progress, power consumption Pavel Machek 2018-05-02 14:41 ` Tony Lindgren 2018-05-02 14:41 ` Tony Lindgren 2018-05-02 19:10 ` Pavel Machek 2018-05-02 19:10 ` Pavel Machek 2018-05-02 19:42 ` Tony Lindgren 2018-05-02 19:42 ` Tony Lindgren 2018-05-02 21:32 ` Pavel Machek 2018-05-02 21:32 ` Pavel Machek 2018-05-03 9:06 ` Pavel Machek 2018-05-03 9:06 ` Pavel Machek 2018-05-03 14:22 ` Tony Lindgren 2018-05-03 14:22 ` Tony Lindgren 2018-05-04 20:20 ` Pavel Machek [this message] 2018-05-04 20:20 ` Pavel Machek 2018-05-04 20:38 ` Tony Lindgren 2018-05-04 20:38 ` Tony Lindgren 2018-05-04 21:47 ` Pavel Machek 2018-05-04 21:47 ` Pavel Machek 2018-05-05 19:44 ` Tony Lindgren 2018-05-05 19:44 ` Tony Lindgren
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180504202044.GC22519@amd \ --to=pavel@ucw.cz \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=merlijn@wizzup.org \ --cc=mpartap@gmx.net \ --cc=nekit1000@gmail.com \ --cc=sre@kernel.org \ --cc=tony@atomide.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.