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: Tue, 05 Apr 2011 12:29:09 +0200	[thread overview]
Message-ID: <ineqtm$cmp$1@dough.gmane.org> (raw)
In-Reply-To: 20110404100132.GQ28811@ltw.loris.tv

>> > Any chance to recover the old broken configuration?
>> > Then we can track down the bug.
>> attached
> 
> This isn't the old configuration, it's the new one. (it has my new magic
> code "HDSPMixer v1" right at the beginning)
> 
> The foremost question is: when did you wrote your old configuration
> file?

hm, good question! i have been using this preset file for years, maybe i've 
stored it with the new hdspmixer. but with this preset file, the 96khz mixer was 
not shown.


> But what I can say for sure is: if you used hdspmixer before 1.0.24 on a
> non-MADI card like Multiface/Digiface, loading this preset file in
> 1.0.24 would break the configuration. Is this what we're talking about?

exactly!


> Reason: defines.h contains HDSP_MAX_CHANNELS, it was 26 until adding
> support for MADI cards, bumping this value to 64.
> And now let's have a
> look at the preset load/restore code:

oh shit :/


> The only possible way to circumvent this is to check the file size.
> 3*3*8*26*some_constant is different than 3*3*8*64*some_constant, and we
> would be able to guess if it's a pre-1.0.24 preset file or a newer one.

that sounds reasonable. at least printing a big fat warning would be 
appreciated!

btw, when working on the hdspmixer it would be cool, if you keep the fireface in 
mind. someone seems to work on ffado support and i suppose the fireface mixer 
can share some code with the hdspmixer.

cheers, tim

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

Contrary to general belief, an artist is never ahead of his time but
most people are far behind theirs.
  Edgar Varèse


_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

      reply	other threads:[~2011-04-05 10:29 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
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 [this message]

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='ineqtm$cmp$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).