From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757410Ab2EHWQY (ORCPT ); Tue, 8 May 2012 18:16:24 -0400 Received: from na3sys009aog130.obsmtp.com ([74.125.149.143]:53422 "EHLO na3sys009aog130.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757231Ab2EHWQW (ORCPT ); Tue, 8 May 2012 18:16:22 -0400 From: Kevin Hilman To: "Woodruff\, Richard" Cc: "AnilKumar\, Chimata" , "J\, KEERTHY" , Mark Brown , "linux-kernel\@vger.kernel.org" , "linux-pm\@lists.linux-foundation.org" , "linux-omap\@vger.kernel.org" , "Pihet-XID\, Jean" , "linux-arm-kernel\@lists.infradead.org" Subject: Re: [linux-pm] [PATCH V3 00/10] PM: Create the AVS(Adaptive Voltage Scaling) Organization: Texas Instruments, Inc. References: <1335462041-4949-1-git-send-email-j-keerthy@ti.com> <20120426191159.GA9415@sirena.org.uk> <20120427175657.GP18260@opensource.wolfsonmicro.com> <87397o3n4y.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B1A67@DBDE01.ent.ti.com> <87mx5jy2li.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B6681@DBDE01.ent.ti.com> Date: Tue, 08 May 2012 15:16:36 -0700 In-Reply-To: (Richard Woodruff's message of "Tue, 8 May 2012 20:38:35 +0000") Message-ID: <87lil2jp2z.fsf@ti.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org "Woodruff, Richard" writes: >> > >> The only thing the higher-level layers might potentially need to >> > >> do is to enable/disable AVS around transitions (e.g. when >> > >> changing OPP, > > >> AVS is disabled before changing OPP and >> > >> only re-enabled when the new > >> >> nominal voltage has been >> > >> acheived.) > > Getting clean baseline in place is huge step but actual production > interfaces will need to comprehend some OPP to AVS dependencies beyond > on/off. A basic OMAP AVS driver has been in mainline for a long time, yet we have not seen support submitted for all of these features. When folks are motivated to propose such changes upstream, we will be happy to discuss them and add support for them to the AVS driver. Until then, the primary purpose of this series is to do some minimal cleanup of an *existing* driver so it can be moved into drivers/*. New features can be added there as easily as they could've been added when it was a driver under arch/arm. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [linux-pm] [PATCH V3 00/10] PM: Create the AVS(Adaptive Voltage Scaling) Date: Tue, 08 May 2012 15:16:36 -0700 Message-ID: <87lil2jp2z.fsf@ti.com> References: <1335462041-4949-1-git-send-email-j-keerthy@ti.com> <20120426191159.GA9415@sirena.org.uk> <20120427175657.GP18260@opensource.wolfsonmicro.com> <87397o3n4y.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B1A67@DBDE01.ent.ti.com> <87mx5jy2li.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B6681@DBDE01.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: (Richard Woodruff's message of "Tue, 8 May 2012 20:38:35 +0000") Sender: linux-omap-owner@vger.kernel.org To: "Woodruff, Richard" Cc: "AnilKumar, Chimata" , "J, KEERTHY" , Mark Brown , "linux-kernel@vger.kernel.org" , "linux-pm@lists.linux-foundation.org" , "linux-omap@vger.kernel.org" , "Pihet-XID, Jean" , "linux-arm-kernel@lists.infradead.org" List-Id: linux-pm@vger.kernel.org "Woodruff, Richard" writes: >> > >> The only thing the higher-level layers might potentially need to >> > >> do is to enable/disable AVS around transitions (e.g. when >> > >> changing OPP, > > >> AVS is disabled before changing OPP and >> > >> only re-enabled when the new > >> >> nominal voltage has been >> > >> acheived.) > > Getting clean baseline in place is huge step but actual production > interfaces will need to comprehend some OPP to AVS dependencies beyond > on/off. A basic OMAP AVS driver has been in mainline for a long time, yet we have not seen support submitted for all of these features. When folks are motivated to propose such changes upstream, we will be happy to discuss them and add support for them to the AVS driver. Until then, the primary purpose of this series is to do some minimal cleanup of an *existing* driver so it can be moved into drivers/*. New features can be added there as easily as they could've been added when it was a driver under arch/arm. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Tue, 08 May 2012 15:16:36 -0700 Subject: [linux-pm] [PATCH V3 00/10] PM: Create the AVS(Adaptive Voltage Scaling) In-Reply-To: (Richard Woodruff's message of "Tue, 8 May 2012 20:38:35 +0000") References: <1335462041-4949-1-git-send-email-j-keerthy@ti.com> <20120426191159.GA9415@sirena.org.uk> <20120427175657.GP18260@opensource.wolfsonmicro.com> <87397o3n4y.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B1A67@DBDE01.ent.ti.com> <87mx5jy2li.fsf@ti.com> <331ABD5ECB02734CA317220B2BBEABC13E9B6681@DBDE01.ent.ti.com> Message-ID: <87lil2jp2z.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org "Woodruff, Richard" writes: >> > >> The only thing the higher-level layers might potentially need to >> > >> do is to enable/disable AVS around transitions (e.g. when >> > >> changing OPP, > > >> AVS is disabled before changing OPP and >> > >> only re-enabled when the new > >> >> nominal voltage has been >> > >> acheived.) > > Getting clean baseline in place is huge step but actual production > interfaces will need to comprehend some OPP to AVS dependencies beyond > on/off. A basic OMAP AVS driver has been in mainline for a long time, yet we have not seen support submitted for all of these features. When folks are motivated to propose such changes upstream, we will be happy to discuss them and add support for them to the AVS driver. Until then, the primary purpose of this series is to do some minimal cleanup of an *existing* driver so it can be moved into drivers/*. New features can be added there as easily as they could've been added when it was a driver under arch/arm. Kevin