Re: Re: Smart Battery System driver

On Tue, Jan 18, 2005 at 10:39:36AM -0500, Rich Townsend wrote:

Bruno Ducrot wrote:

quoted

On Mon, Jan 17, 2005 at 10:03:25PM -0500, Rich Townsend wrote:

quoted

What could be going wrong? Some thoughts:
*) maybe there is already a driver registered for SMBus alarms, that 
prevents them from getting through?
*) I'm assuming its already there, but maybe the functionality described 
in 5.6.2.2.2 actually needs to be implemented in the i2c-acpi-ec code?

Exactly.  This must be implemented in i2c-acpi-ec, and a event notifier
sohuld be done in order to pass this to the children.  Also I have to
modify i2c-acpi-ec in order to make this driver event driven (we write to
the PCRTL, then wait for an event (query number will be 0x20 in this
perticular case) from the EC before processing further), and in the
handler, if the alarm bit is set, then notify the child.
That what I meant in a previous mail with 'overriden the _Q20',
by the fact that I think its only a dummy alarm ack in order to be
sure the smbus is still functionnal even if the OS do not provide
a driver for accessing the smbus.  I never meant to do this in DSDT.
I'm sorry for the confusion.

OK, thanks for the clarification. Out of interest, how do I install 
event handlers for the EC? A quick look through ec.c doesn't show any 
obvious way to do it.

Simple.  Need to add this into ec.c ;)

quoted

quoted

*) perhaps the battery simply isn't generating the alarms?

This may be another trouble.

Yeah, I think that may be the real problem. Even without proper alarm 
handling in i2c-acpi-ec, the alarms sent by the battery should still 
show up in the _Q20 method I hacked into my DSDT. Is it possible to fake 
an SMBus alarm, to track whether it is getting through the EC driver 
code properly

I don't know.

-- 
Bruno Ducrot

--  Which is worse:  ignorance or apathy?
--  Don't know.  Don't care.


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt