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

From: Jakub Kicinski
Date: Fri Jun 12 2020 - 13:49:59 EST


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.
>
> The following test was performed.
> - Issue btmgmt advmon-remove with valid and invalid handles.
>
> Signed-off-by: Miao-chen Chou <mcchou@xxxxxxxxxxxx>

Still doesn't build cleanly with W=1 C=1

net/bluetooth/mgmt.c:4009:46: warning: incorrect type in argument 2 (different base types)
net/bluetooth/mgmt.c:4009:46: expected unsigned short [usertype] handle
net/bluetooth/mgmt.c:4009:46: got restricted __le16 [usertype] monitor_handle
net/bluetooth/mgmt.c:4018:29: warning: cast from restricted __le16