greybus: bundle: separate bundle creation and registration

Separate bundle creation and registration.

Note that the bundle connections still needs to be initialised post
registration as protocol drivers create child devices to the bundle.

This will ultimately allow connection structures to be created while
parsing manifests, but the connections to not be enabled until a driver
is bound.

Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Reviewed-by: Viresh Kumar <viresh.kumar@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 0f3a00d8d4f2813e8786681fde04b4f47f1316db..fb775ff1895d60267cb81f83dbdba48c1023b21f 100644 (file)
--- a/drivers/staging/greybus/bundle.c
+++ b/drivers/staging/greybus/bundle.c
@@ -107,7 +107,6 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
                                   u8 class)
 {
        struct gb_bundle *bundle;
-       int retval;
 
        /*
         * Reject any attempt to reuse a bundle id.  We initialize
@@ -128,8 +127,6 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
        bundle->class = class;
        INIT_LIST_HEAD(&bundle->connections);
 
-       /* Build up the bundle device structures and register it with the
-        * driver core */
        bundle->dev.parent = &intf->dev;
        bundle->dev.bus = &greybus_bus_type;
        bundle->dev.type = &greybus_bundle_type;
@@ -137,18 +134,24 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
        device_initialize(&bundle->dev);
        dev_set_name(&bundle->dev, "%s.%d", dev_name(&intf->dev), bundle_id);
 
-       retval = device_add(&bundle->dev);
-       if (retval) {
-               pr_err("failed to add bundle device for id %u\n", bundle_id);
-               put_device(&bundle->dev);
-               return NULL;
-       }
-
        list_add(&bundle->links, &intf->bundles);
 
        return bundle;
 }
 
+int gb_bundle_add(struct gb_bundle *bundle)
+{
+       int ret;
+
+       ret = device_add(&bundle->dev);
+       if (ret) {
+               dev_err(&bundle->dev, "failed to register bundle: %d\n", ret);
+               return ret;
+       }
+
+       return 0;
+}
+
 static void gb_bundle_connections_exit(struct gb_bundle *bundle)
 {
        struct gb_connection *connection;
@@ -162,10 +165,12 @@ static void gb_bundle_connections_exit(struct gb_bundle *bundle)
  */
 void gb_bundle_destroy(struct gb_bundle *bundle)
 {
-       list_del(&bundle->links);
-
        gb_bundle_connections_exit(bundle);
-       device_unregister(&bundle->dev);
-}
 
+       if (device_is_registered(&bundle->dev))
+               device_del(&bundle->dev);
 
+       list_del(&bundle->links);
+
+       put_device(&bundle->dev);
+}

diff --git a/drivers/staging/greybus/bundle.h b/drivers/staging/greybus/bundle.h
index eae375c12100eac3ad8754cf78529cef7673a723..682ec327f8a36a402985d0b5547754029caabc5e 100644 (file)
--- a/drivers/staging/greybus/bundle.h
+++ b/drivers/staging/greybus/bundle.h
@@ -31,6 +31,7 @@ struct gb_bundle {
 /* Greybus "private" definitions" */
 struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
                                   u8 class);
+int gb_bundle_add(struct gb_bundle *bundle);
 void gb_bundle_destroy(struct gb_bundle *bundle);
 
 #endif /* __BUNDLE_H */

diff --git a/drivers/staging/greybus/manifest.c b/drivers/staging/greybus/manifest.c
index 8310f199f0f12448fa0da4fe2c7baaccfbdc8403..51201128498d59f5ec05a4f2e6ce6329a254e0ff 100644 (file)
--- a/drivers/staging/greybus/manifest.c
+++ b/drivers/staging/greybus/manifest.c
@@ -236,7 +236,6 @@ static u32 gb_manifest_parse_cports(struct gb_bundle *bundle)
        u8 protocol_id;
        u16 cport_id;
        u32 count = 0;
-       int ret;
 
        /* Set up all cport descriptors associated with this bundle */
        list_for_each_entry_safe(desc, next, &intf->manifest_descs, links) {
@@ -262,12 +261,6 @@ static u32 gb_manifest_parse_cports(struct gb_bundle *bundle)
                if (!connection)
                        goto exit;
 
-               ret = gb_connection_init(connection);
-               if (ret) {
-                       gb_connection_destroy(connection);
-                       goto exit;
-               }
-
                count++;
 
                /* Release the cport descriptor */
@@ -293,12 +286,14 @@ exit:
  */
 static u32 gb_manifest_parse_bundles(struct gb_interface *intf)
 {
+       struct gb_connection *connection;
        struct manifest_desc *desc;
        struct gb_bundle *bundle;
        struct gb_bundle *bundle_next;
        u32 count = 0;
        u8 bundle_id;
        u8 class;
+       int ret;
 
        while ((desc = get_next_bundle_desc(intf))) {
                struct greybus_descriptor_bundle *desc_bundle;
@@ -354,6 +349,23 @@ static u32 gb_manifest_parse_bundles(struct gb_interface *intf)
                        continue;
                }
 
+               ret = gb_bundle_add(bundle);
+               if (ret) {
+                       gb_bundle_destroy(bundle);
+                       continue;
+               }
+
+               list_for_each_entry(connection, &bundle->connections,
+                                                       bundle_links) {
+                       ret = gb_connection_init(connection);
+                       if (ret)
+                               break;
+               }
+               if (ret) {
+                       gb_bundle_destroy(bundle);
+                       continue;
+               }
+
                count++;
        }