diff options
author | Torne Wuff <torne@wolfpuppy.org.uk> | 2010-04-13 20:37:04 +0000 |
---|---|---|
committer | Torne Wuff <torne@wolfpuppy.org.uk> | 2010-04-13 20:37:04 +0000 |
commit | 17a4c0715dcfe26af7f8b145d603a9e27589d130 (patch) | |
tree | 6965dc0f42932f644e3d14189c183a6f24f761a1 /firmware/sound.c | |
parent | 8bc3dba6e68331c68adff55222ca9a13b6ede47b (diff) | |
download | rockbox-17a4c0715dcfe26af7f8b145d603a9e27589d130.tar.gz rockbox-17a4c0715dcfe26af7f8b145d603a9e27589d130.zip |
Fix minor mistake in tone control code: UDA1341 was added to the wrong part of the conditional (it has hw bass/treble but not prescaling, like the other wm codecs listed) and TSC2100 has no hw tone controls at all so is already covered by HAVE_SW_TONE_CONTROLS.
Comment has been clarified to make this less likely to happen again :)
Net effect: tone controls should no longer adjust the overall perceived volume in a weird way on mini2440.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@25640 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'firmware/sound.c')
-rw-r--r-- | firmware/sound.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/firmware/sound.c b/firmware/sound.c index d66a93a70d..ca8b79e4d9 100644 --- a/firmware/sound.c +++ b/firmware/sound.c | |||
@@ -200,12 +200,15 @@ static void set_prescaled_volume(void) | |||
200 | int prescale = 0; | 200 | int prescale = 0; |
201 | int l, r; | 201 | int l, r; |
202 | 202 | ||
203 | /* The WM codecs listed don't have suitable prescaler functionality, so we let | 203 | /* The codecs listed use HW tone controls but don't have suitable prescaler |
204 | * the prescaler stay at 0 for these unless SW tone controls are in use */ | 204 | * functionality, so we let the prescaler stay at 0 for these, unless |
205 | * SW tone controls are in use. This is to avoid needing the SW DSP just for | ||
206 | * the prescaling. | ||
207 | */ | ||
205 | #if defined(HAVE_SW_TONE_CONTROLS) || !(defined(HAVE_WM8975) \ | 208 | #if defined(HAVE_SW_TONE_CONTROLS) || !(defined(HAVE_WM8975) \ |
206 | || defined(HAVE_WM8711) || defined(HAVE_WM8721) || defined(HAVE_WM8731) \ | 209 | || defined(HAVE_WM8711) || defined(HAVE_WM8721) || defined(HAVE_WM8731) \ |
207 | || defined(HAVE_WM8751) || defined(HAVE_WM8758) || defined(HAVE_WM8985)) \ | 210 | || defined(HAVE_WM8751) || defined(HAVE_WM8758) || defined(HAVE_WM8985) \ |
208 | || defined(HAVE_TSC2100) || defined(HAVE_UDA1341) | 211 | || defined(HAVE_UDA1341)) |
209 | 212 | ||
210 | prescale = MAX(current_bass, current_treble); | 213 | prescale = MAX(current_bass, current_treble); |
211 | if (prescale < 0) | 214 | if (prescale < 0) |