From: Alex Elder <elder@linaro.org>
Date: Fri, 27 Mar 2015 02:25:06 +0000 (-0500)
Subject: greybus: get rid of {conceal,reveal}_urb()
X-Git-Tag: v4.9-rc1~119^2~378^2~21^2~1649
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=142f8ddf71e2e081955b4f54ba72c78dbb7b7ce8;p=karo-tx-linux.git

greybus: get rid of {conceal,reveal}_urb()

These clever macros were fine for early development, but they're
more of a distraction now.

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

diff --git a/drivers/staging/greybus/es1.c b/drivers/staging/greybus/es1.c
index 8aad4fbe25e6..239358dce9be 100644
--- a/drivers/staging/greybus/es1.c
+++ b/drivers/staging/greybus/es1.c
@@ -21,13 +21,6 @@
 #include "svc_msg.h"
 #include "kernel_ver.h"
 
-/*
- * Macros for making pointers explicitly opaque, such that the result
- * isn't valid but also can't be mistaken for an ERR_PTR() value.
- */
-#define conceal_urb(urb)	((void *)((uintptr_t)(urb) ^ 0xbad))
-#define reveal_urb(cookie)	((void *)((uintptr_t)(cookie) ^ 0xbad))
-
 /* Memory sizes for the buffers sent to/from the ES1 controller */
 #define ES1_SVC_MSG_SIZE	(sizeof(struct svc_msg) + SZ_64K)
 #define ES1_GBUF_MSG_SIZE_MAX	PAGE_SIZE
@@ -274,7 +267,7 @@ static void *buffer_send(struct greybus_host_device *hd, u16 cport_id,
 		return ERR_PTR(retval);
 	}
 
-	return conceal_urb(urb);
+	return urb;
 }
 
 /*
@@ -292,7 +285,7 @@ static void buffer_cancel(void *cookie)
 	 * is valid.  For the time being, this will do.
 	 */
 	if (cookie)
-		usb_kill_urb(reveal_urb(cookie));
+		usb_kill_urb(cookie);
 }
 
 static struct greybus_host_driver es1_driver = {
diff --git a/drivers/staging/greybus/es2.c b/drivers/staging/greybus/es2.c
index d8d45bec4f1d..a73a268c8759 100644
--- a/drivers/staging/greybus/es2.c
+++ b/drivers/staging/greybus/es2.c
@@ -1,8 +1,8 @@
 /*
  * Greybus "AP" USB driver for "ES2" controller chips
  *
- * Copyright 2014 Google Inc.
- * Copyright 2014 Linaro Ltd.
+ * Copyright 2014-2015 Google Inc.
+ * Copyright 2014-2015 Linaro Ltd.
  *
  * Released under the GPLv2 only.
  */
@@ -17,13 +17,6 @@
 #include "svc_msg.h"
 #include "kernel_ver.h"
 
-/*
- * Macros for making pointers explicitly opaque, such that the result
- * isn't valid but also can't be mistaken for an ERR_PTR() value.
- */
-#define conceal_urb(urb)	((void *)((uintptr_t)(urb) ^ 0xbad))
-#define reveal_urb(cookie)	((void *)((uintptr_t)(cookie) ^ 0xbad))
-
 /* Memory sizes for the buffers sent to/from the ES1 controller */
 #define ES1_SVC_MSG_SIZE	(sizeof(struct svc_msg) + SZ_64K)
 #define ES1_GBUF_MSG_SIZE_MAX	PAGE_SIZE
@@ -262,7 +255,7 @@ static void *buffer_send(struct greybus_host_device *hd, u16 cport_id,
 		return ERR_PTR(retval);
 	}
 
-	return conceal_urb(urb);
+	return urb;
 }
 
 /*
@@ -280,7 +273,7 @@ static void buffer_cancel(void *cookie)
 	 * is valid.  For the time being, this will do.
 	 */
 	if (cookie)
-		usb_kill_urb(reveal_urb(cookie));
+		usb_kill_urb(cookie);
 }
 
 static struct greybus_host_driver es1_driver = {