Re: [PATCH 2/5] USB: serial: f81232: add high baud rate support

From: Johan Hovold
Date: Mon Jan 29 2018 - 22:31:57 EST


On Tue, Jan 23, 2018 at 10:08:24AM +0800, Ji-Ze Hong (Peter Hong) wrote:
> Hi Andy,
>
> Andy Shevchenko æ 2018/1/22 äå 10:55 åé:
> > On Mon, Jan 22, 2018 at 9:58 AM, Ji-Ze Hong (Peter Hong)
> > <hpeter@xxxxxxxxx> wrote:
> >> The F81232 had 4 clocksource 1.846/18.46/14.77/24MHz and baud rates
> >> can be up to 1.5Mbits with 24MHz.
> >>
> >> F81232 Clock registers (106h)
> >>
> >> Bit1-0: Clock source selector
> >> 00: 1.846MHz.
> >> 01: 18.46MHz.
> >> 10: 24MHz.
> >> 11: 14.77MHz.
> >
> > Hmm... Why not to provide a proper clk driver (based on table variant
> > of clk-divider) and use it here?
>
> It seems too complex to use clock framework in this driver.
> What do you think about this, Johan ?

Yeah, you don't need to implement a clk driver for this. If anyone
thinks that would simplify things, I'd be happy to consider it as a
follow-on patch.

Thanks,
Johan