greybus: connection: convert connected dev_warn() to dev_err()
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 31 Aug 2015 11:51:10 +0000 (17:21 +0530)
committerJohan Hovold <johan@hovoldconsulting.com>
Thu, 3 Sep 2015 12:41:44 +0000 (14:41 +0200)
Failures from control-connected operations are fatal errors and
must be reported with dev_err() instead of dev_warn().

Fix it.

Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
[ johan: do not promote disconnected warnings, update summary ]
Signed-off-by: Johan Hovold <johan@hovoldconsulting.com>
drivers/staging/greybus/connection.c

index 62cbeb30461694cdd5ad2ea3c93f85cf71aba88d..f89b199230dc717971469696ed2a653bd5d79b58 100644 (file)
@@ -386,9 +386,9 @@ static int gb_connection_init(struct gb_connection *connection)
 
                ret = gb_control_connected_operation(control, cport_id);
                if (ret) {
-                       dev_warn(&connection->dev,
-                                "Failed to connect CPort-%d (%d)\n",
-                                cport_id, ret);
+                       dev_err(&connection->dev,
+                               "Failed to connect CPort-%d (%d)\n",
+                               cport_id, ret);
                        return 0;
                }
        }