From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lin Ming Subject: Re: [PATCH 1/6] ACPI: D3cold state is always valid Date: Wed, 18 Apr 2012 10:15:27 +0800 Message-ID: <1334715327.11188.144.camel@minggr> References: <1334641676-14566-1-git-send-email-ming.m.lin@intel.com> <1334641676-14566-2-git-send-email-ming.m.lin@intel.com> <201204172229.18203.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga02.intel.com ([134.134.136.20]:63579 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751093Ab2DRCPa (ORCPT ); Tue, 17 Apr 2012 22:15:30 -0400 In-Reply-To: <201204172229.18203.rjw@sisk.pl> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Rafael J. Wysocki" Cc: Len Brown , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, Zhang Rui , Huang Ying , Aaron Lu (fix Len's address) On Tue, 2012-04-17 at 22:29 +0200, Rafael J. Wysocki wrote: > On Tuesday, April 17, 2012, Lin Ming wrote: > > ACPI_STATE_D3 actually means ACPI D3hot which is not always valid. > > Instead, ACPI D3cold is always valid. > > > > Signed-off-by: Lin Ming > > Can you just introduce ACPI_STATE_D3_HOT and redefine D3_COLD so that > it's the same as D3, as I said? Do you mean below? diff --git a/include/acpi/actypes.h b/include/acpi/actypes.h index eba6604..f0b25c8 100644 --- a/include/acpi/actypes.h +++ b/include/acpi/actypes.h @@ -499,9 +499,9 @@ typedef u64 acpi_integer; #define ACPI_STATE_D0 (u8) 0 #define ACPI_STATE_D1 (u8) 1 #define ACPI_STATE_D2 (u8) 2 -#define ACPI_STATE_D3 (u8) 3 -#define ACPI_STATE_D3_COLD (u8) 4 -#define ACPI_D_STATES_MAX ACPI_STATE_D3_COLD +#define ACPI_STATE_D3_HOT (u8) 3 +#define ACPI_STATE_D3 (u8) 4 +#define ACPI_D_STATES_MAX ACPI_STATE_D3 #define ACPI_D_STATE_COUNT 5 #define ACPI_STATE_C0 (u8) 0