bluetooth: detect when bitpool has changed on sbc codec

A2DP spec allow bitpool changes midstream which is why sbc configuration
has a range of values for bitpool that the encoder can use and decoder
must support.

Bitpool changes do not affect the state of encoder/decoder so they don't
need to be reinitialize when this happens, so the impact is fairly small,
what it does change is the frame length so encoders may change the
bitpool to use the link more efficiently.
This commit is contained in:
Luiz Augusto von Dentz 2010-12-23 13:13:28 +02:00
parent 96338eec28
commit e4979ab5cf

View file

@ -1005,7 +1005,8 @@ ssize_t sbc_decode(sbc_t *sbc, const void *input, size_t input_len,
priv->frame.codesize = sbc_get_codesize(sbc); priv->frame.codesize = sbc_get_codesize(sbc);
priv->frame.length = framelen; priv->frame.length = framelen;
} } else if (priv->frame.bitpool != sbc->bitpool)
sbc->bitpool = priv->frame.bitpool;
if (!output) if (!output)
return framelen; return framelen;
@ -1076,6 +1077,9 @@ ssize_t sbc_encode(sbc_t *sbc, const void *input, size_t input_len,
sbc_encoder_init(&priv->enc_state, &priv->frame); sbc_encoder_init(&priv->enc_state, &priv->frame);
priv->init = 1; priv->init = 1;
} else if (priv->frame.bitpool != sbc->bitpool) {
priv->frame.length = sbc_get_frame_length(sbc);
priv->frame.bitpool = sbc->bitpool;
} }
/* input must be large enough to encode a complete frame */ /* input must be large enough to encode a complete frame */
@ -1140,7 +1144,7 @@ size_t sbc_get_frame_length(sbc_t *sbc)
struct sbc_priv *priv; struct sbc_priv *priv;
priv = sbc->priv; priv = sbc->priv;
if (priv->init) if (priv->init && priv->frame.bitpool == sbc->bitpool)
return priv->frame.length; return priv->frame.length;
subbands = sbc->subbands ? 8 : 4; subbands = sbc->subbands ? 8 : 4;