From: Pavel Machek <pavel@ucw.cz>
To: Matt Ranostay <mranostay@gmail.com>
Cc: "Andrew F . Davis" <afd@ti.com>,
Sebastian Reichel <sre@kernel.org>,
linux-pm@vger.kernel.org,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Matt Ranostay <matt@ranostay.consulting>
Subject: Re: [PATCH 1/2] power: bq27xxx_battery: add configurable poll_interval by sysfs
Date: Mon, 24 Oct 2016 10:23:31 +0200 [thread overview]
Message-ID: <20161024082331.GA13240@amd> (raw)
In-Reply-To: <CAKzfze95nJMcRU2=FivDa72MX4P0md1VoWbUjUnt-75Vs_ektQ@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 773 bytes --]
> Ok can do in next revision
>
> > Also... what is it good for?
> >
> > Do you have a device that needs non-standard interval?
>
> Basically we need to have the ability to dynamically change the
> intervals. So closer to a battery drain we need to up the reporting
> intervals.
Could you elaborate on that? What intervals do you normally need, what
do you need around empty battery, and why is that?
I'm thinking about adding kernel thread to shut down the system if
battery goes critically empty. It is kernel job to protect the
hardware, and that should include the battery...
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 --]
next prev parent reply other threads:[~2016-10-24 8:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-17 3:42 [PATCH 0/2] power: bq27xxx_battery: add configurable poll_interval Matt Ranostay
2016-09-17 3:42 ` [PATCH 1/2] power: bq27xxx_battery: add configurable poll_interval by sysfs Matt Ranostay
2016-09-19 19:46 ` Sebastian Reichel
2016-09-20 3:22 ` Matt Ranostay
2016-10-07 18:49 ` Pavel Machek
2016-10-24 3:08 ` Matt Ranostay
2016-10-24 8:23 ` Pavel Machek [this message]
2016-10-24 19:51 ` Pavel Machek
2016-10-24 19:55 ` Matt Ranostay
2016-09-17 3:42 ` [PATCH 2/2] power: bq27xxx_battery: allow kernel poll_interval parameter runtime update Matt Ranostay
2016-09-19 19:49 ` Sebastian Reichel
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=20161024082331.GA13240@amd \
--to=pavel@ucw.cz \
--cc=afd@ti.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=matt@ranostay.consulting \
--cc=mranostay@gmail.com \
--cc=sre@kernel.org \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).