Re: [PATCH v3 4/7] Bluetooth: Add handler of MGMT_OP_REMOVE_ADV_MONITOR

Hi Jakub, I uploaded v4 to address these. Thanks for the reminder. Regards, Miao On Fri, Jun 12, 2020 at 10:49 AM Jakub Kicinski wrote: > > On Thu, 11 Jun 2020 23:15:26 -0700 Miao-chen Chou wrote: > > This adds the request handler of MGMT_OP_REMOVE_ADV_MONITOR command. > > Note that the

Re: [PATCH v3 4/7] Bluetooth: Add handler of MGMT_OP_REMOVE_ADV_MONITOR

On Thu, 11 Jun 2020 23:15:26 -0700 Miao-chen Chou wrote: > This adds the request handler of MGMT_OP_REMOVE_ADV_MONITOR command. > Note that the controller-based monitoring is not yet in place. This > removes the internal monitor(s) without sending HCI traffic, so the > request returns immediately.

Re: [PATCH v3 4/7] Bluetooth: Add handler of MGMT_OP_REMOVE_ADV_MONITOR

Hi Miao-chen, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on bluetooth-next/master] [also build test WARNING on net-next/master sparc-next/master net/master next-20200611] [cannot apply to bluetooth/master v5.7] [if your patch is applied to the wrong git

[PATCH v3 4/7] Bluetooth: Add handler of MGMT_OP_REMOVE_ADV_MONITOR

This adds the request handler of MGMT_OP_REMOVE_ADV_MONITOR command. Note that the controller-based monitoring is not yet in place. This removes the internal monitor(s) without sending HCI traffic, so the request returns immediately. The following test was performed. - Issue btmgmt advmon-remove