Browse Source

greybus: control: make disconnecting a core operation

Make the control-protocol disconnecting operation a "core" operation.

This is needed to be able to use the operation in the new connection
tear-down sequence of control connections, which moves disconnecting
after the flush barrier.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Sandeep Patil <sspatil@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Johan Hovold 9 years ago
parent
commit
9f77b80fdf
1 changed files with 21 additions and 5 deletions
  1. 21 5
      drivers/staging/greybus/control.c

+ 21 - 5
drivers/staging/greybus/control.c

@@ -152,13 +152,29 @@ int gb_control_disconnected_operation(struct gb_control *control, u16 cport_id)
 int gb_control_disconnecting_operation(struct gb_control *control,
 					u16 cport_id)
 {
-	struct gb_control_disconnecting_request request;
+	struct gb_control_disconnecting_request *request;
+	struct gb_operation *operation;
+	int ret;
 
-	request.cport_id = cpu_to_le16(cport_id);
+	operation = gb_operation_create_core(control->connection,
+					GB_CONTROL_TYPE_DISCONNECTING,
+					sizeof(*request), 0, 0,
+					GFP_KERNEL);
+	if (!operation)
+		return -ENOMEM;
 
-	return gb_operation_sync(control->connection,
-				 GB_CONTROL_TYPE_DISCONNECTING, &request,
-				 sizeof(request), NULL, 0);
+	request = operation->request->payload;
+	request->cport_id = cpu_to_le16(cport_id);
+
+	ret = gb_operation_request_send_sync(operation);
+	if (ret) {
+		dev_err(&control->dev, "failed to send disconnecting: %d\n",
+				ret);
+	}
+
+	gb_operation_put(operation);
+
+	return ret;
 }
 
 int gb_control_mode_switch_operation(struct gb_control *control)