* Kalle Jokiniemi <ext-kalle.jokiniemi@xxxxxxxxx> [081024 03:50]: > Hi Tony, > > On pe, 2008-10-17 at 09:46 +0300, Kalle Jokiniemi wrote: > > Remove CONFIG_OMAP_SYSOFFMODE flag and do the full initialization of voltage > > controller . Enabling automatic sending of OFF command and selecting wether > > SYS_OFF_MODE signal is used should be determined dynamically. Hence they are > > now not set in the initialization. > > Any chance getting this in? > > It's currently in the pm-branch, but it does not really depend on any of > the pathces there. So I think it could be moved away from the pm-branch > and into the linux-omap master. Well I was kind of waiting for Kevin and Paul to ack this. But looks like this fix is safe to apply, so pushing today. Tony > > regards, > Kalle > > > > > The sleep voltage for OFF mode is changed to default 1,2V for VDD1 and > > 1,15V for VDD2. Using the 0,9V setting causes hangup. > > > > Signed-off-by: Kalle Jokiniemi <ext-kalle.jokiniemi@xxxxxxxxx> > > --- > > arch/arm/mach-omap2/pm34xx.c | 10 +--------- > > arch/arm/mach-omap2/prm-regbits-34xx.h | 4 ++-- > > 2 files changed, 3 insertions(+), 11 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c > > index a828db6..457639f 100644 > > --- a/arch/arm/mach-omap2/pm34xx.c > > +++ b/arch/arm/mach-omap2/pm34xx.c > > @@ -622,10 +622,7 @@ static void __init configure_vc(void) > > OMAP3_PRM_VC_I2C_CFG_OFFSET); > > > > /* Setup voltctrl and other setup times */ > > - > > -#ifdef CONFIG_OMAP_SYSOFFMODE > > - prm_write_mod_reg(OMAP3430_AUTO_OFF | OMAP3430_AUTO_RET | > > - OMAP3430_SEL_OFF, OMAP3430_GR_MOD, > > + prm_write_mod_reg(OMAP3430_AUTO_RET, OMAP3430_GR_MOD, > > OMAP3_PRM_VOLTCTRL_OFFSET); > > > > prm_write_mod_reg(OMAP3430_CLKSETUP_DURATION, OMAP3430_GR_MOD, > > @@ -640,11 +637,6 @@ static void __init configure_vc(void) > > OMAP3_PRM_VOLTOFFSET_OFFSET); > > prm_write_mod_reg(OMAP3430_VOLTSETUP2_DURATION, OMAP3430_GR_MOD, > > OMAP3_PRM_VOLTSETUP2_OFFSET); > > -#else > > - prm_set_mod_reg_bits(OMAP3430_AUTO_RET, OMAP3430_GR_MOD, > > - OMAP3_PRM_VOLTCTRL_OFFSET); > > -#endif > > - > > } > > > > static int __init omap3_pm_early_init(void) > > diff --git a/arch/arm/mach-omap2/prm-regbits-34xx.h b/arch/arm/mach-omap2/prm-regbits-34xx.h > > index f82b5a7..d73eee8 100644 > > --- a/arch/arm/mach-omap2/prm-regbits-34xx.h > > +++ b/arch/arm/mach-omap2/prm-regbits-34xx.h > > @@ -466,13 +466,13 @@ > > #define OMAP3430_VC_CMD_VAL0_ON (0x3 << 4) > > #define OMAP3430_VC_CMD_VAL0_ONLP (0x3 << 3) > > #define OMAP3430_VC_CMD_VAL0_RET (0x3 << 3) > > -#define OMAP3430_VC_CMD_VAL0_OFF (0x3 << 3) > > +#define OMAP3430_VC_CMD_VAL0_OFF (0x3 << 4) > > > > /* PRM_VC_CMD_VAL_1 specific bits */ > > #define OMAP3430_VC_CMD_VAL1_ON (0xB << 2) > > #define OMAP3430_VC_CMD_VAL1_ONLP (0x3 << 3) > > #define OMAP3430_VC_CMD_VAL1_RET (0x3 << 3) > > -#define OMAP3430_VC_CMD_VAL1_OFF (0x3 << 3) > > +#define OMAP3430_VC_CMD_VAL1_OFF (0xB << 2) > > > > /* PRM_VC_CH_CONF */ > > #define OMAP3430_CMD1 (1 << 20) -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html