diff options
author | Axel Castaneda Gonzalez <x0055901@ti.com> | 2012-02-03 14:32:55 -0600 |
---|---|---|
committer | Axel Castaneda Gonzalez <x0055901@ti.com> | 2012-02-13 20:40:30 -0600 |
commit | a6dd4abcbf3007600be621bc886182de7bd2a25d (patch) | |
tree | 9e0f732c437cbe6f5fca7251d8a4a6749afc6365 /pcm.c | |
parent | 43625117cf24ac933819a10c2da7681252042d49 (diff) |
tinyplay: Use buffer size in bytes instead of frames
pcm_write expects to receive buffer size given in bytes,
but tinyplay sends as a parameter the size in frames, which
causes audio sample data loss for some scenarios where more
than 2 channels are used.
For example:
When using 6 channels and 1024 as period size:
-On tinyplay side
size=4096 (buffer_size)
-On pcm_write (pcm.c)
x.frames = count / (pcm->config.channels *
pcm_format_to_bits(pcm->config.format) / 8);
x.frames = 4096 / (6 * 16/8)
x.frames = 341.33 (decimal part is discarded, 4 bytes are lost every
pcm_write)
--
In this case, 4 bytes are lost, which causes a shift
of the channels.
Use buffer size in bytes in order to avoid this problem.
Signed-off-by: Axel Castaneda Gonzalez <x0055901@ti.com>
Diffstat (limited to 'pcm.c')
0 files changed, 0 insertions, 0 deletions