Hi Lizardo, On Tue, May 28, 2013, Anderson Lizardo wrote: > On Tue, May 28, 2013 at 3:07 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > > +int mgmt_set_powered_failed(struct hci_dev *hdev, int err) > > +{ > > + struct pending_cmd *cmd; > > + > > + cmd = mgmt_pending_find(MGMT_OP_SET_POWERED, hdev); > > + if (!cmd) > > + return -ENOENT; > > + > > + err = cmd_status(cmd->sk, hdev->id, MGMT_OP_SET_POWERED, > > + MGMT_STATUS_FAILED); > > + > > + mgmt_pending_remove(cmd); > > + > > + return err; > > +} > > + > > Was the "err" parameter value supposed to be used on this function? As I mentioned in my reply to Marcel it's there so this can be easily extended to have a special mapping to the exact mgmt error returned (and now it seems that this mapping will be implemented before this goes upstream). Johan -- 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