Hi Siarhei, Since only neon is concerned by this, I'd rather add a one liner like this : #ifdef SBC_BUILD_WITH_NEON_SUPPORT sbc_init_primitives_neon(state); + + if (state->increment == 1) + state->sbc_analyze_4b_8s = sbc_analyze_1b_8s_simd; #endif It is more explicit, doesn't change priority and doesn't add needless code to other implementations. And what about sbc_analyze_8s? Regarding point 2, this is the reason why patch 4 was a bit bigger, the simd implementation is complete. 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