From: "Christian Hoene" <christian.hoene@gmx.net>
To: "'BlueZ development'" <bluez-devel@lists.sourceforge.net>
Subject: [Bluez-devel] Bug in SBC
Date: Mon, 17 Nov 2008 16:57:48 +0100 [thread overview]
Message-ID: <000801c948cd$41364120$c3a2c360$@hoene@gmx.net> (raw)
Hello,
Let me forward some comments from Frans de Bont, the inventor of SBC. He
found a bug in the encoder but is not allowed to change the BlueZ
implementation. Thus, I forward you his error description because these
weeks I hardly have any time to look at the source code myself and - I have
to admit - I am getting older and coding lazy.
With best regards,
Christian Hoene
PS:
Here Frans comments:
I also have taken a quick look into the filtering part of the encoder. It
seems that this code does not behave as the reference code. Unfortunately, I
cannot disclose this part of the reference source code. However, if someone
wants to correct this, the flow chart in the MPEG-1 Audio standard could be
taken as an example with slightly difference parameters, i.e. subbands=32
and filterlength=512.
You can find the referred flow chart in Figure C.4 on page 78 (in my version
of the ISO/IEC 11172-3 standard). Please note that the decoder flow charts
in A2DP have been derived from Figure A.2 on page 39.
You can find ... public version on:
http://www.mp3-tech.org/programmer/docs/iso11172-3.zip
For the encoder filterbank, you can find this in annex_c.doc, page 13,
FIGURE 3-C.2, "Layer I, II encoder flow chart".
The decoder filterbank is shown in annex_ab.doc, page 2, Figure 3-A.3,
"Layer III decoder flow chart".
BTW, I noticed that earlier versions of the Bluez SBC codec are indeed based
on the MPEG-1 flow charts. Probably the conversion from floating-point to
fixed-point has introduced the quality degradation.
Best regards,
Frans
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
reply other threads:[~2008-11-17 15:57 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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='000801c948cd$41364120$c3a2c360$@hoene@gmx.net' \
--to=christian.hoene@gmx.net \
--cc=bluez-devel@lists.sourceforge.net \
--cc=hoene@uni-tuebingen.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.