Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Drivers
  • Assigned To
    Buschel
  • Operating System Cowon D2
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jakse - 2010-05-05
Last edited by Buschel - 2010-05-30

FS#11240 - Allow lower volume for Cowon D2+

I just installed Rockbox r25810-100504 on my Cowon D2+ player yesterday. It works quite well.
However, I find the lowest volume, which is -57 dB, too high. If I try -58 dB, the sound is turned off.

I usually listen to music at volume 2/3 with the original firmware, but the lowest volume in rockbox seems to be like volume 5 of the original firmware, which is too high for me when there isn’t any noise .

I don’t know wether it is a driver issue or not.

I don’t know what category is best for this issue.

Closed by  Buschel
2010-05-30 18:29
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Submitted with r26413

This is not a bug but a change request :o)
If you take a look at  FS#10082  you will see that exactly this change was added to another audio codec driver. Are you able and willing to create a patch to attach it here?

jakse commented on 2010-05-06 17:28

Thank you for your response.
I'm unfortunately not able to do the change because I have never writen driver code and I still don't know Cowon D2+ sound hardware. I installed rockbox 2 days ago and have not set any rockbox coding environement yet. I'm not familiar witch patches neither.
However, I can learn to do this and see what I can finely do I'll try to understand the patch you linked and make changes to the Cowon driver, but I'm not confident about testing it myself and it could be long before I can actually write any code…

jakse commented on 2010-05-06 18:22

the audio codec, according to http://www.rockbox.org/wiki/CowonD2Info#Hardware , is the Wolfson WM8985. This could simplify the work as the patch linked in the first post is for the wm8758.

Yes, the Cowon uses a WM8795. If you compare both the WM8985 and the WM8758 driver (especially before my change) you will see the related parts of source code are very very similar. I am confident that the  FS#10082  can be easily transferred to the WM8985 driver.

jakse commented on 2010-05-29 08:36

Hi !
Good news, I come with two patches fixing the issue. I think it works, I only tried to play a mp3 file with.
First patch corrects the file wm8985.c, and the second corrects wm8985.h, where the constant VOLUME_MIN is defined. The first patch is likely to depends on the second, and vice versa.
My modifications are strongly inspired of the  FS#10082  patch.

I think It needs some testing / checking : I did it with lots of intuitions, and this is the first time I :
- patch something
- look at a driver code.

Hope that it will help somebody one day ;)

Your changes look fine and compile :o)
I would like to have another tester to verify the functionality before I submit your changes.

Attached patch also corrects the manual.

jakse commented on 2010-05-30 06:10

Thank you for having tested my patch :)

I had time to test more : It works with Ogg and Flac too. I think 90 dB turn the sound off as expected as I can hear a little noise when switching from 89 dB to 90 dB.
I played Rockblock while listening to music, too.

So yes, another tester would by great.

jakse commented on 2010-05-30 06:10

-90db and -89dB, of course.

Your changes affect the volume in general, this is independent of any codec.

jakse commented on 2010-05-30 07:22

Yes, I know, but sometimes weird things happen.
I had a mp3 player with whitch I could listen mp3s with the volume 1 but not wma !
Strange,isn't it ?

Thanks for working on this dysmann, I saw your comment a week or two ago and was about to try the same thing! I will test your patch later today with a view to getting it committed.

shotofadds, just waiting for your approval of proper functionality. Then I will hit the button :o)

It works like a charm. I can't check the exact ouptut levels, but I can tell you the upper range is unaffected, the lowest volume setting is barely perceptile with my in-ear buds, and the muting at -90 works fine. Go for it.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing