On Tue, 2015-06-30 at 01:45 +0000, Kuninori Morimoto wrote:
> Hi Ben
>
> > > > > SH-MMC is using this style. and I think it is flexible for every
> > > > > speed.
> > > > > Please check sh_mmcif_clock_control(), sh_mmcif_clk_setup()
> > > > > on ${LINUX}/drivers/mmc/host/sh_mmcif.c
> > > >
> > > > That's certainly a nicer way of doing this. The difficulty I see is
> > > > that tmio_mmc doesn't know anything about the input clock, and not all
> > > > of the drivers using it actually use the clock framework.
> > >
> > > More importantly, that algrithm can result in overclocking the card.
> >
> > ...and some of the multiplications overflow!
>
> Can you show us more detail ?
> It seems bug...
The requested clock rate can be up to 52 MHz and the divider can be up
to 1024. With div == 1024 and clk == 52000000, clk * div overflows.
Ben.
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
