Skip to contentSkip to conversation

Patch to correct SBC gain mismatch and to test the encoding performance

Bluez-devel Christian Hoene 3 participants 4 messages Oct 31, 2008 – Nov 3, 2008 Bluetooth drivers

lore
Share & tools
lore
#0 lore raw

Hello,

attached you will find a patch to correct the following issues.

1.) Gain of the decoder is corrected (before it was 6.02dB or twice too low) 2.) Gain of the encoder is corrected (empirical gathered values were used to correct the filter tables)
3.) "sbcenc" supports more command line options.
4.) A testing procedure for SBC is added including a script to test and a remote interface to PEAQ. It requires the reference version of SBC (available at the Bluetooth SIG website) and wine. Please note, that the remote access to PEAQ is restricted and only available by request for a limited time. Requesty from all unknown IP addresses are blocked by default.

With best regards,

Christian

--------------------------------------------------------
Dr.-Ing. Christian Hoene
Computer Networks and Internet, University of Tübingen Sand 13, 72076 Tübingen, Germany, Phone +49 7071 2970532
http://net.informatik.uni-tuebingen.de/~hoene

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

maintainer
#1 lore raw

Hi Christian,

Quoted 8 lines

attached you will find a patch to correct the following issues.

1.) Gain of the decoder is corrected (before it was 6.02dB or twice too low)
2.) Gain of the encoder is corrected (empirical gathered values were used to
correct the filter tables)
3.) "sbcenc" supports more command line options.

it would be nice if you just follow our coding style conventions. I fixed this up for you this time. Please compare the result that I pushed to the repository and your version.

Quoted 9 lines

4.) A testing procedure for SBC is added including a script to test and a
remote interface to PEAQ. It requires the reference version of SBC (available at the Bluetooth SIG website) and wine. Please note, that the
remote access to PEAQ is restricted and only available by request for a
limited time. Requesty from all unknown IP addresses are blocked by default.

So what is wrong with using sbctester program?

Regards

Marcel

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

#2 lore raw

Hello Marcel,

Quoted 3 lines

it would be nice if you just follow our coding style conventions. I fixed this up for you this time. Please compare the result that I pushed to the repository and your version.

Sorry, next time I will read the document beforehand.

Quoted 1 line

So what is wrong with using sbctester program?

The sbctester program is fine with me. Of course, I will do some indeed tests with PEAQ and I will complain until every things works perfect ;-)

Regards,
Christian

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel

maintainer
#3 lore raw

Hi Christian,

Quoted 6 lines
Quoted 1 line

So what is wrong with using sbctester program?

The sbctester program is fine with me. Of course, I will do some indeed
tests with PEAQ and I will complain until every things works perfect ;-)

yes please. Keep sending patches so we get the SBC encoder and decoder passing the SBC conformance tests. I do prefer if we base it around sbctester program so that we have a full open source solution (besides the reference encoder/decoder).

Regards

Marcel

-------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel