>> I thought following is correct:- >> Probe() >> clk_get() followed by clk_enable() >> Remove() >> clk_disable() followed by clk_put() >> Suspend() >> clk_disable() >> Resume() >> clk_enable() > >Yes, that is correct. > >I didn't look at the whole driver. My concern was that if the driver >was enhanced to more aggressive clock management, you shouldn't do a >clk_get() every time you do a clk_enable(), same for put. Got you. I am in sync with your thinking. -Murali > >Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html