From: Alex Elder <elder@linaro.org>
Date: Fri, 12 Jun 2015 15:21:10 +0000 (-0500)
Subject: greybus: bundle: refactor gb_bundle_find()
X-Git-Tag: v4.9-rc1~119^2~378^2~21^2~1466
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=bc942083585da78a5287089023aebcc8952c21f4;p=karo-tx-linux.git

greybus: bundle: refactor gb_bundle_find()

Rearrange gb_bundle_find() so it follows the pattern used by
gb_connection_find().

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

diff --git a/drivers/staging/greybus/bundle.c b/drivers/staging/greybus/bundle.c
index 6e9d03ac7e35..89568b2b0adf 100644
--- a/drivers/staging/greybus/bundle.c
+++ b/drivers/staging/greybus/bundle.c
@@ -283,13 +283,13 @@ struct gb_bundle *gb_bundle_find(struct gb_interface *intf, u8 bundle_id)
 
 	spin_lock_irq(&gb_bundles_lock);
 	list_for_each_entry(bundle, &intf->bundles, links)
-		if (bundle->id == bundle_id) {
-			spin_unlock_irq(&gb_bundles_lock);
-			return bundle;
-		}
+		if (bundle->id == bundle_id)
+			goto found;
+	bundle = NULL;
+found:
 	spin_unlock_irq(&gb_bundles_lock);
 
-	return NULL;
+	return bundle;
 }
 
 static int gb_bundle_connections_init(struct gb_bundle *bundle)