Rockbox

Forgot to mention that the first patch (rec_agc_histogram) is for people who already installed Mmmm’s recording enhancement pack.
The second includes the enhancement pack and AGC.

The ‘graphics’ for HD space and battery life do not take different fontsizes into account..

So here’s the updated patch for AGC.
New features:
- settings are saved.
- localization (english & german)
- limiter versions of presets. allows to set a maximum gain which isn’t succeeded. press left when AGC presets highlighted to select limiter presets. timing is the same as ‘normal’ presets. press down when AGC presets highlighted and a limiter preset is selected to change maximum gain.
- improved precision of peak meter (much lower error in important -12dB to 0dB range, error < 0.2dB).

This patch includes Mmmm’s remote support and high size peak meter.

   rec_agc_enhancementspack2.pat... (74.2 KiB)

hi!
I have no idea how to apply that to my rockbox…..I m very interested since i m a musician and record a lot with my Iriver h320 in live sittuations…Anybody could tip me? thanks a lot and thanks for people developing this great thing!

A quick update against daily build 060313:

- now with user font in recording screen if it fits. remote and main can have different fonts and peak-meter sizes depending on space.
- removed the ‘limiter’ presets and made ‘max. AGC gain’ enabled always.
- slight rearrangment of battery, disk and balance-meter. also enabled these infos for the higher sized histogram views if there is enough space on the screen.
- seperate saved preset & max. agc gain settings for mic and line-in.
- includes remote support (from Mmmm’s patch)
- includes improved power managment (as I was unable to remove it right now, just too busy).
- use patch -p1 to apply to source.

@sebastien: This is for developers only at the moment (you need the development tools installed and compile the source yourself). I’ll compile a recent binary for H120 and H320 when I find some time.

   rec_agc_enhance_pack3.patch (107.1 KiB)

I isolated the recording enhancements part of the patch, and i also made a version which will apply cleanly against paprica’s scroll margins patch (which can be found here : http://www.rockbox.org/bugs/task/2994 ), as they conflict but they are both very nice ;)

Use -p0 to apply

   rec_agc_enhancementspack_2006... (82.1 KiB)
   rec_agc_enhancementspack_scro... (82.1 KiB)

there is a missing part in nicolas_p patch (english.lang)

id: LANG_WARNING_BATTERY_LOW
desc: in recording screen
eng: “Battery is low!” voice:
new:

With the remote support and peakmeter patch added to CVS I guess we need a new patch for ‘just’ the AGC part..

Could you *please* split usefull patches (like this) into one functionality per patch?
I would love to test and commit the AGC and other stuff, but I can’t do it all in one…

OK here it comes. This is only the AGC, the histogram follows in a seperate patch. Since remote support and peakmeter patch got added to CVS there are only a few files left to modify, namely german & englisch language, settings, sound_menu, status (for showing the ‘REC? symbol in statusbar) and recording.

   rec_agc_31.patch (42.4 KiB)

Great… but it seems you just missed the ‘feature freeze’ deadline, so I won’t be able to commit it for another 4 weeks. After the 3.0 release, it ‘ll be one of the first things to go in :)

Yeah that’s sad I know, but it was impossible to do it before.

I splitted agc and batt/disk warnings too, so here are the seperate patches. I still try to update the patch every two weeks or so that they remain updated for easier patching lateron.

   rec_agc_32.patch (37.1 KiB)
   rec_batt_disk_1.patch (5.6 KiB)

Tried your AGC patch and it sure has potential - I only used the safety setting and it worked nicely. The settings will need a fair bit of documentation, maybe change it to a fully parametric control (in true rockbox style) next to the wizard-like presets.

As the lang file changed, it didn’t apply cleanly anymore, so I fixed it for you (AGC patch only)

   agc_20060409.patch (36.2 KiB)

Thanks Peter, I thought about more control for presets too, perhaps some user presets which can be configured like the EQs. There are quite some control parameters that’s why I thought it would be easier just to give presets. But we can make something that lets the user choose target level, control hysteresis, attack and release timing as the most important ones. Perhaps via context menu?
I’ll dive into this…

This is the official patch for the new language format. I moved all graphic changes (like status bar REC symbol etc) to the seperate histogram patch.

And I added a patch which includes AGC and Histogram display because it’s not so easy to apply both patches to the source (conflicts because several functions and other stuff are used by both). So use the ‘rec_agc_hist_pack’ for this.

Patches were done against daily-060408 and use -p1 for patching

   rec_agc_33.patch (34.1 KiB)
   rec_agc_hist_pack.patch (55.3 KiB)

The color (part of the) patch turns the textcolor to black .. not very nice if you have a black background :^)

Paul, you must surely be talking about the agc+histogram patch, as the agc-only patch does no such thing here…

hmm I guess yes :)

Fixed it..

Replace lcd_set_foreground(LCD_DEFAULT_FG) with lcd_set_foreground(global_settings.fg_color) works for me

it appears the patch does not setup LCD width correctly(on my H3x0 at least, the left side (as far as I can see the outline only) of the balance meter is missing and the outline of the histogram is not shown either..

Another thing I noticed is that the box around the histogram and the ‘needle’ in the balancementer is always drawn in black. This makes it hard to see when using a black background..;^)

updated agc patch for current cvs

   agc (36.2 KiB)

Hello Petur, Paul, and Jvo.
I think that there’s a small bug in this lastest AGC patch.
When AGC is set to be in a “non-off” state it seems to only be available if your source is set to “mic”.
Wouldn’t AGC be useful during line-in recording as well?
I understand that it makes no sense to have AGC on when recording from digital, but automatic line-in attenuation would be a good feature too, no?
If you didn’t want it during line-in you could always set the AGC to “off”.

Davide, I think the problem was that the newly added frequency line didn’t compensate for line-in having L/R gain ( ↔ mic only one line). I have fixed this and had quite some work getting the patch compatible with latest CVS.

Please test ;)

   agc (33.6 KiB)

Updated to latest CVS and fixed build issue for sim builds

   agc (33.7 KiB)

Updated to latest CVS

   agc (33.6 KiB)

Updated to latest CVS

   agc_20060703.patch (28.8 KiB)

Updated to latest CVS (but in a hurry, so please check)

   agc_20060801.patch (28.3 KiB)

Regretfully, it breaks when attempting to use it on Line-In.
Seems to work for Mic in. This was the case in an earlier build.
Sorry Petur, it’s broken.

How does it break?

It’s a diff from a version I run here, and I admit I only tested Line-In. But that worked.
Will check tonight…

Using Mic input I monitor and record BOTH channels! This is actually great, a dual mono track is better than left channel only (as it was), but I don’t know whether this new behavior is caused specifically by AGC or not. AGC seems to work fine with the new dual mono mic recording.

Using Line input I can monitor and record both channels, but there is no AGC display and when I set the gain high and clip the input none of the levels change even when AGC is set to Safety in the menu.

I am currently ‘make’ing a clean build from yesterday’s source (20060801) and will report back on the ‘dual mono’ mic input recording status.

CONFIRMED: Dual Mono Recording for Mic Input committed to CVS. Whoever commited this change THANK YOU!
AGC bug still pending, de-activated during Line-Input recording regardless of menu setting. No AGC status disply in WRS when Source=Line-In (someone please corroberate)
Davide-NYC.
PS: thanks for all of the Devs on this project. LLRB (long live Rockbox)

Ouch.. must have overlooked that… will fix

This one should be better. Don’t know about the remote and certainly not what the code will do now for non AGC targets.

The recording screen code is in need of some serious clean-up. I’m trying to have a look at it. *seeks time*

   agc_20060802.patch (28.6 KiB)

Some small updates:
- moved HAVE_AGC to the config_xxx.h files for smoother multi-target implementation
- small fix in recording screen

   agc_20060803.patch (30.3 KiB)

Petur… Here’s a patch based on the one above 3/08

I’ve added display support for digital in for players that support it (although AGC obviously does nothing for digital so on second thoughts maybe we shouldn’t display at all in that mode…hmmmm).

Also I’ve fixed the remote display (and any other screens that are too small to fit all this info on).

   AGC.patch (31.3 KiB)

Here’s a better one. Now the AGC settings don’t appear for digital in (as they don’t do anything anyway) and the cursor handling is better…

Small screens that can’t fit in the AGC info/settings now have the bottom line change into the settings line when the cursor is moved down.

   AGC20060806.patch (31.5 KiB)

my version of the agc patch in sync with cvs. Does not contain recent changes by Mmmm yet. Posted here on request ;)

   agc_20060808.patch (30.3 KiB)

Better version:
- Synced with CVS (again)
- Should incorporate Mmmm’s changes
- Removed 2 constants
- Prepared for other targets

Do test…

   agc_20060809.patch (32.2 KiB)

DLed clean CVS and patched with agc_20060809.patch.
Built SIM: no errors, build NORMAL: no errors.
Installed on H140, tested mic in and line-in.
Digital in shows no AGC.
Tested with the 300LCD-Joystick remote. (buttons are mis-mapped but are all there)
IMO: Looks good! Woohoo!

sync to cvs

   agc_20060816.patch (32.3 KiB)

AGC part has been committed ;)