Hi Arun, On 23.05.2014 07:08, Arun Kumar K wrote: > Adds IDs for MUX clocks to be used by power domain for MFC > for doing re-parenting while pd on/off. > > Signed-off-by: Arun Kumar K <arun.kk@xxxxxxxxxxx> > --- > drivers/clk/samsung/clk-exynos5420.c | 6 ++++-- > include/dt-bindings/clock/exynos5420.h | 2 ++ > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c > index 9d7d7ee..c899dbe 100644 > --- a/drivers/clk/samsung/clk-exynos5420.c > +++ b/drivers/clk/samsung/clk-exynos5420.c > @@ -631,7 +631,8 @@ static struct samsung_mux_clock exynos5x_mux_clks[] __initdata = { > SRC_TOP4, 16, 1), > MUX(0, "mout_user_aclk266", mout_user_aclk266_p, SRC_TOP4, 20, 1), > MUX(0, "mout_user_aclk166", mout_user_aclk166_p, SRC_TOP4, 24, 1), > - MUX(0, "mout_user_aclk333", mout_user_aclk333_p, SRC_TOP4, 28, 1), > + MUX(CLK_MOUT_USER_ACLK333, "mout_user_aclk333", mout_user_aclk333_p, > + SRC_TOP4, 28, 1), Please keep the indentation consistent. As you can see below, just two extra tabs are used on wrapped lines. > > MUX(0, "mout_user_aclk400_disp1", mout_user_aclk400_disp1_p, > SRC_TOP5, 0, 1), > @@ -684,7 +685,8 @@ static struct samsung_mux_clock exynos5x_mux_clks[] __initdata = { > SRC_TOP11, 12, 1), > MUX(0, "mout_sw_aclk266", mout_sw_aclk266_p, SRC_TOP11, 20, 1), > MUX(0, "mout_sw_aclk166", mout_sw_aclk166_p, SRC_TOP11, 24, 1), > - MUX(0, "mout_sw_aclk333", mout_sw_aclk333_p, SRC_TOP11, 28, 1), > + MUX(CLK_MOUT_SW_ACLK333, "mout_sw_aclk333", mout_sw_aclk333_p, > + SRC_TOP11, 28, 1), Ditto. 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