greybus: manifest: use bundle's embedded interface pointer

An initialized bundle structure contains a pointer to its
interface.  Because of this there's no need to provide
the interface pointer to gb_manifest_parse_cports().  This
also precludes the possibility of passing a bad interface
pointer.

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

diff --git a/drivers/staging/greybus/manifest.c b/drivers/staging/greybus/manifest.c
index b937ddecbe4118b88fe037227e1741a7faf18dd3..d20cd80e6cdd37807ac8f05f2b6b1f1125c8f2d0 100644 (file)
--- a/drivers/staging/greybus/manifest.c
+++ b/drivers/staging/greybus/manifest.c
@@ -198,9 +198,9 @@ static char *gb_string_get(struct gb_interface *intf, u8 string_id)
  * them.  Returns the number of cports set up for the bundle, or 0
  * if there is an error.
  */
-static u32 gb_manifest_parse_cports(struct gb_interface *intf,
-                                   struct gb_bundle *bundle)
+static u32 gb_manifest_parse_cports(struct gb_bundle *bundle)
 {
+       struct gb_interface *intf = bundle->intf;
        u32 count = 0;
 
        while (true) {
@@ -270,7 +270,7 @@ static u32 gb_manifest_parse_bundles(struct gb_interface *intf)
                        return 0;       /* Error */
 
                /* Now go set up this bundle's functions and cports */
-               if (!gb_manifest_parse_cports(intf, bundle))
+               if (!gb_manifest_parse_cports(bundle))
                        return 0;       /* Error parsing cports */
 
                count++;