On Thu, Dec 13, 2012 at 11:32 PM, Paul Walmsley <paul@xxxxxxxxx> wrote: > The OMAP4 clock divider "div_iva_hs_clk" is listed in the clock data > as an OMAP HSDIVIDER, but it's actually a power-of-two divider. This > causes a warning during boot on an OMAP4460 Pandaboard-ES with a > recent u-boot: > > WARNING: at arch/arm/mach-omap2/clkt_clksel.c:143 omap2_clksel_recalc+0xf4/0x12c() > clock: div_iva_hs_clk: could not find fieldval 0 for parent dpll_core_m5x2_ck > > Fix by converting the data for this clock to a power-of-two divider. > > Signed-off-by: Paul Walmsley <paul@xxxxxxxxx> > Cc: Mike Turquette <mturquette@xxxxxxxxxx> > --- > arch/arm/mach-omap2/cclock44xx_data.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/mach-omap2/cclock44xx_data.c b/arch/arm/mach-omap2/cclock44xx_data.c > index aa56c3e..a3c54b2 100644 > --- a/arch/arm/mach-omap2/cclock44xx_data.c > +++ b/arch/arm/mach-omap2/cclock44xx_data.c > @@ -286,9 +286,9 @@ DEFINE_CLK_DIVIDER(div_core_ck, "dpll_core_m5x2_ck", &dpll_core_m5x2_ck, 0x0, > OMAP4430_CM_CLKSEL_CORE, OMAP4430_CLKSEL_CORE_SHIFT, > OMAP4430_CLKSEL_CORE_WIDTH, 0x0, NULL); > > -DEFINE_CLK_OMAP_HSDIVIDER(div_iva_hs_clk, "dpll_core_m5x2_ck", > - &dpll_core_m5x2_ck, 0x0, OMAP4430_CM_BYPCLK_DPLL_IVA, > - OMAP4430_CLKSEL_0_1_MASK); > +DEFINE_CLK_DIVIDER(div_iva_hs_clk, "dpll_core_m5x2_ck", &dpll_core_m5x2_ck, > + 0x0, OMAP4430_CM_BYPCLK_DPLL_IVA, OMAP4430_CLKSEL_0_1_SHIFT, > + OMAP4430_CLKSEL_0_1_WIDTH, CLK_DIVIDER_POWER_OF_TWO, NULL); > Hi Paul, Looks good to me. Regards, Mike > DEFINE_CLK_DIVIDER(div_mpu_hs_clk, "dpll_core_m5x2_ck", &dpll_core_m5x2_ck, > 0x0, OMAP4430_CM_BYPCLK_DPLL_MPU, OMAP4430_CLKSEL_0_1_SHIFT, > > -- 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
