Hi Rahul, Please see some minor comments below. On 04.03.2014 12:12, Rahul Sharma wrote:
Add macros which are used as Clock IDs in DT and clock file. It also adds the documentation for the exynos5260 clocks. Signed-off-by: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx> --- .../devicetree/bindings/clock/exynos5260-clock.txt | 55 +++++ include/dt-bindings/clk/exynos5260-clk.h | 233 ++++++++++++++++++++ 2 files changed, 288 insertions(+) create mode 100644 Documentation/devicetree/bindings/clock/exynos5260-clock.txt create mode 100644 include/dt-bindings/clk/exynos5260-clk.h
[snip]
diff --git a/include/dt-bindings/clk/exynos5260-clk.h b/include/dt-bindings/clk/exynos5260-clk.h new file mode 100644 index 0000000..4dc20a8 --- /dev/null +++ b/include/dt-bindings/clk/exynos5260-clk.h
Please use dt-bindings/clock directory.
@@ -0,0 +1,233 @@ +/* + * Copyright (c) 2013 Samsung Electronics Co., Ltd. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * Provides Constants for Exynos5260 clocks. +*/
nit: Misaligned star.
+ +#ifndef _DT_BINDINGS_CLK_EXYNOS5260_H +#define _DT_BINDINGS_CLK_EXYNOS5260_H + +/* + * Clock names: XXXXXX_YYYYY_ZZZZZ + * |------| |----| |----| + * cmu type IP
The alignment looks wrong. Also tabs should be used.
+*/
nit: Misaligned star.
+ +/* list of clocks for CMU_TOP */
nit: Please properly capitalize comments and end them with a dot, e.g. /* List of clocks for CMU_TOP. */
+#define FIN_PLL 1
nit: No tabs between #define and DEF_NAME, please. Best regards, Tomasz -- 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