On 08/05/2014 16:29, Maxime Ripard wrote: > On Thu, May 08, 2014 at 11:40:59AM +0800, Chen-Yu Tsai wrote: >>>> + apb0_gates: apb0_gates_clk { >>>> + compatible = "allwinner,sun6i-a31-apb0-gates-clk"; >>>> + #clock-cells = <1>; >>>> + clocks = <&apb0>; >>>> + clock-output-names = "apb0_pio", "apb0_ir", >>>> + "apb0_timer01", "apb0_p2wi", >>> timer01 ? is this a typo? >> A23 manual lists the clock gate as "r_timer0_1", so I put the name on the wiki. >> Allwinner sun6i code uses "r_tmr" or just "tmr". I see no problem naming this >> clock output as "apb0_timer" though. > Yep, it seems better. Fair enough, I'll change the name for the next version. > > Maxime > -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html