[PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

[PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

[Stephen Boyd]

This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate
CLK_IS_ROOT", 2016-02-02) so remove it.

Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Jason Cooper <jason at lakedaemon.net
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
 arch/arm/mach-mv78xx0/common.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/mach-mv78xx0/common.c b/arch/arm/mach-mv78xx0/common.c
index 99cc93900a24..45a05207b418 100644
--- a/arch/arm/mach-mv78xx0/common.c
+++ b/arch/arm/mach-mv78xx0/common.c
@@ -168,8 +168,7 @@ static struct clk *tclk;
 
 static void __init clk_init(void)
 {
-	tclk = clk_register_fixed_rate(NULL, "tclk", NULL, CLK_IS_ROOT,
-				       get_tclk());
+	tclk = clk_register_fixed_rate(NULL, "tclk", NULL, 0, get_tclk());
 
 	orion_clkdev_init(tclk);
 }
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

[PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

[Arnd Bergmann]

On Tuesday 19 April 2016 18:40:04 Stephen Boyd wrote:
> This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate
> CLK_IS_ROOT", 2016-02-02) so remove it.
> 
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Jason Cooper <jason@lakedaemon.net
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> 

Looks good, like all other instances in arch/arm.

If you want to merge these through your tree, please add my

Acked-by: Arnd Bergmann <arnd@arndb.de>

to all of them, in addition to the other other Acks. I don't
think there is a significant risk for regressions as they all
tend to be "stable" in the sense that those files haven't
been touched in a while.

	Arnd

[PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

[Stephen Boyd]

On 04/20, Arnd Bergmann wrote:
> On Tuesday 19 April 2016 18:40:04 Stephen Boyd wrote:
> > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate
> > CLK_IS_ROOT", 2016-02-02) so remove it.
> > 
> > Cc: Andrew Lunn <andrew@lunn.ch>
> > Cc: Jason Cooper <jason@lakedaemon.net
> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> > 
> 
> Looks good, like all other instances in arch/arm.
> 
> If you want to merge these through your tree, please add my
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> to all of them, in addition to the other other Acks. I don't
> think there is a significant risk for regressions as they all
> tend to be "stable" in the sense that those files haven't
> been touched in a while.
> 

Thanks. If Gregory doesn't pick them up I'll take care of it, but
we're not in any hurry here.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

[Gregory CLEMENT]

Hi Stephen,
 
 On mer., avril 20 2016, Stephen Boyd <sboyd@codeaurora.org> wrote:

> This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate
> CLK_IS_ROOT", 2016-02-02) so remove it.
>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Jason Cooper <jason@lakedaemon.net
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>

Applied on mvebu/soc

(I fixed the email format for Jason: the closing angle bracket was
missing)

Thanks,

Gregory
> ---
>  arch/arm/mach-mv78xx0/common.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-mv78xx0/common.c b/arch/arm/mach-mv78xx0/common.c
> index 99cc93900a24..45a05207b418 100644
> --- a/arch/arm/mach-mv78xx0/common.c
> +++ b/arch/arm/mach-mv78xx0/common.c
> @@ -168,8 +168,7 @@ static struct clk *tclk;
>  
>  static void __init clk_init(void)
>  {
> -	tclk = clk_register_fixed_rate(NULL, "tclk", NULL, CLK_IS_ROOT,
> -				       get_tclk());
> +	tclk = clk_register_fixed_rate(NULL, "tclk", NULL, 0, get_tclk());
>  
>  	orion_clkdev_init(tclk);
>  }
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com