[PATCH v3] arm: remove unused code in delay.S
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Fri Sep 18 15:09:20 EDT 2009
Hello Steve,
On Fri, Sep 18, 2009 at 11:46:58AM -0500, Steve Chen wrote:
> Document #if 0 code block in delay.S and make it selectable for compile.
>
> Signed-off-by: Steve Chen <schen at mvista.com>
>
> ---
> arch/arm/Kconfig | 31 +++++++++++++++++++++++++++++++
> arch/arm/lib/delay.S | 2 +-
> 2 files changed, 32 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index aef63c8..f44cb70 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -813,6 +813,37 @@ config ARM_ERRATA_460075
> ACTLR register. Note that setting specific bits in the ACTLR
> register
> may not be available in non-secure mode.
>
> +config OLD_CPU_DELAY
> + depends on CPU_32v3 || CPU_32v4 || CPU_32v4T
> + bool "Different delay() code for some older CPUs"
> + def_bool n
as already noted, n is the default (that would better be expressed using
"default n" btw).
> + help
> + Enable this if observing longer than expected delays. This code
> + improves delay accuracy for some CPUs. However, it can also cause
> + delay duration to be too short for others which leads to stability
> + issues.
> +
> + In other words, do not enable unless you can guarantee that the
> + processor (or ALL of the processors if building a generic kernel)
> + delays for at least the time requested after enabling.
> +
> + ARM610 and ARM710 are known to benefit from enabling this option.
> + It should not be enabled for StrongARMs, because it is known
> + to produce too short delays on those.
> +
> + Lastly, below are 2 lists. The first list contains the processors
> + that would benefit from enabling this flag, and the second list
> + contains processor that are known to have issues. Please note that
> + both lists are by no means complete. Entries are expected to be
> + added and refined. If you like to update the list, please send a
> + patch to Linux ARM mailing list.
I'd delete the last three sentences.
> + CPUs should enable this flag
> + ARM610
> + ARM710
> +
> + CPUs should disable this flag
> + StrongARM
everything armv5+ ?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-arm-kernel
mailing list