[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20161024195151.GA5989@amd>
Date: Mon, 24 Oct 2016 21:51:51 +0200
From: Pavel Machek <pavel@....cz>
To: Matt Ranostay <mranostay@...il.com>
Cc: "Andrew F . Davis" <afd@...com>,
Sebastian Reichel <sre@...nel.org>, linux-pm@...r.kernel.org,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Matt Ranostay <matt@...ostay.consulting>
Subject: Re: [PATCH 1/2] power: bq27xxx_battery: add configurable
poll_interval by sysfs
On Sun 2016-10-23 20:08:22, Matt Ranostay wrote:
> On Fri, Oct 7, 2016 at 11:49 AM, Pavel Machek <pavel@....cz> wrote:
> > On Fri 2016-09-16 20:42:54, Matt Ranostay wrote:
> >> Allow the poll_interval to be runtime configurable via an sysfs entry.
> >> This is needed for udev control of the poll interval.
> >>
> >> Signed-off-by: Matt Ranostay <matt@...ostay.consulting>
> >
> > working mail address would be nice here.
> >
> > sysfs files should be documented.
> >
>
> 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.
Umm, there seems to be mechanism there to change that already...?
static const struct kernel_param_ops param_ops_poll_interval = {
.get = param_get_uint,
.set = poll_interval_param_set,
};
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Download attachment "signature.asc" of type "application/pgp-signature" (182 bytes)
Powered by blists - more mailing lists