From: Daniel Mack <zonque@gmail.com>
To: Jussi Laako <jussi@sonarnerd.net>
Cc: alsa-devel@alsa-project.org, tiwai@suse.de, clemens@ladisch.de,
demian@auraliti.com, ray@auraliti.com,
Andreas Koch <andreas@akdesigninc.com>
Subject: Re: [PATCH 2/2] alsa-lib: bring pcm.h and pcm.c in sync with the kernel list
Date: Sat, 06 Apr 2013 13:29:50 +0200 [thread overview]
Message-ID: <5160072E.8020500@gmail.com> (raw)
In-Reply-To: <515F39B9.90202@sonarnerd.net>
On 05.04.2013 22:53, Jussi Laako wrote:
>> On 04/05/2013 10:59 PM, Andreas Koch wrote:
>> However, they are interpreting the ASIO spec for DSD differently. They
>> express buffer size in samples, which means that for DSD it is measured
>> in bits.
>
> One more thing.. So the logic should go like this:
>
> DSD sample -> 1-bit -> sample rates in bits per second -> buffer and
> latency sizes in bits.
>
> ...or...
>
> DSD sample -> 8-bit -> sample rates in bytes per second -> buffer and
> latency sizes in bytes.
>
> But please, please, don't mix bits and bytes in the API, always use only
> either of these.
Well, we don't care for the ASIO spec of course, as we do not share any
compatibility layer with them. So we can define whatever we think is
useful and consistent.
All audio formats are specified in *bytes* for sample and buffer sizes,
so we will also keep it that way for DSD. Plus, that way we do not have
to fiddle with sample rate translations but always keep the hardware
state in sync to the user space API. Everything will end in utter
confusion at some point.
I will, however, add 'DOP' to the data type names, so it's clear that
the application stack has to deal with the padding bytes.
I'll send out new patches in a minute.
Thanks,
Daniel
next prev parent reply other threads:[~2013-04-06 11:29 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-27 23:32 [PATCH 1/2] alsa-lib: pcm.c: fix indentation Daniel Mack
2013-03-27 23:32 ` [PATCH 2/2] alsa-lib: bring pcm.h and pcm.c in sync with the kernel list Daniel Mack
2013-03-27 23:45 ` Jussi Laako
[not found] ` <7.0.0.16.2.20130327213632.13b07718@akdesigninc.com>
2013-03-28 7:36 ` Jussi Laako
2013-03-28 7:49 ` Jussi Laako
2013-03-28 9:54 ` Daniel Mack
2013-03-28 9:52 ` Daniel Mack
2013-04-05 7:45 ` Daniel Mack
[not found] ` <7.0.0.16.2.20130405092937.144d10d0@akdesigninc.com>
2013-04-05 20:48 ` Jussi Laako
2013-04-05 20:53 ` Jussi Laako
2013-04-06 11:29 ` Daniel Mack [this message]
2013-04-06 17:52 ` Jussi Laako
2013-04-06 17:58 ` Daniel Mack
2013-04-06 21:48 ` Jussi Laako
-- strict thread matches above, loose matches on Subject: below --
2013-04-18 8:37 [PATCH 1/2] alsa-lib: pcm.c: fix indentation Daniel Mack
2013-04-18 8:37 ` [PATCH 2/2] alsa-lib: bring pcm.h and pcm.c in sync with the kernel list Daniel Mack
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=5160072E.8020500@gmail.com \
--to=zonque@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=andreas@akdesigninc.com \
--cc=clemens@ladisch.de \
--cc=demian@auraliti.com \
--cc=jussi@sonarnerd.net \
--cc=ray@auraliti.com \
--cc=tiwai@suse.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.