[PATCH 1/1] OMAP3: CLOCK: Remove few unnecessary clocks

[PATCH 1/1] OMAP3: CLOCK: Remove few unnecessary clocks

[Jouni Hogander]

dpllx_m2x2_ck parent is dpllx_m2_ck. So remove few useless clocks and
and use right parent for dpllx_m2x2_ck.

Signed-off-by: Jouni Hogander <jouni.hogander@nokia.com>
---
 arch/arm/mach-omap2/clock34xx.h |   31 ++-----------------------------
 1 files changed, 2 insertions(+), 29 deletions(-)

diff --git a/arch/arm/mach-omap2/clock34xx.h b/arch/arm/mach-omap2/clock34xx.h
index 179ea17..4f462ea 100644
--- a/arch/arm/mach-omap2/clock34xx.h
+++ b/arch/arm/mach-omap2/clock34xx.h
@@ -427,18 +427,6 @@ static struct clk dpll3_ck = {
 	.recalc		= &omap3_dpll_recalc,
 };
 
-/*
- * This virtual clock provides the CLKOUTX2 output from the DPLL if the
- * DPLL isn't bypassed
- */
-static struct clk dpll3_x2_ck = {
-	.name		= "dpll3_x2_ck",
-	.parent		= &dpll3_ck,
-	.flags		= CLOCK_IN_OMAP343X | PARENT_CONTROLS_CLOCK,
-	.clkdm		= { .name = "dpll3_clkdm" },
-	.recalc		= &omap3_clkoutx2_recalc,
-};
-
 static const struct clksel_rate div31_dpll3_rates[] = {
 	{ .div = 1, .val = 1, .flags = RATE_IN_343X | DEFAULT_RATE },
 	{ .div = 2, .val = 2, .flags = RATE_IN_343X },
@@ -505,10 +493,10 @@ static struct clk core_ck = {
 
 static struct clk dpll3_m2x2_ck = {
 	.name		= "dpll3_m2x2_ck",
-	.parent		= &dpll3_x2_ck,
+	.parent		= &dpll3_m2_ck,
 	.flags		= CLOCK_IN_OMAP343X | PARENT_CONTROLS_CLOCK,
 	.clkdm		= { .name = "dpll3_clkdm" },
-	.recalc		= &followparent_recalc,
+	.recalc		= &omap3_clkoutx2_recalc,
 };
 
 /* The PWRDN bit is apparently only available on 3430ES2 and above */
@@ -590,19 +578,6 @@ static struct clk dpll4_ck = {
 	.recalc		= &omap3_dpll_recalc,
 };
 
-/*
- * This virtual clock provides the CLKOUTX2 output from the DPLL if the
- * DPLL isn't bypassed --
- * XXX does this serve any downstream clocks?
- */
-static struct clk dpll4_x2_ck = {
-	.name		= "dpll4_x2_ck",
-	.parent		= &dpll4_ck,
-	.flags		= CLOCK_IN_OMAP343X | PARENT_CONTROLS_CLOCK,
-	.clkdm		= { .name = "dpll4_clkdm" },
-	.recalc		= &omap3_clkoutx2_recalc,
-};
-
 static const struct clksel div16_dpll4_clksel[] = {
 	{ .parent = &dpll4_ck, .rates = div16_dpll_rates },
 	{ .parent = NULL }
@@ -3355,14 +3330,12 @@ static struct clk *onchip_34xx_clks[] __initdata = {
 	&dpll2_m2_ck,
 	&dpll3_ck,
 	&core_ck,
-	&dpll3_x2_ck,
 	&dpll3_m2_ck,
 	&dpll3_m2x2_ck,
 	&dpll3_m3_ck,
 	&dpll3_m3x2_ck,
 	&emu_core_alwon_ck,
 	&dpll4_ck,
-	&dpll4_x2_ck,
 	&omap_96m_alwon_fck,
 	&omap_96m_fck,
 	&cm_96m_fck,
-- 
1.6.0.1

Re: [PATCH 1/1] OMAP3: CLOCK: Remove few unnecessary clocks

[Paul Walmsley]

On Fri, 20 Feb 2009, Jouni Hogander wrote:

> dpllx_m2x2_ck parent is dpllx_m2_ck. So remove few useless clocks and
> and use right parent for dpllx_m2x2_ck.
> 
> Signed-off-by: Jouni Hogander <jouni.hogander@nokia.com>

Acked-by: Paul Walmsley <paul@pwsan.com>

Thanks Jouni.



- Paul

Re: [PATCH 1/1] OMAP3: CLOCK: Remove few unnecessary clocks

[Kevin Hilman]

Paul Walmsley <paul@pwsan.com> writes:

> On Fri, 20 Feb 2009, Jouni Hogander wrote:
>
>> dpllx_m2x2_ck parent is dpllx_m2_ck. So remove few useless clocks and
>> and use right parent for dpllx_m2x2_ck.
>> 
>> Signed-off-by: Jouni Hogander <jouni.hogander@nokia.com>
>
> Acked-by: Paul Walmsley <paul@pwsan.com>
>

Pulling into PM branch while waiting for upstream clock sync up.

Kevin

[APPLIED] [PATCH 1/1] OMAP3: CLOCK: Remove few unnecessary clocks

[Tony Lindgren]

This patch has been applied to the linux-omap
by youw fwiendly patch wobot.

Commit: 4cf4cac3647323b24a2cc3cdd6c5accfd98f4921

PatchWorks
http://patchwork.kernel.org/patch/8102/

Git
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=4cf4cac3647323b24a2cc3cdd6c5accfd98f4921