From: Alex Elder <elder@linaro.org>
Date: Sat, 18 Oct 2014 02:03:49 +0000 (-0500)
Subject: greybus: report gbuf errors
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=f012a520e1a8bb5d05de3307334a8de4dd95afdf;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git

greybus: report gbuf errors

If a gbuf completion indicates an error has occurred, report it.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
---

diff --git a/drivers/staging/greybus/operation.c b/drivers/staging/greybus/operation.c
index 4d19eec790f0..4cbe33e21254 100644
--- a/drivers/staging/greybus/operation.c
+++ b/drivers/staging/greybus/operation.c
@@ -224,10 +224,29 @@ static void gb_operation_recv_work(struct work_struct *recv_work)
  * we'll be done with everything we need to do before we mark it
  * finished.
  *
- * XXX We may want to record that a buffer is (or is no longer) in flight.
+ * XXX We may want to record that a request is (or is no longer) in flight.
  */
 static void gb_operation_gbuf_complete(struct gbuf *gbuf)
 {
+	if (gbuf->status) {
+		struct gb_operation *operation = gbuf->context;
+		struct gb_operation_msg_hdr *header;
+		int id;
+		int type;
+
+		if (gbuf == operation->request)
+			header = operation->request_payload;
+		else if (gbuf == operation->response)
+			header = operation->response_payload;
+		else
+			header = NULL;
+		id = header ? (int)header->id : -1;
+		type = header ? (int)header->type : -1;
+
+		gb_connection_err(operation->connection,
+			"operation %d type %d gbuf error %d",
+			id, type, gbuf->status);
+	}
 	return;
 }