Hi Johan,
On Thu, Nov 03, 2011 at 12:58:51PM +0200, johan.hedberg@xxxxxxxxx wrote:
> From: Johan Hedberg <johan.hedberg@xxxxxxxxx>
>
> When an adapter gets powered off or is removed any pending commands
> should receive a ENETDOWN or ENODEV status response.
>
> Signed-off-by: Johan Hedberg <johan.hedberg@xxxxxxxxx>
> ---
> net/bluetooth/mgmt.c | 19 ++++++++++++++++++-
> 1 files changed, 18 insertions(+), 1 deletions(-)
>
> diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> index cbc8a6d..26b58ad 100644
> --- a/net/bluetooth/mgmt.c
> +++ b/net/bluetooth/mgmt.c
> @@ -262,7 +262,7 @@ static void mgmt_pending_foreach(u16 opcode, int index,
>
> cmd = list_entry(p, struct pending_cmd, list);
>
> - if (cmd->opcode != opcode)
> + if (opcode > 0 && cmd->opcode != opcode)
> continue;
>
> if (index >= 0 && cmd->index != index)
> @@ -1949,6 +1949,14 @@ done:
> return err;
> }
>
> +static void cmd_status_rsp(struct pending_cmd *cmd, void *data)
> +{
> + uint8_t *status = data;
> +
> + cmd_status(cmd->sk, cmd->index, cmd->opcode, *status);
> + mgmt_pending_remove(cmd);
> +}
> +
> int mgmt_index_added(u16 index)
> {
> return mgmt_event(MGMT_EV_INDEX_ADDED, index, NULL, 0, NULL);
> @@ -1956,6 +1964,10 @@ int mgmt_index_added(u16 index)
>
> int mgmt_index_removed(u16 index)
> {
> + uint8_t status = ENODEV;
Does it make sense to mix type naming here and below? (u8, u16, uint8_t, uint16_t)
Best regards
Andrei Emeltchenko
> +
> + mgmt_pending_foreach(0, index, cmd_status_rsp, &status);
> +
> return mgmt_event(MGMT_EV_INDEX_REMOVED, index, NULL, 0, NULL);
> }
>
> @@ -1992,6 +2004,11 @@ int mgmt_powered(u16 index, u8 powered)
>
> mgmt_pending_foreach(MGMT_OP_SET_POWERED, index, mode_rsp, &match);
>
> + if (!powered) {
> + uint8_t status = ENETDOWN;
> + mgmt_pending_foreach(0, index, cmd_status_rsp, &status);
> + }
> +
> ev.val = powered;
>
> ret = mgmt_event(MGMT_EV_POWERED, index, &ev, sizeof(ev), match.sk);
> --
> 1.7.7.1
>
> --
> 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
--
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
