From: Johan Hovold <johan@hovoldconsulting.com>
Date: Wed, 3 Aug 2016 12:09:33 +0000 (+0200)
Subject: greybus: es2: allow ARPC result to be ignored
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=121bae2bec47a9944049f2583b1d31f479110f0f;p=linux-beck.git

greybus: es2: allow ARPC result to be ignored

Allow user of ARPC to pass a NULL pointer for the ARPC result.

For simple requests there may only be one error code for remote errors,
or the caller may simply not care to differentiate them.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
---

diff --git a/drivers/staging/greybus/es2.c b/drivers/staging/greybus/es2.c
index c31124d7defc..8d0e52d46a2b 100644
--- a/drivers/staging/greybus/es2.c
+++ b/drivers/staging/greybus/es2.c
@@ -1184,7 +1184,8 @@ static int arpc_sync(struct es2_ap_dev *es2, u8 type, void *payload,
 	unsigned long flags;
 	int retval;
 
-	*result = 0;
+	if (result)
+		*result = 0;
 
 	rpc = arpc_alloc(payload, size, type);
 	if (!rpc)
@@ -1209,7 +1210,8 @@ static int arpc_sync(struct es2_ap_dev *es2, u8 type, void *payload,
 
 	if (rpc->resp->result) {
 		retval = -EREMOTEIO;
-		*result = rpc->resp->result;
+		if (result)
+			*result = rpc->resp->result;
 	} else {
 		retval = 0;
 	}