Hi Andrei, On 08/28/2012 09:40 AM, Andrei Emeltchenko wrote:
Hi Andrzej, On Mon, Aug 27, 2012 at 04:47:30PM +0200, Andrzej Kaczmarek wrote:Set LE called when hdev is up but still has HCI_AUTO_OFF flag set will only change dev_flags and enable feature in host but won't enable feature in controller. As a resulit it's not possible to e.g. start LE only or interleaved discovery due to missing LE Supported flag in local features. This patch ensures HCI Write LE Host Supported is sent when Set Powered is called to clear HCI_AUTO_OFF flag. Signed-off-by: Andrzej Kaczmarek <andrzej.kaczmarek@xxxxxxxxx> --- net/bluetooth/mgmt.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index ccc6716..f73441f 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -2897,6 +2897,17 @@ int mgmt_powered(struct hci_dev *hdev, u8 powered) hci_send_cmd(hdev, HCI_OP_WRITE_SSP_MODE, 1, &ssp); } + if (test_bit(HCI_LE_ENABLED, &hdev->dev_flags)) { + struct hci_cp_write_le_host_supported cp; + + memset(&cp, 0, sizeof(cp));Why memset is needed here?
Not really needed, I forgot to remove it after copy&paste from other code. I'll send v2 later with memset removed.
BR, Andrzej -- 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
