On Wed, Jun 29, 2011 at 11:26 PM, Kevin Hilman <khilman@xxxxxx> wrote: > "T Krishnamoorthy, Balaji" <balajitk@xxxxxx> writes: > >> On Wed, Jun 29, 2011 at 5:02 AM, Kevin Hilman <khilman@xxxxxx> wrote: >>> +Rajendra >>> >>> Balaji T K <balajitk@xxxxxx> writes: >>> >>>> add runtime pm support to HSMMC host controller >>>> Use runtime pm API to enable/disable HSMMC clock >>>> Use runtime autosuspend APIs to enable auto suspend delay >>>> >>>> Based on OMAP HSMMC runtime implementation by Kevin Hilman, Kishore Kadiyala >>>> >>>> Signed-off-by: Balaji T K <balajitk@xxxxxx> >>> >>> I tried to test this series along with Benoit's clkdm/modulemode/hwmod >>> cleanups and something strange is happening on OMAP4. >>> >>> First, this series by itself is working as I would expect, but testing >>> in combination with Benoit's series, it's different... >>> >>> First, I'm using Benoit's branch: >>> >>> git://gitorious.org/omap-pm/linux.git for_3.0.1/7_hwmod_modulemode >>> >>> in combination with your series. >>> >>> I've also reverted these two commits: >>> >>> OMAP4: PM: TEMP: Prevent l3init from idling/force sleep >>> OMAP3+: hwmod data: TEMP: Do not idle MMC1 & MMC2 after boot >>> >>> which are temporary workarounds for not having MMC runtime PM. >>> >>> I turned the dev_dbg calls in the runtime PM callbacks into dev_info >>> callbacks to see exactly when the device is enabled/disabled via runtime >>> PM. >>> >>> To my surprise, I didn't see the device being enabled/disabled when >>> writing do the device. >> >> I tried for_3.0.1/7_hwmod_modulemode as you mentioned above, >> changing dev_dbg to dev_err and I am not observing the delayed write >> during umount. Can you let me know if the files are there in MMC after >> remounting ? > > Did you also revert the above two commits? yes > > I've pushed a 'tmp/mmc' branch to my git tree[1] which is my PM branch > (including various PM stuff queued for upstream) as well as Benoit's > series with the TEMP patches above reverted and your series on top of > it. There's one additional patch to convert the dev_dbg into dev_info > to see the transitions. > > Testing this on my OMAP4430 ES2.2 EMU device, I still see the above > behavior. I used OMAP4430 ES2.1 GP > > Can you try out this branch on your board using the default > omap2plus_defconfig? Ok, I will retry on EMU device. > > Thanks, > > Kevin > > [1] git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git > -- 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
