From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] OMAP3: SR: Wait for VP idle before a VP disable Date: Tue, 30 Jun 2009 17:51:15 -0700 Message-ID: <87ws6t1b0c.fsf@deeprootsystems.com> References: <1246355137-13600-1-git-send-email-rnayak@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pz0-f188.google.com ([209.85.222.188]:62489 "EHLO mail-pz0-f188.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761871AbZGAAvV (ORCPT ); Tue, 30 Jun 2009 20:51:21 -0400 Received: by pzk26 with SMTP id 26so473151pzk.33 for ; Tue, 30 Jun 2009 17:51:24 -0700 (PDT) In-Reply-To: <1246355137-13600-1-git-send-email-rnayak@ti.com> (Rajendra Nayak's message of "Tue\, 30 Jun 2009 15\:15\:37 +0530") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Rajendra Nayak Cc: linux-omap@vger.kernel.org Rajendra Nayak writes: > In the SR disable path it is recommended to wait > for Voltage processor to idle before disabling the > VP. > > Signed-off-by: Rajendra Nayak Thanks, pushing to PM branch, and pm-2.6.29 (compile tested only for .29) Kevin > --- > arch/arm/mach-omap2/smartreflex.c | 47 +++++++++++++++++++++++++++++++++++++ > 1 files changed, 47 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-omap2/smartreflex.c b/arch/arm/mach-omap2/smartreflex.c > index 9d462e3..1407783 100644 > --- a/arch/arm/mach-omap2/smartreflex.c > +++ b/arch/arm/mach-omap2/smartreflex.c > @@ -36,6 +36,8 @@ > #include "smartreflex.h" > #include "prm-regbits-34xx.h" > > +#define MAX_TRIES 100 > + > struct omap_sr { > int srid; > int is_sr_reset; > @@ -620,16 +622,39 @@ static int sr_enable(struct omap_sr *sr, u32 target_opp_no) > > static void sr_disable(struct omap_sr *sr) > { > + u32 i = 0; > + > sr->is_sr_reset = 1; > > /* SRCONFIG - disable SR */ > sr_modify_reg(sr, SRCONFIG, SRCONFIG_SRENABLE, ~SRCONFIG_SRENABLE); > > if (sr->srid == SR1) { > + /* Wait for VP idle before disabling VP */ > + while ((!prm_read_mod_reg(OMAP3430_GR_MOD, > + OMAP3_PRM_VP1_STATUS_OFFSET)) > + && i++ < MAX_TRIES) > + udelay(1); > + > + if (i >= MAX_TRIES) > + pr_warning("VP1 not idle, still going ahead with \ > + VP1 disable\n"); > + > /* Disable VP1 */ > prm_clear_mod_reg_bits(PRM_VP1_CONFIG_VPENABLE, OMAP3430_GR_MOD, > OMAP3_PRM_VP1_CONFIG_OFFSET); > + > } else if (sr->srid == SR2) { > + /* Wait for VP idle before disabling VP */ > + while ((!prm_read_mod_reg(OMAP3430_GR_MOD, > + OMAP3_PRM_VP2_STATUS_OFFSET)) > + && i++ < MAX_TRIES) > + udelay(1); > + > + if (i >= MAX_TRIES) > + pr_warning("VP2 not idle, still going ahead with \ > + VP2 disable\n"); > + > /* Disable VP2 */ > prm_clear_mod_reg_bits(PRM_VP2_CONFIG_VPENABLE, OMAP3430_GR_MOD, > OMAP3_PRM_VP2_CONFIG_OFFSET); > @@ -731,6 +756,8 @@ void enable_smartreflex(int srid) > > void disable_smartreflex(int srid) > { > + u32 i = 0; > + > struct omap_sr *sr = NULL; > > if (srid == SR1) > @@ -751,11 +778,31 @@ void disable_smartreflex(int srid) > /* Disable SR clk */ > sr_clk_disable(sr); > if (sr->srid == SR1) { > + /* Wait for VP idle before disabling VP */ > + while ((!prm_read_mod_reg(OMAP3430_GR_MOD, > + OMAP3_PRM_VP1_STATUS_OFFSET)) > + && i++ < MAX_TRIES) > + udelay(1); > + > + if (i >= MAX_TRIES) > + pr_warning("VP1 not idle, still going \ > + ahead with VP1 disable\n"); > + > /* Disable VP1 */ > prm_clear_mod_reg_bits(PRM_VP1_CONFIG_VPENABLE, > OMAP3430_GR_MOD, > OMAP3_PRM_VP1_CONFIG_OFFSET); > } else if (sr->srid == SR2) { > + /* Wait for VP idle before disabling VP */ > + while ((!prm_read_mod_reg(OMAP3430_GR_MOD, > + OMAP3_PRM_VP2_STATUS_OFFSET)) > + && i++ < MAX_TRIES) > + udelay(1); > + > + if (i >= MAX_TRIES) > + pr_warning("VP2 not idle, still going \ > + ahead with VP2 disable\n"); > + > /* Disable VP2 */ > prm_clear_mod_reg_bits(PRM_VP2_CONFIG_VPENABLE, > OMAP3430_GR_MOD, > -- > 1.5.4.7 > > -- > 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