Re: [PATCH] tools/btgatt-client: Fixed register-notify prompt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Maxime,

On Thu, Dec 22, 2016 at 11:25 AM, Maxime Chevallier
<maxime.chevallier@xxxxxxxx> wrote:
> When using the register-notify command in btgatt-client, the prompt
> would be broken :
>
> [GATT client]# register-notify 0x005f
> Registering notify handler with id: 2
> [GATT client]# [GATT client]# Registered notify handler![GATT client]#
>
> This commit fixes the prompt by using the correct printing function and
> adding an extra '\n' :
>
> [GATT client]# register-notify 0x005f
> Registering notify handler with id: 2
> [GATT client]# Registered notify handler
> [GATT client]#
> ---
>  tools/btgatt-client.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/btgatt-client.c b/tools/btgatt-client.c
> index 4c8c9dd..40a03b1 100644
> --- a/tools/btgatt-client.c
> +++ b/tools/btgatt-client.c
> @@ -1114,7 +1114,7 @@ static void register_notify_cb(uint16_t att_ecode, void *user_data)
>                 return;
>         }
>
> -       PRLOG("Registered notify handler!");
> +       PRLOG("Registered notify handler\n");
>  }
>
>  static void cmd_register_notify(struct client *cli, char *cmd_str)
> @@ -1149,7 +1149,7 @@ static void cmd_register_notify(struct client *cli, char *cmd_str)
>                 return;
>         }
>
> -       PRLOG("Registering notify handler with id: %u\n", id);
> +       printf("Registering notify handler with id: %u\n", id);
>  }
>
>  static void unregister_notify_usage(void)
> --
> 2.1.4

Applied, thanks.


-- 
Luiz Augusto von Dentz
--
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



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux