Hi Luiz,
> This makes sure BT_CODEC_MSBC is used by default if socket user attempt
> to use BT_VOICE_TRANSPARENT.
>
> Fixes: b2af264ad3af ("Bluetooth: Add support for HCI_Enhanced_Setup_Synchronous_Connection command")
> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
> ---
> net/bluetooth/sco.c | 12 +++---------
> 1 file changed, 3 insertions(+), 9 deletions(-)
>
> diff --git a/net/bluetooth/sco.c b/net/bluetooth/sco.c
> index 8eabf41b2993..b35c772efc7e 100644
> --- a/net/bluetooth/sco.c
> +++ b/net/bluetooth/sco.c
> @@ -879,15 +879,9 @@ static int sco_sock_setsockopt(struct socket *sock, int level, int optname,
> }
>
> sco_pi(sk)->setting = voice.setting;
> - hdev = hci_get_route(&sco_pi(sk)->dst, &sco_pi(sk)->src,
> - BDADDR_BREDR);
> - if (!hdev) {
> - err = -EBADFD;
> - break;
> - }
> - if (enhanced_sco_capable(hdev) &&
> - voice.setting == BT_VOICE_TRANSPARENT)
> - sco_pi(sk)->codec.id = BT_CODEC_TRANSPARENT;
> + if (voice.setting == BT_VOICE_TRANSPARENT)
> + sco_pi(sk)->codec.id = BT_CODEC_MSBC;
> +
> hci_dev_put(hdev);
> break;
why are you removing the rest and especially the eSCO check?
Regards
Marcel
