public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arch: arm: mm: cache-feroceon-l2:  Remove some unused functions
@ 2015-01-01 17:56 Rickard Strandqvist
  2015-01-05 14:32 ` Andrew Lunn
  0 siblings, 1 reply; 2+ messages in thread
From: Rickard Strandqvist @ 2015-01-01 17:56 UTC (permalink / raw)
  To: Russell King, Jason Cooper
  Cc: Rickard Strandqvist, Andrew Lunn, Jason Gunthorpe,
	linux-arm-kernel, linux-kernel

Removes some functions that are not used anywhere:
l2_inv_pa() l2_clean_pa()

This was partially found by using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
---
 arch/arm/mm/cache-feroceon-l2.c |   10 ----------
 1 file changed, 10 deletions(-)

diff --git a/arch/arm/mm/cache-feroceon-l2.c b/arch/arm/mm/cache-feroceon-l2.c
index e028a7f..c2eac97 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -65,11 +65,6 @@ static inline void l2_put_va(unsigned long vaddr)
 #endif
 }
 
-static inline void l2_clean_pa(unsigned long addr)
-{
-	__asm__("mcr p15, 1, %0, c15, c9, 3" : : "r" (addr));
-}
-
 static inline void l2_clean_pa_range(unsigned long start, unsigned long end)
 {
 	unsigned long va_start, va_end, flags;
@@ -96,11 +91,6 @@ static inline void l2_clean_inv_pa(unsigned long addr)
 	__asm__("mcr p15, 1, %0, c15, c10, 3" : : "r" (addr));
 }
 
-static inline void l2_inv_pa(unsigned long addr)
-{
-	__asm__("mcr p15, 1, %0, c15, c11, 3" : : "r" (addr));
-}
-
 static inline void l2_inv_pa_range(unsigned long start, unsigned long end)
 {
 	unsigned long va_start, va_end, flags;
-- 
1.7.10.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] arch: arm: mm: cache-feroceon-l2: Remove some unused functions
  2015-01-01 17:56 [PATCH] arch: arm: mm: cache-feroceon-l2: Remove some unused functions Rickard Strandqvist
@ 2015-01-05 14:32 ` Andrew Lunn
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Lunn @ 2015-01-05 14:32 UTC (permalink / raw)
  To: Rickard Strandqvist
  Cc: Russell King, Jason Cooper, Jason Gunthorpe, linux-kernel,
	linux-arm-kernel

On Thu, Jan 01, 2015 at 06:56:09PM +0100, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> l2_inv_pa() l2_clean_pa()
> 
> This was partially found by using a static code analysis program called cppcheck.
> 
> Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>


Acked-by: Andrew Lunn <andrew@lunn.ch>

Please could you submit this to Russell Kings patch tracker.

Thanks
	Andrew

> ---
>  arch/arm/mm/cache-feroceon-l2.c |   10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/arch/arm/mm/cache-feroceon-l2.c b/arch/arm/mm/cache-feroceon-l2.c
> index e028a7f..c2eac97 100644
> --- a/arch/arm/mm/cache-feroceon-l2.c
> +++ b/arch/arm/mm/cache-feroceon-l2.c
> @@ -65,11 +65,6 @@ static inline void l2_put_va(unsigned long vaddr)
>  #endif
>  }
>  
> -static inline void l2_clean_pa(unsigned long addr)
> -{
> -	__asm__("mcr p15, 1, %0, c15, c9, 3" : : "r" (addr));
> -}
> -
>  static inline void l2_clean_pa_range(unsigned long start, unsigned long end)
>  {
>  	unsigned long va_start, va_end, flags;
> @@ -96,11 +91,6 @@ static inline void l2_clean_inv_pa(unsigned long addr)
>  	__asm__("mcr p15, 1, %0, c15, c10, 3" : : "r" (addr));
>  }
>  
> -static inline void l2_inv_pa(unsigned long addr)
> -{
> -	__asm__("mcr p15, 1, %0, c15, c11, 3" : : "r" (addr));
> -}
> -
>  static inline void l2_inv_pa_range(unsigned long start, unsigned long end)
>  {
>  	unsigned long va_start, va_end, flags;
> -- 
> 1.7.10.4
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-01-05 14:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-01 17:56 [PATCH] arch: arm: mm: cache-feroceon-l2: Remove some unused functions Rickard Strandqvist
2015-01-05 14:32 ` Andrew Lunn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox