Bad tone quality when floating point is disabled (thanks Olle Frimanson for the report)

[bennylp]

Issues with the tone generator:

1. The sine approximation algorithm especially for the fixed point version is not good enough, causing aliasing frequencies to occur
2. The volume perhaps can be made louder
3. The volume is different between float and fixed point version
4. There is a thump noise at the beginning and end of a tone (more noticeable at the end), due to the sharp transition between off-on and on-off (tone wave signal being terminated before it is fully complete).

[bennylp]

In r2281:

• Added PJMEDIA_TONEGEN_FORCE_FLOAT to force the tone generator to use floating point calculation even when floating point is disabled, to improve the tone quality.
• Added PJMEDIA_TONEGEN_FADE_IN_TIME and PJMEDIA_TONEGEN_FADE_OUT_TIME to control tone fade-in and fade-out feature. Default value is 1 msec and 2 msec respectively.

[bennylp]

In r2284:

• Fixed the volume difference between fixed and floating point tone generation algorithm (the fixed point was significantly quieter)
• Added PJMEDIA_TONEGEN_VOLUME setting to control the default tone volume.
• Increased default tone volume by 50% (from approx -12dB to approx -8dB)

[bennylp]

In r2292:
Large reorganization of the tonegen:

• Deprecate the automatic selection of algorithm. The old PJMEDIA_USE_HIGH_QUALITY_TONEGEN macro and the newly introduced PJMEDIA_TONEGEN_FORCE_FLOAT macro are now deprecated and new algorithm selection is introduced (see below)
• Introduced various constants for tonegen backends in config.h:
  • PJMEDIA_TONEGEN_SINE
  • PJMEDIA_TONEGEN_FLOATING_POINT
  • PJMEDIA_TONEGEN_FIXED_POINT_CORDIC
  • PJMEDIA_TONEGEN_FAST_FIXED_POINT
• Introduced PJMEDIA_TONEGEN_ALG setting to control which algorithm to be used. With this macro, user can override which algorithm to be used.
• Some fixes to the floating-point approximation backend:
  • the sine signal started with 1 while it should start with 0.
  • Use float rather than double for better performance.
• Updated the benchmark results in tonegen.c

[bennylp]

Additional info:

• the r2292 also introduced new fixed-point tone generation backend based on Cordic algorithm.

[bennylp]

In r2294:

• changed default Cordic loop from 8 to 10 to gain more accuracy, and
• updated tonegen.c with the results fro m ARM9 tests

[bennylp]

Done.

More documentation can be found in Analyzing the PJMEDIA Tone Generator Algorithms Performance

[bennylp]

In r2305:

• disable fade-in and fade-out if the tone off duration is zero. We assume that application wants to play continuous tone in this case, and the fade in/out feature will cause click noise to be generated