commit 0c33c99102bc00e49327ce9ce30a5b43c811f424
author Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Mon Dec 18 10:03:03 2017 -0200
committer Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Fri Jan 19 11:44:09 2018 -0200
tree f77cd4f0cd445047392b70165a9985aad77a3b0f
parent 045e6b9a10c006a2ff93bbc405bb83c6a62e7c98

client: Rename set-tx-power to tx-power

Make the command return the current value if no parameters:

[bluetooth]# tx-power on
[bluetooth]# tx-power
Tx Power: on

client/advertising.c

diff --git a/client/advertising.c b/client/advertising.c
index 304ae68..05c6910 100644
--- a/client/advertising.c
+++ b/client/advertising.c
@@ -587,12 +587,17 @@
 							"ManufacturerData");
 }
 
-void ad_advertise_tx_power(DBusConnection *conn, bool value)
+void ad_advertise_tx_power(DBusConnection *conn, dbus_bool_t *value)
 {
-	if (ad.tx_power == value)
+	if (!value) {
+		bt_shell_printf("Tx Power: %s\n", ad.tx_power ? "on" : "off");
+		return;
+	}
+
+	if (ad.tx_power == *value)
 		return;
 
-	ad.tx_power = value;
+	ad.tx_power = *value;
 
 	g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, "Includes");
 }

client/advertising.h

diff --git a/client/advertising.h b/client/advertising.h
index d7472e1..6c73b63 100644
--- a/client/advertising.h
+++ b/client/advertising.h
@@ -27,7 +27,7 @@
 void ad_advertise_uuids(DBusConnection *conn, int argc, char *argv[]);
 void ad_advertise_service(DBusConnection *conn, int argc, char *argv[]);
 void ad_advertise_manufacturer(DBusConnection *conn, int argc, char *argv[]);
-void ad_advertise_tx_power(DBusConnection *conn, bool value);
+void ad_advertise_tx_power(DBusConnection *conn, dbus_bool_t *value);
 void ad_advertise_name(DBusConnection *conn, bool value);
 void ad_advertise_appearance(DBusConnection *conn, bool value);
 void ad_advertise_local_name(DBusConnection *conn, const char *name);

client/main.c

diff --git a/client/main.c b/client/main.c
index 26bbc7e..195a861 100644
--- a/client/main.c
+++ b/client/main.c
@@ -2206,14 +2206,19 @@
 	ad_advertise_manufacturer(dbus_conn, argc, argv);
 }
 
-static void cmd_set_advertise_tx_power(int argc, char *argv[])
+static void cmd_advertise_tx_power(int argc, char *argv[])
 {
 	dbus_bool_t powered;
 
+	if (argc < 2) {
+		ad_advertise_tx_power(dbus_conn, NULL);
+		return;
+	}
+
 	if (!parse_argument(argc, argv, NULL, NULL, &powered, NULL))
 		return;
 
-	ad_advertise_tx_power(dbus_conn, powered);
+	ad_advertise_tx_power(dbus_conn, &powered);
 }
 
 static void cmd_set_advertise_name(int argc, char *argv[])
@@ -2294,7 +2299,7 @@
 	{ "manufacturer", "[id] [data=xx xx ...]",
 			cmd_advertise_manufacturer,
 			"Set advertise manufacturer data" },
-	{ "set-tx-power", "<on/off>", cmd_set_advertise_tx_power,
+	{ "tx-power", "[on/off]", cmd_advertise_tx_power,
 			"Enable/disable TX power to be advertised",
 							mode_generator },
 	{ "set-name", "<on/off/name>", cmd_set_advertise_name,