staging/rdma/hfi1: Make firmware failure messages warnings
authorDean Luick <dean.luick@intel.com>
Wed, 3 Feb 2016 22:32:06 +0000 (14:32 -0800)
committerDoug Ledford <dledford@redhat.com>
Fri, 11 Mar 2016 01:37:48 +0000 (20:37 -0500)
Make firmware validation failure and missing firmware messages
a warning since alternates can be tried.  Add an error message
when all attempts fail.

Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Dean Luick <dean.luick@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/staging/rdma/hfi1/firmware.c

index d954e1ab4bbead3c3a9af9d28f481cd9eb59f56f..817cbf94f973b0abd99bb36ddc545b382b51b991 100644 (file)
@@ -433,8 +433,8 @@ static int obtain_one_firmware(struct hfi1_devdata *dd, const char *name,
 
        ret = request_firmware(&fdet->fw, name, &dd->pcidev->dev);
        if (ret) {
-               dd_dev_err(dd, "cannot find firmware \"%s\", err %d\n",
-                          name, ret);
+               dd_dev_warn(dd, "cannot find firmware \"%s\", err %d\n",
+                           name, ret);
                return ret;
        }
 
@@ -572,7 +572,7 @@ retry:
                 * We tried the original and it failed.  Move to the
                 * alternate.
                 */
-               dd_dev_info(dd, "using alternate firmware names\n");
+               dd_dev_warn(dd, "using alternate firmware names\n");
                /*
                 * Let others run.  Some systems, when missing firmware, does
                 * something that holds for 30 seconds.  If we do that twice
@@ -626,6 +626,7 @@ done:
                        fw_state = FW_TRY;
                        goto retry;
                }
+               dd_dev_err(dd, "unable to obtain working firmware\n");
                fw_state = FW_ERR;
                fw_err = -ENOENT;
        } else {
@@ -896,16 +897,17 @@ static int run_rsa(struct hfi1_devdata *dd, const char *who,
                        MISC_ERR_STATUS_MISC_FW_AUTH_FAILED_ERR_SMASK
                        | MISC_ERR_STATUS_MISC_KEY_MISMATCH_ERR_SMASK);
        /*
-        * All that is left are the current errors.  Print failure details,
-        * if any.
+        * All that is left are the current errors.  Print warnings on
+        * authorization failure details, if any.  Firmware authorization
+        * can be retried, so these are only warnings.
         */
        reg = read_csr(dd, MISC_ERR_STATUS);
        if (ret) {
                if (reg & MISC_ERR_STATUS_MISC_FW_AUTH_FAILED_ERR_SMASK)
-                       dd_dev_err(dd, "%s firmware authorization failed\n",
-                               who);
+                       dd_dev_warn(dd, "%s firmware authorization failed\n",
+                                   who);
                if (reg & MISC_ERR_STATUS_MISC_KEY_MISMATCH_ERR_SMASK)
-                       dd_dev_err(dd, "%s firmware key mismatch\n", who);
+                       dd_dev_warn(dd, "%s firmware key mismatch\n", who);
        }
 
        return ret;