From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rich Townsend Subject: Re: Re: What's up with the Embedded Controller driver? [was Re: [ACPI] patched sms-cm acpi/ec.c to 2.6.11.5] Date: Tue, 29 Mar 2005 10:09:21 -0500 Message-ID: <42496FA1.90105@bartol.udel.edu> References: <3ACA40606221794F80A5670F0AF15F84041AC161@pdsmsx403> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <3ACA40606221794F80A5670F0AF15F84041AC161@pdsmsx403> Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: "Yu, Luming" , acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-acpi@vger.kernel.org Yu, Luming wrote: > I'm sorry, what is sbs-cm packag?=20 > Please make sure sbs-cm package is using ec_read/write functions define= d in ec.c The sbs-cm package is a patchset for DSDT, that implements the Control Method battery API for systems that use Smart Battery technology; see her= e: http://sourceforge.net/projects/sbs-linux/ The ec_read/write functions are NOT used in sbs-cm, because all accesses to the Embedded Controller are done from within the AML virtual machine, using the EC OperationRegion. So in fact sbs-cm accesses the EC via the address space handler in ec.c, NOT the read/write routines. Note that many of the people reporting problems with ec.c (including myself, Pedro and Francois) are using this package. So it may be that the problem with ec.c is in the address space handler. cheers, Rich >=20 > Thanks > Luming=20 >=20 >=20 >>-----Original Message----- >>From: Rich Townsend [mailto:rhdt-OBnUx95tOyn10jlvfTC4gA@public.gmane.org]=20 >>Sent: 2005=E5=B9=B43=E6=9C=8829=E6=97=A5 22:50 >>To: Yu, Luming >>Cc: Fran=C3=A7ois Valenduc; ACPI devel >>Subject: Re: [ACPI] Re: What's up with the Embedded Controller=20 >>driver? [was Re: [ACPI] patched sms-cm acpi/ec.c to 2.6.11.5] >> >>Yu, Luming wrote: >> >>>You modified DSDT just remove some AML methods that related to EC. >>>That means it just delete the problem.=20 >>>I believe if you think this is the regression, just because=20 >> >>the changeset >> >>http://linux-acpi.bkbits.net:8080/26-latest-test/diffs/include/ >>acpi/acconfig.h@1.55?nav=3Dindex.html|src/.|src/include|src/inclu >=20 > de/acpi|hist/include/acpi/acconfig.h >=20 >>>make _those AML methods just run. >>>So, this is NOT a regression problem. >>> >>>Please cat /proc/interrupts to me before accessing EC and=20 >> >>after accessing EC.(using burst mode EC patch) >> >>Hi Yuming -- >> >>I'm using your ec patch, plus a DSDT modified using my sbs-cm package >>(which accesses the Smart Battery system through the EC address space >>handler). Here's what happens to the interrupts when I access=20 >>the battery: >> >>rhdt@sto ~ $ cat /proc/interrupts >> CPU0 >> 0: 127116981 XT-PIC timer >> 1: 213426 XT-PIC i8042 >> 2: 0 XT-PIC cascade >> 6: 171382 XT-PIC i915@pci:0000:00:02.0, uhci_hcd, >>uhci_hcd, uhci_hcd, eth1 >> 8: 8 XT-PIC rtc >> 9: 3305062 XT-PIC acpi >>10: 1965890 XT-PIC Intel 82801DB-ICH4, yenta, ehci_hcd, >>ipw2200 >>12: 4335843 XT-PIC i8042 >>14: 466033 XT-PIC ide0 >>15: 28 XT-PIC ide1 >>NMI: 0 >>LOC: 0 >>ERR: 0 >>rhdt@sto ~ $ cat /proc/acpi/battery/BAT0/state >>present: yes >>capacity state: ok >>charging state: charged >>present rate: 0 mA >>remaining capacity: 4081 mAh >>present voltage: 16699 mV >>rhdt@sto ~ $ cat /proc/interrupts >> CPU0 >> 0: 127131974 XT-PIC timer >> 1: 213488 XT-PIC i8042 >> 2: 0 XT-PIC cascade >> 6: 171462 XT-PIC i915@pci:0000:00:02.0, uhci_hcd, >>uhci_hcd, uhci_hcd, eth1 >> 8: 8 XT-PIC rtc >> 9: 3305297 XT-PIC acpi >>10: 1966133 XT-PIC Intel 82801DB-ICH4, yenta, ehci_hcd, >>ipw2200 >>12: 4335843 XT-PIC i8042 >>14: 466065 XT-PIC ide0 >>15: 28 XT-PIC ide1 >>NMI: 0 >>LOC: 0 >>ERR: 0 >> >>Note that the battery access took over 5 seconds! And note, of course, >>the big jump in the acpi interrupt count. >> >>cheers, >> >>Rich >> ------------------------------------------------------- SF email is sponsored by - The IT Product Guide Read honest & candid reviews on hundreds of IT Products from real users. Discover which products truly live up to the hype. Start reading now. http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click