Hi Geert,

On 19/02/12 20:12, Geert Uytterhoeven wrote:
On Tue, Feb 7, 2012 at 04:25,<[email protected]>  wrote:
--- a/arch/m68k/kernel/ptrace_mm.c
+++ b/arch/m68k/kernel/ptrace.c
@@ -277,7 +287,6 @@ asmlinkage void syscall_trace(void)
        }
}

-#ifdef CONFIG_COLDFIRE
asmlinkage int syscall_trace_enter(void)
{
        int ret = 0;
@@ -292,4 +301,3 @@ asmlinkage void syscall_trace_leave(void)
        if (test_thread_flag(TIF_SYSCALL_TRACE))
                tracehook_report_syscall_exit(task_pt_regs(current), 0);
}
-#endif /* CONFIG_COLDFIRE */

Why have you dropped the #ifdef?

Purely to have less "#ifdef"s in the code.
Does bloat the code a little, would you prefer I leave the #ifdef there?

Regards
Greg


------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     [email protected]
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to