Hi, I am not sure if this is needed. All calls within configure_dispc() ensure that clocks are enabled before a register write. The functions which read/write to registers and don't enable/disable clocks have names which start with a "_"(for most cases). Regards, Archit linux-omap-owner@xxxxxxxxxxxxxxx wrote: > From 086e3454c8f154cd90a4669899f2179f16ef32cd Mon Sep 17 00:00:00 > 2001 From: Ben Tucker <btucker@xxxxxxxxxxxxxx> > Date: Thu, 13 Jan 2011 12:56:45 +0000 > Subject: [PATCH] OMAP DSS Enable clocks in > dss_setup_partial_planes Enable the interface clocks while > calling configure_dispc(). > > --- > drivers/video/omap2/dss/manager.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/video/omap2/dss/manager.c > b/drivers/video/omap2/dss/manager.c > index 545e9b9..cb90dac 100644 > --- a/drivers/video/omap2/dss/manager.c > +++ b/drivers/video/omap2/dss/manager.c > @@ -1106,7 +1106,9 @@ void dss_setup_partial_planes(struct > omap_dss_device *dssdev, mc->w = w; > mc->h = h; > > + dss_clk_enable(DSS_CLK_ICK | DSS_CLK_FCK1); > configure_dispc(); + dss_clk_disable(DSS_CLK_ICK | > DSS_CLK_FCK1); > > mc->do_manual_update = false; > > -- > 1.7.3.2-- 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
