aboutsummaryrefslogtreecommitdiff
path: root/mixer.c
diff options
context:
space:
mode:
authorAxel Castaneda Gonzalez <x0055901@ti.com>2012-02-03 14:32:55 -0600
committerAxel Castaneda Gonzalez <x0055901@ti.com>2012-02-13 20:40:30 -0600
commita6dd4abcbf3007600be621bc886182de7bd2a25d (patch)
tree9e0f732c437cbe6f5fca7251d8a4a6749afc6365 /mixer.c
parent43625117cf24ac933819a10c2da7681252042d49 (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 'mixer.c')
0 files changed, 0 insertions, 0 deletions