All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Preparing patches for pca9552, ad5259, and bq27350
Date: Tue, 19 Feb 2008 10:19:08 +0000	[thread overview]
Message-ID: <20080219111908.4085dfdd@hyperion.delvare> (raw)
In-Reply-To: <FD2217BC-9A9C-4CFE-843F-15F1600B4CCE@codetastic.com>

On Mon, 18 Feb 2008 23:51:59 -0800, Philip Pokorny wrote:
> There are several monitoring chips that monitor the CMOS battery
> voltage.  That's similar...

While I agree that battery monitoring drivers and hardware monitoring
drivers have some features in common (in particular measuring an input
voltage) I still believe that having separate subsystems for these,
with different sysfs interfaces, makes sense. The user needs are
different. The CMOS battery monitoring is only there to tell you when
you should change the battery. A "real" battery monitor OTOH will tell
you a lot more information on the battery, how much energy there is
left inside, how much time you have before the power runs out, how old
the battery is, how hot it is, etc. Using the hwmon standard sysfs
interface for this would probably put restrictions on what can be done.
This doesn't mean though that battery drivers can't additionally export
their sensor values as standard, read-only hwmon sysfs values (maybe
even using common code for all battery drivers) but I don't think that
this should be their primary interface.

Anyway, I didn't look deep enough into the power subsystem to make
myself a definitive opinion, so the above is really only a first
thought. I might be wrong.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

      parent reply	other threads:[~2008-02-19 10:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-18 18:54 [lm-sensors] Preparing patches for pca9552, ad5259, and bq27350 Matthew Bosworth
2008-02-18 19:45 ` Jean Delvare
2008-02-18 21:26 ` Matthew Bosworth
2008-02-18 21:41 ` Jean Delvare
2008-02-19  7:51 ` Philip Pokorny
2008-02-19 10:19 ` Jean Delvare [this message]

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=20080219111908.4085dfdd@hyperion.delvare \
    --to=khali@linux-fr.org \
    --cc=lm-sensors@vger.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 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.