Hi Robert,
On Mon, Apr 2, 2018 at 10:29 PM, Robert Lubaś <robert.lubas@xxxxxxxxxxx> wrote:
> This patch adds two commands (set and get) to menu config and handles the
> incomming status message.
> ---
> mesh/config-client.c | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
> diff --git a/mesh/config-client.c b/mesh/config-client.c
> index 95297bfee..4e5e06258 100644
> --- a/mesh/config-client.c
> +++ b/mesh/config-client.c
> @@ -179,6 +179,13 @@ static bool client_msg_recvd(uint16_t src, uint8_t *data,
> mesh_status_str(data[0]));
> break;
>
> + case OP_CONFIG_BEACON_STATUS:
> + if (len != 1)
> + return true;
> + bt_shell_printf("Node %4.4x Config Beacon Status 0x%02x\n",
> + src, data[0]);
> + break;
> +
> case OP_CONFIG_RELAY_STATUS:
> if (len != 2)
> return true;
> @@ -689,6 +696,37 @@ static void cmd_bind(int argc, char *argv[])
> return bt_shell_noninteractive_quit(EXIT_SUCCESS);
> }
>
> +static void cmd_beacon_set(int argc, char *argv[])
> +{
> + uint16_t n;
> + uint8_t msg[2 + 1];
> + int parm_cnt;
> +
> + if (!verify_config_target(target))
> + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> +
> + n = mesh_opcode_set(OP_CONFIG_BEACON_SET, msg);
> +
> + parm_cnt = read_input_parameters(argc, argv);
> + if (parm_cnt != 1) {
> + bt_shell_printf("bad arguments\n");
> + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> + }
> +
> + msg[n++] = parms[0];
> +
> + if (!config_send(msg, n)) {
> + bt_shell_printf("Failed to send \"SET BEACON\"\n");
> + return;
> + }
> +
> + return bt_shell_noninteractive_quit(EXIT_SUCCESS);}
You should have a like break before }
> +
> +static void cmd_beacon_get(int argc, char *argv[])
> +{
> + cmd_default(OP_CONFIG_BEACON_GET);
> +}
> +
> static void cmd_ident_set(int argc, char *argv[])
> {
> uint16_t n;
> @@ -1178,6 +1216,10 @@ static const struct bt_shell_menu cfg_menu = {
> "Set node identity state"},
> {"ident-get", "<net_idx>", cmd_ident_get,
> "Get node identity state"},
> + {"beacon-set", "<state>", cmd_beacon_set,
> + "Set node identity state"},
> + {"beacon-get", NULL, cmd_beacon_get,
> + "Get node beacon state"},
> {"relay-set", "<relay> <rexmt count> <rexmt steps>",
> cmd_relay_set,
> "Set relay"},
> --
> 2.11.0
Other than this looks good, lets see if Inga has anything to comment.
--
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
