[PATCH] clk: ux500: Remove BML8580 clock

[PATCH] clk: ux500: Remove BML8580 clock

[Linus Walleij]

From: Lee Jones <lee.jones@linaro.org>

There is no mention of the PRCMU_BML8580CLK in any of the Design
Specifications for the chips supported in Mainline. In fact, where it
is incorrectly used in the u8540 clock definition driver it would
have the side effect of using the incorrect clock management address
([PRCM_BML8580CLK_MGT] 0x108 instead of the correct value 0x04C).

Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/clk/ux500/u8540_clk.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/ux500/u8540_clk.c b/drivers/clk/ux500/u8540_clk.c
index f262588..20c8add 100644
--- a/drivers/clk/ux500/u8540_clk.c
+++ b/drivers/clk/ux500/u8540_clk.c
@@ -83,7 +83,7 @@ void u8540_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
 	clk_register_clkdev(clk, NULL, "lcd");
 	clk_register_clkdev(clk, "lcd", "mcde");
 
-	clk = clk_reg_prcmu_opp_gate("bmlclk", NULL, PRCMU_BML8580CLK,
+	clk = clk_reg_prcmu_opp_gate("bmlclk", NULL, PRCMU_BMLCLK,
 				CLK_IS_ROOT);
 	clk_register_clkdev(clk, NULL, "bml");
 
-- 
1.8.1.4

[PATCH] clk: ux500: Remove BML8580 clock

[Mike Turquette]

Quoting Linus Walleij (2013-08-11 14:49:16)
> From: Lee Jones <lee.jones@linaro.org>
> 
> There is no mention of the PRCMU_BML8580CLK in any of the Design
> Specifications for the chips supported in Mainline. In fact, where it
> is incorrectly used in the u8540 clock definition driver it would
> have the side effect of using the incorrect clock management address
> ([PRCM_BML8580CLK_MGT] 0x108 instead of the correct value 0x04C).
> 
> Signed-off-by: Lee Jones <lee.jones@linaro.org>

Pulled into clk-next.

Thanks,
Mike

> ---
>  drivers/clk/ux500/u8540_clk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clk/ux500/u8540_clk.c b/drivers/clk/ux500/u8540_clk.c
> index f262588..20c8add 100644
> --- a/drivers/clk/ux500/u8540_clk.c
> +++ b/drivers/clk/ux500/u8540_clk.c
> @@ -83,7 +83,7 @@ void u8540_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
>         clk_register_clkdev(clk, NULL, "lcd");
>         clk_register_clkdev(clk, "lcd", "mcde");
>  
> -       clk = clk_reg_prcmu_opp_gate("bmlclk", NULL, PRCMU_BML8580CLK,
> +       clk = clk_reg_prcmu_opp_gate("bmlclk", NULL, PRCMU_BMLCLK,
>                                 CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "bml");
>  
> -- 
> 1.8.1.4

[PATCH] clk: ux500: Remove BML8580 clock

[Linus Walleij]

On Thu, Aug 15, 2013 at 8:42 PM, Mike Turquette <mturquette@linaro.org> wrote:
> Quoting Linus Walleij (2013-08-11 14:49:16)
>> From: Lee Jones <lee.jones@linaro.org>
>>
>> There is no mention of the PRCMU_BML8580CLK in any of the Design
>> Specifications for the chips supported in Mainline. In fact, where it
>> is incorrectly used in the u8540 clock definition driver it would
>> have the side effect of using the incorrect clock management address
>> ([PRCM_BML8580CLK_MGT] 0x108 instead of the correct value 0x04C).
>>
>> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>
> Pulled into clk-next.


Hm it turns out that I need this to be in the middle of Lee's patch
set for device tree clock that need to go through ARM SoC.

Can you (please) drop this patch and provide an ACK instead?

Yours,
Linus Walleij

[PATCH] clk: ux500: Remove BML8580 clock

[Mike Turquette]

Quoting Linus Walleij (2013-08-16 02:15:13)
> On Thu, Aug 15, 2013 at 8:42 PM, Mike Turquette <mturquette@linaro.org> wrote:
> > Quoting Linus Walleij (2013-08-11 14:49:16)
> >> From: Lee Jones <lee.jones@linaro.org>
> >>
> >> There is no mention of the PRCMU_BML8580CLK in any of the Design
> >> Specifications for the chips supported in Mainline. In fact, where it
> >> is incorrectly used in the u8540 clock definition driver it would
> >> have the side effect of using the incorrect clock management address
> >> ([PRCM_BML8580CLK_MGT] 0x108 instead of the correct value 0x04C).
> >>
> >> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> >
> > Pulled into clk-next.
> 
> 
> Hm it turns out that I need this to be in the middle of Lee's patch
> set for device tree clock that need to go through ARM SoC.
> 
> Can you (please) drop this patch and provide an ACK instead?

I've dropped the patch from clk-next. Please add my Ack.

Regards,
Mike

> 
> Yours,
> Linus Walleij