greybus: loopback: drop unneeded casts for void pointers

There is no need to cast a void pointer to a particular type.
Drop the casts used in this way, mainly in the attribute definition
macros.

Signed-off-by: Alex Elder <elder@linaro.org>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c
index 08f7780..66059a9 100644 (file)
--- a/drivers/staging/greybus/loopback.c
+++ b/drivers/staging/greybus/loopback.c
@@ -84,8 +84,7 @@ static ssize_t field##_show(struct device *dev,                               \
                            char *buf)                                  \
 {                                                                      \
        struct gb_connection *connection = to_gb_connection(dev);       \
-       struct gb_loopback *gb =                                        \
-               (struct gb_loopback *)connection->private;              \
+       struct gb_loopback *gb = connection->private;                   \
        return sprintf(buf, "%"#type"\n", gb->field);                   \
 }                                                                      \
 static DEVICE_ATTR_RO(field)
@@ -96,8 +95,7 @@ static ssize_t name##_##field##_show(struct device *dev,              \
                            char *buf)                                  \
 {                                                                      \
        struct gb_connection *connection = to_gb_connection(dev);       \
-       struct gb_loopback *gb =                                        \
-               (struct gb_loopback *)connection->private;              \
+       struct gb_loopback *gb = connection->private;                   \
        return sprintf(buf, "%"#type"\n", gb->name.field);              \
 }                                                                      \
 static DEVICE_ATTR_RO(name##_##field)
@@ -113,8 +111,7 @@ static ssize_t field##_show(struct device *dev,                             \
                            char *buf)                                  \
 {                                                                      \
        struct gb_connection *connection = to_gb_connection(dev);       \
-       struct gb_loopback *gb =                                        \
-               (struct gb_loopback *)connection->private;              \
+       struct gb_loopback *gb = connection->private;                   \
        return sprintf(buf, "%"#type"\n", gb->field);                   \
 }                                                                      \
 static ssize_t field##_store(struct device *dev,                       \
@@ -124,8 +121,7 @@ static ssize_t field##_store(struct device *dev,                    \
 {                                                                      \
        int ret;                                                        \
        struct gb_connection *connection = to_gb_connection(dev);       \
-       struct gb_loopback *gb =                                        \
-               (struct gb_loopback *)connection->private;              \
+       struct gb_loopback *gb = connection->private;                   \
        mutex_lock(&gb->mutex);                                         \
        ret = sscanf(buf, "%"#type, &gb->field);                        \
        if (ret != 1)                                                   \
@@ -423,7 +419,7 @@ static int gb_loopback_fn(void *data)
 {
        int error = 0;
        int ms_wait;
-       struct gb_loopback *gb = (struct gb_loopback *)data;
+       struct gb_loopback *gb = data;
 
        while (1) {
                if (!gb->type)