Hi Marcel,
On Wed, Oct 16, 2013 at 3:16 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> +static void update_scan_rsp_data(struct hci_request *req)
> +{
> + struct hci_dev *hdev = req->hdev;
> + struct hci_cp_le_set_scan_rsp_data cp;
> + u8 len;
> +
> + if (!lmp_le_capable(hdev))
> + return;
> +
> + memset(&cp, 0, sizeof(cp));
> +
> + len = create_scan_rsp_data(hdev, cp.data);
> +
> + if (hdev->adv_data_len == len &&
> + memcmp(cp.data, hdev->adv_data, len) == 0)
> + return;
> +
> + memcpy(hdev->adv_data, cp.data, sizeof(cp.data));
> + hdev->adv_data_len = len;
Shouldn't you be using hdev->scan_rsp_data/scan_rsp_data_len here?
(I still haven't read the later patches, so this may already be fixed.)
> +
> + cp.length = len;
> +
> + hci_req_add(req, HCI_OP_LE_SET_SCAN_RSP_DATA, sizeof(cp), &cp);
> +}
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
