From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1423664AbcFMVA5 (ORCPT ); Mon, 13 Jun 2016 17:00:57 -0400 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:41496 "HELO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1161222AbcFMVAy (ORCPT ); Mon, 13 Jun 2016 17:00:54 -0400 From: "Rafael J. Wysocki" To: Sudeep Holla Cc: linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Vikas Sajjan , Sunil , Prashanth Prakash , Ashwin Chaugule , Al Stone , Lorenzo Pieralisi Subject: Re: [PATCH v5 2/5] ACPI / processor_idle: Add support for Low Power Idle(LPI) states Date: Mon, 13 Jun 2016 23:05:01 +0200 Message-ID: <6013371.oAQ1sZvyYq@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/4.5.0-rc1+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <575AFAF9.3020606@arm.com> References: <1462981062-24909-1-git-send-email-sudeep.holla@arm.com> <1462981062-24909-3-git-send-email-sudeep.holla@arm.com> <575AFAF9.3020606@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday, June 10, 2016 06:38:01 PM Sudeep Holla wrote: > Hi Rafael, Hi, > On 11/05/16 16:37, Sudeep Holla wrote: > > ACPI 6.0 introduced an optional object _LPI that provides an alternate > > method to describe Low Power Idle states. It defines the local power > > states for each node in a hierarchical processor topology. The OSPM can > > use _LPI object to select a local power state for each level of processor > > hierarchy in the system. They used to produce a composite power state > > request that is presented to the platform by the OSPM. > > > > Since multiple processors affect the idle state for any non-leaf hierarchy > > node, coordination of idle state requests between the processors is > > required. ACPI supports two different coordination schemes: Platform > > coordinated and OS initiated. > > > > This patch adds initial support for Platform coordination scheme of LPI. > > > > I have added support for autopromote states(basically skip flattening or > creating composite state). I have also fixed the bug discussed in this > thread with Prashant. Do you have any other feedback on this version > that I incorporate before posting next version. I'd really preferred it if you posted the next version without waiting for my feedback to the previous one (as the feedback may not be relevant any more among other things). Thanks, Rafael