Add register bit defines for as3514 and clean stuff up. Reduce poppiness at startup and shutdown (and even powerup for e200). Really, I can't honestly say it will help anything but an e200v1 but I'm sick of the noises. ;)

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

5 files changed, 287 insertions, 103 deletions

diff --git a/firmware/drivers/audio/as3514.c b/firmware/drivers/audio/as3514.c
index db4cb0699f..3985ea1177 100644
--- a/firmware/drivers/audio/as3514.c
+++ b/firmware/drivers/audio/as3514.c
@@ -49,11 +49,12 @@ const struct sound_settings_info audiohw_settings[] = {
 /* Shadow registers */
 static struct as3514_info
 {
-    int          vol_r;       /* Cached volume level (R) */
-    int          vol_l;       /* Cached volume level (L) */
-    unsigned int regs[0x1e];  /* last audio register: PLLMODE 0x1d */
+    int     vol_r;       /* Cached volume level (R) */
+    int     vol_l;       /* Cached volume level (L) */
+    uint8_t regs[AS3514_NUM_AUDIO_REGS]; /* 8-bit registers */
 } as3514;
 
+/* In order to keep track of source for combining volume ranges */
 enum
 {
     SOURCE_DAC = 0,
@@ -87,14 +88,20 @@ static void as3514_write(unsigned int reg, unsigned int value)
 }
 
 /* Helpers to set/clear bits */
-static void as3514_write_or(unsigned int reg, unsigned int bits)
+static void as3514_set(unsigned int reg, unsigned int bits)
 {
     as3514_write(reg, as3514.regs[reg] | bits);
 }
 
-static void as3514_write_and(unsigned int reg, unsigned int bits)
+static void as3514_clear(unsigned int reg, unsigned int bits)
 {
-    as3514_write(reg, as3514.regs[reg] & bits);
+    as3514_write(reg, as3514.regs[reg] & ~bits);
+}
+
+static void as3514_write_masked(unsigned int reg, unsigned int bits,
+                                unsigned int mask)
+{
+    as3514_write(reg, (as3514.regs[reg] & ~mask) | (bits & mask));
 }
 
 /* convert tenth of dB volume to master volume register value */
@@ -139,63 +146,66 @@ void audiohw_preinit(void)
 {
     unsigned int i;
 
+    /* read all reg values */
+    for (i = 0; i < ARRAYLEN(as3514.regs); i++)
+    {
+        as3514.regs[i] = ascodec_read(i);
+    }
+
     /* Set ADC off, mixer on, DAC on, line out off, line in off, mic off */
 
     /* Turn on SUM, DAC */
-    as3514_write(AS3514_AUDIOSET1, (1 << 6) | (1 << 5));
+    as3514_write(AS3514_AUDIOSET1, AUDIOSET1_DAC_on | AUDIOSET1_SUM_on);
 
     /* Set BIAS on, DITH on, AGC on, IBR_DAC max, LSP_LP on, IBR_LSP min */
-    as3514_write(AS3514_AUDIOSET2, (1 << 2) | (3 << 0));
+    as3514_write(AS3514_AUDIOSET2,
+                 AUDIOSET2_IBR_DAC_0 | AUDIOSET2_LSP_LP |
+                 AUDIOSET2_IBR_LSP_50);
 
-    /* Set HPCM off, ZCU off*/
-    as3514_write(AS3514_AUDIOSET3, (1 << 2) | (1 << 0));
+    /* Set HPCM on, ZCU on */
+    as3514_write(AS3514_AUDIOSET3, 0);
 
     /* Mute and disable speaker */
-    as3514_write(AS3514_LSP_OUT_R, 0);
-    as3514_write(AS3514_LSP_OUT_L, (1 << 7));
+    as3514_write(AS3514_LSP_OUT_R, LSP_OUT_R_SP_OVC_TO_256MS | 0x00);
+    as3514_write(AS3514_LSP_OUT_L, LSP_OUT_L_SP_MUTE | 0x00);
 
-    /* set vol and set headphone over-current to 0 */
-    as3514_write(AS3514_HPH_OUT_R, (0x3 << 6) | 0x16);
-    /* set default vol for headphone */
-    as3514_write(AS3514_HPH_OUT_L, 0x16);
+    /* Set headphone over-current to 0, Min volume */
+    as3514_write(AS3514_HPH_OUT_R,
+                 HPH_OUT_R_HP_OVC_TO_0MS | 0x00);
+
+    /* Headphone ON, MUTE, Min volume */
+    as3514_write(AS3514_HPH_OUT_L,
+                 HPH_OUT_L_HP_ON | HPH_OUT_L_HP_MUTE | 0x00);
 
     /* LRCK 24-48kHz */
-    as3514_write(AS3514_PLLMODE, 0x00);
+    as3514_write(AS3514_PLLMODE, PLLMODE_LRCK_24_48);
 
     /* DAC_Mute_off */
-    as3514_write_or(AS3514_DAC_L, (1 << 6));
+    as3514_set(AS3514_DAC_L, DAC_L_DAC_MUTE_off);
 
     /* M1_Sup_off */
-    as3514_write_or(AS3514_MIC1_L, (1 << 7));
+    as3514_set(AS3514_MIC1_L, MIC1_L_M1_SUP_off);
     /* M2_Sup_off */
-    as3514_write_or(AS3514_MIC2_L, (1 << 7));
-
-    /* read all reg values */
-    for (i = 0; i < ARRAYLEN(as3514.regs); i++)
-    {
-        as3514.regs[i] = ascodec_read(i);
-    }
+    as3514_set(AS3514_MIC2_L, MIC2_L_M2_SUP_off);
 }
 
-/* Silently enable / disable audio output */
-void audiohw_enable_output(bool enable)
+void audiohw_postinit(void)
 {
-    if (enable) {
-        /* reset the I2S controller into known state */
-        i2s_reset();
+    /* wait until outputs have stabilized */
+    sleep(HZ/4);
 
-        as3514_write_or(AS3514_HPH_OUT_L, (1 << 6)); /* power on */
-        audiohw_mute(0);
-    } else {
-        audiohw_mute(1);
-        as3514_write_and(AS3514_HPH_OUT_L, ~(1 << 6)); /* power off */
-    }
+    as3514_write(AS3514_AUDIOSET3, AUDIOSET3_HPCM_off);
+
+#ifdef CPU_PP
+    ascodec_supressor_on(false);
+#endif
+
+    audiohw_mute(false);
 }
 
 void audiohw_set_master_vol(int vol_l, int vol_r)
 {
-    unsigned int hph_r = as3514.regs[AS3514_HPH_OUT_R] & ~0x1f;
-    unsigned int hph_l = as3514.regs[AS3514_HPH_OUT_L] & ~0x1f;
+    unsigned int hph_r, hph_l;
     unsigned int mix_l, mix_r;
     unsigned int mix_reg_r, mix_reg_l;
 
@@ -211,45 +221,44 @@ void audiohw_set_master_vol(int vol_l, int vol_r)
         mix_reg_l = AS3514_DAC_L;
     }
 
-    mix_r = as3514.regs[mix_reg_r] & ~0x1f;    
-    mix_l = as3514.regs[mix_reg_l] & ~0x1f;    
-
-    /* we combine the mixer channel volume range with the headphone volume
-       range */
+    /* We combine the mixer channel volume range with the headphone volume
+       range - keep first stage as loud as possible */
     if (vol_r <= 0x16) {
-        mix_r |= vol_r;
-        /* hph_r - set 0 */
+        mix_r = vol_r;
+        hph_r = 0;
     } else {
-        mix_r |= 0x16;
-        hph_r += vol_r - 0x16;
+        mix_r = 0x16;
+        hph_r = vol_r - 0x16;
     }
 
     if (vol_l <= 0x16) {
-        mix_l |= vol_l;
-        /* hph_l - set 0 */
+        mix_l = vol_l;
+        hph_l = 0;
     } else {
-        mix_l |= 0x16;
-        hph_l += vol_l - 0x16;
+        mix_l = 0x16;
+        hph_l = vol_l - 0x16;
     }
 
-    as3514_write(mix_reg_r, mix_r);
-    as3514_write(mix_reg_l, mix_l);
-    as3514_write(AS3514_HPH_OUT_R, hph_r);
-    as3514_write(AS3514_HPH_OUT_L, hph_l);
+    as3514_write_masked(mix_reg_r, mix_r, AS3514_VOL_MASK);
+    as3514_write_masked(mix_reg_l, mix_l, AS3514_VOL_MASK);
+    as3514_write_masked(AS3514_HPH_OUT_R, hph_r, AS3514_VOL_MASK);
+    as3514_write_masked(AS3514_HPH_OUT_L, hph_l, AS3514_VOL_MASK);
 }
 
 void audiohw_set_lineout_vol(int vol_l, int vol_r)
 {
-    as3514_write(AS3514_LINE_OUT_R, vol_r);
-    as3514_write(AS3514_LINE_OUT_L, (1 << 6) | vol_l);
+    as3514_write_masked(AS3514_LINE_OUT_R, vol_r,
+                        AS3514_VOL_MASK);
+    as3514_write_masked(AS3514_LINE_OUT_L, vol_l,
+                        AS3514_VOL_MASK);
 }
 
 void audiohw_mute(bool mute)
 {
     if (mute) {
-        as3514_write_or(AS3514_HPH_OUT_L, (1 << 7));
+        as3514_set(AS3514_HPH_OUT_L, HPH_OUT_L_HP_MUTE);
     } else {
-        as3514_write_and(AS3514_HPH_OUT_L, ~(1 << 7));
+        as3514_clear(AS3514_HPH_OUT_L, HPH_OUT_L_HP_MUTE);
     }
 }
 
@@ -259,8 +268,19 @@ void audiohw_close(void)
     /* mute headphones */
     audiohw_mute(true);
 
+#ifdef CPU_PP
+    ascodec_supressor_on(true);
+#endif
+
+    /* turn on common */
+    as3514_clear(AS3514_AUDIOSET3, AUDIOSET3_HPCM_off);
+
     /* turn off everything */
+    as3514_clear(AS3514_HPH_OUT_L, HPH_OUT_L_HP_ON);
     as3514_write(AS3514_AUDIOSET1, 0x0);
+
+    /* Allow caps to discharge */
+    sleep(HZ/4);
 }
 
 void audiohw_set_sample_rate(int sampling_control)
@@ -278,29 +298,32 @@ void audiohw_enable_recording(bool source_mic)
         audiohw_set_master_vol(as3514.vol_l, as3514.vol_r);
 
         /* ADCmux = Stereo Microphone */
-        as3514_write_and(AS3514_ADC_R, ~(0x3 << 6));
+        as3514_write_masked(AS3514_ADC_R, ADC_R_ADCMUX_ST_MIC,
+                            ADC_R_ADCMUX);
+
         /* MIC1_on, LIN1_off */
-        as3514_write(AS3514_AUDIOSET1,
-            (as3514.regs[AS3514_AUDIOSET1] & ~(1 << 2)) | (1 << 0));
+        as3514_write_masked(AS3514_AUDIOSET1, AUDIOSET1_MIC1_on,
+                            AUDIOSET1_MIC1_on | AUDIOSET1_LIN1_on);
         /* M1_AGC_off */
-        as3514_write_and(AS3514_MIC1_R, ~(1 << 7));
+        as3514_clear(AS3514_MIC1_R, MIC1_R_M1_AGC_off);
     } else {
         source = SOURCE_LINE_IN1;
 
         audiohw_set_master_vol(as3514.vol_l, as3514.vol_r);
 
         /* ADCmux = Line_IN1 */
-        as3514_write(AS3514_ADC_R,
-            (as3514.regs[AS3514_ADC_R] & ~(0x3 << 6)) | (0x1 << 6));
+        as3514_write_masked(AS3514_ADC_R, ADC_R_ADCMUX_LINE_IN1,
+                            ADC_R_ADCMUX);
+
         /* MIC1_off, LIN1_on */
-        as3514_write(AS3514_AUDIOSET1,
-            (as3514.regs[AS3514_AUDIOSET1] & ~(1 << 0)) | (1 << 2));
+        as3514_write_masked(AS3514_AUDIOSET1, AUDIOSET1_LIN1_on,
+                            AUDIOSET1_MIC1_on | AUDIOSET1_LIN1_on);
     }
 
     /* ADC_Mute_off */
-    as3514_write_or(AS3514_ADC_L, (1 << 6));
+    as3514_set(AS3514_ADC_L, ADC_L_ADC_MUTE_off);
     /* ADC_on */
-    as3514_write_or(AS3514_AUDIOSET1, (1 << 7));
+    as3514_set(AS3514_AUDIOSET1, AUDIOSET1_ADC_on);
 }
 
 void audiohw_disable_recording(void)
@@ -308,10 +331,12 @@ void audiohw_disable_recording(void)
     source = SOURCE_DAC;
 
     /* ADC_Mute_on */
-    as3514_write_and(AS3514_ADC_L, ~(1 << 6));
+    as3514_clear(AS3514_ADC_L, ADC_L_ADC_MUTE_off);
 
     /* ADC_off, LIN1_off, MIC_off */
-    as3514_write_and(AS3514_AUDIOSET1, ~((1 << 7) | (1 << 2) | (1 << 0)));
+    as3514_clear(AS3514_AUDIOSET1,
+                 AUDIOSET1_ADC_on | AUDIOSET1_LIN1_on |
+                 AUDIOSET1_MIC1_on);
 
     audiohw_set_master_vol(as3514.vol_l, as3514.vol_r);
 }
@@ -332,25 +357,27 @@ void audiohw_set_recvol(int left, int right, int type)
     case AUDIO_GAIN_MIC:
     {
         /* Combine MIC gains seamlessly with ADC levels */
-        unsigned int mic1_r = as3514.regs[AS3514_MIC1_R] & ~(0x3 << 5);
+        unsigned int mic1_r;
 
         if (left >= 36) {
             /* M1_Gain = +40db, ADR_Vol = +7.5dB .. +12.0 dB =>
                +19.5 dB .. +24.0 dB */
             left -= 8;
-            mic1_r |= (0x2 << 5);
+            mic1_r = MIC1_R_M1_GAIN_40DB;
         } else if (left >= 32) {
             /* M1_Gain = +34db, ADR_Vol = +7.5dB .. +12.0 dB =>
                +13.5 dB .. +18.0 dB */
             left -= 4; 
-            mic1_r |= (0x1 << 5);
-        }
+            mic1_r = MIC1_R_M1_GAIN_34DB;
+        } else {
             /* M1_Gain = +28db, ADR_Vol = -34.5dB .. +12.0 dB =>
                -34.5 dB .. +12.0 dB */
+            mic1_r = MIC1_R_M1_GAIN_28DB;
+        }
 
         right = left;
 
-        as3514_write(AS3514_MIC1_R, mic1_r);
+        as3514_write_masked(AS3514_MIC1_R, mic1_r, MIC1_R_M1_GAIN);
         break;
         }
     case AUDIO_GAIN_LINEIN:
@@ -359,8 +386,8 @@ void audiohw_set_recvol(int left, int right, int type)
         return;
     }
 
-    as3514_write(AS3514_ADC_R, (as3514.regs[AS3514_ADC_R] & ~0x1f) | right);
-    as3514_write(AS3514_ADC_L, (as3514.regs[AS3514_ADC_L] & ~0x1f) | left);
+    as3514_write_masked(AS3514_ADC_R, right, AS3514_VOL_MASK);
+    as3514_write_masked(AS3514_ADC_L, left, AS3514_VOL_MASK);
 }
 
 /**
@@ -369,27 +396,18 @@ void audiohw_set_recvol(int left, int right, int type)
  */
 void audiohw_set_monitor(bool enable)
 {
-    /* LI1R_Mute_on - default */
-    unsigned int line_in1_r = as3514.regs[AS3514_LINE_IN1_R] & ~(1 << 5);
-    /* LI1L_Mute_on - default */
-    unsigned int line_in1_l = as3514.regs[AS3514_LINE_IN1_L] & ~(1 << 5);
-    /* LIN1_off - default */
-    unsigned int audioset1 = as3514.regs[AS3514_AUDIOSET1] & ~(1 << 2);
-
     if (enable) {
         source = SOURCE_LINE_IN1_ANALOG;
 
-        /* LI1R_Mute_off */
-        line_in1_r |= (1 << 5);
-        /* LI1L_Mute_off */
-        line_in1_l |= (1 << 5);
-        /* LIN1_on */
-        audioset1 |= (1 << 2);
+        as3514_set(AS3514_AUDIOSET1, AUDIOSET1_LIN1_on);
+        as3514_set(AS3514_LINE_IN1_R, LINE_IN1_R_LI1R_MUTE_off);
+        as3514_set(AS3514_LINE_IN1_L, LINE_IN1_L_LI1L_MUTE_off);
+    }
+    else {
+        as3514_clear(AS3514_AUDIOSET1, AUDIOSET1_LIN1_on);
+        as3514_clear(AS3514_LINE_IN1_R, LINE_IN1_R_LI1R_MUTE_off);
+        as3514_clear(AS3514_LINE_IN1_L, LINE_IN1_L_LI1L_MUTE_off);
     }
-
-    as3514_write(AS3514_AUDIOSET1, audioset1);
-    as3514_write(AS3514_LINE_IN1_R, line_in1_r);
-    as3514_write(AS3514_LINE_IN1_L, line_in1_l);
 
     /* Sync mixer volume */
     audiohw_set_master_vol(as3514.vol_l, as3514.vol_r);
diff --git a/firmware/export/as3514.h b/firmware/export/as3514.h
index 83e911f42b..af4615a15e 100644
--- a/firmware/export/as3514.h
+++ b/firmware/export/as3514.h
@@ -78,6 +78,158 @@ extern void audiohw_set_sample_rate(int sampling_control);
 #define VOLUME_MIN -735
 #define VOLUME_MAX   60
 
+/*** Audio Registers ***/
+
+/* 00h (LINE_OUT_R) to 1Dh (PLLMODE) */
+#define AS3514_NUM_AUDIO_REGS   (0x1e)
+
+/* Common registers masks */
+#define AS3514_VOL_MASK         (0x1f << 0)
+
+/* LINE_OUT_R (0x00) */
+/* Only has volume control bits */
+
+/* LINE_OUT_L (0x01) */
+#define LINE_OUT_L_LO_SES_DM    (0x3 << 6)
+    #define LINE_OUT_L_LO_SES_DM_MUTE   (0x0 << 6)
+    #define LINE_OUT_L_LO_SES_DM_DF_MO  (0x1 << 6)
+    #define LINE_OUT_L_LO_SES_DM_SE_ST  (0x2 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* HPH_OUT_R (0x02) */
+#define HPH_OUT_R_HP_OVC_TO     (0x3 << 6)
+    #define HPH_OUT_R_HP_OVC_TO_0MS     (0x3 << 6)
+    #define HPH_OUT_R_HP_OVC_TO_128MS   (0x1 << 6)
+    #define HPH_OUT_R_HP_OVC_TO_256MS   (0x0 << 6)
+    #define HPH_OUT_R_HP_OVC_TO_512MS   (0x2 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* HPH_OUT_L (0x03) */
+#define HPH_OUT_L_HP_MUTE       (0x1 << 7)
+#define HPH_OUT_L_HP_ON         (0x1 << 6)
+#define HPH_OUT_L_HP_DET_ON     (0x1 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* LSP_OUT_R (0x04) */
+#define LSP_OUT_R_SP_OVC_TO     (0x3 << 6)
+    #define LSP_OUT_R_SP_OVC_TO_256MS (0x0 << 6)
+    #define LSP_OUT_R_SP_OVC_TO_128MS (0x1 << 6)
+    #define LSP_OUT_R_SP_OVC_TO_512MS (0x2 << 6)
+    #define LSP_OUT_R_SP_OVC_TO_0MS   (0x3 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* LSP_OUT_L (0x05) */
+#define LSP_OUT_L_SP_MUTE       (0x1 << 7)
+#define LSP_OUT_L_SP_ON         (0x1 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* MIC1_R (0x06) */
+#define MIC1_R_M1_AGC_off       (0x1 << 7)
+#define MIC1_R_M1_GAIN          (0x3 << 5)
+    #define MIC1_R_M1_GAIN_28DB (0x0 << 5)
+    #define MIC1_R_M1_GAIN_34DB (0x1 << 5)
+    #define MIC1_R_M1_GAIN_40DB (0x2 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* MIC1_L (0x07) */
+#define MIC1_L_M1_SUP_off       (0x1 << 7)
+#define MIC1_L_M1_MUTE_off      (0x1 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* MIC2_R (0x08) */
+#define MIC2_R_M2_AGC_off       (0x1 << 7)
+#define MIC2_R_M2_GAIN          (0x3 << 5)
+    #define MIC2_R_M2_GAIN_28DB (0x0 << 5)
+    #define MIC2_R_M2_GAIN_34DB (0x1 << 5)
+    #define MIC2_R_M2_GAIN_40DB (0x2 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* MIC2_L (0x09) */
+#define MIC2_L_M2_SUP_off       (0x1 << 7)
+#define MIC2_L_M2_MUTE_off      (0x1 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* LINE_IN1_R (0Ah) */
+#define LINE_IN1_R_LI1R_MUTE_off    (0x1 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* LINE_IN1_L (0Bh) */
+#define LINE_IN1_L_LI1_MODE      (0x3 << 6)
+    #define LINE_IN1_L_LI1_MODE_SE_ST   (0x0 << 6)
+    #define LINE_IN1_L_LI1_MODE_DF_MO   (0x1 << 6)
+    #define LINE_IN1_L_LI1_MODE_SE_MO   (0x2 << 6)
+#define LINE_IN1_L_LI1L_MUTE_off (0x1 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* LINE_IN2_R (0Ch) */
+#define LINE_IN2_R_LI2R_MUTE_off (0x1 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* LINE_IN2_L (0Dh) */
+#define LINE_IN2_L_LI2_MODE      (0x3 << 6)
+    #define LINE_IN2_L_LI2_MODE_SE_ST   (0x0 << 6)
+    #define LINE_IN2_L_LI2_MODE_DF_MO   (0x1 << 6)
+    #define LINE_IN2_L_LI2_MODE_SE_MO   (0x2 << 6)
+#define LINE_IN2_L_LI2L_MUTE_off (0x1 << 5)
+/* Use AS3514_VOL_MASK */
+
+/* DAC_R (0Eh) */
+/* Only has volume control bits */
+/* Use AS3514_VOL_MASK */
+
+/* DAC_L (0Fh) */
+#define DAC_L_DAC_MUTE_off      (0x1 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* ADC_R (10h) */
+#define ADC_R_ADCMUX            (0x3 << 6)
+    #define ADC_R_ADCMUX_ST_MIC     (0x0 << 6)
+    #define ADC_R_ADCMUX_LINE_IN1   (0x1 << 6)
+    #define ADC_R_ADCMUX_LINE_IN2   (0x2 << 6)
+    #define ADC_R_ADCMUX_AUDIO_SUM  (0x3 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* ADC_L (11h) */
+#define ADC_L_FS_2              (0x1 << 7)
+#define ADC_L_ADC_MUTE_off      (0x1 << 6)
+/* Use AS3514_VOL_MASK */
+
+/* AUDIOSET1 (14h)*/
+#define AUDIOSET1_ADC_on        (0x1 << 7)
+#define AUDIOSET1_SUM_on        (0x1 << 6)
+#define AUDIOSET1_DAC_on        (0x1 << 5)
+#define AUDIOSET1_LOUT_on       (0x1 << 4)
+#define AUDIOSET1_LIN2_on       (0x1 << 3)
+#define AUDIOSET1_LIN1_on       (0x1 << 2)
+#define AUDIOSET1_MIC2_on       (0x1 << 1)
+#define AUDIOSET1_MIC1_on       (0x1 << 0)
+
+/* AUDIOSET2 (15h) */
+#define AUDIOSET2_BIAS_off      (0x1 << 7)
+#define AUDIOSET2_DITH_off      (0x1 << 6)
+#define AUDIOSET2_AGC_off       (0x1 << 5)
+#define AUDIOSET2_IBR_DAC       (0x3 << 3)
+    #define AUDIOSET2_IBR_DAC_0     (0x0 << 3)
+    #define AUDIOSET2_IBR_DAC_25    (0x1 << 3)
+    #define AUDIOSET2_IBR_DAC_40    (0x2 << 3)
+    #define AUDIOSET2_IBR_DAC_50    (0x3 << 3)
+#define AUDIOSET2_LSP_LP        (0x1 << 2)
+#define AUDIOSET2_IBR_LSP       (0x3 << 0)
+    #define AUDIOSET2_IBR_LSP_0     (0x0 << 0)
+    #define AUDIOSET2_IBR_LSP_17    (0x1 << 0)
+    #define AUDIOSET2_IBR_LSP_34    (0x2 << 0)
+    #define AUDIOSET2_IBR_LSP_50    (0x3 << 0)
+
+/* AUDIOSET3 (16h) */
+#define AUDIOSET3_ZCU_off       (0x1 << 2)
+#define AUDIOSET3_IBR_HPH       (0x1 << 1)
+#define AUDIOSET3_HPCM_off      (0x1 << 0)
+
+/* PLLMODE (1Dh) */
+#define PLLMODE_LRCK             (0x3 << 1)
+    #define PLLMODE_LRCK_24_48      (0x0 << 1)
+    #define PLLMODE_LRCK_8_23       (0x2 << 1)
+
 /* ADC channels */
 #define NUM_ADC_CHANNELS 13
 
diff --git a/firmware/target/arm/ascodec-pp.c b/firmware/target/arm/ascodec-pp.c
index 30b6b1f8de..ac45cddc14 100644
--- a/firmware/target/arm/ascodec-pp.c
+++ b/firmware/target/arm/ascodec-pp.c
@@ -39,10 +39,6 @@ void audiohw_init(void)
     DEV_INIT1 &=~0x3000000;
     /*mini2?*/
 
-    /* device reset */
-    DEV_RS |= DEV_I2S;
-    DEV_RS &=~DEV_I2S;
-
     /* I2S device reset */
     DEV_RS |= DEV_I2S;
     DEV_RS &=~DEV_I2S;
@@ -56,11 +52,27 @@ void audiohw_init(void)
     /* external dev clock to 24MHz */
     outl(inl(0x70000018) & ~0xc, 0x70000018);
 
+#ifdef SANSA_E200
+    /* Prevent pops on startup */
+    GPIOG_ENABLE |= 0x08;
+    GPIO_SET_BITWISE(GPIOG_OUTPUT_VAL, 0x08);
+    GPIOG_OUTPUT_EN |= 0x08;
+#endif
+
     i2s_reset();
 
     audiohw_preinit();
 }
 
-void audiohw_postinit(void)
+void ascodec_supressor_on(bool on)
 {
-}
+#ifdef SANSA_E200
+    if (on) {
+        /* Set pop prevention */
+        GPIO_SET_BITWISE(GPIOG_OUTPUT_VAL, 0x08);
+    } else {
+        /* Release pop prevention */
+        GPIO_CLEAR_BITWISE(GPIOG_OUTPUT_VAL, 0x08);
+    }
+#endif
+} 
diff --git a/firmware/target/arm/ascodec-target.h b/firmware/target/arm/ascodec-target.h
index 3337cb78b9..e26afa840c 100644
--- a/firmware/target/arm/ascodec-target.h
+++ b/firmware/target/arm/ascodec-target.h
@@ -59,6 +59,8 @@ static inline void ascodec_unlock(void)
     i2c_unlock();
 }
 
+extern void ascodec_supressor_on(bool on);
+
 #endif /* CPU_PP */
 
 #endif /* !_ASCODEC_TARGET_H */
diff --git a/firmware/target/arm/pcm-pp.c b/firmware/target/arm/pcm-pp.c
index 2f25353fff..c572578383 100644
--- a/firmware/target/arm/pcm-pp.c
+++ b/firmware/target/arm/pcm-pp.c
@@ -374,7 +374,7 @@ void pcm_play_dma_init(void)
     audiohw_init();
 
 #if !defined(HAVE_WM8731) && !defined(HAVE_WM8751) && !defined(HAVE_WM8975) \
- && !defined(HAVE_WM8758)
+ && !defined(HAVE_WM8758) && !defined(HAVE_AS3514)
     /* Power on */
     audiohw_enable_output(true);
     /* Unmute the master channel (DAC should be at zero point now). */