Re: [PATCH 5/6] cris: remove unused wp_works_ok macro

2017-02-13 Thread Jesper Nilsson
On Sun, Feb 12, 2017 at 10:12:11PM +0100, Mathias Krause wrote:
> It had no use since it's introduction in v2.4.1.2. Get rid of it.

Agreed.

Acked-by: Jesper Nilsson 

> Cc: Mikael Starvik 
> Signed-off-by: Mathias Krause 
> ---
>  arch/cris/include/arch-v10/arch/processor.h |3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/arch/cris/include/arch-v10/arch/processor.h 
> b/arch/cris/include/arch-v10/arch/processor.h
> index 93feb2a487d8..58f75bee1d6c 100644
> --- a/arch/cris/include/arch-v10/arch/processor.h
> +++ b/arch/cris/include/arch-v10/arch/processor.h
> @@ -7,9 +7,6 @@
>   */
>  #define current_text_addr() ({void *pc; __asm__ ("move.d $pc,%0" : "=rm" 
> (pc)); pc; })
>  
> -/* CRIS has no problems with write protection */
> -#define wp_works_ok 1
> -
>  /* CRIS thread_struct. this really has nothing to do with the processor 
> itself, since
>   * CRIS does not do any hardware task-switching, but it's here for legacy 
> reasons.
>   * The thread_struct here is used when task-switching using _resume defined 
> in entry.S.
> -- 
> 1.7.10.4

/^JN - Jesper Nilsson
-- 
   Jesper Nilsson -- jesper.nils...@axis.com


[PATCH 5/6] cris: remove unused wp_works_ok macro

2017-02-12 Thread Mathias Krause
It had no use since it's introduction in v2.4.1.2. Get rid of it.

Cc: Jesper Nilsson 
Cc: Mikael Starvik 
Signed-off-by: Mathias Krause 
---
 arch/cris/include/arch-v10/arch/processor.h |3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/cris/include/arch-v10/arch/processor.h 
b/arch/cris/include/arch-v10/arch/processor.h
index 93feb2a487d8..58f75bee1d6c 100644
--- a/arch/cris/include/arch-v10/arch/processor.h
+++ b/arch/cris/include/arch-v10/arch/processor.h
@@ -7,9 +7,6 @@
  */
 #define current_text_addr() ({void *pc; __asm__ ("move.d $pc,%0" : "=rm" 
(pc)); pc; })
 
-/* CRIS has no problems with write protection */
-#define wp_works_ok 1
-
 /* CRIS thread_struct. this really has nothing to do with the processor 
itself, since
  * CRIS does not do any hardware task-switching, but it's here for legacy 
reasons.
  * The thread_struct here is used when task-switching using _resume defined in 
entry.S.
-- 
1.7.10.4