On Thu, 10 Mar 2011, Kalle Jokiniemi wrote: > These patches fix the issue where the wdtimer blocks CORE idle > transitions on n900 (OMAP3). The root cause was that SMART idle > mode in wdtimer did not allow the CORE idle transition to happen. > > Paul, I added your S-o-bs to the patches already. Please check > that they are ok by you.
Thanks, they look mostly fine to me. What do you think about saying something like this in the commit message on the first one? This patch is based on a patch created by Kalle Jokiniemi: https://patchwork.kernel.org/patch/618231/ which is itself based on a patch created by Atal Shargorodsky: http://lkml.org/lkml/2009/3/10/266. Also, I guess we should get an Acked-by for the second patch from Wim Van Sebroeck <[email protected]> since he is the maintainer. Could you forward a copy of the second patch to [email protected] and Wim and request his Ack? thanks, - Paul > > Paul Walmsley (2): > Watchdog: omap_wdt: add fine grain runtime-pm > OMAP3: wdtimer: Fix CORE idle transition > > arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 1 + > drivers/watchdog/omap_wdt.c | 25 +++++++++++++++++++++++-- > 2 files changed, 24 insertions(+), 2 deletions(-) > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
