Re: [PATCH 0/2] OMAP3 ISP: Simplify clock usage

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Mike,

On Monday 14 January 2013 17:10:15 Mike Turquette wrote:
> Quoting Laurent Pinchart (2013-01-08 05:43:52)
> 
> > Hello,
> > 
> > Now that the OMAP3 supports the common clock framework, clock rate
> > back-propagation is available for the ISP clocks. Instead of setting the
> > cam_mclk parent clock rate to control the cam_mclk clock rate, we can mark
> > the dpll4_m5x2_ck_3630 and cam_mclk clocks as supporting
> > back-propagation, and set the cam_mclk rate directly. This simplifies the
> > ISP clocks configuration.
>
> I'm pleased to see this feature get used on OMAP.  Plus your driver gets
> a negative diffstat :)
> 
> Reviewed-by: Mike Turquette <mturquette@linaro.org>

Thanks.

Would you like to take the arch/ patch in your tree, or should I push it 
through the linux-media tree along with the omap3isp patch ?

> > Laurent Pinchart (2):
> >   ARM: OMAP3: clock: Back-propagate rate change from cam_mclk to
> >     dpll4_m5
> >   omap3isp: Set cam_mclk rate directly
> >  
> >  arch/arm/mach-omap2/cclock3xxx_data.c |   10 +++++++++-
> >  drivers/media/platform/omap3isp/isp.c |   18 ++----------------
> >  drivers/media/platform/omap3isp/isp.h |    8 +++-----
> >  3 files changed, 14 insertions(+), 22 deletions(-)

-- 
Regards,

Laurent Pinchart