• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Settings
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes 1
  • Private
Attached to Project: Rockbox
Opened by Will Robertson - 2007-01-21
Last edited by Jonathan Gordon - 2007-08-05

FS#6555 - Volume limiter

As per a request from a friend I've implemented a feature from the iPod original firmware, volume limiting.
You can set a maximum volume, and while in the WPS you can't increase the volume past that point, to avoid blowing up your ears.
The patch has only been tested in the sims, and it bumps the config block version so you'll need to save your settings.

Closed by  Jonathan Gordon
2007-08-05 10:53
Reason for closing:  Out of Date
Additional comments about closing:  

if someone wants to start this again, imo it shuold be added to setvol() in misc.c and not touch any firmware/ code

Hepdog commented on 2007-03-15 17:32

This patch is out of sync with current svn. Please update it because it would be really usfull!

Will Robertson commented on 2007-03-17 09:34

Your wish is my command.

Hepdog commented on 2007-03-17 14:54

Just to tell you, it is still out of sync! it still works becasue it is just a hunk that changes the manual! lol, how did that get in there?

Will Robertson commented on 2007-03-18 08:17

Ugh, that file is strangeā€¦ It's been getting into my patches lately for some reason - svn update doesn't see any difference but svn diff obviously does.
Anyway here's the fixed patch.

Sacha commented on 2007-03-18 15:58

Very useful!!!! Safety First^^

Nikkhil commented on 2007-04-16 14:40

sync'ed, can someone please add this already

Dominik Riebeling commented on 2007-04-16 17:21

The patch contains unnecessary (and unwanted) whitespace changes.

Nikkhil commented on 2007-04-16 17:40

can anyone fix it , i have no clue what it means

Will Robertson commented on 2007-04-17 00:33

When you're making a patch you have to be careful not to include tabs. Rockbox source uses 4 spaces for indentation.
Here's the fixed patch.

Nikkhil commented on 2007-05-04 14:16


Nikkhil commented on 2007-05-28 18:58

patch needs sync, sound.c & sound.h have bin changed

Mark commented on 2007-06-30 10:20

Can someone please add this to the regular builds?


Available keyboard shortcuts


Task Details

Task Editing