commit 6d05ad3c4226024ad78dfbd2ab683b760ab56389
author Viresh Kumar <viresh.kumar@linaro.org> Thu Aug 06 12:44:54 2015 +0530
committer Greg Kroah-Hartman <gregkh@google.com> Mon Aug 10 17:29:27 2015 -0700
tree 7281770d8927147cd0df97b07a9e1cb6a9679521
parent 76ecf73ee4e3f15db202e4ea76816f155c478051

greybus: svc: Print expected/actual payload size on mismatch

Print (expected-payload-size actual-payload-size), when the size doesn't
match for requests received by the module. This gives more details
required for debugging the issue.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

drivers/staging/greybus/svc.c

diff --git a/drivers/staging/greybus/svc.c b/drivers/staging/greybus/svc.c
index b89e889..4d59cbb 100644
--- a/drivers/staging/greybus/svc.c
+++ b/drivers/staging/greybus/svc.c
@@ -269,7 +269,8 @@
 	int ret;
 
 	if (request->payload_size < sizeof(*hotplug)) {
-		dev_err(dev, "%s: short hotplug request received\n", __func__);
+		dev_err(dev, "%s: short hotplug request received (%zu < %zu)\n",
+			__func__, request->payload_size, sizeof(*hotplug));
 		return -EINVAL;
 	}
 
@@ -372,7 +373,8 @@
 
 	if (request->payload_size < sizeof(*hot_unplug)) {
 		dev_err(&op->connection->dev,
-			"short hot unplug request received\n");
+			"short hot unplug request received (%zu < %zu)\n",
+			request->payload_size, sizeof(*hot_unplug));
 		return -EINVAL;
 	}
 
@@ -400,7 +402,8 @@
 
 	if (request->payload_size < sizeof(*reset)) {
 		dev_err(&op->connection->dev,
-			"short reset request received\n");
+			"short reset request received (%zu < %zu)\n",
+			request->payload_size, sizeof(*reset));
 		return -EINVAL;
 	}
 	reset = request->payload;