From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Subject: Re: [PATCH 12/15] OMAP4: powerdomain data: add wake-up latency figures Date: Tue, 16 Aug 2011 20:28:08 +0530 Message-ID: <4E4A8580.4080806__42365.8932363198$1313506765$gmane$org@ti.com> References: <1313502198-9298-1-git-send-email-j-pihet@ti.com> <1313502198-9298-13-git-send-email-j-pihet@ti.com> <4E4A7E25.8020701@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Jean Pihet Cc: markgross@thegnar.org, Mark Brown , Linux PM mailing list , linux-omap@vger.kernel.org, Jean Pihet List-Id: linux-pm@vger.kernel.org On Tuesday 16 August 2011 08:08 PM, Jean Pihet wrote: > On Tue, Aug 16, 2011 at 4:26 PM, Santosh wrote: >> On Tuesday 16 August 2011 07:13 PM, jean.pihet@newoldbits.com wrote: >>> >>> From: Vishwanath BS >>> >>> This patch adds wake up latency numbers for OMAP4. Note that these are >>> preliminary numbers and need to be relooked. >>> >>> Signed-off-by: Vishwanath BS >>> >>> The INACTIVE state is added as unsupported. >>> >> In that case, don't add that support in first place. When INA support is >> getting added, you can update these as well. > No. A value is needed for all states, even if unsupported at the > moment. Omitting a value causes it to be set to '0', which means 'no > latency'. > What I am saying is don't add "PWRDM_FUNC_PWRST_INACTIVE" which is not supported. Then you won't even have that state and no need of latency number for that in the current series. Regards Santosh