linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH BlueZ] mesh: Fix names of dbus management interface input arguments
@ 2019-11-20  9:09 Rafał Gajda
  0 siblings, 0 replies; only message in thread
From: Rafał Gajda @ 2019-11-20  9:09 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Rafał Gajda

Previously names for method arguments in "org.bluez.mesh.Management1"
would not register properly, being shifted by one:
     <method name="ImportRemoteNode">
		 <arg name="" type="q" direction="in"/>
		 <arg name="primary" type="y" direction="in"/>
		 <arg name="count" type="ay" direction="in"/>
	 </method>

 This fixes this issue:
	 <method name="ImportRemoteNode">
		 <arg name="primary" type="q" direction="in"/>
		 <arg name="count" type="y" direction="in"/>
		 <arg name="dev_key" type="ay" direction="in"/>
	 </method>
---
 mesh/manager.c | 60 ++++++++++++++++++++++++--------------------------
 1 file changed, 29 insertions(+), 31 deletions(-)

diff --git a/mesh/manager.c b/mesh/manager.c
index adbb01280..1ad6c126d 100644
--- a/mesh/manager.c
+++ b/mesh/manager.c
@@ -750,37 +750,35 @@ static struct l_dbus_message *set_key_phase_call(struct l_dbus *dbus,
 
 static void setup_management_interface(struct l_dbus_interface *iface)
 {
-	l_dbus_interface_method(iface, "AddNode", 0, add_node_call, "", "ay",
-								"uuid");
-	l_dbus_interface_method(iface, "ImportRemoteNode", 0, import_node_call,
-						"", "qyay", "", "primary",
-						"count", "dev_key");
-	l_dbus_interface_method(iface, "DeleteRemoteNode", 0, delete_node_call,
-					"", "qy", "", "primary", "count");
-	l_dbus_interface_method(iface, "UnprovisionedScan", 0, start_scan_call,
-							"", "q", "", "seconds");
-	l_dbus_interface_method(iface, "UnprovisionedScanCancel", 0,
-						cancel_scan_call, "", "");
-	l_dbus_interface_method(iface, "CreateSubnet", 0, create_subnet_call,
-						"", "q", "", "net_index");
-	l_dbus_interface_method(iface, "UpdateSubnet", 0, update_subnet_call,
-						"", "q", "", "net_index");
-	l_dbus_interface_method(iface, "DeleteSubnet", 0, delete_subnet_call,
-						"", "q", "", "net_index");
-	l_dbus_interface_method(iface, "ImportSubnet", 0, import_subnet_call,
-					"", "qay", "", "net_index", "net_key");
-	l_dbus_interface_method(iface, "CreateAppKey", 0, create_appkey_call,
-					"", "qq", "", "net_index", "app_index");
-	l_dbus_interface_method(iface, "UpdateAppKey", 0, update_appkey_call,
-						"", "q", "", "app_index");
-	l_dbus_interface_method(iface, "DeleteAppKey", 0, delete_appkey_call,
-						"", "q", "", "app_index");
-	l_dbus_interface_method(iface, "ImportAppKey", 0, import_appkey_call,
-				"", "qqay", "", "net_index", "app_index",
-								"app_key");
-	l_dbus_interface_method(iface, "SetKeyPhase", 0, set_key_phase_call,
-					"", "qy", "", "net_index", "phase");
-}
+    l_dbus_interface_method(iface, "AddNode", 0, add_node_call, "",
+                            "ay", "uuid");
+    l_dbus_interface_method(iface, "ImportRemoteNode", 0, import_node_call, "",
+                            "qyay", "primary", "count", "dev_key");
+    l_dbus_interface_method(iface, "DeleteRemoteNode", 0, delete_node_call, "",
+                            "qy", "primary", "count");
+    l_dbus_interface_method(iface, "UnprovisionedScan", 0, start_scan_call, "",
+                            "q", "seconds");
+    l_dbus_interface_method(iface, "UnprovisionedScanCancel", 0, cancel_scan_call, "",
+                            "");
+    l_dbus_interface_method(iface, "CreateSubnet", 0, create_subnet_call, "",
+                            "q", "net_index");
+    l_dbus_interface_method(iface, "UpdateSubnet", 0, update_subnet_call, "",
+                            "q", "net_index");
+    l_dbus_interface_method(iface, "DeleteSubnet", 0, delete_subnet_call, "",
+                            "q", "net_index");
+    l_dbus_interface_method(iface, "ImportSubnet", 0, import_subnet_call, "",
+                            "qay", "net_index", "net_key");
+    l_dbus_interface_method(iface, "CreateAppKey", 0, create_appkey_call, "",
+                            "qq", "net_index", "app_index");
+    l_dbus_interface_method(iface, "UpdateAppKey", 0, update_appkey_call, "",
+                            "q", "app_index");
+    l_dbus_interface_method(iface, "DeleteAppKey", 0, delete_appkey_call, "",
+                            "q", "app_index");
+    l_dbus_interface_method(iface, "ImportAppKey", 0, import_appkey_call, "",
+                            "qqay", "net_index", "app_index", "app_key");
+    l_dbus_interface_method(iface, "SetKeyPhase", 0, set_key_phase_call, "",
+                            "qy", "net_index", "phase");
+ }
 
 bool manager_dbus_init(struct l_dbus *bus)
 {
-- 
2.22.0


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2019-11-20  9:10 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-20  9:09 [PATCH BlueZ] mesh: Fix names of dbus management interface input arguments Rafał Gajda

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).