remoteproc: Remove null character write of shared mem

remoteproc is writing '\0' in the shared mem region. This
region is shared among multiple clients that are also trying
to read. Hence they miss first character.

Remove this null character write, as this mem area is
supposed to be Read only.

Further during every subsystem reboot, this region is
initialized with default, hence no need to write this
region.

Signed-off-by: Jitendra Sharma <shajit@codeaurora.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

diff --git a/drivers/remoteproc/qcom_adsp_pil.c b/drivers/remoteproc/qcom_adsp_pil.c
index 373c167892d7d4754ccef7161f1f3fea1054afc6..4a2ee6c5816cd4469e228dfd1022a5ff94ebbcc4 100644 (file)
--- a/drivers/remoteproc/qcom_adsp_pil.c
+++ b/drivers/remoteproc/qcom_adsp_pil.c
@@ -201,9 +201,6 @@ static irqreturn_t adsp_fatal_interrupt(int irq, void *dev)
 
        rproc_report_crash(adsp->rproc, RPROC_FATAL_ERROR);
 
-       if (!IS_ERR(msg))
-               msg[0] = '\0';
-
        return IRQ_HANDLED;
 }
 

diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c
index b4e5e725848d263936d93b7bbe1645dd94e37ce1..7293d45c26715310a106ecacbd9bee5bb76ceaa6 100644 (file)
--- a/drivers/remoteproc/qcom_q6v5_pil.c
+++ b/drivers/remoteproc/qcom_q6v5_pil.c
@@ -939,9 +939,6 @@ static irqreturn_t q6v5_wdog_interrupt(int irq, void *dev)
 
        rproc_report_crash(qproc->rproc, RPROC_WATCHDOG);
 
-       if (!IS_ERR(msg))
-               msg[0] = '\0';
-
        return IRQ_HANDLED;
 }
 
@@ -959,9 +956,6 @@ static irqreturn_t q6v5_fatal_interrupt(int irq, void *dev)
 
        rproc_report_crash(qproc->rproc, RPROC_FATAL_ERROR);
 
-       if (!IS_ERR(msg))
-               msg[0] = '\0';
-
        return IRQ_HANDLED;
 }
 

diff --git a/drivers/remoteproc/qcom_wcnss.c b/drivers/remoteproc/qcom_wcnss.c
index 3f0609236a769c995f75a411c37b638177254400..043f3d3dea7d321fc831d323aec4e3c9ae270480 100644 (file)
--- a/drivers/remoteproc/qcom_wcnss.c
+++ b/drivers/remoteproc/qcom_wcnss.c
@@ -332,9 +332,6 @@ static irqreturn_t wcnss_fatal_interrupt(int irq, void *dev)
 
        rproc_report_crash(wcnss->rproc, RPROC_FATAL_ERROR);
 
-       if (!IS_ERR(msg))
-               msg[0] = '\0';
-
        return IRQ_HANDLED;
 }