Hi Johan, On Wed, Apr 3, 2013 at 9:52 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > + switch (hdev->req_status) { > + case HCI_REQ_DONE: > + err = -bt_to_errno(hdev->req_result); > + break; > + > + case HCI_REQ_CANCELED: > + err = -hdev->req_result; > + break; > + > + default: > + err = -ETIMEDOUT; > + break; > + } > + > + hdev->req_status = hdev->req_result = 0; > + > + BT_DBG("%s end: err %d", hdev->name, err); > + > + if (err < 0) > + return ERR_PTR(err); Isn't this always true (due to the switch() avove) ? Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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