alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Adrian Knoth <adi@drcomp.erfurt.thur.de>
To: patch@alsa-project.org
Cc: Adrian Knoth <adi@drcomp.erfurt.thur.de>, alsa-devel@alsa-project.org
Subject: [PATCH 0/4] hdspmixer: rescue current mixer settings when switching cards
Date: Mon,  4 Apr 2011 14:34:26 +0200	[thread overview]
Message-ID: <1301920470-4615-1-git-send-email-adi@drcomp.erfurt.thur.de> (raw)

Hi!


This series of patches was developed for EMAtech. They discovered that
when running with multiple cards, switching between cards would lose all
current changes made to the last active card.

Obviously, this is undesired behaviour. Though rather small, it nearly
took me 6hrs to get this right, and it could still be improved, but I
like to take hdspmixer into a completely different direction:

I want to remove the card switching code completely and simply run a
single instance per card, that is, if you have more than one card,
you'll say something like hdspmixer -c 1 (or even a card name).

Likewise, the preset files should then only contain the settings for a
single card. Right now, it stores everything for three cards, and if you
have four, then you'd be left in the rain. Also, when the order of cards
changes, the mixer settings would end up on the wrong card.

Long story short: the current approach is a dead-end.


Cheers

Adrian Knoth (4):
  hdspmixer: Add a 9th pseudo preset
  hdspmixer: Save preset before switching cards
  hdspmixer: Recall 1st preset on all cards, not just on the first
  hdspmixer: Initialize headphones out in presets

 hdspmixer/src/HDSPMixerCard.cxx         |    1 +
 hdspmixer/src/HDSPMixerCard.h           |    2 +
 hdspmixer/src/HDSPMixerCardSelector.cxx |    2 +
 hdspmixer/src/HDSPMixerIOMixer.cxx      |    2 +-
 hdspmixer/src/HDSPMixerIOMixer.h        |    2 +-
 hdspmixer/src/HDSPMixerOutput.cxx       |    4 +-
 hdspmixer/src/HDSPMixerOutput.h         |    2 +-
 hdspmixer/src/HDSPMixerWindow.cxx       |   48 ++++++++++++++++++++++++++++--
 hdspmixer/src/HDSPMixerWindow.h         |    4 ++-
 hdspmixer/src/defines.h                 |    6 ++++
 10 files changed, 63 insertions(+), 10 deletions(-)

-- 
1.7.4.1

             reply	other threads:[~2011-04-04 12:34 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-04 12:34 Adrian Knoth [this message]
2011-04-04 12:34 ` [PATCH 1/4] hdspmixer: Add a 9th pseudo preset Adrian Knoth
2011-04-04 12:34 ` [PATCH 2/4] hdspmixer: Save preset before switching cards Adrian Knoth
2011-04-04 12:34 ` [PATCH 3/4] hdspmixer: Recall 1st preset on all cards, not just on the first Adrian Knoth
2011-04-04 12:34 ` [PATCH 4/4] hdspmixer: Initialize headphones out in presets Adrian Knoth
2011-04-06  6:28 ` [PATCH 0/4] hdspmixer: rescue current mixer settings when switching cards Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1301920470-4615-1-git-send-email-adi@drcomp.erfurt.thur.de \
    --to=adi@drcomp.erfurt.thur.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=patch@alsa-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).