All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Guilhem Tardy <guilhem_tardy@yahoo.com>
Cc: alsa-devel@alsa-project.org
Subject: Re: new pcm format(s)
Date: Thu, 22 Aug 2002 16:49:20 +0200	[thread overview]
Message-ID: <s5hy9azrlsf.wl@alsa2.suse.de> (raw)
In-Reply-To: <20020822143742.83136.qmail@web11505.mail.yahoo.com>

At Thu, 22 Aug 2002 07:37:42 -0700 (PDT),
Guilhem Tardy wrote:
> 
> > > Anyway, I found out that there's not so much space left for more formats.
> > 
> > the space must be enough.  at the last change of hw_params struct, we
> > extended the space for the sample format.
> 
> Indeed, I managed to build RC3 at last (I'll port all my stuff to RC3 over the
> WE) and found out that the SNDRV_PCM_FMTBIT_ are now defined as 64-bit unsigned
> integers. But I think it would be worthwhile in the long run to use subformats.

it's only on the kernel side for ease of use.
the api itself allows up to 256 (or more) formats.
so we can change it at any time.

> 
> > > Therefore, I added one for G726 and started adding subformats (a field that
> > > was unused until now). This also means that I need to update the function
> > > snd_pcm_format_width() in order to deal with subformats.
> > 
> > yes.
> > if you finished, please send the patch.
> > the patch is always welcome.
> 
> I built the ALSA framework (RC2) with those changes, but then all drivers must
> be updated (e.g. dummy.c at line 207) to call snd_pcm_format_width() with 2
> parameters: the format AND the subformat. Is this something you think would be
> possible?

is it needed to be a subformat?

also, note that runtime->format is not a mask but a value.
hence you can combine the format and subformat in each 16bit if
necessary.


Takashi


-------------------------------------------------------
This sf.net email is sponsored by: OSDN - Tired of that same old
cell phone?  Get a new here for FREE!
https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390

  reply	other threads:[~2002-08-22 14:49 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-21 20:47 new pcm format(s) Guilhem Tardy
2002-08-22 12:59 ` Takashi Iwai
2002-08-22 14:37   ` Guilhem Tardy
2002-08-22 14:49     ` Takashi Iwai [this message]
2002-08-22 15:12       ` Guilhem Tardy

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=s5hy9azrlsf.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=guilhem_tardy@yahoo.com \
    /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.