From: Jassi Brar <jassisinghbrar@gmail.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: Sangbeom Kim <sbkim73@samsung.com>,
alsa-devel@alsa-project.org, broonie@opensource.wolfsonmicro.com,
lrg@slimlogic.co.uk, Seungwhan Youn <claude.youn@gmail.com>
Subject: Re: [PATCH v5 4/4] ASoC: SAMSUNG: Add I2S0 internal dma driver
Date: Wed, 13 Jul 2011 19:50:21 +0530 [thread overview]
Message-ID: <CABb+yY0CuNj45w-dAJR49Y3ms_g-4wsOe5GbPMavevCwqJ2MNQ@mail.gmail.com> (raw)
In-Reply-To: <s5hk4bmfw8x.wl%tiwai@suse.de>
On Wed, Jul 13, 2011 at 1:27 PM, Takashi Iwai <tiwai@suse.de> wrote:
>> An important part missing in Claude's quote is
>> + .periods_min = 1,
>> + .periods_max = 2,
>>
>> When the periods_max is only 2, the calculations in ALSA mid-layer
>> seem to always
>> decide on using two equal size periods.
>>
>> I didn't dig in the logic but wasn't able to get any application to
>> configure something
>> like {Period_size = 90% of Buffer_size} which is actually beneficial for
>> Low-Power-Audio-Mode.
>
> Even you don't set the integer constraint, right?
Right, I didn't set set the integer constraint in this idma driver.
> Does it happen with aplay with -Dhw?
Yes, I remember testing without plugins.
> In the driver side, there is a define RULES_DEBUG in
> sound/core/pcm_native.c, which will enable debug prints of hw-refines.
> A similar flag is found in alsa-lib/src/pcm/pcm_params.c, too.
I don't have access to the h/w anymore.
Maybe someone from Samsung could dig into it and find something useful ?
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2011-07-13 14:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-13 1:25 [PATCH v5 4/4] ASoC: SAMSUNG: Add I2S0 internal dma driver Sangbeom Kim
2011-07-13 3:08 ` Seungwhan Youn
2011-07-13 5:41 ` Jassi Brar
2011-07-13 5:54 ` Takashi Iwai
2011-07-13 6:14 ` Jassi Brar
2011-07-13 6:18 ` Jassi Brar
2011-07-13 7:57 ` Takashi Iwai
2011-07-13 14:20 ` Jassi Brar [this message]
2011-07-18 23:44 ` Seungwhan Youn
2011-07-18 11:49 ` Jassi Brar
2011-07-18 12:38 ` Liam Girdwood
2011-07-19 15:04 ` Mark Brown
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=CABb+yY0CuNj45w-dAJR49Y3ms_g-4wsOe5GbPMavevCwqJ2MNQ@mail.gmail.com \
--to=jassisinghbrar@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=claude.youn@gmail.com \
--cc=lrg@slimlogic.co.uk \
--cc=sbkim73@samsung.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.