From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932695Ab3LDORx (ORCPT ); Wed, 4 Dec 2013 09:17:53 -0500 Received: from mail-pb0-f42.google.com ([209.85.160.42]:39154 "EHLO mail-pb0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932358Ab3LDORw (ORCPT ); Wed, 4 Dec 2013 09:17:52 -0500 Message-ID: <529F3983.5000200@linaro.org> Date: Wed, 04 Dec 2013 22:17:39 +0800 From: Hanjun Guo User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: One Thousand Gnomes CC: "Rafael J. Wysocki" , Catalin Marinas , Will Deacon , Russell King - ARM Linux , Daniel Lezcano , linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Grant Likely , Matthew Garrett , Olof Johansson , Linus Walleij , Bjorn Helgaas , Rob Herring , Mark Rutland , Jon Masters , patches@linaro.org, linux-kernel@vger.kernel.org, linaro-kernel@lists.linaro.org, linaro-acpi@lists.linaro.org, Al Stone , Graeme Gregory Subject: Re: [RFC part1 PATCH 4/7] ARM64 / ACPI: Introduce the skeleton of _PDC related for ARM64 References: <1386088611-2801-1-git-send-email-hanjun.guo@linaro.org> <1386088611-2801-5-git-send-email-hanjun.guo@linaro.org> <20131203165318.2c04a8bf@alan.etchedpixels.co.uk> In-Reply-To: <20131203165318.2c04a8bf@alan.etchedpixels.co.uk> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2013年12月04日 00:53, One Thousand Gnomes wrote: > O> +enum idle_boot_override { IDLE_NO_OVERRIDE = 0, IDLE_HALT, IDLE_NOMWAIT, >> + IDLE_POLL, IDLE_FORCE_MWAIT }; > This should probably move out of the arch directory to be a single enum > including both platforms values. That will make it rather easier to keep > sane and avoid ifdefs around which definitions are for which processor. Thank you very much for the suggestion, it makes sense to me, will figure out how to implement it. Thanks Hanjun