On Tue, Oct 25, 2016 at 11:47 AM, Matt Ranostay [off-list ref] wrote:

On Mon, Oct 24, 2016 at 1:14 PM, Pavel Machek [off-list ref] wrote:

quoted

On Mon 2016-10-24 12:58:25, Matt Ranostay wrote:

quoted

Pavel + Sebastian this is the patchset that need I some input on :)

Better then previous one.

But my version of bq27xxx_battery.c already contains this:

This is for allowing udev rule to set the properties as well.
otherwise a kinda crude RUN = " echo value >
/sys/module/bq27xxx_battery/parameters/poll_interval" is required.

Any thoughts on this?

Thanks,

Matt

quoted

static const struct kernel_param_ops param_ops_poll_interval = {
        .get = param_get_uint,
        .set = poll_interval_param_set,
};

...so it should be possible to set poll interval already.

                                                                        Pavel

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