Message-ID: <56E15032.5060302@linux.vnet.ibm.com>
Date: Thu, 10 Mar 2016 16:15:06 +0530
From: Shreyas B Prabhu <shreyas@linux.vnet.ibm.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0
MIME-Version: 1.0
To: mpe@ellerman.id.au
CC: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        benh@kernel.crashing.org, paulus@samba.org, mahesh@linux.vnet.ibm.com,
        maddy@linux.vnet.ibm.com
Subject: Re: [PATCH 0/3] powerpc/powernv: Cpuidle related cleanup
References: <1456748580-10519-1-git-send-email-shreyas@linux.vnet.ibm.com>
In-Reply-To: <1456748580-10519-1-git-send-email-shreyas@linux.vnet.ibm.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 961
Lines: 24

Hi,
Any thoughts on this?


On 02/29/2016 05:52 PM, Shreyas B. Prabhu wrote:
> These patches are purely code movement and cleanup.
> There is no functionality change.
> 
> Note, there are multiple style error reported for patch 1 and 2.
> I think this is because checkpatch script mistakes the patches
> as C code because of the presence of semi-colon in each line. I
> for now have largely ignored these errors.
> 
> Shreyas B. Prabhu (3):
>   powerpc/powernv: Move CHECK_HMI_INTERRUPT to exception-64s header
>   powerpc/powernv: Encapsulate idle preparation steps in a macro
>   powerpc/powernv: make hypervisor state restore a function
> 
>  arch/powerpc/include/asm/cpuidle.h       |  68 ++++++++++++++
>  arch/powerpc/include/asm/exception-64s.h |  19 ++++
>  arch/powerpc/kernel/exceptions-64s.S     |  29 +-----
>  arch/powerpc/kernel/idle_power7.S        | 148 ++++++++-----------------------
>  4 files changed, 129 insertions(+), 135 deletions(-)
>