Re: Re: [PATCH] MIPS: ftrace: Remove redundant #ifdef CONFIG_DYNAMIC_FTRAC

2020-08-13 Thread Zejiang Tang
okļ¼Œthank you very much

Florian Fainelli f.faine...@gmail.comwrote:
> 
> 
> On 8/13/2020 7:08 PM, Zejiang Tang wrote:
> > There exists redundant #ifdef CONFIG_DYNAMIC_FTRAC in ftrace.c, remove it.
> 
> There is a missing E at the end of CONFIG_DYNAMIC_FTRAC in your subject 
> FWIWW.
> -- 
> Florian


Re: [PATCH] MIPS: ftrace: Remove redundant #ifdef CONFIG_DYNAMIC_FTRAC

2020-08-13 Thread Florian Fainelli




On 8/13/2020 7:08 PM, Zejiang Tang wrote:

There exists redundant #ifdef CONFIG_DYNAMIC_FTRAC in ftrace.c, remove it.


There is a missing E at the end of CONFIG_DYNAMIC_FTRAC in your subject 
FWIWW.

--
Florian


Re: [PATCH] MIPS: ftrace: Remove redundant #ifdef CONFIG_DYNAMIC_FTRAC

2020-08-13 Thread Steven Rostedt
On Fri, 14 Aug 2020 10:08:15 +0800
Zejiang Tang  wrote:

> There exists redundant #ifdef CONFIG_DYNAMIC_FTRAC in ftrace.c, remove it.
> 
> Signed-off-by: Zejiang Tang 

Reviewed-by: Steven Rostedt (VMware) 

-- Steve

> ---
>  arch/mips/kernel/ftrace.c | 4 
>  1 file changed, 4 deletions(-)
> 
> diff --git a/arch/mips/kernel/ftrace.c b/arch/mips/kernel/ftrace.c
> index 2625232..f57e68f 100644
> --- a/arch/mips/kernel/ftrace.c
> +++ b/arch/mips/kernel/ftrace.c
> @@ -37,10 +37,6 @@ void arch_ftrace_update_code(int command)
>   ftrace_modify_all_code(command);
>  }
>  
> -#endif
> -
> -#ifdef CONFIG_DYNAMIC_FTRACE
> -
>  #define JAL 0x0c00   /* jump & link: ip --> ra, jump to 
> target */
>  #define ADDR_MASK 0x03ff /*  op_code|addr : 31...26|25 0 */
>  #define JUMP_RANGE_MASK ((1UL << 28) - 1)