Tomi Valkeinen <tomi.valkeinen@xxxxxxxxx> writes: > Hi, > > Mans Rullgard has the patches mentioned below in his kernel tree at > http://git.mansr.com/?p=linux-omap;a=summary . I currently have those > patches and use the following sequence to set the dss1_alwon_fck clock: > > /* get the rate of the parent of dpll4_m4_ck, which is dpll4_ck */ > prate = clk_get_rate(clk_get_parent(dispc.dpll4_m4_ck)); > > /* calculate divisor so that wanted dss1_alwon_fck is prate / div * 2. > * x2 because dss1_alwon_fck comes from dpll4_m4x2_ck */ > div = ...; > > /* set the rate */ > clk_set_rate(dispc.dpll4_m4_ck, prate / div); > > Is this the correct way to set dss1_alwon_fck? It's the only way I could find. I'd prefer if the mucking about with get_parent() could be avoided, but Paul (I think) told me there was no other way, at least not then. Maybe it has changed since. I'd also prefer to not do those divisor calculations. However, I could think of no other way to get the desired result, since clock_set_rate() isn't aware of the cascaded dividers. > Are the patches below needed, or can dss1_alwon_fck be set some > other way? At the time I wrote them, they were necessary. If later changes to the clock subsystem have made them obsolete, all the better. I don't recall seeing anything like that going in, though. > commit e2de5e5578fbaa9b4b75074796da0608fc93e6ae > Author: Mans Rullgard <mans@xxxxxxxxx> > Date: Tue Jul 22 01:58:18 2008 +0100 > > ARM: OMAP: add clk_get_parent() for OMAP2/3 > > Signed-off-by: Mans Rullgard <mans@xxxxxxxxx> > > :100644 100644 577be44... 28aec36... M arch/arm/mach-omap2/clock.c > :100644 100644 49245f7... 4aa69d5... M arch/arm/mach-omap2/clock.h > :100644 100644 54cc6e1... ed7af21... M arch/arm/mach-omap2/clock24xx.c > :100644 100644 04dedec... 08c8c46... M arch/arm/mach-omap2/clock34xx.c > > commit 2b7b958dc79e51127d7a4ecf88ce12dbc6c31426 > Author: Mans Rullgard <mans@xxxxxxxxx> > Date: Tue Jul 22 01:31:11 2008 +0100 > > ARM: OMAP: make dpll4_m4_ck programmable with clk_set_rate() > > Filling the set_rate and round_rate fields of dpll4_m4_ck makes > this clock programmable through clk_set_rate(). This is needed > to give omapfb control over the dss1_alwon_fck rate. > > :100644 100644 161da12... 876eb13... M arch/arm/mach-omap2/clock34xx.h I'll send the patches as replies to this mail for easier reference. -- Måns Rullgård mans@xxxxxxxxx -- 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