From: Viresh Kumar <viresh.kumar@linaro.org>
Date: Mon, 31 Aug 2015 11:51:08 +0000 (+0530)
Subject: greybus: bundle: kill unnecessary forward declaration of routine
X-Git-Tag: v4.14-rc1~2366^2~378^2~21^2~1228
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ad14b9e9ec4973416a911a5d3c59caedc048a279;p=platform%2Fkernel%2Flinux-rpi.git

greybus: bundle: kill unnecessary forward declaration of routine

Move the function to an earlier place, to kill the unnecessary forward
declaration.

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
---

diff --git a/drivers/staging/greybus/bundle.c b/drivers/staging/greybus/bundle.c
index 694bcce..459c62a 100644
--- a/drivers/staging/greybus/bundle.c
+++ b/drivers/staging/greybus/bundle.c
@@ -9,8 +9,6 @@
 
 #include "greybus.h"
 
-static void gb_bundle_connections_exit(struct gb_bundle *bundle);
-
 static ssize_t class_show(struct device *dev, struct device_attribute *attr,
 			  char *buf)
 {
@@ -197,6 +195,18 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
 	return bundle;
 }
 
+static void gb_bundle_connections_exit(struct gb_bundle *bundle)
+{
+	struct gb_connection *connection;
+	struct gb_connection *next;
+
+	list_for_each_entry_safe(connection, next, &bundle->connections,
+				 bundle_links) {
+		gb_connection_exit(connection);
+		gb_connection_destroy(connection);
+	}
+}
+
 /*
  * Tear down a previously set up bundle.
  */
@@ -224,15 +234,3 @@ found:
 
 	return bundle;
 }
-
-static void gb_bundle_connections_exit(struct gb_bundle *bundle)
-{
-	struct gb_connection *connection;
-	struct gb_connection *next;
-
-	list_for_each_entry_safe(connection, next, &bundle->connections,
-				 bundle_links) {
-		gb_connection_exit(connection);
-		gb_connection_destroy(connection);
-	}
-}