From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Date: Tue, 27 Oct 2015 14:09:34 +0000 Subject: Re: [PATCH v2 5/5] ACPI / processor_idle: Add support for Low Power Idle(LPI) states Message-Id: <562F859E.4010105@arm.com> List-Id: References: <1438710406-3822-1-git-send-email-sudeep.holla@arm.com> <1442411963-14398-1-git-send-email-sudeep.holla@arm.com> <1442411963-14398-6-git-send-email-sudeep.holla@arm.com> <562EC3A5.2070402@codeaurora.org> In-Reply-To: <562EC3A5.2070402@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Prakash, Prashanth" , Ashwin Chaugule Cc: Sudeep Holla , linux acpi , "Rafael J. Wysocki" , lkml , linux-ia64@vger.kernel.org, x86@kernel.org, Al Stone , Lorenzo Pieralisi , Mahesh Sivasubramanian Hi Prashanth, On 27/10/15 00:21, Prakash, Prashanth wrote: > Hi Sudeep, > > While testing with these patches everything looked fine except that we > are flattening all the LPI > states even if it is disabled. I added a simple check to fix it. Please > let me know if you have any > feedback on the same. > Looks good to me and thanks for the nice catch. I completely missed to handle that case. Is it OK if I fold this patch into the next version ? I have worked on Ashwin's comments, but was waiting to get some feedback from Rafael before I post newer version. -- Regards, Sudeep