From 15b7321fe1345a9efd296fe135f19e1da45f0e7c Mon Sep 17 00:00:00 2001
From: Michael Sevakis <jethead71@rockbox.org>
Date: Tue, 2 Oct 2007 07:48:50 +0000
Subject: Split wm8731 and wm8721 drivers. wm8721 can be much simpler since it
 has no recording. Provide reg bit definitions for those. Add some temporary
 hacks pcm-pp to prevent crashing when playing after recording. Make playback
 start after recording on pp5020. Get wm8731 to monitor recording but plans
 are to do that digitally.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@14948 a1c6a512-1295-4272-9138-f99709370657
---
 firmware/target/arm/pcm-pp.c     | 49 +++++++++++++++++++++++++++++-----------
 firmware/target/arm/wmcodec-pp.c | 10 ++++++--
 2 files changed, 44 insertions(+), 15 deletions(-)

(limited to 'firmware/target/arm')

diff --git a/firmware/target/arm/pcm-pp.c b/firmware/target/arm/pcm-pp.c
index 5ac15fe00c..42b9aeb6cb 100644
--- a/firmware/target/arm/pcm-pp.c
+++ b/firmware/target/arm/pcm-pp.c
@@ -168,7 +168,8 @@ void fiq(void)
 
     do {
         while (p_size) {
-            if (FIFO_FREE_COUNT < 2) {
+            //if (FIFO_FREE_COUNT < 2) {
+            if (((IISFIFO_CFG & (0x1f << 16)) >> 16) < 2) {
                 /* Enable interrupt */
 #ifdef CPU_PP502x
                 IISCONFIG |= (1 << 1);
@@ -219,8 +220,10 @@ void pcm_play_dma_start(const void *addr, size_t size)
     set_fiq_handler(fiq);
     enable_fiq();
 
+#if CONFIG_CPU == PP5020
+    /* Do nothing */
+#elif defined(CPU_PP502x)
     /* Enable playback FIFO */
-#ifdef CPU_PP502x
     IISCONFIG |= (1 << 29);
 #elif CONFIG_CPU == PP5002
     IISCONFIG |= 0x4;
@@ -257,11 +260,13 @@ void pcm_play_dma_stop(void)
     if (!audio_status())
         pcm_paused = false;
 
-#ifdef CPU_PP502x
+#if CONFIG_CPU == PP5020
+    /* Disable TX interrupt */
+    IISCONFIG &= ~(1 << 1);
+#elif defined(CPU_PP502x)
     /* Disable playback FIFO and interrupt */
     IISCONFIG &= ~((1 << 29) | (1 << 1));
 #elif CONFIG_CPU == PP5002
-
     /* Disable playback FIFO */
     IISCONFIG &= ~0x4;
 
@@ -274,7 +279,10 @@ void pcm_play_dma_stop(void)
 
 void pcm_play_pause_pause(void)
 {
-#ifdef CPU_PP502x
+#if CONFIG_CPU == PP5020
+    /* Disable TX interrupt */
+    IISCONFIG &= ~(1 << 1);
+#elif defined(CPU_PP502x)
     /* Disable playback FIFO and interrupt */
     IISCONFIG &= ~((1 << 29) | (1 << 1));
 #elif CONFIG_CPU == PP5002
@@ -293,8 +301,10 @@ void pcm_play_pause_unpause(void)
     set_fiq_handler(fiq);
     enable_fiq();
 
+#if CONFIG_CPU == PP5020
+    /* Do nothing */
+#elif defined(CPU_PP502x)
     /* Enable playback FIFO */
-#ifdef CPU_PP502x
     IISCONFIG |= (1 << 29);
 #elif CONFIG_CPU == PP5002
     IISCONFIG |= 0x4;
@@ -344,14 +354,20 @@ void pcm_init(void)
     /* Initialize default register values. */
     audiohw_init();
 
+#ifndef HAVE_WM8731
     /* Power on */
     audiohw_enable_output(true);
-
     /* Unmute the master channel (DAC should be at zero point now). */
     audiohw_mute(false);
+#endif
 
     /* Call pcm_play_dma_stop to initialize everything. */
     pcm_play_dma_stop();
+
+#if CONFIG_CPU == PP5020
+    /* This processor doesn't like this disabled */
+    IISCONFIG |= (1 << 29);
+#endif
 }
 
 void pcm_postinit(void)
@@ -438,12 +454,15 @@ fiq_record_exit:
 #else
 static short peak_l, peak_r IBSS_ATTR;
 
-void fiq_record(void) ICODE_ATTR __attribute__ ((interrupt ("FIQ")));
+/* Temporary to stop playback crashing after record */
+void fiq_record(void) ICODE_ATTR __attribute__((naked));
 void fiq_record(void)
 {
-    short value;
-    pcm_more_callback_type2 more_ready;
-    int status = 0;
+    asm volatile ("stmfd sp!, {r0-r7, r11, ip, lr} \n");   /* Store context */
+
+    register short value;
+    register pcm_more_callback_type2 more_ready;
+    register int status = 0;
 
     /* Clear interrupt */
 #ifdef CPU_PP502x
@@ -460,7 +479,7 @@ void fiq_record(void)
 #elif CONFIG_CPU == PP5002
             /* TODO */
 #endif
-            return;
+            goto fiq_record_exit;
         }
 
         value = (unsigned short)(IISFIFO_RD >> 16);
@@ -486,10 +505,14 @@ void fiq_record(void)
     more_ready = pcm_callback_more_ready;
 
     if (more_ready != NULL && more_ready(status) >= 0)
-        return;
+        goto fiq_record_exit;
 
     /* Finished recording */
     pcm_rec_dma_stop();
+
+fiq_record_exit:
+    asm volatile("ldmfd sp!, {r0-r7, r11, ip, lr} \n"   /* Restore context */
+                 "subs  pc, lr, #4                \n"); /* Return from FIQ */
 }
 
 #endif /* HAVE_AS3514 */
diff --git a/firmware/target/arm/wmcodec-pp.c b/firmware/target/arm/wmcodec-pp.c
index 455e3e1087..a2c83f33b3 100644
--- a/firmware/target/arm/wmcodec-pp.c
+++ b/firmware/target/arm/wmcodec-pp.c
@@ -25,6 +25,7 @@
  ****************************************************************************/
 
 #include "system.h"
+#include "audiohw.h"
 #include "i2c-pp.h"
 
 #if defined(IRIVER_H10) || defined(IRIVER_H10_5GB)
@@ -39,7 +40,7 @@
 /*
  * Initialise the PP I2C and I2S.
  */
-int audiohw_init(void) {
+void audiohw_init(void) {
     /* reset I2C */
     i2c_init();
 
@@ -95,12 +96,17 @@ int audiohw_init(void) {
 #endif /* IPOD_1G2G/3G */
 #endif
     
-    return 0;
+#ifdef HAVE_WM8731
+    audiohw_preinit();
+#endif
+   
 }
 
+#ifndef HAVE_WM8731
 void audiohw_postinit(void)
 {
 }
+#endif
 
 void wmcodec_write(int reg, int data)
 {
-- 
cgit v1.2.3