[oe-issues] [Bug 2618] New: Clean spitz.c ALSA driver

[bugzilla-daemon at tinman.treke.net]

http://bugs.openembedded.org/show_bug.cgi?id=2618

           Summary: Clean spitz.c ALSA driver
           Product: Zaurus Kernel
           Version: unspecified
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: C3xxx
        AssignedTo: openembedded-issues at lists.openembedded.org
        ReportedBy: utx at penguin.cz
         QAContact: openembedded-issues at lists.openembedded.org


spitz.c driver is based on wm8750 codec driver.

To make things even less understandable, it mimics some mixer with it Zaurus
meanings (Speaker, Line Jack), but other mixers still use wm8750 names.

Because most wires of wm8750 are not connected, many mixers have no use.

Other mixers lose its usability, because only one input is available and the
input is physically bind with the input.

Also wm8750 have some problems: Some capture mixers are in Playback section.

And finally, it seems that only left input line is wired. It means that stereo
recording is not possible in fact.

Here is my proposal:

- Drop all "Line Mux" mixers - only one input is wired, there is no way to
multiplex. Set it to LINE 1.

- Drop all "Bypass" mixers - it either does nothing or causes analog loopback.

- Drop all "Differential" mixers - INPUT 2 is not connected.

- Drop all "PGA" mixers - only one input is wired, there is no way to mix using
PGA.  Set it to LINE 1.

- Drop "Out3 mux" - OUT3 is not connected.

- Drop "Speaker Playback Invert" - speaker is wired between left and right
output. It is hard to imagine, that anybody wants to play L-R on the speaker
instead of L+R. Suggest to turn it on.

- And finally drop input routing and either route both left jack to left and
right channel or change behavior to act as a mono input card.

References:

http://www.wolfsonmicro.com/uploads/documents/WM8750.pdf for audio chip
reference

Bug 2617 for user space solution.

Bug 2571 comment 4 for temporary user space patch.


-- 
Configure bugmail: http://bugs.openembedded.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.