Rockbox

  • Status Closed
  • Percent Complete
    0%
  • Task Type Patches
  • Category Settings
  • Assigned To No-one
  • Operating System
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by bigdan42 - 2006-02-04
Last edited by bigdan42 - 2006-02-07

FS#2982 - EQ Menu User Interface

This patch provides a menu based interface for
configuring the EQ on software codec platforms. It uses
the existing settings system to store EQ settings and
adds an additional .rockbox/eqs directory for loading
EQ configs.

The menu is split in to a simple and an advance
interface. The simple interface provides control over
gain only. The advanced interface provides control over
all parameters for all five bands of the EQ.

There is also the beginnings of a graphical version of
the equalizer, though this is very preliminary and
doesn’t work at all well.

Closed by  bigdan42
2006-02-07 18:11
Reason for closing:  Accepted

New patch with the following changes:

* Changed default equalizer cutoffs to those suggested by t0mas
* Added graphical settings method. Use left and right to
change the cutoff, press “NAVI” or “SELECT” to change
setting mode. Move up and down to change band.
* Fixed bug with setting the first EQ band values (found by
curio in the forums)

New patch

  • Enable faster EQ coef calculation on Coldfire
  • Rearrange EQ coef calculation so changes are updated

more often

  • Added more comments
  • Remove some repetitive code in eq_menu.c with the use of

macros

  • Rename some functions for consistency
Anonymous Submitter commented on 2006-02-04 22:19

Very thanks safetydan !!!!
Your are one god ;-)

V8 works like magic… Only one request.
Can you make the steps in GAIN mode in 0.1 untis except for
0.5?
I modified this for myself, but it would be nice if this
was default, or user selectable…

New patch v9:

  • Fixed restoration of EQ enabled setting after power-on

(found by bk on forum.rockbox.org)

  • Handle USB connection event in the graphical EQ setting page
  • Changed gain step size from 0.5 dB to 0.1 dB (suggested

by needleboy)

  • Changed cutoff step size from 20 Hz to 10 Hz
  • Some lang file corrections
  • Added easy way to save EQ presets
  • Fix warning while compiling dsp.c

New patch v10:

  • Fix saving of settings to include all 5 bands (bug found

by tracer at forums.rockbox.org)

I’ve just been reading the commments…

I think the step sizes are right, however a method to
increase the step size would be nice, i.e. a button
combination (play+left/right) to increment the step size
from 0.1db to 1db, likewise for cutoff go from 10hz to 100hz
steps.

Or do what the pitch control does and accelerate the step
size over time.

Great patch btw :)

New v11:

  • Incorporate changes from bk on formums.rockbox.org
    1. Fix display corruption on exit from graphical UI
    2. Add WPS context menu entries for quick EQ access
  • Sync with current CVS
  • More comments
  • Fix some C++ style comments

another UI suggestion, if its possible invert the color of
the currently selected paramater, and don’t reset the mode
to gain when flicking through the bands.

Cheers.

New EQ UI v12 changes:

* Band selection in UI now wraps when moving up and down
(suggested by ghode on #rockbox)
* Don’t switch modes when scrolling through options
(suggested by ep0ch1978)
* Use native frequency for EQ coef calculation since EQ
processing is after the resampler. Fixes oddities with low
frequency MP3s
* Invert the current item being changed in the GUI (suggested by ep0ch1978)
* Clearer labels for EQ bands, distinguishing between low
shelf (LS), high shelf (HS) and peak (PK)

EQ UI v13 patch:

  • Update EQ filters when the user changes them.
  • i18n fixes
  • Some misc other stuff I forget now.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing