All of lore.kernel.org
 help / color / mirror / Atom feed
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 19:58:34 +0200	[thread overview]
Message-ID: <5160624A.7070804@gmail.com> (raw)
In-Reply-To: <516060C1.9000101@sonarnerd.net>

On 06.04.2013 19:52, Jussi Laako wrote:
>> All audio formats are specified in *bytes* for sample and buffer sizes,
>> so we will also keep it that way for DSD.
> 
> Umm, IIRC, all the ALSA PCM API snd_pcm_*_period_size() stuff is in 
> number of frames? (number of samples per channel)

I was talking about the SND_PCM_FORMAT_ definitions that I augmented.
What they denote is bytes per sample frame, not bits.

> Thus size of the 
> buffer is period size * sample size * channels (* nperiods). So if we 
> would use 2.8 MHz as sampling rate for DSD, then buffer size would be 
> period size * 1/8 * channels, since one sample of DSD is eight' of a byte.

No, you would choose 1/8 of the actual sample rate, because the actual
transport goes in multiple of 8 bits.

> And sampling rate is number of frames per second. So it is quite clear 
> and non-confusing.

... and one frame is also just 8 bits (or 16, for the second format). I
think that matches the logic quite well.


Daniel

  reply	other threads:[~2013-04-06 17:58 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
2013-04-06 17:52                 ` Jussi Laako
2013-04-06 17:58                   ` Daniel Mack [this message]
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=5160624A.7070804@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.