Clean unused stuff out of thread.h and config.h and reorganize thread-pp.c to simplify the preprocessor blocks.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@26743 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 263 insertions, 263 deletions

diff --git a/firmware/target/arm/thread-pp.c b/firmware/target/arm/thread-pp.c
index 8dfbd64080..335f1f3e0a 100644
--- a/firmware/target/arm/thread-pp.c
+++ b/firmware/target/arm/thread-pp.c
@@ -26,7 +26,21 @@
 #define IF_NO_SKIP_YIELD(...) __VA_ARGS__
 #endif
 
-#if NUM_CORES > 1
+#if NUM_CORES == 1
+/* Single-core variants for FORCE_SINGLE_CORE */
+static inline void core_sleep(void)
+{
+    sleep_core(CURRENT_CORE);
+    enable_irq();
+}
+
+/* Shared single-core build debugging version */
+void core_wake(void)
+{
+    /* No wakey - core already wakey (because this is it) */
+}
+#else /* NUM_CORES > 1 */
+/** Model-generic PP dual-core code **/
 extern uintptr_t cpu_idlestackbegin[];
 extern uintptr_t cpu_idlestackend[];
 extern uintptr_t cop_idlestackbegin[];
@@ -37,23 +51,7 @@ static uintptr_t * const idle_stacks[NUM_CORES] =
     [COP] = cop_idlestackbegin
 };
 
-#if CONFIG_CPU == PP5002
-/* Bytes to emulate the PP502x mailbox bits */
-struct core_semaphores
-{
-    volatile uint8_t intend_wake;  /* 00h */
-    volatile uint8_t stay_awake;   /* 01h */
-    volatile uint8_t intend_sleep; /* 02h */
-    volatile uint8_t unused;       /* 03h */
-};
-
-static struct core_semaphores core_semaphores[NUM_CORES] IBSS_ATTR;
-#endif /* CONFIG_CPU == PP5002 */
-
-#endif /* NUM_CORES */
-
-#if CONFIG_CORELOCK == SW_CORELOCK
-/* Software core locks using Peterson's mutual exclusion algorithm */
+/* Core locks using Peterson's mutual exclusion algorithm */
 
 /*---------------------------------------------------------------------------
  * Initialize the corelock structure.
@@ -69,8 +67,7 @@ void corelock_init(struct corelock *cl)
  * Wait for the corelock to become free and acquire it when it does.
  *---------------------------------------------------------------------------
  */
-void corelock_lock(struct corelock *cl) __attribute__((naked));
-void corelock_lock(struct corelock *cl)
+void __attribute__((naked)) corelock_lock(struct corelock *cl)
 {
     /* Relies on the fact that core IDs are complementary bitmasks (0x55,0xaa) */
     asm volatile (
@@ -96,8 +93,7 @@ void corelock_lock(struct corelock *cl)
  * Try to aquire the corelock. If free, caller gets it, otherwise return 0.
  *---------------------------------------------------------------------------
  */
-int corelock_try_lock(struct corelock *cl) __attribute__((naked));
-int corelock_try_lock(struct corelock *cl)
+int __attribute__((naked)) corelock_try_lock(struct corelock *cl)
 {
     /* Relies on the fact that core IDs are complementary bitmasks (0x55,0xaa) */
     asm volatile (
@@ -125,8 +121,7 @@ int corelock_try_lock(struct corelock *cl)
  * Release ownership of the corelock
  *---------------------------------------------------------------------------
  */
-void corelock_unlock(struct corelock *cl) __attribute__((naked));
-void corelock_unlock(struct corelock *cl)
+void __attribute__((naked)) corelock_unlock(struct corelock *cl)
 {
     asm volatile (
         "mov    r1, %0               \n" /* r1 = PROCESSOR_ID */
@@ -138,11 +133,9 @@ void corelock_unlock(struct corelock *cl)
     );
     (void)cl;
 }
+
 #else /* C versions for reference */
-/*---------------------------------------------------------------------------
- * Wait for the corelock to become free and aquire it when it does.
- *---------------------------------------------------------------------------
- */
+
 void corelock_lock(struct corelock *cl)
 {
     const unsigned int core = CURRENT_CORE;
@@ -158,10 +151,6 @@ void corelock_lock(struct corelock *cl)
     }
 }
 
-/*---------------------------------------------------------------------------
- * Try to aquire the corelock. If free, caller gets it, otherwise return 0.
- *---------------------------------------------------------------------------
- */
 int corelock_try_lock(struct corelock *cl)
 {
     const unsigned int core = CURRENT_CORE;
@@ -179,85 +168,141 @@ int corelock_try_lock(struct corelock *cl)
     return 0;
 }
 
-/*---------------------------------------------------------------------------
- * Release ownership of the corelock
- *---------------------------------------------------------------------------
- */
 void corelock_unlock(struct corelock *cl)
 {
     cl->myl[CURRENT_CORE] = 0;
 }
 #endif /* ASM / C selection */
 
-#endif /* CONFIG_CORELOCK == SW_CORELOCK */
-
 /*---------------------------------------------------------------------------
- * Put core in a power-saving state if waking list wasn't repopulated and if
- * no other core requested a wakeup for it to perform a task.
+ * Do any device-specific inits for the threads and synchronize the kernel
+ * initializations.
  *---------------------------------------------------------------------------
  */
-#ifdef CPU_PP502x
-#if NUM_CORES == 1
-static inline void core_sleep(void)
+static void INIT_ATTR core_thread_init(unsigned int core) 
 {
-    sleep_core(CURRENT_CORE);
-    enable_irq();
+    if (core == CPU)
+    {
+        /* Wake up coprocessor and let it initialize kernel and threads */
+#ifdef CPU_PP502x
+        MBX_MSG_CLR = 0x3f;
+#endif
+        wake_core(COP);
+        /* Sleep until COP has finished */
+        sleep_core(CPU);
+    }
+    else
+    {
+        /* Wake the CPU and return */
+        wake_core(CPU);
+    }
 }
-#else
-static inline void core_sleep(unsigned int core)
+
+/*---------------------------------------------------------------------------
+ * Switches to a stack that always resides in the Rockbox core.
+ *
+ * Needed when a thread suicides on a core other than the main CPU since the
+ * stack used when idling is the stack of the last thread to run. This stack
+ * may not reside in the core firmware in which case the core will continue
+ * to use a stack from an unloaded module until another thread runs on it.
+ *---------------------------------------------------------------------------
+ */
+static inline void switch_to_idle_stack(const unsigned int core)
 {
-#if 1
     asm volatile (
-        "mov    r0, #4                     \n" /* r0 = 0x4 << core */
-        "mov    r0, r0, lsl %[c]           \n"
-        "str    r0, [%[mbx], #4]           \n" /* signal intent to sleep */
-        "ldr    r1, [%[mbx], #0]           \n" /* && !(MBX_MSG_STAT & (0x10<<core)) ? */
-        "tst    r1, r0, lsl #2             \n"  
-        "moveq  r1, #0x80000000            \n" /* Then sleep */
-        "streq  r1, [%[ctl], %[c], lsl #2] \n"
-        "moveq  r1, #0                     \n" /* Clear control reg */
-        "streq  r1, [%[ctl], %[c], lsl #2] \n"
-        "orr    r1, r0, r0, lsl #2         \n" /* Signal intent to wake - clear wake flag */
-        "str    r1, [%[mbx], #8]           \n"
-    "1:                                    \n" /* Wait for wake procedure to finish */
-        "ldr    r1, [%[mbx], #0]           \n"
-        "tst    r1, r0, lsr #2             \n"
-        "bne    1b                         \n"
-        :
-        :  [ctl]"r"(&CPU_CTL), [mbx]"r"(MBX_BASE), [c]"r"(core)
-        : "r0", "r1");
-#else /* C version for reference */
-    /* Signal intent to sleep */
-    MBX_MSG_SET = 0x4 << core;
+        "str  sp, [%0] \n" /* save original stack pointer on idle stack */
+        "mov  sp, %0   \n" /* switch stacks */
+        : : "r"(&idle_stacks[core][IDLE_STACK_WORDS-1]));
+    (void)core;
+}
 
-    /* Something waking or other processor intends to wake us? */
-    if ((MBX_MSG_STAT & (0x10 << core)) == 0)
-    {
-        sleep_core(core);
-        wake_core(core);
-    }
+/*---------------------------------------------------------------------------
+ * Perform core switch steps that need to take place inside switch_thread.
+ *
+ * These steps must take place while before changing the processor and after
+ * having entered switch_thread since switch_thread may not do a normal return
+ * because the stack being used for anything the compiler saved will not belong
+ * to the thread's destination core and it may have been recycled for other
+ * purposes by the time a normal context load has taken place. switch_thread
+ * will also clobber anything stashed in the thread's context or stored in the
+ * nonvolatile registers if it is saved there before the call since the
+ * compiler's order of operations cannot be known for certain.
+ */
+static void core_switch_blk_op(unsigned int core, struct thread_entry *thread)
+{
+    /* Flush our data to ram */
+    cpucache_flush();
+    /* Stash thread in r4 slot */
+    thread->context.r[0] = (uint32_t)thread;
+    /* Stash restart address in r5 slot */
+    thread->context.r[1] = thread->context.start;
+    /* Save sp in context.sp while still running on old core */
+    thread->context.sp = idle_stacks[core][IDLE_STACK_WORDS-1];
+}
 
-    /* Signal wake - clear wake flag */
-    MBX_MSG_CLR = 0x14 << core;
+/*---------------------------------------------------------------------------
+ * Machine-specific helper function for switching the processor a thread is
+ * running on. Basically, the thread suicides on the departing core and is
+ * reborn on the destination. Were it not for gcc's ill-behavior regarding
+ * naked functions written in C where it actually clobbers non-volatile
+ * registers before the intended prologue code, this would all be much
+ * simpler.  Generic setup is done in switch_core itself.
+ */
 
-    /* Wait for other processor to finish wake procedure */
-    while (MBX_MSG_STAT & (0x1 << core));
-#endif /* ASM/C selection */
-    enable_irq();
-}
-#endif /* NUM_CORES */
-#elif CONFIG_CPU == PP5002
-#if NUM_CORES == 1
-static inline void core_sleep(void)
+/*---------------------------------------------------------------------------
+ * This actually performs the core switch.
+ */
+static void __attribute__((naked))
+    switch_thread_core(unsigned int core, struct thread_entry *thread)
 {
-    sleep_core(CURRENT_CORE);
-    enable_irq();
+    /* Pure asm for this because compiler behavior isn't sufficiently predictable.
+     * Stack access also isn't permitted until restoring the original stack and
+     * context. */
+    asm volatile (
+        "stmfd  sp!, { r4-r11, lr }      \n" /* Stack all non-volatile context on current core */
+        "ldr    r2, =idle_stacks         \n" /* r2 = &idle_stacks[core][IDLE_STACK_WORDS] */
+        "ldr    r2, [r2, r0, lsl #2]     \n"
+        "add    r2, r2, %0*4             \n"
+        "stmfd  r2!, { sp }              \n" /* save original stack pointer on idle stack */
+        "mov    sp, r2                   \n" /* switch stacks */
+        "adr    r2, 1f                   \n" /* r2 = new core restart address */
+        "str    r2, [r1, #40]            \n" /* thread->context.start = r2 */
+        "ldr    pc, =switch_thread       \n" /* r0 = thread after call - see load_context */
+    "1:                                  \n"
+        "ldr    sp, [r0, #32]            \n" /* Reload original sp from context structure */
+        "mov    r1, #0                   \n" /* Clear start address */
+        "str    r1, [r0, #40]            \n"
+        "ldr    r0, =cpucache_invalidate \n" /* Invalidate new core's cache */
+        "mov    lr, pc                   \n"
+        "bx     r0                       \n"
+        "ldmfd  sp!, { r4-r11, pc }      \n" /* Restore non-volatile context to new core and return */
+        : : "i"(IDLE_STACK_WORDS)
+    );
+    (void)core; (void)thread;
 }
-#else
-/* PP5002 has no mailboxes - emulate using bytes */
+
+/** PP-model-specific dual-core code **/
+
+#if CONFIG_CPU == PP5002
+/* PP5002 has no mailboxes - Bytes to emulate the PP502x mailbox bits */
+struct core_semaphores
+{
+    volatile uint8_t intend_wake;  /* 00h */
+    volatile uint8_t stay_awake;   /* 01h */
+    volatile uint8_t intend_sleep; /* 02h */
+    volatile uint8_t unused;       /* 03h */
+};
+
+static struct core_semaphores core_semaphores[NUM_CORES] IBSS_ATTR;
+
+#if 1 /* Select ASM */
+/*---------------------------------------------------------------------------
+ * Put core in a power-saving state if waking list wasn't repopulated and if
+ * no other core requested a wakeup for it to perform a task.
+ *---------------------------------------------------------------------------
+ */
 static inline void core_sleep(unsigned int core)
 {
-#if 1
     asm volatile (
         "mov    r0, #1                     \n" /* Signal intent to sleep */
         "strb   r0, [%[sem], #2]           \n"
@@ -288,7 +333,50 @@ static inline void core_sleep(unsigned int core)
           [ctl]"r"(&CPU_CTL)
         : "r0"
         );
+    enable_irq();
+}
+
+/*---------------------------------------------------------------------------
+ * Wake another processor core that is sleeping or prevent it from doing so
+ * if it was already destined. FIQ, IRQ should be disabled before calling.
+ *---------------------------------------------------------------------------
+ */
+void core_wake(unsigned int othercore)
+{
+    /* avoid r0 since that contains othercore */
+    asm volatile (
+        "mrs    r3, cpsr                \n" /* Disable IRQ */
+        "orr    r1, r3, #0x80           \n"
+        "msr    cpsr_c, r1              \n"
+        "mov    r1, #1                  \n" /* Signal intent to wake other core */
+        "orr    r1, r1, r1, lsl #8      \n" /* and set stay_awake */
+        "strh   r1, [%[sem], #0]        \n"
+        "mov    r2, #0x8000             \n"
+    "1:                                 \n" /* If it intends to sleep, let it first */
+        "ldrb   r1, [%[sem], #2]        \n" /* intend_sleep != 0 ? */
+        "cmp    r1, #1                  \n"
+        "ldr    r1, [%[st]]             \n" /* && not sleeping ? */
+        "tsteq  r1, r2, lsr %[oc]       \n"
+        "beq    1b                      \n" /* Wait for sleep or wake */
+        "tst    r1, r2, lsr %[oc]       \n"
+        "ldrne  r2, =0xcf004054         \n" /* If sleeping, wake it */
+        "movne  r1, #0xce               \n"
+        "strne  r1, [r2, %[oc], lsl #2] \n"
+        "mov    r1, #0                  \n" /* Done with wake procedure */
+        "strb   r1, [%[sem], #0]        \n"
+        "msr    cpsr_c, r3              \n" /* Restore IRQ */
+        :
+        : [sem]"r"(&core_semaphores[othercore]),
+          [st]"r"(&PROC_STAT),
+          [oc]"r"(othercore)
+        : "r1", "r2", "r3"
+    );
+}
+
 #else /* C version for reference */
+
+static inline void core_sleep(unsigned int core)
+{
     /* Signal intent to sleep */
     core_semaphores[core].intend_sleep = 1;
 
@@ -306,27 +394,71 @@ static inline void core_sleep(unsigned int core)
     while (core_semaphores[core].intend_wake != 0);
 
     /* Enable IRQ */
-#endif /* ASM/C selection */
     enable_irq();
 }
-#endif /* NUM_CORES */
-#endif /* PP CPU type */
 
+void core_wake(unsigned int othercore)
+{
+    /* Disable interrupts - avoid reentrancy from the tick */
+    int oldlevel = disable_irq_save();
+
+    /* Signal intent to wake other processor - set stay awake */
+    core_semaphores[othercore].intend_wake = 1;
+    core_semaphores[othercore].stay_awake = 1;
+
+    /* If it intends to sleep, wait until it does or aborts */
+    while (core_semaphores[othercore].intend_sleep != 0 &&
+           (PROC_STAT & PROC_SLEEPING(othercore)) == 0);
+
+    /* If sleeping, wake it up */
+    if (PROC_STAT & PROC_SLEEPING(othercore))
+        wake_core(othercore);
+
+    /* Done with wake procedure */
+    core_semaphores[othercore].intend_wake = 0;
+    restore_irq(oldlevel);
+}
+#endif  /* ASM/C selection */
+
+#elif defined (CPU_PP502x)
+
+#if 1 /* Select ASM */
 /*---------------------------------------------------------------------------
- * Wake another processor core that is sleeping or prevent it from doing so
- * if it was already destined. FIQ, IRQ should be disabled before calling.
+ * Put core in a power-saving state if waking list wasn't repopulated and if
+ * no other core requested a wakeup for it to perform a task.
  *---------------------------------------------------------------------------
  */
-#if NUM_CORES == 1
-/* Shared single-core build debugging version */
-void core_wake(void)
+static inline void core_sleep(unsigned int core)
 {
-    /* No wakey - core already wakey */
+    asm volatile (
+        "mov    r0, #4                     \n" /* r0 = 0x4 << core */
+        "mov    r0, r0, lsl %[c]           \n"
+        "str    r0, [%[mbx], #4]           \n" /* signal intent to sleep */
+        "ldr    r1, [%[mbx], #0]           \n" /* && !(MBX_MSG_STAT & (0x10<<core)) ? */
+        "tst    r1, r0, lsl #2             \n"  
+        "moveq  r1, #0x80000000            \n" /* Then sleep */
+        "streq  r1, [%[ctl], %[c], lsl #2] \n"
+        "moveq  r1, #0                     \n" /* Clear control reg */
+        "streq  r1, [%[ctl], %[c], lsl #2] \n"
+        "orr    r1, r0, r0, lsl #2         \n" /* Signal intent to wake - clear wake flag */
+        "str    r1, [%[mbx], #8]           \n"
+    "1:                                    \n" /* Wait for wake procedure to finish */
+        "ldr    r1, [%[mbx], #0]           \n"
+        "tst    r1, r0, lsr #2             \n"
+        "bne    1b                         \n"
+        :
+        :  [ctl]"r"(&CPU_CTL), [mbx]"r"(MBX_BASE), [c]"r"(core)
+        : "r0", "r1");
+    enable_irq();
 }
-#elif defined (CPU_PP502x)
+
+/*---------------------------------------------------------------------------
+ * Wake another processor core that is sleeping or prevent it from doing so
+ * if it was already destined. FIQ, IRQ should be disabled before calling.
+ *---------------------------------------------------------------------------
+ */
 void core_wake(unsigned int othercore)
 {
-#if 1
     /* avoid r0 since that contains othercore */
     asm volatile (
         "mrs    r3, cpsr                    \n" /* Disable IRQ */
@@ -352,190 +484,58 @@ void core_wake(unsigned int othercore)
         : [ctl]"r"(&PROC_CTL(CPU)), [mbx]"r"(MBX_BASE),
           [oc]"r"(othercore)
         : "r1", "r2", "r3");
+}
+
 #else /* C version for reference */
-    /* Disable interrupts - avoid reentrancy from the tick */
-    int oldlevel = disable_irq_save();
 
-    /* Signal intent to wake other processor - set stay awake */
-    MBX_MSG_SET = 0x11 << othercore;
+static inline void core_sleep(unsigned int core)
+{
+    /* Signal intent to sleep */
+    MBX_MSG_SET = 0x4 << core;
 
-    /* If it intends to sleep, wait until it does or aborts */
-    while ((MBX_MSG_STAT & (0x4 << othercore)) != 0 &&
-           (PROC_CTL(othercore) & PROC_SLEEP) == 0);
+    /* Something waking or other processor intends to wake us? */
+    if ((MBX_MSG_STAT & (0x10 << core)) == 0)
+    {
+        sleep_core(core);
+        wake_core(core);
+    }
 
-    /* If sleeping, wake it up */
-    if (PROC_CTL(othercore) & PROC_SLEEP)
-        PROC_CTL(othercore) = 0;
+    /* Signal wake - clear wake flag */
+    MBX_MSG_CLR = 0x14 << core;
 
-    /* Done with wake procedure */
-    MBX_MSG_CLR = 0x1 << othercore;
-    restore_irq(oldlevel);
-#endif /* ASM/C selection */
+    /* Wait for other processor to finish wake procedure */
+    while (MBX_MSG_STAT & (0x1 << core));
+    enable_irq();
 }
-#elif CONFIG_CPU == PP5002
-/* PP5002 has no mailboxes - emulate using bytes */
+
 void core_wake(unsigned int othercore)
 {
-#if 1
-    /* avoid r0 since that contains othercore */
-    asm volatile (
-        "mrs    r3, cpsr                \n" /* Disable IRQ */
-        "orr    r1, r3, #0x80           \n"
-        "msr    cpsr_c, r1              \n"
-        "mov    r1, #1                  \n" /* Signal intent to wake other core */
-        "orr    r1, r1, r1, lsl #8      \n" /* and set stay_awake */
-        "strh   r1, [%[sem], #0]        \n"
-        "mov    r2, #0x8000             \n"
-    "1:                                 \n" /* If it intends to sleep, let it first */
-        "ldrb   r1, [%[sem], #2]        \n" /* intend_sleep != 0 ? */
-        "cmp    r1, #1                  \n"
-        "ldr    r1, [%[st]]             \n" /* && not sleeping ? */
-        "tsteq  r1, r2, lsr %[oc]       \n"
-        "beq    1b                      \n" /* Wait for sleep or wake */
-        "tst    r1, r2, lsr %[oc]       \n"
-        "ldrne  r2, =0xcf004054         \n" /* If sleeping, wake it */
-        "movne  r1, #0xce               \n"
-        "strne  r1, [r2, %[oc], lsl #2] \n"
-        "mov    r1, #0                  \n" /* Done with wake procedure */
-        "strb   r1, [%[sem], #0]        \n"
-        "msr    cpsr_c, r3              \n" /* Restore IRQ */
-        :
-        : [sem]"r"(&core_semaphores[othercore]),
-          [st]"r"(&PROC_STAT),
-          [oc]"r"(othercore)
-        : "r1", "r2", "r3"
-    );
-#else /* C version for reference */
     /* Disable interrupts - avoid reentrancy from the tick */
     int oldlevel = disable_irq_save();
 
     /* Signal intent to wake other processor - set stay awake */
-    core_semaphores[othercore].intend_wake = 1;
-    core_semaphores[othercore].stay_awake = 1;
+    MBX_MSG_SET = 0x11 << othercore;
 
     /* If it intends to sleep, wait until it does or aborts */
-    while (core_semaphores[othercore].intend_sleep != 0 &&
-           (PROC_STAT & PROC_SLEEPING(othercore)) == 0);
+    while ((MBX_MSG_STAT & (0x4 << othercore)) != 0 &&
+           (PROC_CTL(othercore) & PROC_SLEEP) == 0);
 
     /* If sleeping, wake it up */
-    if (PROC_STAT & PROC_SLEEPING(othercore))
-        wake_core(othercore);
+    if (PROC_CTL(othercore) & PROC_SLEEP)
+        PROC_CTL(othercore) = 0;
 
     /* Done with wake procedure */
-    core_semaphores[othercore].intend_wake = 0;
+    MBX_MSG_CLR = 0x1 << othercore;
     restore_irq(oldlevel);
-#endif  /* ASM/C selection */
-}
-#endif /* CPU type */
-
-#if NUM_CORES > 1
-/*---------------------------------------------------------------------------
- * Switches to a stack that always resides in the Rockbox core.
- *
- * Needed when a thread suicides on a core other than the main CPU since the
- * stack used when idling is the stack of the last thread to run. This stack
- * may not reside in the core firmware in which case the core will continue
- * to use a stack from an unloaded module until another thread runs on it.
- *---------------------------------------------------------------------------
- */
-static inline void switch_to_idle_stack(const unsigned int core)
-{
-    asm volatile (
-        "str  sp, [%0] \n" /* save original stack pointer on idle stack */
-        "mov  sp, %0   \n" /* switch stacks */
-        : : "r"(&idle_stacks[core][IDLE_STACK_WORDS-1]));
-    (void)core;
-}
-
-/*---------------------------------------------------------------------------
- * Perform core switch steps that need to take place inside switch_thread.
- *
- * These steps must take place while before changing the processor and after
- * having entered switch_thread since switch_thread may not do a normal return
- * because the stack being used for anything the compiler saved will not belong
- * to the thread's destination core and it may have been recycled for other
- * purposes by the time a normal context load has taken place. switch_thread
- * will also clobber anything stashed in the thread's context or stored in the
- * nonvolatile registers if it is saved there before the call since the
- * compiler's order of operations cannot be known for certain.
- */
-static void core_switch_blk_op(unsigned int core, struct thread_entry *thread)
-{
-    /* Flush our data to ram */
-    cpucache_flush();
-    /* Stash thread in r4 slot */
-    thread->context.r[0] = (uint32_t)thread;
-    /* Stash restart address in r5 slot */
-    thread->context.r[1] = thread->context.start;
-    /* Save sp in context.sp while still running on old core */
-    thread->context.sp = idle_stacks[core][IDLE_STACK_WORDS-1];
 }
+#endif /* ASM/C selection */
 
-/*---------------------------------------------------------------------------
- * Machine-specific helper function for switching the processor a thread is
- * running on. Basically, the thread suicides on the departing core and is
- * reborn on the destination. Were it not for gcc's ill-behavior regarding
- * naked functions written in C where it actually clobbers non-volatile
- * registers before the intended prologue code, this would all be much
- * simpler.  Generic setup is done in switch_core itself.
- */
+#endif /* CPU_PPxxxx */
 
-/*---------------------------------------------------------------------------
- * This actually performs the core switch.
- */
-static void __attribute__((naked))
-    switch_thread_core(unsigned int core, struct thread_entry *thread)
+/* Keep constant pool in range of inline ASM */
+static void __attribute__((naked, used)) dump_ltorg(void)
 {
-    /* Pure asm for this because compiler behavior isn't sufficiently predictable.
-     * Stack access also isn't permitted until restoring the original stack and
-     * context. */
-    asm volatile (
-        "stmfd  sp!, { r4-r11, lr }      \n" /* Stack all non-volatile context on current core */
-        "ldr    r2, =idle_stacks         \n" /* r2 = &idle_stacks[core][IDLE_STACK_WORDS] */
-        "ldr    r2, [r2, r0, lsl #2]     \n"
-        "add    r2, r2, %0*4             \n"
-        "stmfd  r2!, { sp }              \n" /* save original stack pointer on idle stack */
-        "mov    sp, r2                   \n" /* switch stacks */
-        "adr    r2, 1f                   \n" /* r2 = new core restart address */
-        "str    r2, [r1, #40]            \n" /* thread->context.start = r2 */
-        "ldr    pc, =switch_thread       \n" /* r0 = thread after call - see load_context */
-    "1:                                  \n"
-        "ldr    sp, [r0, #32]            \n" /* Reload original sp from context structure */
-        "mov    r1, #0                   \n" /* Clear start address */
-        "str    r1, [r0, #40]            \n"
-        "ldr    r0, =cpucache_invalidate \n" /* Invalidate new core's cache */
-        "mov    lr, pc                   \n"
-        "bx     r0                       \n"
-        "ldmfd  sp!, { r4-r11, pc }      \n" /* Restore non-volatile context to new core and return */
-        ".ltorg                          \n" /* Dump constant pool */
-        : : "i"(IDLE_STACK_WORDS)
-    );
-    (void)core; (void)thread;
+    asm volatile (".ltorg");
 }
 
-/*---------------------------------------------------------------------------
- * Do any device-specific inits for the threads and synchronize the kernel
- * initializations.
- *---------------------------------------------------------------------------
- */
-static void core_thread_init(unsigned int core) INIT_ATTR;
-static void core_thread_init(unsigned int core) 
-{
-    if (core == CPU)
-    {
-        /* Wake up coprocessor and let it initialize kernel and threads */
-#ifdef CPU_PP502x
-        MBX_MSG_CLR = 0x3f;
-#endif
-        wake_core(COP);
-        /* Sleep until COP has finished */
-        sleep_core(CPU);
-    }
-    else
-    {
-        /* Wake the CPU and return */
-        wake_core(CPU);
-    }
-}
 #endif /* NUM_CORES */
-