Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20200619225623.10200-1-michaelfsun@google.com>
In-Reply-To: <20200619225623.10200-1-michaelfsun@google.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Mon, 22 Jun 2020 12:29:29 -0700
Message-ID: <CABBYNZ+a_PteA5zpUx_qWL-QFST9hLQnXY7HLCzu82fx_PFORw@mail.gmail.com>
Subject: Re: [bluez PATCH v4 0/3] Add new commands in btmgmt to support adv monitor
To:     Michael Sun <michaelfsun@google.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        Luiz Augusto Von Dentz <luiz.von.dentz@intel.com>,
        Miao-chen Chou <mcchou@chromium.org>,
        ChromeOS Bluetooth Upstreaming 
        <chromeos-bluetooth-upstreaming@chromium.org>,
        Alain Michaud <alainm@chromium.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Michael,

On Fri, Jun 19, 2020 at 9:55 PM Michael Sun <michaelfsun@google.com> wrote:
>
> Hi linux-bluetooth,
>
> This series of patches add support for new advertisement monitor mgmt
> opcodes by introducing new btmgmt commands into a new submenu
> 'monitor'. The new commands are =E2=80=98features=E2=80=99, =E2=80=98add=
=E2=80=99, and =E2=80=98remove=E2=80=99. They
> provide the ability for users to query supported advertisement
> monitoring features and add/remove monitor filters through btmgmt
> console.
>
> Changes in v4:
> - Fix endianness
> - Use regular options instead of optget options per feedback, and fix
> endianness
>
> Changes in v3:
> - Fix build errors
>
> Changes in v2:
> - Move features command into submenu and fix build warnings
> - Move remove command into submenu and fix build warnings
> - Move add command into submenu and fix build warnings
>
> Michael Sun (3):
>   btmgmt: Add "monitor" btmgmt submenu and command "features"
>   btmgmt: Add command "remove" into "monitor" btmgmt submenu
>   btmgmt: Add command "add" into "monitor" btmgmt submenu
>
>  tools/btmgmt.c | 268 +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 268 insertions(+)
>
> --
> 2.27.0.111.gc72c7da667-goog
>

Applied, thanks.

--=20
Luiz Augusto von Dentz