* [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
@ 2017-01-21 17:36 ` Andrew Murray
0 siblings, 0 replies; 8+ messages in thread
From: Andrew Murray @ 2017-01-21 17:36 UTC (permalink / raw)
To: linux-omap
Cc: Tony Lindgren, Tero Kristo, Paul Walmsley, Andrew Murray,
linux-arm-kernel
The 'mpurate' option, historically used for specifying an initial
MPU rate at boot, no longer has any effect due to the supporting
code being removed as it was 'wrong and dangerous' [1].
This patch removes the remaining dead code associated with the
__setup macros to avoid confusion and reduce bloat.
[1] https://patchwork.kernel.org/patch/5954631/
Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
---
arch/arm/mach-omap1/clock.c | 20 --------------------
arch/arm/mach-omap1/clock.h | 2 --
arch/arm/mach-omap2/clock.c | 20 --------------------
3 files changed, 42 deletions(-)
diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
index 4f5fd4a..43e3e18 100644
--- a/arch/arm/mach-omap1/clock.c
+++ b/arch/arm/mach-omap1/clock.c
@@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
* OMAP specific clock functions shared between omap1 and omap2
*/
-int __initdata mpurate;
-
-/*
- * By default we use the rate set by the bootloader.
- * You can override this with mpurate= cmdline option.
- */
-static int __init omap_clk_setup(char *str)
-{
- get_option(&str, &mpurate);
-
- if (!mpurate)
- return 1;
-
- if (mpurate < 1000)
- mpurate *= 1000000;
-
- return 1;
-}
-__setup("mpurate=", omap_clk_setup);
-
/* Used for clocks that always have same value as the parent clock */
unsigned long followparent_recalc(struct clk *clk)
{
diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
index 1e4918a..f2d1be2 100644
--- a/arch/arm/mach-omap1/clock.h
+++ b/arch/arm/mach-omap1/clock.h
@@ -173,8 +173,6 @@ struct clk_functions {
void (*clk_disable_unused)(struct clk *clk);
};
-extern int mpurate;
-
extern int clk_init(struct clk_functions *custom_clocks);
extern void clk_preinit(struct clk *clk);
extern int clk_register(struct clk *clk);
diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
index d058125..d999295 100644
--- a/arch/arm/mach-omap2/clock.c
+++ b/arch/arm/mach-omap2/clock.c
@@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
}
}
-static int __initdata mpurate;
-
-/*
- * By default we use the rate set by the bootloader.
- * You can override this with mpurate= cmdline option.
- */
-static int __init omap_clk_setup(char *str)
-{
- get_option(&str, &mpurate);
-
- if (!mpurate)
- return 1;
-
- if (mpurate < 1000)
- mpurate *= 1000000;
-
- return 1;
-}
-__setup("mpurate=", omap_clk_setup);
-
/**
* omap2_clk_print_new_rates - print summary of current clock tree rates
* @hfclkin_ck_name: clk name for the off-chip HF oscillator
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
@ 2017-01-21 17:36 ` Andrew Murray
0 siblings, 0 replies; 8+ messages in thread
From: Andrew Murray @ 2017-01-21 17:36 UTC (permalink / raw)
To: linux-arm-kernel
The 'mpurate' option, historically used for specifying an initial
MPU rate at boot, no longer has any effect due to the supporting
code being removed as it was 'wrong and dangerous' [1].
This patch removes the remaining dead code associated with the
__setup macros to avoid confusion and reduce bloat.
[1] https://patchwork.kernel.org/patch/5954631/
Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
---
arch/arm/mach-omap1/clock.c | 20 --------------------
arch/arm/mach-omap1/clock.h | 2 --
arch/arm/mach-omap2/clock.c | 20 --------------------
3 files changed, 42 deletions(-)
diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
index 4f5fd4a..43e3e18 100644
--- a/arch/arm/mach-omap1/clock.c
+++ b/arch/arm/mach-omap1/clock.c
@@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
* OMAP specific clock functions shared between omap1 and omap2
*/
-int __initdata mpurate;
-
-/*
- * By default we use the rate set by the bootloader.
- * You can override this with mpurate= cmdline option.
- */
-static int __init omap_clk_setup(char *str)
-{
- get_option(&str, &mpurate);
-
- if (!mpurate)
- return 1;
-
- if (mpurate < 1000)
- mpurate *= 1000000;
-
- return 1;
-}
-__setup("mpurate=", omap_clk_setup);
-
/* Used for clocks that always have same value as the parent clock */
unsigned long followparent_recalc(struct clk *clk)
{
diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
index 1e4918a..f2d1be2 100644
--- a/arch/arm/mach-omap1/clock.h
+++ b/arch/arm/mach-omap1/clock.h
@@ -173,8 +173,6 @@ struct clk_functions {
void (*clk_disable_unused)(struct clk *clk);
};
-extern int mpurate;
-
extern int clk_init(struct clk_functions *custom_clocks);
extern void clk_preinit(struct clk *clk);
extern int clk_register(struct clk *clk);
diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
index d058125..d999295 100644
--- a/arch/arm/mach-omap2/clock.c
+++ b/arch/arm/mach-omap2/clock.c
@@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
}
}
-static int __initdata mpurate;
-
-/*
- * By default we use the rate set by the bootloader.
- * You can override this with mpurate= cmdline option.
- */
-static int __init omap_clk_setup(char *str)
-{
- get_option(&str, &mpurate);
-
- if (!mpurate)
- return 1;
-
- if (mpurate < 1000)
- mpurate *= 1000000;
-
- return 1;
-}
-__setup("mpurate=", omap_clk_setup);
-
/**
* omap2_clk_print_new_rates - print summary of current clock tree rates
* @hfclkin_ck_name: clk name for the off-chip HF oscillator
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
2017-01-21 17:36 ` Andrew Murray
@ 2017-01-23 15:10 ` Tony Lindgren
-1 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2017-01-23 15:10 UTC (permalink / raw)
To: Andrew Murray
Cc: Tero Kristo, Paul Walmsley, linux-omap, linux-arm-kernel, Aaro Koskinen
* Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
> The 'mpurate' option, historically used for specifying an initial
> MPU rate at boot, no longer has any effect due to the supporting
> code being removed as it was 'wrong and dangerous' [1].
>
> This patch removes the remaining dead code associated with the
> __setup macros to avoid confusion and reduce bloat.
>
> [1] https://patchwork.kernel.org/patch/5954631/
Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
Cc for mach-omap1 in case there might be issues with this
patch.
Regards,
Tony
> Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
> ---
> arch/arm/mach-omap1/clock.c | 20 --------------------
> arch/arm/mach-omap1/clock.h | 2 --
> arch/arm/mach-omap2/clock.c | 20 --------------------
> 3 files changed, 42 deletions(-)
>
> diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
> index 4f5fd4a..43e3e18 100644
> --- a/arch/arm/mach-omap1/clock.c
> +++ b/arch/arm/mach-omap1/clock.c
> @@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
> * OMAP specific clock functions shared between omap1 and omap2
> */
>
> -int __initdata mpurate;
> -
> -/*
> - * By default we use the rate set by the bootloader.
> - * You can override this with mpurate= cmdline option.
> - */
> -static int __init omap_clk_setup(char *str)
> -{
> - get_option(&str, &mpurate);
> -
> - if (!mpurate)
> - return 1;
> -
> - if (mpurate < 1000)
> - mpurate *= 1000000;
> -
> - return 1;
> -}
> -__setup("mpurate=", omap_clk_setup);
> -
> /* Used for clocks that always have same value as the parent clock */
> unsigned long followparent_recalc(struct clk *clk)
> {
> diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
> index 1e4918a..f2d1be2 100644
> --- a/arch/arm/mach-omap1/clock.h
> +++ b/arch/arm/mach-omap1/clock.h
> @@ -173,8 +173,6 @@ struct clk_functions {
> void (*clk_disable_unused)(struct clk *clk);
> };
>
> -extern int mpurate;
> -
> extern int clk_init(struct clk_functions *custom_clocks);
> extern void clk_preinit(struct clk *clk);
> extern int clk_register(struct clk *clk);
> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
> index d058125..d999295 100644
> --- a/arch/arm/mach-omap2/clock.c
> +++ b/arch/arm/mach-omap2/clock.c
> @@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
> }
> }
>
> -static int __initdata mpurate;
> -
> -/*
> - * By default we use the rate set by the bootloader.
> - * You can override this with mpurate= cmdline option.
> - */
> -static int __init omap_clk_setup(char *str)
> -{
> - get_option(&str, &mpurate);
> -
> - if (!mpurate)
> - return 1;
> -
> - if (mpurate < 1000)
> - mpurate *= 1000000;
> -
> - return 1;
> -}
> -__setup("mpurate=", omap_clk_setup);
> -
> /**
> * omap2_clk_print_new_rates - print summary of current clock tree rates
> * @hfclkin_ck_name: clk name for the off-chip HF oscillator
> --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
@ 2017-01-23 15:10 ` Tony Lindgren
0 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2017-01-23 15:10 UTC (permalink / raw)
To: linux-arm-kernel
* Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
> The 'mpurate' option, historically used for specifying an initial
> MPU rate at boot, no longer has any effect due to the supporting
> code being removed as it was 'wrong and dangerous' [1].
>
> This patch removes the remaining dead code associated with the
> __setup macros to avoid confusion and reduce bloat.
>
> [1] https://patchwork.kernel.org/patch/5954631/
Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
Cc for mach-omap1 in case there might be issues with this
patch.
Regards,
Tony
> Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
> ---
> arch/arm/mach-omap1/clock.c | 20 --------------------
> arch/arm/mach-omap1/clock.h | 2 --
> arch/arm/mach-omap2/clock.c | 20 --------------------
> 3 files changed, 42 deletions(-)
>
> diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
> index 4f5fd4a..43e3e18 100644
> --- a/arch/arm/mach-omap1/clock.c
> +++ b/arch/arm/mach-omap1/clock.c
> @@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
> * OMAP specific clock functions shared between omap1 and omap2
> */
>
> -int __initdata mpurate;
> -
> -/*
> - * By default we use the rate set by the bootloader.
> - * You can override this with mpurate= cmdline option.
> - */
> -static int __init omap_clk_setup(char *str)
> -{
> - get_option(&str, &mpurate);
> -
> - if (!mpurate)
> - return 1;
> -
> - if (mpurate < 1000)
> - mpurate *= 1000000;
> -
> - return 1;
> -}
> -__setup("mpurate=", omap_clk_setup);
> -
> /* Used for clocks that always have same value as the parent clock */
> unsigned long followparent_recalc(struct clk *clk)
> {
> diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
> index 1e4918a..f2d1be2 100644
> --- a/arch/arm/mach-omap1/clock.h
> +++ b/arch/arm/mach-omap1/clock.h
> @@ -173,8 +173,6 @@ struct clk_functions {
> void (*clk_disable_unused)(struct clk *clk);
> };
>
> -extern int mpurate;
> -
> extern int clk_init(struct clk_functions *custom_clocks);
> extern void clk_preinit(struct clk *clk);
> extern int clk_register(struct clk *clk);
> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
> index d058125..d999295 100644
> --- a/arch/arm/mach-omap2/clock.c
> +++ b/arch/arm/mach-omap2/clock.c
> @@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
> }
> }
>
> -static int __initdata mpurate;
> -
> -/*
> - * By default we use the rate set by the bootloader.
> - * You can override this with mpurate= cmdline option.
> - */
> -static int __init omap_clk_setup(char *str)
> -{
> - get_option(&str, &mpurate);
> -
> - if (!mpurate)
> - return 1;
> -
> - if (mpurate < 1000)
> - mpurate *= 1000000;
> -
> - return 1;
> -}
> -__setup("mpurate=", omap_clk_setup);
> -
> /**
> * omap2_clk_print_new_rates - print summary of current clock tree rates
> * @hfclkin_ck_name: clk name for the off-chip HF oscillator
> --
> 2.5.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
2017-01-23 15:10 ` Tony Lindgren
@ 2017-01-23 19:51 ` Andrew Murray
-1 siblings, 0 replies; 8+ messages in thread
From: Andrew Murray @ 2017-01-23 19:51 UTC (permalink / raw)
To: Tony Lindgren
Cc: Tero Kristo, Paul Walmsley, linux-omap, linux-arm-kernel, Aaro Koskinen
On 23 January 2017 at 15:10, Tony Lindgren <tony@atomide.com> wrote:
> * Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
>> The 'mpurate' option, historically used for specifying an initial
>> MPU rate at boot, no longer has any effect due to the supporting
>> code being removed as it was 'wrong and dangerous' [1].
>>
>> This patch removes the remaining dead code associated with the
>> __setup macros to avoid confusion and reduce bloat.
>>
>> [1] https://patchwork.kernel.org/patch/5954631/
>
> Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
> Cc for mach-omap1 in case there might be issues with this
> patch.
I've also noticed that omap2_clk_print_new_rates in
arch/arm/mach-omap2/clock.c is also unused - this was previously used
to print the new clock rates upon change after use of mpurate.
I'll update the patch to include the removal of this function as well
- unless you have any objections.
Thanks,
Andrew Murray
>
>
>> Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
>> ---
>> arch/arm/mach-omap1/clock.c | 20 --------------------
>> arch/arm/mach-omap1/clock.h | 2 --
>> arch/arm/mach-omap2/clock.c | 20 --------------------
>> 3 files changed, 42 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
>> index 4f5fd4a..43e3e18 100644
>> --- a/arch/arm/mach-omap1/clock.c
>> +++ b/arch/arm/mach-omap1/clock.c
>> @@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
>> * OMAP specific clock functions shared between omap1 and omap2
>> */
>>
>> -int __initdata mpurate;
>> -
>> -/*
>> - * By default we use the rate set by the bootloader.
>> - * You can override this with mpurate= cmdline option.
>> - */
>> -static int __init omap_clk_setup(char *str)
>> -{
>> - get_option(&str, &mpurate);
>> -
>> - if (!mpurate)
>> - return 1;
>> -
>> - if (mpurate < 1000)
>> - mpurate *= 1000000;
>> -
>> - return 1;
>> -}
>> -__setup("mpurate=", omap_clk_setup);
>> -
>> /* Used for clocks that always have same value as the parent clock */
>> unsigned long followparent_recalc(struct clk *clk)
>> {
>> diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
>> index 1e4918a..f2d1be2 100644
>> --- a/arch/arm/mach-omap1/clock.h
>> +++ b/arch/arm/mach-omap1/clock.h
>> @@ -173,8 +173,6 @@ struct clk_functions {
>> void (*clk_disable_unused)(struct clk *clk);
>> };
>>
>> -extern int mpurate;
>> -
>> extern int clk_init(struct clk_functions *custom_clocks);
>> extern void clk_preinit(struct clk *clk);
>> extern int clk_register(struct clk *clk);
>> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
>> index d058125..d999295 100644
>> --- a/arch/arm/mach-omap2/clock.c
>> +++ b/arch/arm/mach-omap2/clock.c
>> @@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
>> }
>> }
>>
>> -static int __initdata mpurate;
>> -
>> -/*
>> - * By default we use the rate set by the bootloader.
>> - * You can override this with mpurate= cmdline option.
>> - */
>> -static int __init omap_clk_setup(char *str)
>> -{
>> - get_option(&str, &mpurate);
>> -
>> - if (!mpurate)
>> - return 1;
>> -
>> - if (mpurate < 1000)
>> - mpurate *= 1000000;
>> -
>> - return 1;
>> -}
>> -__setup("mpurate=", omap_clk_setup);
>> -
>> /**
>> * omap2_clk_print_new_rates - print summary of current clock tree rates
>> * @hfclkin_ck_name: clk name for the off-chip HF oscillator
>> --
>> 2.5.0
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Andrew Murray
Witekio UK
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
@ 2017-01-23 19:51 ` Andrew Murray
0 siblings, 0 replies; 8+ messages in thread
From: Andrew Murray @ 2017-01-23 19:51 UTC (permalink / raw)
To: linux-arm-kernel
On 23 January 2017 at 15:10, Tony Lindgren <tony@atomide.com> wrote:
> * Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
>> The 'mpurate' option, historically used for specifying an initial
>> MPU rate at boot, no longer has any effect due to the supporting
>> code being removed as it was 'wrong and dangerous' [1].
>>
>> This patch removes the remaining dead code associated with the
>> __setup macros to avoid confusion and reduce bloat.
>>
>> [1] https://patchwork.kernel.org/patch/5954631/
>
> Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
> Cc for mach-omap1 in case there might be issues with this
> patch.
I've also noticed that omap2_clk_print_new_rates in
arch/arm/mach-omap2/clock.c is also unused - this was previously used
to print the new clock rates upon change after use of mpurate.
I'll update the patch to include the removal of this function as well
- unless you have any objections.
Thanks,
Andrew Murray
>
>
>> Signed-off-by: Andrew Murray <amurray@embedded-bits.co.uk>
>> ---
>> arch/arm/mach-omap1/clock.c | 20 --------------------
>> arch/arm/mach-omap1/clock.h | 2 --
>> arch/arm/mach-omap2/clock.c | 20 --------------------
>> 3 files changed, 42 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap1/clock.c b/arch/arm/mach-omap1/clock.c
>> index 4f5fd4a..43e3e18 100644
>> --- a/arch/arm/mach-omap1/clock.c
>> +++ b/arch/arm/mach-omap1/clock.c
>> @@ -720,26 +720,6 @@ EXPORT_SYMBOL(clk_get_parent);
>> * OMAP specific clock functions shared between omap1 and omap2
>> */
>>
>> -int __initdata mpurate;
>> -
>> -/*
>> - * By default we use the rate set by the bootloader.
>> - * You can override this with mpurate= cmdline option.
>> - */
>> -static int __init omap_clk_setup(char *str)
>> -{
>> - get_option(&str, &mpurate);
>> -
>> - if (!mpurate)
>> - return 1;
>> -
>> - if (mpurate < 1000)
>> - mpurate *= 1000000;
>> -
>> - return 1;
>> -}
>> -__setup("mpurate=", omap_clk_setup);
>> -
>> /* Used for clocks that always have same value as the parent clock */
>> unsigned long followparent_recalc(struct clk *clk)
>> {
>> diff --git a/arch/arm/mach-omap1/clock.h b/arch/arm/mach-omap1/clock.h
>> index 1e4918a..f2d1be2 100644
>> --- a/arch/arm/mach-omap1/clock.h
>> +++ b/arch/arm/mach-omap1/clock.h
>> @@ -173,8 +173,6 @@ struct clk_functions {
>> void (*clk_disable_unused)(struct clk *clk);
>> };
>>
>> -extern int mpurate;
>> -
>> extern int clk_init(struct clk_functions *custom_clocks);
>> extern void clk_preinit(struct clk *clk);
>> extern int clk_register(struct clk *clk);
>> diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c
>> index d058125..d999295 100644
>> --- a/arch/arm/mach-omap2/clock.c
>> +++ b/arch/arm/mach-omap2/clock.c
>> @@ -112,26 +112,6 @@ void omap2_init_clk_clkdm(struct clk_hw *hw)
>> }
>> }
>>
>> -static int __initdata mpurate;
>> -
>> -/*
>> - * By default we use the rate set by the bootloader.
>> - * You can override this with mpurate= cmdline option.
>> - */
>> -static int __init omap_clk_setup(char *str)
>> -{
>> - get_option(&str, &mpurate);
>> -
>> - if (!mpurate)
>> - return 1;
>> -
>> - if (mpurate < 1000)
>> - mpurate *= 1000000;
>> -
>> - return 1;
>> -}
>> -__setup("mpurate=", omap_clk_setup);
>> -
>> /**
>> * omap2_clk_print_new_rates - print summary of current clock tree rates
>> * @hfclkin_ck_name: clk name for the off-chip HF oscillator
>> --
>> 2.5.0
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>> the body of a message to majordomo at vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Andrew Murray
Witekio UK
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
2017-01-23 19:51 ` Andrew Murray
@ 2017-01-23 20:34 ` Tony Lindgren
-1 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2017-01-23 20:34 UTC (permalink / raw)
To: Andrew Murray
Cc: Tero Kristo, Paul Walmsley, linux-omap, linux-arm-kernel, Aaro Koskinen
* Andrew Murray <amurray@embedded-bits.co.uk> [170123 11:52]:
> On 23 January 2017 at 15:10, Tony Lindgren <tony@atomide.com> wrote:
> > * Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
> >> The 'mpurate' option, historically used for specifying an initial
> >> MPU rate at boot, no longer has any effect due to the supporting
> >> code being removed as it was 'wrong and dangerous' [1].
> >>
> >> This patch removes the remaining dead code associated with the
> >> __setup macros to avoid confusion and reduce bloat.
> >>
> >> [1] https://patchwork.kernel.org/patch/5954631/
> >
> > Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
> > Cc for mach-omap1 in case there might be issues with this
> > patch.
>
> I've also noticed that omap2_clk_print_new_rates in
> arch/arm/mach-omap2/clock.c is also unused - this was previously used
> to print the new clock rates upon change after use of mpurate.
>
> I'll update the patch to include the removal of this function as well
> - unless you have any objections.
Sure.
Thanks,
Tony
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option
@ 2017-01-23 20:34 ` Tony Lindgren
0 siblings, 0 replies; 8+ messages in thread
From: Tony Lindgren @ 2017-01-23 20:34 UTC (permalink / raw)
To: linux-arm-kernel
* Andrew Murray <amurray@embedded-bits.co.uk> [170123 11:52]:
> On 23 January 2017 at 15:10, Tony Lindgren <tony@atomide.com> wrote:
> > * Andrew Murray <amurray@embedded-bits.co.uk> [170121 09:38]:
> >> The 'mpurate' option, historically used for specifying an initial
> >> MPU rate at boot, no longer has any effect due to the supporting
> >> code being removed as it was 'wrong and dangerous' [1].
> >>
> >> This patch removes the remaining dead code associated with the
> >> __setup macros to avoid confusion and reduce bloat.
> >>
> >> [1] https://patchwork.kernel.org/patch/5954631/
> >
> > Yeah AFAIK this is unused for mach-omap2. Adding Aaro to
> > Cc for mach-omap1 in case there might be issues with this
> > patch.
>
> I've also noticed that omap2_clk_print_new_rates in
> arch/arm/mach-omap2/clock.c is also unused - this was previously used
> to print the new clock rates upon change after use of mpurate.
>
> I'll update the patch to include the removal of this function as well
> - unless you have any objections.
Sure.
Thanks,
Tony
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-01-23 20:34 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-21 17:36 [PATCH] ARM: OMAP1/2: clock: Remove unused mpurate cmdline option Andrew Murray
2017-01-21 17:36 ` Andrew Murray
2017-01-23 15:10 ` Tony Lindgren
2017-01-23 15:10 ` Tony Lindgren
2017-01-23 19:51 ` Andrew Murray
2017-01-23 19:51 ` Andrew Murray
2017-01-23 20:34 ` Tony Lindgren
2017-01-23 20:34 ` Tony Lindgren
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.