From: Greg KH <greg@kroah.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: can't seem to ignore a battery
Date: Fri, 13 Nov 2009 20:33:49 +0000 [thread overview]
Message-ID: <20091113203349.GC26546@kroah.com> (raw)
In-Reply-To: <29507.1258042014@foxharp.boston.ma.us>
On Fri, Nov 13, 2009 at 01:43:44PM -0500, Paul Fox wrote:
> well, this probably isn't the right list. but you asked. :-)
>
> the XO-1 didn't use ACPI under linux. it did have some minimal
> DSDT support in openfirmware to support Windows [1], in the form
> of simple battery reporting. for the XO-1.5, the decision was
> made to fully support ACPI, so among other things, yours truly
> has been enhancing our DSDT support to support the lid, the ebook
> switch, etc, and trying to get the EC events tied in. one of the
> things still on the list is to make the battery support more
> correct -- someone tried powertop on a 1.5 XO said it reported 35W
> being used. i don't think that sounds right. :-) so work needs
> to be done there. if there are any DSDT experts in the house,
> i'd love to hear from you, if only for fielding occasional
> questions.
>
> the CONFIG_POWER_SUPPLY-based battery driver we used on XO-1
> still works fine on XO-1.5, and correctly reports our (accurate)
> charge counter, and our (accurate) capacity level. we need the
> charge_counter value in particular, as it's used in all of our
> power analysis scripts. i confess i'm hazy on the details right
> now, but the way our charge counting works isn't directly
> translatable to the values that ACPI provides -- it's a frame of
> reference issue, having to do with where or when "full" is
> assumed to be. richard smith and i have gone through it in the
> past -- i just can't remember right now. eventually we need to
> figure out how to get the old values to come out of the new
> driver, but this is why i was hoping i could get udev to ignore
> the device -- we need it for some specific purposes, but don't
> necessarily want most "normal" code to have to deal with it.
>
> i'd actually be tempted to simply use the old battery driver
> (i.e., turn off CONFIG_ACPI_BATTERY), but it would of course be
> nice if things like powertop and g-p-m worked.
Well as things will not work properly anyway, I would suggest this
option as well. It's the easiest for you to do right now, and seems to
solve your root problems until you get your acpi code up and running
properly.
thanks,
greg k-h
prev parent reply other threads:[~2009-11-13 20:33 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-12 16:06 can't seem to ignore a battery Paul Fox
2009-11-12 16:50 ` David Zeuthen
2009-11-12 17:33 ` Richard Hughes
2009-11-12 17:40 ` Paul Fox
2009-11-12 17:43 ` David Zeuthen
2009-11-13 2:01 ` Kay Sievers
2009-11-13 2:44 ` Paul Fox
2009-11-13 4:54 ` Greg KH
2009-11-13 14:13 ` Kay Sievers
2009-11-13 14:33 ` Paul Fox
2009-11-13 14:43 ` Justin Schoeman
2009-11-13 16:48 ` Greg KH
2009-11-13 16:49 ` Greg KH
2009-11-13 17:29 ` Justin Schoeman
2009-11-13 17:38 ` Greg KH
2009-11-13 18:22 ` Justin Schoeman
2009-11-13 18:43 ` Paul Fox
2009-11-13 20:33 ` Greg KH [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=20091113203349.GC26546@kroah.com \
--to=greg@kroah.com \
--cc=linux-hotplug@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 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).