[PATCH V2 13/36] smartpqi: eliminate redundant error messages

[Don Brace]

From: Kevin Barnett <kevin.barn...@hpe.com>

eliminate redundant error message during initialization
if the controller has crashed.

Reviewed-by: Scott Benesh <scott.ben...@microsemi.com>
Signed-off-by: Kevin Barnett <kevin.barn...@microsemi.com>
Signed-off-by: Don Brace <don.br...@microsemi.com>
---
 drivers/scsi/smartpqi/smartpqi_init.c |    5 +----
 drivers/scsi/smartpqi/smartpqi_sis.c  |    5 ++++-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/scsi/smartpqi/smartpqi_init.c 
b/drivers/scsi/smartpqi/smartpqi_init.c
index 0083ea4..fa21dd4 100644
--- a/drivers/scsi/smartpqi/smartpqi_init.c
+++ b/drivers/scsi/smartpqi/smartpqi_init.c
@@ -5796,11 +5796,8 @@ static int pqi_ctrl_init(struct pqi_ctrl_info *ctrl_info)
         * commands.
         */
        rc = sis_wait_for_ctrl_ready(ctrl_info);
-       if (rc) {
-               dev_err(&ctrl_info->pci_dev->dev,
-                       "error initializing SIS interface\n");
+       if (rc)
                return rc;
-       }
 
        /*
         * Get the controller properties.  This allows us to determine
diff --git a/drivers/scsi/smartpqi/smartpqi_sis.c 
b/drivers/scsi/smartpqi/smartpqi_sis.c
index 3155bda..d4b1cc7 100644
--- a/drivers/scsi/smartpqi/smartpqi_sis.c
+++ b/drivers/scsi/smartpqi/smartpqi_sis.c
@@ -102,8 +102,11 @@ static int sis_wait_for_ctrl_ready_with_timeout(struct 
pqi_ctrl_info *ctrl_info,
                        if (status & SIS_CTRL_KERNEL_UP)
                                break;
                }
-               if (time_after(jiffies, timeout))
+               if (time_after(jiffies, timeout)) {
+                       dev_err(&ctrl_info->pci_dev->dev,
+                               "controller not ready\n");
                        return -ETIMEDOUT;
+               }
                msleep(SIS_CTRL_READY_POLL_INTERVAL_MSECS);
        }