Messages in this thread | | | Subject | Re: [PATCH v6 5/8] remoteproc: qcom: Replace hard coded values with macros | From | Sibi Sankar <> | Date | Wed, 14 Sep 2022 15:07:19 +0530 |
| |
On 9/8/22 6:53 PM, Srinivasa Rao Mandadapu wrote: > Replace hard coded values of QDSP6 boot control reg params > with appropriate macro names. > > Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Sibi Sankar <quic_sibis@quicinc.com>
> --- > drivers/remoteproc/qcom_q6v5_adsp.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/remoteproc/qcom_q6v5_adsp.c b/drivers/remoteproc/qcom_q6v5_adsp.c > index 207270d4..389b2c0 100644 > --- a/drivers/remoteproc/qcom_q6v5_adsp.c > +++ b/drivers/remoteproc/qcom_q6v5_adsp.c > @@ -54,6 +54,9 @@ > > #define QCOM_Q6V5_RPROC_PROXY_PD_MAX 3 > > +#define LPASS_BOOT_CORE_START BIT(0) > +#define LPASS_BOOT_CMD_START BIT(0) > + > struct adsp_pil_data { > int crash_reason_smem; > const char *firmware_name; > @@ -366,10 +369,10 @@ static int adsp_start(struct rproc *rproc) > writel(adsp->mem_phys >> 4, adsp->qdsp6ss_base + RST_EVB_REG); > > /* De-assert QDSP6 stop core. QDSP6 will execute after out of reset */ > - writel(0x1, adsp->qdsp6ss_base + CORE_START_REG); > + writel(LPASS_BOOT_CORE_START, adsp->qdsp6ss_base + CORE_START_REG); > > /* Trigger boot FSM to start QDSP6 */ > - writel(0x1, adsp->qdsp6ss_base + BOOT_CMD_REG); > + writel(LPASS_BOOT_CMD_START, adsp->qdsp6ss_base + BOOT_CMD_REG); > > /* Wait for core to come out of reset */ > ret = readl_poll_timeout(adsp->qdsp6ss_base + BOOT_STATUS_REG, >
| |