Hi Johan, On 00:09 Thu 25 Oct, Johan Hedberg wrote: > diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c > index dc60d31..ed6b1e2 100644 > --- a/net/bluetooth/hci_event.c > +++ b/net/bluetooth/hci_event.c > @@ -790,9 +790,24 @@ static void hci_set_le_support(struct hci_dev *hdev) > cp.simul = !!lmp_le_br_capable(hdev); > } > > + /* If the host features don't reflect the desired state for LE > + * then send the write_le_host_supported command. The command > + * complete handler for it will take care of any necessary > + * subsequent commands like set_adv_enable. > + * > + * If the host features for LE are already correct and > + * peripheral mode is enabled directly send the le_set_adv > + * command. The value of &cp.le is used so that advertising will > + * not be enabled in the exceptional case that LE for some > + * reason isn't enabled - something that should only be possible > + * if someone is doing direct raw access to HCI. > + */ > if (cp.le != !!lmp_host_le_capable(hdev)) > hci_send_cmd(hdev, HCI_OP_WRITE_LE_HOST_SUPPORTED, sizeof(cp), > &cp); > + else if (test_bit(HCI_LE_PERIPHERAL, &hdev->dev_flags)) > + hci_send_cmd(hdev, HCI_OP_LE_SET_ADV_ENABLE, sizeof(cp.le), > + &cp.le); > } I agree with Marcel, and one point that worried me was this unconditional advertising, I feel that we should be smarter about when to start advertising, for example, here we are not taking into account the user's visiblity settings. Cheers, -- Vinicius -- 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
