diff options
author | Johan Hovold <johan@hovoldconsulting.com> | 2015-12-07 15:05:46 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2015-12-08 15:56:38 -0500 |
commit | 5626e0bf99a065d53a0f9966ee335fb0707bd005 (patch) | |
tree | d57a3ced7067976aa8b54d8a554938815b440fd5 /drivers | |
parent | ab66dd786b4b56875baabef7c188cff862231717 (diff) |
greybus: interface: clean up error messages
Drop redundant __func__ and add missing errno values.
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/greybus/interface.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/staging/greybus/interface.c b/drivers/staging/greybus/interface.c index 1c6a8a4ac4b5..85fcae76928c 100644 --- a/drivers/staging/greybus/interface.c +++ b/drivers/staging/greybus/interface.c @@ -179,8 +179,7 @@ int gb_interface_init(struct gb_interface *intf, u8 device_id) /* Get manifest size using control protocol on CPort */ size = gb_control_get_manifest_size_operation(intf); if (size <= 0) { - dev_err(&intf->dev, "%s: Failed to get manifest size (%d)\n", - __func__, size); + dev_err(&intf->dev, "failed to get manifest size: %d\n", size); if (size) return size; else @@ -194,7 +193,7 @@ int gb_interface_init(struct gb_interface *intf, u8 device_id) /* Get manifest using control protocol on CPort */ ret = gb_control_get_manifest_operation(intf, manifest, size); if (ret) { - dev_err(&intf->dev, "%s: Failed to get manifest\n", __func__); + dev_err(&intf->dev, "failed to get manifest: %d\n", ret); goto free_manifest; } @@ -203,7 +202,7 @@ int gb_interface_init(struct gb_interface *intf, u8 device_id) * what's in it. */ if (!gb_manifest_parse(intf, manifest, size)) { - dev_err(&intf->dev, "%s: Failed to parse manifest\n", __func__); + dev_err(&intf->dev, "failed to parse manifest\n"); ret = -EINVAL; goto free_manifest; } |