accel/habanalabs: change COMMS warning messages to error level

COMMS protocol is used for LKD <--> FW communication, and any
communication failure between the two might turn out to be
destructive, hence, it should be well emphasized.

Signed-off-by: Koby Elbaz <kelbaz@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
Reviewed-by: Stanislaw Gruszka <stanislaw.gruszka@linux.intel.com>

diff --git a/drivers/accel/habanalabs/common/firmware_if.c b/drivers/accel/habanalabs/common/firmware_if.c
index 7ea611392f8cac63578378462e8092c89e86d4e4..96027a1c124dda94f916f74cc97b29d2c530c079 100644 (file)
--- a/drivers/accel/habanalabs/common/firmware_if.c
+++ b/drivers/accel/habanalabs/common/firmware_if.c
@@ -1263,7 +1263,7 @@ void hl_fw_ask_hard_reset_without_linux(struct hl_device *hdev)
                                COMMS_RST_DEV, 0, false,
                                hdev->fw_loader.cpu_timeout);
                if (rc)
-                       dev_warn(hdev->dev, "Failed sending COMMS_RST_DEV\n");
+                       dev_err(hdev->dev, "Failed sending COMMS_RST_DEV\n");
        } else {
                WREG32(static_loader->kmd_msg_to_cpu_reg, KMD_MSG_RST_DEV);
        }
@@ -1284,7 +1284,7 @@ void hl_fw_ask_halt_machine_without_linux(struct hl_device *hdev)
                                COMMS_GOTO_WFE, 0, true,
                                hdev->fw_loader.cpu_timeout);
                if (rc)
-                       dev_warn(hdev->dev, "Failed sending COMMS_GOTO_WFE\n");
+                       dev_err(hdev->dev, "Failed sending COMMS_GOTO_WFE\n");
        } else {
                WREG32(static_loader->kmd_msg_to_cpu_reg, KMD_MSG_GOTO_WFE);
                msleep(static_loader->cpu_reset_wait_msec);