Fixed audio webpage issues found testing #16
Conversation
After testing with DRA-PI-zero hat I found a few issues. Audio levels did not allways match percentage. Fixed by adding -M Enum controls were not supported volume controls (neither cvolume nor pvolume) were not supported fixed whitespace issue Fixed copy paste error in capture regex Signed-off-by: Daniel Riechers <[email protected]>
|
So, this isn't working exactly for the fe-pi audio board. it has a dozen capture and a dozen output controls. The one show stopper is the displayed "Capture" slider doesn't have a check mark, even though it is active. when the user hits submit, it effectively mutes the Capture audio without (the user) changing anything on the page. |
|
|
|
additionally, you can see many controls are missing, Mic for example. I haven't looked closely at the code yet, but are these being filtered somehow? |
|
It’s not super clean code. It’s mostly regex scraping command line utils. It isn’t exactly filtering but I’ve learned through this experience with alsa that not all cards behave as I would have assumed. What have you tested on? Id be happy to order a board/boards to reproduce and fix my end. |
|
It also looks from your screenshot you may be running my original code because the url says alsa and not audio |
|
yah, i renamed a few tags |
|
I haven't taken a close look at your code yet, it might be something silly. I just wanted to run it by you to see if it was something obvious, why the html shows an unchecked box for a device that has CAPTURE enabled.... and why many/most controls are missing. It'd be cool to do some filtering, as many/most controls are useless/confusing. line/pcm/volume/mic/capture. The board in question is an fe-pi audio board, in common use. Bud Churchward sells them now. I was just doing some regression testing, getting ready for a new release, but I can't release the Audio stuff if it generates tickets. thanks again - great stuff, |
|
It looks like it might be a bug in "amixer scontrols" Simple mixer control 'Headphone',0
|
|
pi@digipi4:~ $ sudo amixer controls | grep Capt |
|
And you can enable the CAPTURE switch on the Capture device with amixer set "Capture" cap But it doesn't appear to be possible to detect which device has CAPTURE switched on. If alsa is this bad, we might want to disable the switching, and just present sliders? |
After testing with DRA-PI-zero hat I found a few issues.
Audio levels did not allways match percentage. Fixed by adding -M Enum controls were not supported
volume controls (neither cvolume nor pvolume) were not supported fixed whitespace issue
Fixed copy paste error in capture regex
I'm happy to make any changes perform additional testing. I have now tested with digipi hat and dra-pi-zero
73
KF0NRV