FreeCalypso > hg > freecalypso-citrine
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,