[PATCH 1/6] ARM: prefetch: remove redundant "cc" clobber

Nicolas Pitre nico at fluxnic.net
Tue Jul 23 15:48:11 EDT 2013


On Tue, 23 Jul 2013, Will Deacon wrote:

> The pld instruction does not affect the condition flags, so don't bother
> clobbering them.
> 
> Signed-off-by: Will Deacon <will.deacon at arm.com>

Acked-by: Nicolas Pitre <nico at linaro.org>

You know the legacy reason why the cc clobber was there, right?
It certainly doesn't apply anymore.



> ---
>  arch/arm/include/asm/processor.h | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/arch/arm/include/asm/processor.h b/arch/arm/include/asm/processor.h
> index 06e7d50..91cfe08 100644
> --- a/arch/arm/include/asm/processor.h
> +++ b/arch/arm/include/asm/processor.h
> @@ -101,9 +101,7 @@ static inline void prefetch(const void *ptr)
>  {
>  	__asm__ __volatile__(
>  		"pld\t%a0"
> -		:
> -		: "p" (ptr)
> -		: "cc");
> +		:: "p" (ptr));
>  }
>  
>  #define ARCH_HAS_PREFETCHW
> -- 
> 1.8.2.2
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 



More information about the linux-arm-kernel mailing list