From: Marcel Holtmann <marcel@holtmann.org>
To: Howard Chung <howardchung@google.com>
Cc: linux-bluetooth <linux-bluetooth@vger.kernel.org>,
Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
mmandlik@chromium.orgi, Miao-chen Chou <mcchou@chromium.org>,
Alain Michaud <alainm@chromium.org>,
"David S. Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
Johan Hedberg <johan.hedberg@gmail.com>,
open list <linux-kernel@vger.kernel.org>,
netdev@vger.kernel.org
Subject: Re: [PATCH v5 4/4] Bluetooth: Add toggle to switch off interleave scan
Date: Fri, 25 Sep 2020 19:36:33 +0200 [thread overview]
Message-ID: <D9C4CE14-371B-46DD-8FF7-FBA4593788B6@holtmann.org> (raw)
In-Reply-To: <20200923172129.v5.4.I756c1fecc03bcc0cd94400b4992cd7e743f4b3e2@changeid>
Hi Howard,
> This patch add a configurable parameter to switch off the interleave
> scan feature.
>
> Signed-off-by: Howard Chung <howardchung@google.com>
> Reviewed-by: Alain Michaud <alainm@chromium.org>
> ---
>
> (no changes since v4)
>
> Changes in v4:
> - Set EnableAdvMonInterleaveScan default to Disable
> - Fix 80 chars limit in mgmt_config.c
>
> include/net/bluetooth/hci_core.h | 1 +
> net/bluetooth/hci_core.c | 1 +
> net/bluetooth/hci_request.c | 3 ++-
> net/bluetooth/mgmt_config.c | 5 +++++
> 4 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
> index cfede18709d8f..b0225b80152cc 100644
> --- a/include/net/bluetooth/hci_core.h
> +++ b/include/net/bluetooth/hci_core.h
> @@ -363,6 +363,7 @@ struct hci_dev {
> __u32 clock;
> __u16 advmon_allowlist_duration;
> __u16 advmon_no_filter_duration;
> + __u16 enable_advmon_interleave_scan;
I really have to see the patch for doc/mgmt-api.txt first and I am certainly not really in favor of using an uint16 for a simple boolean on/off value just because the kernel code is simpler that way.
Regards
Marcel
prev parent reply other threads:[~2020-09-25 17:36 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-23 9:22 [PATCH v5 1/4] Bluetooth: Interleave with allowlist scan Howard Chung
2020-09-23 9:22 ` [PATCH v5 2/4] Bluetooth: Handle system suspend resume case Howard Chung
2020-09-23 9:22 ` [PATCH v5 3/4] Bluetooth: Handle active scan case Howard Chung
2020-09-23 9:22 ` [PATCH v5 4/4] Bluetooth: Add toggle to switch off interleave scan Howard Chung
2020-09-25 17:36 ` Marcel Holtmann [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=D9C4CE14-371B-46DD-8FF7-FBA4593788B6@holtmann.org \
--to=marcel@holtmann.org \
--cc=alainm@chromium.org \
--cc=davem@davemloft.net \
--cc=howardchung@google.com \
--cc=johan.hedberg@gmail.com \
--cc=kuba@kernel.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=mcchou@chromium.org \
--cc=mmandlik@chromium.orgi \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.