On Tue, Oct 29, 2013 at 03:55:10PM +0100, Laurent Pinchart wrote:
> MSTP clocks are gate clocks controlled through a register that handles
> up to 32 clocks. The register is often sparsely populated.
> 
> Those clocks are found on Renesas ARM SoCs.
> 
> Signed-off-by: Laurent Pinchart <[email protected]>
> ---
>  .../bindings/clock/renesas,cpg-mstp-clocks.txt     |  47 +++++
>  drivers/clk/shmobile/Makefile                      |   1 +
>  drivers/clk/shmobile/clk-mstp.c                    | 229 
> +++++++++++++++++++++
>  include/dt-bindings/clock/r8a7790-clock.h          |  56 +++++
>  4 files changed, 333 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/clock/renesas,cpg-mstp-clocks.txt
>  create mode 100644 drivers/clk/shmobile/clk-mstp.c
>  create mode 100644 include/dt-bindings/clock/r8a7790-clock.h

[snip]

> diff --git a/include/dt-bindings/clock/r8a7790-clock.h 
> b/include/dt-bindings/clock/r8a7790-clock.h
> new file mode 100644
> index 0000000..19f2b48
> --- /dev/null
> +++ b/include/dt-bindings/clock/r8a7790-clock.h

I wonder if it would be best to put this in a separate patch
as it is SoC-specific.

> @@ -0,0 +1,56 @@
> +/*
> + * Copyright 2013 Ideas On Board SPRL
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + */
> +
> +#ifndef __DT_BINDINGS_CLOCK_R8A7790_H__
> +#define __DT_BINDINGS_CLOCK_R8A7790_H__
> +
> +/* MSTP1 */
> +#define R8A7790_CLK_CMT0             20

I believe that R8A7790_CLK_CMT0 should be 24.

> +
> +/* MSTP2 */
> +#define R8A7790_CLK_SCIFA2           2
> +#define R8A7790_CLK_SCIFA1           3
> +#define R8A7790_CLK_SCIFA0           4
> +#define R8A7790_CLK_SCIFB0           6
> +#define R8A7790_CLK_SCIFB1           7
> +#define R8A7790_CLK_SCIFB2           16
> +
> +/* MSTP3 */
> +#define R8A7790_CLK_TPU0             4
> +#define R8A7790_CLK_MMCIF1           5
> +#define R8A7790_CLK_SDHI3            11
> +#define R8A7790_CLK_SDHI2            12
> +#define R8A7790_CLK_SDHI1            13
> +#define R8A7790_CLK_SDHI0            14
> +#define R8A7790_CLK_MMCIF0           15
> +
> +/* MSTP5 */
> +#define R8A7790_CLK_THERMAL          22
> +
> +/* MSTP7 */
> +#define R8A7790_CLK_HSCIF1           16
> +#define R8A7790_CLK_HSCIF0           17
> +#define R8A7790_CLK_SCIF1            20
> +#define R8A7790_CLK_SCIF0            21
> +#define R8A7790_CLK_DU2                      22
> +#define R8A7790_CLK_DU1                      23
> +#define R8A7790_CLK_DU0                      24
> +#define R8A7790_CLK_LVDS1            25
> +#define R8A7790_CLK_LVDS0            26
> +
> +/* MSTP8 */
> +#define R8A7790_CLK_ETHER            13
> +
> +/* MSTP9 */
> +#define R8A7790_CLK_I2C3             28
> +#define R8A7790_CLK_I2C2             29
> +#define R8A7790_CLK_I2C1             30
> +#define R8A7790_CLK_I2C0             31
> +
> +#endif /* __DT_BINDINGS_CLOCK_R8A7790_H__ */
> -- 
> 1.8.1.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to