greybus: connection: no need to verify connection->protocol

connection->protocol will always be valid in gb_connection_init() as it
is called only from a single routine, after initializing the 'protocol'
field.

No need to check it again.

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>

diff --git a/drivers/staging/greybus/connection.c b/drivers/staging/greybus/connection.c
index f89b199230dc717971469696ed2a653bd5d79b58..557fe6d6b7afeb4afd9dc0de0b120b1ae4263930 100644 (file)
--- a/drivers/staging/greybus/connection.c
+++ b/drivers/staging/greybus/connection.c
@@ -371,11 +371,6 @@ static int gb_connection_init(struct gb_connection *connection)
        int cport_id = connection->intf_cport_id;
        int ret;
 
-       if (!connection->protocol) {
-               dev_warn(&connection->dev, "init without protocol.\n");
-               return 0;
-       }
-
        /*
         * Inform Interface about Active CPorts. We don't need to do this
         * operation for control cport.