Hi Jakub,
> This patch splits triggering of le and interleaved scan into
> separate cases.
>
> Signed-off-by: Jakub Pawlowski <jpawlowski@xxxxxxxxxx>
> ---
> net/bluetooth/mgmt.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> index 92fa8e88..9d3c7a1 100644
> --- a/net/bluetooth/mgmt.c
> +++ b/net/bluetooth/mgmt.c
> @@ -4009,9 +4009,13 @@ static bool trigger_discovery(struct hci_request *req, u8 *status)
> break;
>
> case DISCOV_TYPE_LE:
> + if (!trigger_le_scan(req, status,
> + cpu_to_le16(DISCOV_LE_SCAN_INT)))
> + return false;
> + break;
> +
> case DISCOV_TYPE_INTERLEAVED:
> - if (hdev->discovery.type == DISCOV_TYPE_INTERLEAVED &&
> - !hci_dev_test_flag(hdev, HCI_BREDR_ENABLED)) {
> + if (!hci_dev_test_flag(hdev, HCI_BREDR_ENABLED)) {
> *status = MGMT_STATUS_NOT_SUPPORTED;
> return false;
> }
see my re-ordering with a fall through statement from my previous comment. Lets split them out properly first and then the sequence on how things evolve will make more sense. And I think this patch is then no longer needed.
Regards
Marcel
--
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
