alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Tim Blechmann <tim@klingt.org>
To: alsa-devel@alsa-project.org
Subject: Re: hdspmixer broken for 96khz
Date: Mon, 04 Apr 2011 09:21:24 +0200	[thread overview]
Message-ID: <inbrhl$icc$1@dough.gmane.org> (raw)
In-Reply-To: 20110403102516.GJ28811@ltw.loris.tv

> I wonder what went wrong. If you like, feel free to further dig into
> this.
> 
> However, the current on-disk format is utterly stupid, it has no
> meta-information at all, it will break if the order of cards change.
> 
> The whole hdspmixer is a dead-end, even supporting more than one card in
> a single app complicates things like hell (I have an upcoming patch
> series. Took me 6hrs or so just to get switching back/forth between two
> cards right).

the hdspmixer is not perfect, but it more or less worked for years and it is 
used in `production' environments. so breaking the parts that are currently 
working is quite annoying for users.

> So the next thing I'd like to implement is to limit hdspmixer to a
> single card and then run a new instance on the second, third, 4th a.s.o.
> card. Like alsamixer -c 1: hdspmixer -c 1, maybe -c reflecting the ALSA
> card number as found in /proc/asound/cards.
> 
> Anyway, the whole codebase is subtle broken, and the best approach would
> be a rewrite from scratch. Maybe it makes sense to merge hdsp and hdspm,
> first. Comments welcome. ;)

rewriting the hdspmixer doesn't sound like a bad idea to me. reworking the 
sampling rate switching would be a great help as well: the hdsp is the only 
device i know, which requires an external program to switch the sampling rate. a 
few years ago i had a brief look at the driver, but this seemed to be a non-
trivial problem.

cheers, tim

-- 
tim@klingt.org
http://tim.klingt.org

Most of the trouble in this world has been caused by folks who can't
mind their own business, because they have no business of their own to
mind, any more than a smallpox virus has.
  William S. Burroughs

  reply	other threads:[~2011-04-04  7:21 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-02 12:28 hdspmixer broken for 96khz Tim Blechmann
2011-04-03  8:16 ` Adrian Knoth
2011-04-03  9:39   ` Tim Blechmann
2011-04-03 10:25     ` Adrian Knoth
2011-04-04  7:21       ` Tim Blechmann [this message]
2011-04-04  7:58     ` Takashi Iwai
2011-04-04  8:05       ` Tim Blechmann
2011-04-04 10:01         ` Adrian Knoth
2011-04-05 10:29           ` Tim Blechmann

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='inbrhl$icc$1@dough.gmane.org' \
    --to=tim@klingt.org \
    --cc=alsa-devel@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).