> -----Original Message-----
> From: Peter 'p2' De Schrijver [mailto:peter.de-schrij...@nokia.com]
> Sent: Monday, May 10, 2010 8:01 PM
> To: Lesly A M
> Cc: linux-omap@vger.kernel.org; Lesly A M; Nishanth Menon; David Derrick;
> Samuel Ortiz
> Subject: Re: [PATCH v5 3/5] omap3: pm: Generic TRITON power scripts
> forOMAP3 based boards
> 
> On Mon, Apr 19, 2010 at 01:43:28PM +0200, ext Lesly A M wrote:
> > This pacth will create the generic TRITON power scripts which can be
> used
> > by different OMAP3 boards with the same power companion chip (TWL4030
> series).
> >
> > Added the api(twl4030_get_scripts/twl4030_get_vc_timings) to update
> > the sleep/wakeup/warm_rest sequence & voltsetup_time in the board file.
> >
> 
> any reason why you use a different type for the messages sent for P3 and
> P1/P2
> transitions ?
> 
> > +/*
> > + * Sequence to control the TRITON Power resources,
> > + * when the system goes into sleep.
> > + * Executed upon P1_P2/P3 transition for sleep.
> > + */
> > +static struct twl4030_ins __initdata sleep_on_seq[] = {
> > +   /* Broadcast message to put res to sleep */
> > +   {MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_ALL, RES_TYPE_R0, RES_TYPE2_R1,
> > +                                                   RES_STATE_SLEEP),
2},
> > +   {MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_ALL, RES_TYPE_R0, RES_TYPE2_R2,
> > +                                                   RES_STATE_SLEEP),
2},
> > +};
> > +
> > +static struct twl4030_script sleep_on_script __initdata = {
> > +   .script = sleep_on_seq,
> > +   .size   = ARRAY_SIZE(sleep_on_seq),
> > +   .flags  = TWL4030_SLEEP_SCRIPT,
> > +};
> > +
> > +/*
> > + * Sequence to control the TRITON Power resources,
> > + * when the system wakeup from sleep.
> > + * Executed upon P1_P2 transition for wakeup.
> > + */
> > +static struct twl4030_ins wakeup_p12_seq[] __initdata = {
> > +   /* Broadcast message to put res to active */
> > +   {MSG_BROADCAST(DEV_GRP_NULL, RES_GRP_ALL, RES_TYPE_R0, RES_TYPE2_R1,
> > +                                                   RES_STATE_ACTIVE),
2},
> > +};
> > +
> > +static struct twl4030_script wakeup_p12_script __initdata = {
> > +   .script = wakeup_p12_seq,
> > +   .size   = ARRAY_SIZE(wakeup_p12_seq),
> > +   .flags  = TWL4030_WAKEUP12_SCRIPT,
> > +};
> 
> Cheers,
> Peter.

Hi Peter,

The RES_TYPE field is same for all messages.

The resource which can go to low-power mode with clk_req sig de-asserting is
configured as RES_TYPE2 = '2'. And the resource which can go to low-power
mode with sys_off sig de-asserting is configured as RES_TYPE2 = '1'.

The RES_TYPE2 is '2' for P3 wakeup and the msg will applicable for Resource
which have their TYPE2 field configured '2' (RES_VINTANA1, RES_VINTANA2,
RES_VINTDIG, VIO, RES_CLKEN, RES_HFCLKOUT).

RES_TYPE2 = '1' for P1_P2 wakeup.
(Applicable for res: RES_VPLL1, RES_VDD1, RES_VDD2, RES_REGEN,
RES_NRES_PWRON, RES_SYSEN)

Regards,
Lesly A M


--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to