On Wed, Jan 30, 2013 at 22:09:51, Paul Walmsley wrote: > On Wed, 30 Jan 2013, Hebbar, Gururaja wrote: > > > On Thu, Dec 20, 2012 at 13:05:27, Paul Walmsley wrote: > > > On Thu, 20 Dec 2012, Hebbar, Gururaja wrote: > > > > > > > On Wed, Dec 19, 2012 at 07:43:50, Paul Walmsley wrote: > > > > > > > > > We've got a few hwmods on OMAP44xx that don't have clkctrl_offs registers > > > > > listed in the hwmod data, and which are not marked with HWMOD_NO_IDLEST. > > > > > What's going to happen in those cases? > > > > > > > > Hmm. This is a special case to me. Let me go back and do some review. > > > > Will be back with some more details. > > > > > > Probably what needs to happen is that those hwmods need to be marked with > > > HWMOD_NO_IDLEST, in a separate patch before yours. As far as I can tell, > > > that's how they should have been marked initially. Then your patch should > > > be fine. > > > > Looking at latest kernel v3.8-rc5, there is separate cm33xx.c file which > > handles module ready checking for am33xx platform. So I will update this > > patch to work on this file instead of touching omap4 related file > > (cminst44xx.c) > > The same problem still exists. According to the TRM, there's at least one > AM335x CLKCTRL register that doesn't have IDLEST bits. So to avoid > reading reserved bits, either this patch or a preceding patch should > mark those hwmods with HWMOD_NO_IDLEST. Ok. I will look into this now. > > > - Paul > Regards, Gururaja -- 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
