Hi Hans, 2014/1/10 Hans de Goede <hdego...@redhat.com>: > From: arokux <aro...@gmail.com>
I thought this was settled already :) You should use a real name here. > Add register definitions for the usb-clk register found on sun4i, sun5i and > sun7i SoCs. > > Signed-off-by: Hans de Goede <hdego...@redhat.com> > --- > Documentation/devicetree/bindings/clock/sunxi.txt | 5 +++++ > drivers/clk/sunxi/clk-sunxi.c | 12 ++++++++++++ > 2 files changed, 17 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/sunxi.txt > b/Documentation/devicetree/bindings/clock/sunxi.txt > index 79c7197..8bccb6a 100644 > --- a/Documentation/devicetree/bindings/clock/sunxi.txt > +++ b/Documentation/devicetree/bindings/clock/sunxi.txt > @@ -37,6 +37,8 @@ Required properties: > "allwinner,sun6i-a31-apb2-gates-clk" - for the APB2 gates on A31 > "allwinner,sun4i-mod0-clk" - for the module 0 family of clocks > "allwinner,sun7i-a20-out-clk" - for the external output clocks > + "allwinner,sun4i-usb-gates-clk" - for usb gates + resets on A10 / A20 > + "allwinner,sun5i-a13-usb-gates-clk" - for usb gates + resets on A13 > > Required properties for all clocks: > - reg : shall be the control register address for the clock. > @@ -49,6 +51,9 @@ Required properties for all clocks: > Additionally, "allwinner,*-gates-clk" clocks require: > - clock-output-names : the corresponding gate names that the clock controls > > +And "allwinner,*-usb-gates-clk" clocks also require: > +- reset-cells : shall be set to 1 > + > Clock consumers should specify the desired clocks they use with a > "clocks" phandle cell. Consumers that are using a gated clock should > provide an additional ID in their clock property. This ID is the > diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c > index b5d0a7a..82d75c0 100644 > --- a/drivers/clk/sunxi/clk-sunxi.c > +++ b/drivers/clk/sunxi/clk-sunxi.c > @@ -813,6 +813,16 @@ static const struct gates_data sun4i_ahb_gates_data > __initconst = { > .mask = {0x7F77FFF, 0x14FB3F}, > }; > > +static const struct gates_data sun4i_usb_gates_data __initconst = { > + .mask = {0x1C0}, > + .reset_mask = 0x07, > +}; > + > +static const struct gates_data sun5i_a13_usb_gates_data __initconst = { > + .mask = {0x140}, > + .reset_mask = 0x03, > +}; > + > static const struct gates_data sun5i_a10s_ahb_gates_data __initconst = { > .mask = {0x147667e7, 0x185915}, > }; > @@ -1159,6 +1169,8 @@ static const struct of_device_id clk_gates_match[] > __initconst = { > {.compatible = "allwinner,sun6i-a31-apb1-gates-clk", .data = > &sun6i_a31_apb1_gates_data,}, > {.compatible = "allwinner,sun7i-a20-apb1-gates-clk", .data = > &sun7i_a20_apb1_gates_data,}, > {.compatible = "allwinner,sun6i-a31-apb2-gates-clk", .data = > &sun6i_a31_apb2_gates_data,}, > + {.compatible = "allwinner,sun4i-usb-gates-clk", .data = > &sun4i_usb_gates_data,}, > + {.compatible = "allwinner,sun5i-a13-usb-gates-clk", .data = > &sun5i_a13_usb_gates_data,}, > {} > }; It looks ok otherwise. Cheers, Emilio -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.