linux-kernel - Re: [PATCH v3 14/21] arm64: dts: hi3660: enable idle states

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 15 Jun 2017 08:23:58 +0800
From:   Leo Yan <leo.yan@...aro.org>
To:     Guodong Xu <guodong.xu@...aro.org>
Cc:     robh+dt@...nel.org, mark.rutland@....com, xuwei5@...ilicon.com,
        catalin.marinas@....com, will.deacon@....com,
        wangkefeng.wang@...wei.com, xuejiancheng@...ilicon.com,
        peter.griffin@...aro.org, puck.chen@...ilicon.com,
        lee.jones@...aro.org, ulf.hansson@...aro.org, bhelgaas@...gle.com,
        arnd@...db.de, zhangfei.gao@...aro.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-mmc@...r.kernel.org, linux-pci@...r.kernel.org,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Kevin Wang <jean.wangtao@...aro.org>
Subject: Re: [PATCH v3 14/21] arm64: dts: hi3660: enable idle states

On Wed, Jun 14, 2017 at 04:23:31PM +0800, Guodong Xu wrote:
> From: Leo Yan <leo.yan@...aro.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