[PATCH 0/4] rockchip: fix serial output on rk3036
Heiko Stuebner
heiko at sntech.de
Tue Feb 28 09:25:50 PST 2017
Am Montag, 27. Februar 2017, 23:32:40 CET schrieb Stephen Boyd:
> On 02/28, Heiko Stuebner wrote:
> > Recent changes to the 8250-dw variant revealed issues concerning
> > how the clock rates are handled on the rk3036 uart.
> >
> > For one, there was an error in the clock declaration, but also the
> > shared uart-pll-select-mux also as default got supplied from the apll
> > that also supplies the cpu and thus gets frequency scaled.
> >
> > The patches in this series remedy this and make the debug uart
> > function again on 4.10 + current merge window.
>
> What's the merge path? The last patch is sort of questionable
> because it fixes a regression by changing assigned clocks in DT,
> which doesn't really make sense from a DT perspective (it should
> have been right already or can be configured from the clk driver
> itself in software).
yeah, I was (and somewhat still am) debating on the dt vs. clk driver
positioning. As you can see, there are multiple sources, most of them are
somewhat questionable. I.e. apll and dpll are the cpu + ddr supplies and we
will want to do ddr-scaling at some point too.
So the gpll really is the only really sane option and I guess simply doing it
in the driver (similar to what rk3188 does) might be better.
If some boards really want to select a different source, they then can do that
via assigned-clocks in the board-dts.
More information about the Linux-rockchip
mailing list