diff options
| author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-08-31 17:21:10 +0530 |
|---|---|---|
| committer | Johan Hovold <johan@hovoldconsulting.com> | 2015-09-03 14:41:44 +0200 |
| commit | 3dca03de9db7c467811bdfcb0b3b90cb9a184a8f (patch) | |
| tree | 7103b4c60c2b252720f0abd283de0a332440bbf5 /drivers/staging/greybus/connection.c | |
| parent | 630096899120321791f0ca6fea7ad7f9bd572e20 (diff) | |
| download | linux-stable-3dca03de9db7c467811bdfcb0b3b90cb9a184a8f.tar.gz linux-stable-3dca03de9db7c467811bdfcb0b3b90cb9a184a8f.tar.bz2 linux-stable-3dca03de9db7c467811bdfcb0b3b90cb9a184a8f.zip | |
greybus: connection: convert connected dev_warn() to dev_err()
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>
Diffstat (limited to 'drivers/staging/greybus/connection.c')
| -rw-r--r-- | drivers/staging/greybus/connection.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/staging/greybus/connection.c b/drivers/staging/greybus/connection.c index 62cbeb304616..f89b199230dc 100644 --- a/drivers/staging/greybus/connection.c +++ b/drivers/staging/greybus/connection.c @@ -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; } } |