From owner-freebsd-acpi@FreeBSD.ORG  Fri Sep  3 16:11:41 2010
Return-Path: <owner-freebsd-acpi@FreeBSD.ORG>
Delivered-To: freebsd-acpi@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id DB518106570C
	for <freebsd-acpi@freebsd.org>; Fri,  3 Sep 2010 16:11:41 +0000 (UTC)
	(envelope-from avg@icyb.net.ua)
Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140])
	by mx1.freebsd.org (Postfix) with ESMTP id 24DF98FC20
	for <freebsd-acpi@freebsd.org>; Fri,  3 Sep 2010 16:11:40 +0000 (UTC)
Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua
	[212.40.38.101])
	by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id TAA19549;
	Fri, 03 Sep 2010 19:11:30 +0300 (EEST)
	(envelope-from avg@icyb.net.ua)
Message-ID: <4C811E31.2030900@icyb.net.ua>
Date: Fri, 03 Sep 2010 19:11:29 +0300
From: Andriy Gapon <avg@icyb.net.ua>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US;
	rv:1.9.2.8) Gecko/20100823 Lightning/1.0b2 Thunderbird/3.1.2
MIME-Version: 1.0
To: Ian Smith <smithi@nimnet.asn.au>
References: <AANLkTiknRR_PDWfXbhM3iivb4bDnaQ-JBmXyuFP_2knk@mail.gmail.com>
	<20100829235431.J86162@sola.nimnet.asn.au>
	<AANLkTi=Cpg5k1b+iqxsz=Ys8syCv7yOfv2Z_KHYer2p6@mail.gmail.com>
	<20100830183330.Y29840@sola.nimnet.asn.au>
	<4C7BBCFE.4030804@icyb.net.ua>
	<20100831003546.K29840@sola.nimnet.asn.au>
	<AANLkTikkuZJh6FBE3As0wxdXM3Gh3GYRF3=6eNZmP71D@mail.gmail.com>
	<4C8095A2.90506@icyb.net.ua> <4C809912.1080205@FreeBSD.org>
	<4C80AB00.4020008@icyb.net.ua>
	<20100903234130.A32216@sola.nimnet.asn.au>
	<4C810D5A.5040705@icyb.net.ua>
	<20100904014842.T32216@sola.nimnet.asn.au>
In-Reply-To: <20100904014842.T32216@sola.nimnet.asn.au>
X-Enigmail-Version: 1.1.2
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: freebsd-acpi@freebsd.org
Subject: Re: acpi shows wrong battery state
X-BeenThere: freebsd-acpi@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: ACPI and power management development <freebsd-acpi.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-acpi>,
	<mailto:freebsd-acpi-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-acpi>
List-Post: <mailto:freebsd-acpi@freebsd.org>
List-Help: <mailto:freebsd-acpi-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-acpi>,
	<mailto:freebsd-acpi-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 03 Sep 2010 16:11:41 -0000

on 03/09/2010 19:03 Ian Smith said the following:
> On Fri, 3 Sep 2010, Andriy Gapon wrote:
>  > on 03/09/2010 17:51 Ian Smith said the following:
>  > > +acpi_ec0: warning: EC done before starting event wait
>  > 
>  > Well, it's messages of these kind that led me to suggest the patch.
>  > Perhaps, for some reason, in this case EC has response ready before we expect it.
> 
> Ok, well if I'm reading this right, acpi_ec0 definitely seems to have 
> failed the first time on 8.0, but succeeded with a warning later.  I 
> know you suggested not mixing the issues, but re kern/147858 you said:
> 
>  > I guess that I "broke" this driver when I removed second pass of 
>  > probe+attach of acpi bus children, but it is the driver that should 
>  > be fixed.
> 
> .. so I can't help wondering if it was this second pass that you removed 
> that may have fluked succeeding (with warning) on 8.0 on this machine?
> 
> Please tell me to shutup and swim when I'm too far out of my depth :)
> 

Well, I suspect that on his system acpi_ec needs acpi_cpu to be attached to
succeeded, so it (sometimes) worked on the second pass, but I guess that never on
the first pass.
In 8.1 acpi_cpu is always attached before acpi_ec.

-- 
Andriy Gapon