From owner-freebsd-acpi@FreeBSD.ORG  Fri Nov 12 17:12:00 2010
Return-Path: <owner-freebsd-acpi@FreeBSD.ORG>
Delivered-To: freebsd-acpi@freebsd.org
Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28])
	by hub.freebsd.org (Postfix) with ESMTP id AA84B106566C;
	Fri, 12 Nov 2010 17:11:59 +0000 (UTC)
	(envelope-from jkim@FreeBSD.org)
From: Jung-uk Kim <jkim@FreeBSD.org>
To: Andriy Gapon <avg@freebsd.org>
Date: Fri, 12 Nov 2010 12:11:44 -0500
User-Agent: KMail/1.6.2
References: <4CDC25F2.6080409@freebsd.org>
	<201011112048.31127.jkim@FreeBSD.org>
	<4CDCE1F0.9010806@freebsd.org>
In-Reply-To: <4CDCE1F0.9010806@freebsd.org>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <201011121211.46833.jkim@FreeBSD.org>
Cc: freebsd-acpi@freebsd.org
Subject: Re: fixup for missing C1 in _CST
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, 12 Nov 2010 17:12:00 -0000

On Friday 12 November 2010 01:42 am, Andriy Gapon wrote:
> on 12/11/2010 03:48 Jung-uk Kim said the following:
> > On Thursday 11 November 2010 05:37 pm, Jung-uk Kim wrote:
> >> On Thursday 11 November 2010 12:20 pm, Andriy Gapon wrote:
> >>> Dear fellow FreeBSD ACPI hackers,
> >>> what is your opinion about the following patch?
> >>>
> >>> The idea is to add a C1 state to available states if a bugggy
> >>> BIOS supplies us with _CST that has states with C2, C3, etc
> >>> types, but no state with C1 type.
> >>
> >> Can you please try the attached patch instead?
> >
> > It seems C1 state became mandatory years ago.  Please ignore the
> > previous patches and try this instead.
>
> The patch looks OK to me.

Committed as r215188.

Thanks!

Jung-uk Kim