Rockbox

  • Status Unconfirmed
  • Percent Complete
    0%
  • Task Type Patches
  • Category Drivers
  • Assigned To No-one
  • Operating System All players
  • 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 ChristianGmeiner - 2008-02-12

FS#8602 - Remove audiohw_set_lineout_vol call from sound.c

Hi all.

I have found a funny thing, which should get fixed. In set_prescaled_volume there is this call:
audiohw_set_lineout_vol(tenthdb2master(0), tenthdb2master(0));

As far as I can see this call is not needed here. So this patch removes it, but as I own only a sansa e280 I can not test this change. I hope to commit this soon.

If this call is needed… do we need it every time audio changes or only once at init?

Thanks,
AustrianCOder

This call sets the line out volume to 0 dB always, which is how line outs usually works. It doesn't need to be set in every set_prescaled_volume(), but it should be set some time during init, at least.

Okay.. this is my second try.
I make now use of audiohw_postinit to set lineout volume. Maybe we should check datasheet if we
need to set linout volume - maybe default value is okay?

Since you now call audiohw_set_lineout_vol() from the audio codec specific files you could work out the values and registers and set them directly

The closest corresponding datasheet to wm8758 indicates 0 dB is the default, but I can't be bothered to check the others. Since these data sheets aren't exact matches, we'd need to verify with owners of the devices what the actual defaults are anyway.

I did a quick internet research and found this:

- wm8751 - http://www.wolfsonmicro.com/uploads/documents/en/WM8751.pdf –> 0db default → see page 26
for wm8975 and wm8758 I found no datasheet. But I think that they have all 0db as default.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing