From bc942083585da78a5287089023aebcc8952c21f4 Mon Sep 17 00:00:00 2001
From: Alex Elder <elder@linaro.org>
Date: Fri, 12 Jun 2015 10:21:10 -0500
Subject: [PATCH] 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>
---
drivers/staging/greybus/bundle.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/greybus/bundle.c b/drivers/staging/greybus/bundle.c
index 6e9d03ac7e35a..89568b2b0adf8 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)
--
2.30.2