diff services/pcm/pcmdata.c @ 33:6a2b09d3b1b4

disable L1_DYN_DSP_DWNLD and AMR by default
author Mychaela Falconia <falcon@freecalypso.org>
date Sat, 15 Oct 2016 04:23:58 +0000
parents da9056962fb7
children
line wrap: on
line diff
--- a/services/pcm/pcmdata.c	Sat Oct 15 04:07:40 2016 +0000
+++ b/services/pcm/pcmdata.c	Sat Oct 15 04:23:58 2016 +0000
@@ -135,17 +135,16 @@
    * Field MOBILE CAPABILITIES - MSCAP
    ********************************************************************/
   /*
-   * FreeCalypso: the change of L1 to the reconstructed TCS211 version
-   * has fixed the EFR codec, so we can re-enable it now, but AMR is
-   * still broken, hence we are going to advertise as non-AMR-capable
-   * despite running on AMR-capable silicon.
-   *
-   * If you would like to experiment with different codec
-   * configurations, you can do so without having to recompile
-   * and reflash the firmware each time: just write a /pcm/MSCAP
-   * file into FFS with whatever setting you wish to try.
+   * FreeCalypso: this Citrine firmware only runs on AMR-capable
+   * silicon, but we have a build configuration setting to allow or
+   * disallow AMR.  As far as we can tell, the AMR codec support
+   * in TI's TCS211 fw that serves as our reference depends on a
+   * dynamically downloaded DSP patch (L1_DYN_DSP_DWNLD), but we are
+   * having some reliability problems with the latter functionality,
+   * hence we have made AMR enabling a conditional feature to err
+   * on the side of safety.
    */
-  #if 0 //((DSP == 34) || (DSP == 35) || (DSP == 36)) // ROM Codes including AMR feature.
+  #if ALLOW_AMR_CODEC
     #if (STD == 1) // GSM 900
       #if defined (FAX_AND_DATA) 
         0xB1,   0xC7,   0x00,