greybus: connection: add helper to create control connections

Add dedicated helper to create control connections.

This will allow us to simplify the generic (dynamic) interface.

Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>

diff --git a/drivers/staging/greybus/connection.c b/drivers/staging/greybus/connection.c
index 9b8112b..f81f053 100644 (file)
--- a/drivers/staging/greybus/connection.c
+++ b/drivers/staging/greybus/connection.c
@@ -223,6 +223,12 @@ gb_connection_create_static(struct gb_host_device *hd, u16 hd_cport_id)
 }
 
 struct gb_connection *
+gb_connection_create_control(struct gb_interface *intf)
+{
+       return gb_connection_create(intf->hd, -1, intf, NULL, 0, 0);
+}
+
+struct gb_connection *
 gb_connection_create_dynamic(struct gb_interface *intf,
                                        struct gb_bundle *bundle,
                                        u16 cport_id, u8 protocol_id)

diff --git a/drivers/staging/greybus/connection.h b/drivers/staging/greybus/connection.h
index 562bd2b..cd4a093 100644 (file)
--- a/drivers/staging/greybus/connection.h
+++ b/drivers/staging/greybus/connection.h
@@ -57,6 +57,7 @@ struct gb_connection {
 
 struct gb_connection *gb_connection_create_static(struct gb_host_device *hd,
                                u16 hd_cport_id);
+struct gb_connection *gb_connection_create_control(struct gb_interface *intf);
 struct gb_connection *gb_connection_create_dynamic(struct gb_interface *intf,
                                struct gb_bundle *bundle, u16 cport_id,
                                u8 protocol_id);

diff --git a/drivers/staging/greybus/control.c b/drivers/staging/greybus/control.c
index 0e50fd4..a766ec8 100644 (file)
--- a/drivers/staging/greybus/control.c
+++ b/drivers/staging/greybus/control.c
@@ -186,9 +186,7 @@ struct gb_control *gb_control_create(struct gb_interface *intf)
        if (!control)
                return NULL;
 
-       control->connection = gb_connection_create_dynamic(intf, NULL,
-                                               GB_CONTROL_CPORT_ID,
-                                               GREYBUS_PROTOCOL_CONTROL);
+       control->connection = gb_connection_create_control(intf);
        if (!control->connection) {
                dev_err(&intf->dev, "failed to create control connection\n");
                kfree(control);