On Fri, Sep 17, 2010 at 01:46:58PM +0200, ext Emeltchenko Andrei wrote: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > Check eSCO / SCO type before setting retransmission effort flag > in Setup SCO command. We found that 0x01 is better for power > consupmtion but we cannot use it always since controller tries > to setup eSCO even for old devices. > > Might be controller-specific. > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> Revieved-by: Ville Tervo <ville.tervo@xxxxxxxxx> > --- > net/bluetooth/hci_conn.c | 8 +++++++- > 1 files changed, 7 insertions(+), 1 deletions(-) > > diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c > index 0b1e460..145993f 100644 > --- a/net/bluetooth/hci_conn.c > +++ b/net/bluetooth/hci_conn.c > @@ -150,7 +150,13 @@ void hci_setup_sync(struct hci_conn *conn, __u16 handle) > cp.rx_bandwidth = cpu_to_le32(0x00001f40); > cp.max_latency = cpu_to_le16(0xffff); > cp.voice_setting = cpu_to_le16(hdev->voice_setting); > - cp.retrans_effort = 0xff; > + > + /* If remote device supports eSCO use optimization for power > + retransmission effort, otherwise use standard flag do-not-care */ > + if (conn->link->features[3] & LMP_ESCO) > + cp.retrans_effort = 0x01; > + else > + cp.retrans_effort = 0xff; > > hci_send_cmd(hdev, HCI_OP_SETUP_SYNC_CONN, sizeof(cp), &cp); > } > -- > 1.7.0.4 > > -- > 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 -- 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
