On Monday 22 July 2013 11:18 PM, Paul Walmsley wrote: > Hi > > On Mon, 22 Jul 2013, Rajendra Nayak wrote: > >> Memory controllers in OMAP (like GPMC and EMIF) have the hwmods marked with >> HWMOD_INIT_NO_IDLE and are left in enabled state post initial setup. >> >> Even if they have drivers missing, avoid idling them as part of >> omap_device_late_idle() > > Since this patch is intended for the -rc series, it's best if you explain > here what effects this patch will fix. i.e., does this fix a crash, a > warning, etc? Also maybe update your series description message? This patch infact is needed because of 'PATCH 4/4' in the series which makes sure the omap_device and pm_runtime state are in sync with the hwmod state. Without that we always had gpmc and emif having the hwmod state as enabled but omap_device state as disabled and we never hit the issue of omap_device_late_idle() trying to disable gpmc and emif. So in short, this does not fix anything in the existing code, but something that was uncovered by syncing the states across hwmod and omap_device. regards, Rajendra > > > - Paul > -- 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
