Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751888AbdFOAYO (ORCPT <rfc822;w@1wt.eu>);
        Wed, 14 Jun 2017 20:24:14 -0400
Received: from mail-pg0-f48.google.com ([74.125.83.48]:34893 "EHLO
        mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751682AbdFOAYM (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 14 Jun 2017 20:24:12 -0400
Date: Thu, 15 Jun 2017 08:23:58 +0800
From: Leo Yan <leo.yan@linaro.org>
To: Guodong Xu <guodong.xu@linaro.org>
Cc: robh+dt@kernel.org, mark.rutland@arm.com, xuwei5@hisilicon.com,
        catalin.marinas@arm.com, will.deacon@arm.com,
        wangkefeng.wang@huawei.com, xuejiancheng@hisilicon.com,
        peter.griffin@linaro.org, puck.chen@hisilicon.com,
        lee.jones@linaro.org, ulf.hansson@linaro.org, bhelgaas@google.com,
        arnd@arndb.de, zhangfei.gao@linaro.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mmc@vger.kernel.org, linux-pci@vger.kernel.org,
        Daniel Lezcano <daniel.lezcano@linaro.org>,
        Kevin Wang <jean.wangtao@linaro.org>
Subject: Re: [PATCH v3 14/21] arm64: dts: hi3660: enable idle states
Message-ID: <20170615002358.GD28496@leoy-ThinkPad-T440>
References: <20170614082338.15673-1-guodong.xu@linaro.org>
 <20170614082338.15673-15-guodong.xu@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170614082338.15673-15-guodong.xu@linaro.org>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1287
Lines: 37

On Wed, Jun 14, 2017 at 04:23:31PM +0800, Guodong Xu wrote:
> From: Leo Yan <leo.yan@linaro.org>
> 
> On Hi3660 there have two clusters, one is CA53 cluster and another is
> CA73 cluster. This two clusters have different idle states separately.
> With Daniel Lezcano's patch (ARM: cpuidle: Support asymmetric idle
> definition), now ARM idle driver can support different clusters with
> different idle states.
> 
> Base on this, this patch is to bind two clusters idle states on Hi3660.
> Except the "WFI" states are enabled by default for all CPUs, this patch
> also binds below extra idle states:
> 
> - CA53 CPUs have two more states:
>   CPU_SLEEP:       CPU power off state
>   CLUSTER_SLEEP_0: Cluster power off state
> 
> - CA73 CPUs have three more states:
>   CPU_NAP:         CPU retention state
>   CPU_SLEEP:       CPU power off state
>   CLUSTER_SLEEP_1: Cluster power off state

Hi Guodong, Wei and all,

As Daniel reminded, this patch is dependent on patch (ARM: cpuidle:
Support asymmetric idle definition) [1] but his patch has not been
finally merged yet. So let us hold on this patch for Daniel's patch
is picked up firstly.

At the meantime welcome the review and comment for this patch ahead.

[1] https://patchwork.kernel.org/patch/9781869/

[...]

Thanks,
Leo Yan