From: "Éric Piel" <Eric.Piel@tremplin-utc.net>
To: Pavel Machek <pavel@suse.cz>
Cc: Andrew Morton <akpm@osdl.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Arjan van de Ven <arjan@infradead.org>,
"Mark M. Hoffman" <mhoffman@lightlink.com>,
Yan Burman <burman.yan@gmail.com>,
LKML <linux-kernel@vger.kernel.org>,
HWMON <lm-sensors@lm-sensors.org>
Subject: Re: [PATCH 2.6.25.4] hwmon: HP Mobile Data Protection System 3D ACPI driver -- please review!
Date: Thu, 05 Jun 2008 01:29:21 +0200 [thread overview]
Message-ID: <48472551.9020202@tremplin-utc.net> (raw)
In-Reply-To: <20080604230326.GA30475@elf.ucw.cz>
05-06-08 01:03, Pavel Machek wrote/a écrit:
> Hi!
>>> 3) Makes it possible to power the device off.
>
> But this part is what I don't understand. powering off does not seem
> to belong there. Why can't you just use acpi to power down?
Not sure what you mean. Are you saying acpi has already a user interface
to power up/down a device? If so, I'm not aware of it, is it in /sys ?
Or did you mean that there is already a kernel api to turn on/off a
device via acpi, no need to access the device directly?
That said, this special /sys interface to power the device off is
probably going to be used rarely. It was put there because it is costly
to wake the device up each time we want to read it. What I could do is
put a timer to turn the device off after 10s of non-usage, and power it
on automatically again as soon as the user wants to read some info. That
would permit to remove this interface.
Eric
next prev parent reply other threads:[~2008-06-04 23:29 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-31 12:05 [PATCH 2.6.25.4] hwmon: HP Mobile Data Protection System 3D ACPI driver Yan Burman
2008-06-04 19:24 ` [PATCH 2.6.25.4] hwmon: HP Mobile Data Protection System 3D ACPI driver -- please review! Éric Piel
2008-06-04 20:58 ` Jean Delvare
2008-06-04 22:57 ` Éric Piel
2008-06-05 6:27 ` Jean Delvare
2008-06-04 23:05 ` Pavel Machek
2008-06-04 23:43 ` Éric Piel
2008-06-05 6:29 ` Jean Delvare
2008-06-05 9:38 ` [lm-sensors] " Jonathan Cameron
2008-06-05 20:34 ` Éric Piel
2008-06-05 22:27 ` Pau Oliva Fora
2008-06-06 14:24 ` Dmitry Torokhov
2008-06-06 15:29 ` Pau Oliva Fora
2008-06-06 14:56 ` Eric Piel
2008-06-06 15:30 ` Pau Oliva Fora
2008-06-06 8:51 ` Jonathan Cameron
2008-06-04 23:03 ` Pavel Machek
2008-06-04 23:29 ` Éric Piel [this message]
2008-06-05 1:06 ` Henrique de Moraes Holschuh
2008-06-05 8:19 ` Pavel Machek
2008-06-06 14:27 ` Dmitry Torokhov
2008-06-06 14:59 ` Eric Piel
2008-06-06 15:11 ` Dmitry Torokhov
2008-06-06 15:37 ` Pau Oliva Fora
2008-06-05 5:17 ` [PATCH 2.6.25.4] hwmon: HP Mobile Data Protection System 3D ACPI driver Andrew Morton
2008-06-05 7:43 ` [lm-sensors] " Riku Voipio
2008-06-05 8:28 ` Eric Piel
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=48472551.9020202@tremplin-utc.net \
--to=eric.piel@tremplin-utc.net \
--cc=akpm@osdl.org \
--cc=arjan@infradead.org \
--cc=burman.yan@gmail.com \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=mhoffman@lightlink.com \
--cc=pavel@suse.cz \
/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