From: Viresh Kumar <viresh.kumar@linaro.org>
Date: Fri, 22 Jul 2016 21:13:41 +0000 (-0700)
Subject: greybus: bootrom: Don't print duplicate error messages
X-Git-Tag: v4.9.8~1233^2~378^2~21^2~113
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2d6f1c29988c0f5b51949bb674fe9dcdb923c4a6;p=platform%2Fkernel%2Flinux-rpi3.git

greybus: bootrom: Don't print duplicate error messages

On failing to find a firmware image, we get these today:

bootrom 1-3.3.1: Firmware request for ara_00000126_00001001_fffe0001_ffe5001a_s2l.tftf has failed : -12
bootrom 1-3.3.1: gb_bootrom_firmware_size_request: failed to download firmware (-12)

Which are more or less duplicate, as they print error for the same root
cause.

With this patch this is all we get now:

bootrom 1-3.3.1: failed to download ara_00000126_00001001_fffe0001_ffe5001a_s2l.tftf firmware (-12)

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

diff --git a/drivers/staging/greybus/bootrom.c b/drivers/staging/greybus/bootrom.c
index d16f13e..c5b3971 100644
--- a/drivers/staging/greybus/bootrom.c
+++ b/drivers/staging/greybus/bootrom.c
@@ -171,10 +171,11 @@ static int download_firmware(struct gb_bootrom *bootrom, u8 stage)
 
 	rc = request_firmware(&bootrom->fw, firmware_name,
 		&connection->bundle->dev);
-	if (rc)
-		dev_err(&connection->bundle->dev,
-			"Firmware request for %s has failed : %d",
+	if (rc) {
+		dev_err(&connection->bundle->dev, "failed to download %s firmware (%d)\n",
 			firmware_name, rc);
+	}
+
 	return rc;
 }
 
@@ -200,11 +201,8 @@ static int gb_bootrom_firmware_size_request(struct gb_operation *op)
 	mutex_lock(&bootrom->mutex);
 
 	ret = download_firmware(bootrom, size_request->stage);
-	if (ret) {
-		dev_err(dev, "%s: failed to download firmware (%d)\n", __func__,
-			ret);
+	if (ret)
 		goto unlock;
-	}
 
 	if (!gb_operation_response_alloc(op, sizeof(*size_response),
 					 GFP_KERNEL)) {