From patchwork Wed Apr 15 12:55:27 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rajendra Nayak X-Patchwork-Id: 18356 X-Patchwork-Delegate: khilman@deeprootsystems.com Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n3FCtHBZ017100 for ; Wed, 15 Apr 2009 12:55:39 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752671AbZDOMzi (ORCPT ); Wed, 15 Apr 2009 08:55:38 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753799AbZDOMzi (ORCPT ); Wed, 15 Apr 2009 08:55:38 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:55305 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752671AbZDOMzh convert rfc822-to-8bit (ORCPT ); Wed, 15 Apr 2009 08:55:37 -0400 Received: from dbdp20.itg.ti.com ([172.24.170.38]) by devils.ext.ti.com (8.13.7/8.13.7) with ESMTP id n3FCtUUZ013681 for ; Wed, 15 Apr 2009 07:55:36 -0500 Received: from dbde71.ent.ti.com (localhost [127.0.0.1]) by dbdp20.itg.ti.com (8.13.8/8.13.8) with ESMTP id n3FCtTwD021653 for ; Wed, 15 Apr 2009 18:25:29 +0530 (IST) Received: from dbde02.ent.ti.com ([172.24.170.145]) by dbde71.ent.ti.com ([172.24.170.149]) with mapi; Wed, 15 Apr 2009 18:25:29 +0530 From: "Nayak, Rajendra" To: linux-omap Date: Wed, 15 Apr 2009 18:25:27 +0530 Subject: [PATCH 01/10] OMAP3: SR: Fix init voltage on OPP change Thread-Topic: [PATCH 01/10] OMAP3: SR: Fix init voltage on OPP change Thread-Index: Acm9yXNiwtCCjc0lQdq/hYttrpWueg== Message-ID: <5A47E75E594F054BAF48C5E4FC4B92AB02FB25180C@dbde02.ent.ti.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org From: Rajendra Nayak This patch fixes a bug wherein the inital voltage was not set correctly on a OPP change Signed-off-by: Rajendra Nayak Signed-off-by: Jouni Hogander --- arch/arm/mach-omap2/smartreflex.c | 42 ++++++++++++++++++++++++++++++++++---- 1 files changed, 38 insertions(+), 4 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Index: linux-omap-2.6/arch/arm/mach-omap2/smartreflex.c =================================================================== --- linux-omap-2.6.orig/arch/arm/mach-omap2/smartreflex.c 2009-04-15 11:44:51.000000000 +0530 +++ linux-omap-2.6/arch/arm/mach-omap2/smartreflex.c 2009-04-15 11:45:57.000000000 +0530 @@ -30,6 +30,7 @@ #include #include #include +#include #include "prm.h" #include "smartreflex.h" @@ -183,7 +184,6 @@ static void sr_set_efuse_nvalues(struct sr->senn_mod = (omap_ctrl_readl(OMAP343X_CONTROL_FUSE_SR) & OMAP343X_SR1_SENNENABLE_MASK) >> OMAP343X_SR1_SENNENABLE_SHIFT; - sr->senp_mod = (omap_ctrl_readl(OMAP343X_CONTROL_FUSE_SR) & OMAP343X_SR1_SENPENABLE_MASK) >> OMAP343X_SR1_SENPENABLE_SHIFT; @@ -364,7 +364,12 @@ static void sr_configure(struct omap_sr static int sr_enable(struct omap_sr *sr, u32 target_opp_no) { - u32 nvalue_reciprocal; + u32 nvalue_reciprocal, v; + + if (!(mpu_opps && l3_opps)) { + pr_notice("VSEL values not found\n"); + return false; + } sr->req_opp_no = target_opp_no; @@ -418,14 +423,43 @@ static int sr_enable(struct omap_sr *sr, sr_modify_reg(sr, ERRCONFIG, (ERRCONFIG_VPBOUNDINTEN | ERRCONFIG_VPBOUNDINTST), (ERRCONFIG_VPBOUNDINTEN | ERRCONFIG_VPBOUNDINTST)); + if (sr->srid == SR1) { + /* set/latch init voltage */ + v = prm_read_mod_reg(OMAP3430_GR_MOD, + OMAP3_PRM_VP1_CONFIG_OFFSET); + v &= ~(OMAP3430_INITVOLTAGE_MASK | OMAP3430_INITVDD); + v |= mpu_opps[target_opp_no].vsel << + OMAP3430_INITVOLTAGE_SHIFT; + prm_write_mod_reg(v, OMAP3430_GR_MOD, + OMAP3_PRM_VP1_CONFIG_OFFSET); + /* write1 to latch */ + prm_set_mod_reg_bits(OMAP3430_INITVDD, OMAP3430_GR_MOD, + OMAP3_PRM_VP1_CONFIG_OFFSET); + /* write2 clear */ + prm_clear_mod_reg_bits(OMAP3430_INITVDD, OMAP3430_GR_MOD, + OMAP3_PRM_VP1_CONFIG_OFFSET); /* Enable VP1 */ prm_set_mod_reg_bits(PRM_VP1_CONFIG_VPENABLE, OMAP3430_GR_MOD, - OMAP3_PRM_VP1_CONFIG_OFFSET); + OMAP3_PRM_VP1_CONFIG_OFFSET); } else if (sr->srid == SR2) { + /* set/latch init voltage */ + v = prm_read_mod_reg(OMAP3430_GR_MOD, + OMAP3_PRM_VP2_CONFIG_OFFSET); + v &= ~(OMAP3430_INITVOLTAGE_MASK | OMAP3430_INITVDD); + v |= l3_opps[target_opp_no].vsel << + OMAP3430_INITVOLTAGE_SHIFT; + prm_write_mod_reg(v, OMAP3430_GR_MOD, + OMAP3_PRM_VP2_CONFIG_OFFSET); + /* write1 to latch */ + prm_set_mod_reg_bits(OMAP3430_INITVDD, OMAP3430_GR_MOD, + OMAP3_PRM_VP2_CONFIG_OFFSET); + /* write2 clear */ + prm_clear_mod_reg_bits(OMAP3430_INITVDD, OMAP3430_GR_MOD, + OMAP3_PRM_VP2_CONFIG_OFFSET); /* Enable VP2 */ prm_set_mod_reg_bits(PRM_VP2_CONFIG_VPENABLE, OMAP3430_GR_MOD, - OMAP3_PRM_VP2_CONFIG_OFFSET); + OMAP3_PRM_VP2_CONFIG_OFFSET); } /* SRCONFIG - enable SR */