Hi Marcel, On Sat, Oct 19, 2013 at 10:09 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > +static int conn_min_interval_set(void *data, u64 val) > +{ > + struct hci_dev *hdev = data; > + > + if (val < 0x0006 || val > 0x0c80 || val > hdev->le_conn_max_interval) > + return -EINVAL; > + > + hci_dev_lock(hdev); > + hdev->le_conn_min_interval= val; Minor style issue: missing space before "=". > + hci_dev_unlock(hdev); > + > + return 0; > +} > [snip] > +static int conn_max_interval_set(void *data, u64 val) > +{ > + struct hci_dev *hdev = data; > + > + if (val < 0x0006 || val > 0x0c80 || val < hdev->le_conn_min_interval) > + return -EINVAL; > + > + hci_dev_lock(hdev); > + hdev->le_conn_max_interval= val; Same above. > + hci_dev_unlock(hdev); > + > + return 0; > +} Best 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