On Wed, May 06, 2015 at 12:18:11PM +0200, Jens Kuske wrote:
> >> /**
> >> * sunxi_divs_clk_setup() - Setup function for leaf divisors on clocks
> >> *
> >> @@ -1252,6 +1288,7 @@ static const struct of_device_id clk_factors_match[] __initconst = {
> >> {.compatible = "allwinner,sun5i-a13-ahb-clk", .data = &sun5i_a13_ahb_data,},
> >> {.compatible = "allwinner,sun4i-a10-apb1-clk", .data = &sun4i_apb1_data,},
> >> {.compatible = "allwinner,sun7i-a20-out-clk", .data = &sun7i_a20_out_data,},
> >> + {.compatible = "allwinner,sun8i-h3-pll8-clk", .data = &sun8i_h3_pll8_data,},
> >
> > Matching comment above, no need to add a new compatible for something that's
> > the same.
>
> PLL8 doesn't have a x2 output according to User Manual, but I did it
> wrong anyway, see first comment.
Don't worry too much about that output.
It's not really an output, it's a pre-multiplier on a single
child. We're discussing with Chen-Yu about refactoring this anyway, so
it will probably go away in a near future.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
Attachment:
signature.asc
Description: Digital signature
