[PATCH v7 4/6] OMAP4: hwmod data: Add watchdog timer

[Varadarajan, Charulatha]

> -----Original Message-----
> From: Kevin Hilman [mailto:khilman at deeprootsystems.com]
> 
> "Cousson, Benoit" <b-cousson at ti.com> writes:
> 
> > On 9/21/2010 3:37 PM, Varadarajan, Charulatha wrote:
> 
> [...]
> 
> >>   static __initdata struct omap_hwmod *omap44xx_hwmods[] = {
> >>   	/* dmm class */
> >>   	&omap44xx_dmm_hwmod,
> >> @@ -472,6 +602,11 @@ static __initdata struct omap_hwmod
> *omap44xx_hwmods[] = {
> >>
> >>   	/* mpu class */
> >>   	&omap44xx_mpu_hwmod,
> >> +	/* wd_timer class */
> >> +	&omap44xx_wd_timer2_hwmod,
> >> +
> >> +	/* wd_timer3 not yet supported */
> >> +/*	&omap44xx_wd_timer3_hwmod, */
> >
> > Sorry, I forgot to comment that part in your previous revision.
> > What we agreed with Kevin is to enable that hwmod watchdog as well in
> > order to allow the hwmod core to initialize properly this module, even
> > if it not used by any driver yet.
> >
> > Beside that you can add my Acked-by: for the whole series.
> 
> Charu,
> 
> If this is OK with you, can you test with timer3 hwmod enabled, and
> repost an updated version of this patch?
> 
> Then, I will add Benoit's ack and queue this up for 2.6.37.

Already posted the series with this review comment fixed [1].

The latest series (version v8) [1] may also be pulled from [2]

[1] https://patchwork.kernel.org/patch/202152/
[2] git://gitorious.org/omap-pm/linux.git wdt_for_2.6.37

-V Charulatha