Blackfin arch: smp patch cleanup from LKML review

1. Use inline get_l1_... functions instead of macro
2. Fix compile issue about smp barrier functions

Signed-off-by: Graf Yang <graf.yang@analog.com>
Signed-off-by: Bryan Wu <cooloney@kernel.org>

diff --git a/arch/blackfin/include/asm/mem_map.h b/arch/blackfin/include/asm/mem_map.h
index 88d04a707708d85de25a078496628346a6c43d55..e92b31051bb7ad1e5015e2110d6ac80239d1d4f6 100644 (file)
--- a/arch/blackfin/include/asm/mem_map.h
+++ b/arch/blackfin/include/asm/mem_map.h
@@ -9,4 +9,79 @@
 
 #include <mach/mem_map.h>
 
+#ifndef __ASSEMBLY__
+
+#ifdef CONFIG_SMP
+static inline ulong get_l1_scratch_start_cpu(int cpu)
+{
+       return (cpu) ? COREB_L1_SCRATCH_START : COREA_L1_SCRATCH_START;
+}
+static inline ulong get_l1_code_start_cpu(int cpu)
+{
+       return (cpu) ? COREB_L1_CODE_START : COREA_L1_CODE_START;
+}
+static inline ulong get_l1_data_a_start_cpu(int cpu)
+{
+       return (cpu) ? COREB_L1_DATA_A_START : COREA_L1_DATA_A_START;
+}
+static inline ulong get_l1_data_b_start_cpu(int cpu)
+{
+       return (cpu) ? COREB_L1_DATA_B_START : COREA_L1_DATA_B_START;
+}
+
+static inline ulong get_l1_scratch_start(void)
+{
+       return get_l1_scratch_start_cpu(blackfin_core_id());
+}
+static inline ulong get_l1_code_start(void)
+{
+       return get_l1_code_start_cpu(blackfin_core_id());
+}
+static inline ulong get_l1_data_a_start(void)
+{
+       return get_l1_data_a_start_cpu(blackfin_core_id());
+}
+static inline ulong get_l1_data_b_start(void)
+{
+       return get_l1_data_b_start_cpu(blackfin_core_id());
+}
+
+#else /* !CONFIG_SMP */
+
+static inline ulong get_l1_scratch_start_cpu(int cpu)
+{
+       return L1_SCRATCH_START;
+}
+static inline ulong get_l1_code_start_cpu(int cpu)
+{
+       return L1_CODE_START;
+}
+static inline ulong get_l1_data_a_start_cpu(int cpu)
+{
+       return L1_DATA_A_START;
+}
+static inline ulong get_l1_data_b_start_cpu(int cpu)
+{
+       return L1_DATA_B_START;
+}
+static inline ulong get_l1_scratch_start(void)
+{
+       return get_l1_scratch_start_cpu(0);
+}
+static inline ulong get_l1_code_start(void)
+{
+       return  get_l1_code_start_cpu(0);
+}
+static inline ulong get_l1_data_a_start(void)
+{
+       return get_l1_data_a_start_cpu(0);
+}
+static inline ulong get_l1_data_b_start(void)
+{
+       return get_l1_data_b_start_cpu(0);
+}
+
+#endif /* CONFIG_SMP */
+#endif /* __ASSEMBLY__ */
+
 #endif                         /* _MEM_MAP_H_ */

diff --git a/arch/blackfin/include/asm/smp.h b/arch/blackfin/include/asm/smp.h
index 233cb8c3cfb74284c4c7d943cab395e41eebf164..118deeeae7c05ad1f042ffa4bf75eb2cd1112ec0 100644 (file)
--- a/arch/blackfin/include/asm/smp.h
+++ b/arch/blackfin/include/asm/smp.h
@@ -32,6 +32,8 @@
 
 #define raw_smp_processor_id()  blackfin_core_id()
 
+extern char coreb_trampoline_start, coreb_trampoline_end;
+
 struct corelock_slot {
        int lock;
 };

diff --git a/arch/blackfin/include/asm/system.h b/arch/blackfin/include/asm/system.h
index e8bcfa4ee5c0ef5426e5c16410fb94d5738dd08c..dea92037dff588c9e26410cdd5ecb646e71fd440 100644 (file)
--- a/arch/blackfin/include/asm/system.h
+++ b/arch/blackfin/include/asm/system.h
@@ -66,10 +66,13 @@ asmlinkage unsigned long __raw_cmpxchg_4_asm(volatile void *ptr,
 # define smp_mb()      do { barrier(); smp_check_barrier(); smp_mark_barrier(); } while (0)
 # define smp_rmb()     do { barrier(); smp_check_barrier(); } while (0)
 # define smp_wmb()     do { barrier(); smp_mark_barrier(); } while (0)
+#define smp_read_barrier_depends()     do { barrier(); smp_check_barrier(); } while (0)
+
 #else
 # define smp_mb()      barrier()
 # define smp_rmb()     barrier()
 # define smp_wmb()     barrier()
+#define smp_read_barrier_depends()     barrier()
 #endif
 
 static inline unsigned long __xchg(unsigned long x, volatile void *ptr,
@@ -120,8 +123,6 @@ static inline unsigned long __cmpxchg(volatile void *ptr, unsigned long old,
        ((__typeof__(*(ptr)))__cmpxchg((ptr), (unsigned long)(o), \
                (unsigned long)(n), sizeof(*(ptr))))
 
-#define smp_read_barrier_depends()     smp_check_barrier()
-
 #else /* !CONFIG_SMP */
 
 #define smp_mb()       barrier()
@@ -192,6 +193,7 @@ static inline unsigned long __xchg(unsigned long x, volatile void *ptr,
  */
 
 #include <asm/l1layout.h>
+#include <asm/mem_map.h>
 
 asmlinkage struct task_struct *resume(struct task_struct *prev, struct task_struct *next);
 

diff --git a/arch/blackfin/kernel/bfin_ksyms.c b/arch/blackfin/kernel/bfin_ksyms.c
index 763c31531e9e0b466ce49a52acef62c675068c6a..01f917d58b5908e65f3feeda75b89ec9c6fec407 100644 (file)
--- a/arch/blackfin/kernel/bfin_ksyms.c
+++ b/arch/blackfin/kernel/bfin_ksyms.c
@@ -99,6 +99,8 @@ EXPORT_SYMBOL(__raw_bit_test_set_asm);
 EXPORT_SYMBOL(__raw_bit_test_clear_asm);
 EXPORT_SYMBOL(__raw_bit_test_toggle_asm);
 EXPORT_SYMBOL(__raw_uncached_fetch_asm);
+#ifdef __ARCH_SYNC_CORE_DCACHE
 EXPORT_SYMBOL(__raw_smp_mark_barrier_asm);
 EXPORT_SYMBOL(__raw_smp_check_barrier_asm);
 #endif
+#endif

diff --git a/arch/blackfin/kernel/cplb-mpu/cplbinit.c b/arch/blackfin/kernel/cplb-mpu/cplbinit.c
index 269d2a3530a686a60baf9f2d333a7e5645fa1370..1ea7c18435ae39272ff95cbacb1792a0dee23c03 100644 (file)
--- a/arch/blackfin/kernel/cplb-mpu/cplbinit.c
+++ b/arch/blackfin/kernel/cplb-mpu/cplbinit.c
@@ -25,6 +25,7 @@
 #include <asm/blackfin.h>
 #include <asm/cplb.h>
 #include <asm/cplbinit.h>
+#include <asm/mem_map.h>
 
 #if ANOMALY_05000263
 # error the MPU will not function safely while Anomaly 05000263 applies

diff --git a/arch/blackfin/kernel/cplb-nompu/cplbinit.c b/arch/blackfin/kernel/cplb-nompu/cplbinit.c
index 735413d2c228b1c4c1b362f07b2bff98e2e6a15b..4c010ba50a80efa60a2fedc4a05730518c49facb 100644 (file)
--- a/arch/blackfin/kernel/cplb-nompu/cplbinit.c
+++ b/arch/blackfin/kernel/cplb-nompu/cplbinit.c
@@ -27,6 +27,7 @@
 #include <asm/cacheflush.h>
 #include <asm/cplb.h>
 #include <asm/cplbinit.h>
+#include <asm/mem_map.h>
 
 u_long icplb_tables[NR_CPUS][CPLB_TBL_ENTRIES+1];
 u_long dcplb_tables[NR_CPUS][CPLB_TBL_ENTRIES+1];

diff --git a/arch/blackfin/kernel/process.c b/arch/blackfin/kernel/process.c
index 4359ea25301060b947482080f61029fe80e084d3..1ec0faa8c68dbb53ffccdf70db2564ea869e468e 100644 (file)
--- a/arch/blackfin/kernel/process.c
+++ b/arch/blackfin/kernel/process.c
@@ -39,6 +39,7 @@
 
 #include <asm/blackfin.h>
 #include <asm/fixed_code.h>
+#include <asm/mem_map.h>
 
 asmlinkage void ret_from_fork(void);
 

diff --git a/arch/blackfin/kernel/ptrace.c b/arch/blackfin/kernel/ptrace.c
index d5e6be2d5ed83b361b81ef0e20180819b49cc94d..d2d3885366304ea38826c0c2897c0a4008184c20 100644 (file)
--- a/arch/blackfin/kernel/ptrace.c
+++ b/arch/blackfin/kernel/ptrace.c
@@ -45,6 +45,7 @@
 #include <asm/asm-offsets.h>
 #include <asm/dma.h>
 #include <asm/fixed_code.h>
+#include <asm/mem_map.h>
 
 #define TEXT_OFFSET 0
 /*

diff --git a/arch/blackfin/mach-bf518/include/mach/mem_map.h b/arch/blackfin/mach-bf518/include/mach/mem_map.h
index ac95d334d08298edcfe808081a8fd449b8a63f68..62bcc781bfaa884c7db1f293761ab228b42f4358 100644 (file)
--- a/arch/blackfin/mach-bf518/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf518/include/mach/mem_map.h
@@ -99,15 +99,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf527/include/mach/mem_map.h b/arch/blackfin/mach-bf527/include/mach/mem_map.h
index bd7fe0fda9e8c7a7a7553d9d31f2af0f05b27f2b..019e0017ad81790c17478288c6b3f2e2c2835c22 100644 (file)
--- a/arch/blackfin/mach-bf527/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf527/include/mach/mem_map.h
@@ -99,15 +99,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf533/include/mach/mem_map.h b/arch/blackfin/mach-bf533/include/mach/mem_map.h
index d5eaef21edb7d6dd0fbcb208dbe8ae895da34078..fc33b7cb993752af0e80b67871c0ab76e5a4e7f5 100644 (file)
--- a/arch/blackfin/mach-bf533/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf533/include/mach/mem_map.h
@@ -168,15 +168,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf537/include/mach/mem_map.h b/arch/blackfin/mach-bf537/include/mach/mem_map.h
index be4de761be96ee2a2630c171fb5d8e7836768847..f9010c4b4bf3d1eafea05b90224f6edf9185013f 100644 (file)
--- a/arch/blackfin/mach-bf537/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf537/include/mach/mem_map.h
@@ -176,15 +176,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf538/include/mach/mem_map.h b/arch/blackfin/mach-bf538/include/mach/mem_map.h
index c134057c91ff33d9edf15cfa5aaf9041d00ff112..76811966690e133040ce32cc25d7702ebeff7420 100644 (file)
--- a/arch/blackfin/mach-bf538/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf538/include/mach/mem_map.h
@@ -104,15 +104,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf548/include/mach/mem_map.h b/arch/blackfin/mach-bf548/include/mach/mem_map.h
index 361eb0e85bc062174236653cefe2cd7a5b03a76d..70b9c11940243f5f2a160b022389d8471b23d9e1 100644 (file)
--- a/arch/blackfin/mach-bf548/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf548/include/mach/mem_map.h
@@ -108,15 +108,6 @@
 #define L1_SCRATCH_START       0xFFB00000
 #define L1_SCRATCH_LENGTH      0x1000
 
-#define get_l1_scratch_start_cpu(cpu)          L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)             L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)           L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)           L1_DATA_B_START
-#define get_l1_scratch_start()                 L1_SCRATCH_START
-#define get_l1_code_start()                    L1_CODE_START
-#define get_l1_data_a_start()                  L1_DATA_A_START
-#define get_l1_data_b_start()                  L1_DATA_B_START
-
 #define GET_PDA_SAFE(preg)             \
        preg.l = _cpu_pda;              \
        preg.h = _cpu_pda;

diff --git a/arch/blackfin/mach-bf561/include/mach/mem_map.h b/arch/blackfin/mach-bf561/include/mach/mem_map.h
index 488c3bda65bf16329d3b0f22096a1c20f2701d2f..419dffdc96ebd4ad6c6bbf80b0228786f692c054 100644 (file)
--- a/arch/blackfin/mach-bf561/include/mach/mem_map.h
+++ b/arch/blackfin/mach-bf561/include/mach/mem_map.h
@@ -85,47 +85,7 @@
 #define L1_SCRATCH_START       COREA_L1_SCRATCH_START
 #define L1_SCRATCH_LENGTH      0x1000
 
-#ifndef __ASSEMBLY__
-
-#ifdef CONFIG_SMP
-
-#define get_l1_scratch_start_cpu(cpu)                          \
-       ({ unsigned long __addr;                                \
-          __addr = (cpu) ? COREB_L1_SCRATCH_START : COREA_L1_SCRATCH_START;\
-          __addr; })
-
-#define get_l1_code_start_cpu(cpu)                             \
-       ({ unsigned long __addr;                                \
-          __addr = (cpu) ? COREB_L1_CODE_START : COREA_L1_CODE_START;  \
-          __addr; })
-
-#define get_l1_data_a_start_cpu(cpu)                           \
-       ({ unsigned long __addr;                                \
-          __addr = (cpu) ? COREB_L1_DATA_A_START : COREA_L1_DATA_A_START;\
-          __addr; })
-
-#define get_l1_data_b_start_cpu(cpu)                           \
-       ({ unsigned long __addr;                                \
-          __addr = (cpu) ? COREB_L1_DATA_B_START : COREA_L1_DATA_B_START;\
-          __addr; })
-
-#define get_l1_scratch_start() get_l1_scratch_start_cpu(blackfin_core_id())
-#define get_l1_code_start()    get_l1_code_start_cpu(blackfin_core_id())
-#define get_l1_data_a_start()  get_l1_data_a_start_cpu(blackfin_core_id())
-#define get_l1_data_b_start()  get_l1_data_b_start_cpu(blackfin_core_id())
-
-#else /* !CONFIG_SMP */
-#define get_l1_scratch_start_cpu(cpu)  L1_SCRATCH_START
-#define get_l1_code_start_cpu(cpu)     L1_CODE_START
-#define get_l1_data_a_start_cpu(cpu)   L1_DATA_A_START
-#define get_l1_data_b_start_cpu(cpu)   L1_DATA_B_START
-#define get_l1_scratch_start()         L1_SCRATCH_START
-#define get_l1_code_start()            L1_CODE_START
-#define get_l1_data_a_start()          L1_DATA_A_START
-#define get_l1_data_b_start()          L1_DATA_B_START
-#endif /* !CONFIG_SMP */
-
-#else /* __ASSEMBLY__ */
+#ifdef __ASSEMBLY__
 
 /*
  * The following macros both return the address of the PDA for the

diff --git a/arch/blackfin/mach-bf561/smp.c b/arch/blackfin/mach-bf561/smp.c
index 23fd4c13897f039fc83a48e7d1a36d4acf9137d6..9b27e698c0b2e717bd97ff8033bb630112097750 100644 (file)
--- a/arch/blackfin/mach-bf561/smp.c
+++ b/arch/blackfin/mach-bf561/smp.c
@@ -27,11 +27,6 @@
 #include <asm/smp.h>
 #include <asm/dma.h>
 
-#define COREB_SRAM_BASE  0xff600000
-#define COREB_SRAM_SIZE  0x4000
-
-extern char coreb_trampoline_start, coreb_trampoline_end;
-
 static DEFINE_SPINLOCK(boot_lock);
 
 static cpumask_t cpu_callin_map;
@@ -54,15 +49,15 @@ void __init platform_prepare_cpus(unsigned int max_cpus)
        int len;
 
        len = &coreb_trampoline_end - &coreb_trampoline_start + 1;
-       BUG_ON(len > COREB_SRAM_SIZE);
+       BUG_ON(len > L1_CODE_LENGTH);
 
-       dma_memcpy((void *)COREB_SRAM_BASE, &coreb_trampoline_start, len);
+       dma_memcpy((void *)COREB_L1_CODE_START, &coreb_trampoline_start, len);
 
        /* Both cores ought to be present on a bf561! */
        cpu_set(0, cpu_present_map); /* CoreA */
        cpu_set(1, cpu_present_map); /* CoreB */
 
-       printk(KERN_INFO "CoreB bootstrap code to SRAM %p via DMA.\n", (void *)COREB_SRAM_BASE);
+       printk(KERN_INFO "CoreB bootstrap code to SRAM %p via DMA.\n", (void *)COREB_L1_CODE_START);
 }
 
 int __init setup_profiling_timer(unsigned int multiplier) /* not supported */

diff --git a/arch/blackfin/mm/sram-alloc.c b/arch/blackfin/mm/sram-alloc.c
index 1ca5deb7934b0b52dfdb081c7b65185131bfcbdf..834cab7438a8fa3293f2c8e1ef4269c6740d8e13 100644 (file)
--- a/arch/blackfin/mm/sram-alloc.c
+++ b/arch/blackfin/mm/sram-alloc.c
@@ -39,6 +39,7 @@
 #include <linux/spinlock.h>
 #include <linux/rtc.h>
 #include <asm/blackfin.h>
+#include <asm/mem_map.h>
 #include "blackfin_sram.h"
 
 static DEFINE_PER_CPU(spinlock_t, l1sram_lock) ____cacheline_aligned_in_smp;