Hi,
On 11/14/2012 08:27 PM, Siarhei Siamashka wrote:
On Tue, 30 Oct 2012 10:39:28 +0100
+ if (state->increment == 1)
+ state->sbc_analyze_8s = sbc_analyze_1b_8s_simd;
#endif
}
This is not enough. As I commented earlier in
http://permalink.gmane.org/gmane.linux.bluez.kernel/31567
"neon code also provides optimized "sbc_enc_process_input_*" functions,
which are not going to work correctly for mSBC:
state->sbc_enc_process_input_8s_le = sbc_enc_process_input_8s_le_neon;
state->sbc_enc_process_input_8s_be = sbc_enc_process_input_8s_be_neon;"
Indeed, this is a mistake :
I wanted to use the neon analysis in conjonction with simd input processing.
Instead, the patch would look like this :
+ if (state->increment == 1) {
+ state->sbc_enc_process_input_8s_be =
+ sbc_enc_process_input_8s_be;
+ state->sbc_enc_process_input_8s_le =
+ sbc_enc_process_input_8s_le;
+ }
Damned it is 82 chars long!
Unfortunately I can't test this one. I can't even build it. I could but
that's gonna take a lot of time.
Regards,
Frédéric
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
