Skip to contentSkip to conversation

HCI Le time out

DISCUSSION Marco Sinigaglia 2 participants 2 messages Mar 1, 2011 Bluetooth drivers

lore
Share & tools
lore

Marco Sinigaglia

author
#0 lore raw

Hi All,

I am using CSR 8505 A06 and, using LE, I have some timeout problems raised on hci.c. The following LE functions use a 100ms for the hci_send_req cmd.
- hci_le_set_scan_parameters
- hci_le_set_advertise_enable
- hci_le_set_scan_enable

The 100ms is not enough becouse the chip can be in deep sleep. Indeed, increasing it to 1000 the problem is fixed.

Would be possible to check in this modifications?

Cheers
Marco Sinigaglia

@@ -2646,7 +2645,7 @@

     rq.rparam = &status;
     rq.rlen = 1;

- if (hci_send_req(dd, &rq, 100) < 0)
+ if (hci_send_req(dd, &rq, 1000) < 0)

         return -1;

     if (status) {
@@ -2680,7 +2679,7 @@
     rq.rparam = &status;
     rq.rlen = 1;

- if (hci_send_req(dd, &rq, 100) < 0)
+ if (hci_send_req(dd, &rq, 1000) < 0)

         return -1;

     if (status) {
@@ -2708,7 +2707,7 @@
     rq.rparam = &status;
     rq.rlen = 1;

- if (hci_send_req(dd, &rq, 100) < 0)
+ if (hci_send_req(dd, &rq, 1000) < 0)

         return -1;

Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom

#1 lore raw

Hi Marco,

Quoted 9 lines · Marco Sinigaglia

Hi All,

I am using CSR 8505 A06 and, using LE, I have some timeout problems raised on hci.c.
The following LE functions use a 100ms for the hci_send_req cmd.
- hci_le_set_scan_parameters
- hci_le_set_advertise_enable
- hci_le_set_scan_enable

Quoted 4 lines

The 100ms is not enough becouse the chip can be in deep sleep. Indeed, increasing it to 1000 the problem is fixed.

Would be possible to check in this modifications?

I would suggest you to follow the patch submission guidelines [1] before send modifications to this list. And if you are just starting in bluez development, should read this:
http://www.bluez.org/development/

Regards,

Anderson Briglia

[1] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob_plain;f=Documentation/SubmittingPatches;hb=HEAD

Quoted 42 lines

Cheers
Marco Sinigaglia

(quoted diff) +3 -3
@@ -2646,7 +2645,7 @@
     rq.rparam = &status;
     rq.rlen = 1;
 
-    if (hci_send_req(dd, &rq, 100) < 0)
+    if (hci_send_req(dd, &rq, 1000) < 0)
         return -1;
 
     if (status) {

@@ -2680,7 +2679,7 @@
     rq.rparam = &status;
     rq.rlen = 1;
 
-    if (hci_send_req(dd, &rq, 100) < 0)
+    if (hci_send_req(dd, &rq, 1000) < 0)
         return -1;
 
     if (status) {

@@ -2708,7 +2707,7 @@
     rq.rparam = &status;
     rq.rlen = 1;
 
-    if (hci_send_req(dd, &rq, 100) < 0)
+    if (hci_send_req(dd, &rq, 1000) < 0)
         return -1;

Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom --
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Signature · 3 lines
INdT - Instituto Nokia de tecnologia
+55 2126 1122
http://techblog.briglia.net