[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <gaxovmf4uy4qz2zlfx5x3bdjtslhldfedmwqdlanxgl4t22ku5@mrvrdvwpo43j>
Date: Wed, 4 Sep 2024 17:05:30 -0500
From: Bjorn Andersson <andersson@...nel.org>
To: Unnathi Chalicheemala <quic_uchalich@...cinc.com>
Cc: Konrad Dybcio <konradybcio@...nel.org>, linux-arm-msm@...r.kernel.org,
linux-kernel@...r.kernel.org, kernel@...cinc.com
Subject: Re: [PATCH v2 1/2] firmware: qcom_scm: Add API to get waitqueue IRQ
info
On Thu, Aug 29, 2024 at 03:15:54PM GMT, Unnathi Chalicheemala wrote:
> Bootloader and firmware for SM8650 and older chipsets expect node
> name as "qcom_scm".
Perhaps we can add the reason why bootloader/firmware is looking for
this node? Perhaps "looks for a node named qcom_scm, in order to patch
the wait queue IRQ information." ?
> However, DeviceTree uses node name "scm" and this
> mismatch prevents firmware from correctly identifying waitqueue IRQ
> information. Waitqueue IRQ is used for signaling between secure and
> non-secure worlds.
>
> To resolve this, introduce qcom_scm_get_waitq_irq() that'll get the
> hardware irq number to be used from firmware instead of relying on data
> provided by devicetree, thereby bypassing the DeviceTree node name
> mismatch.
>
> This hardware irq number is converted to a linux irq number using newly
Capitalize IRQ, and Linux throughout the message.
Regards,
Bjorn
> defined fill_irq_fwspec_params(). This linux irq number is then supplied to
> the threaded_irq call.
>
> Signed-off-by: Unnathi Chalicheemala <quic_uchalich@...cinc.com>
> ---
> drivers/firmware/qcom/qcom_scm.c | 59 +++++++++++++++++++++++++++++++-
> drivers/firmware/qcom/qcom_scm.h | 1 +
> 2 files changed, 59 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/firmware/qcom/qcom_scm.c b/drivers/firmware/qcom/qcom_scm.c
> index 00c379a3cceb..ed51fbb1c065 100644
> --- a/drivers/firmware/qcom/qcom_scm.c
> +++ b/drivers/firmware/qcom/qcom_scm.c
> @@ -32,6 +32,14 @@
> #include "qcom_scm.h"
> #include "qcom_tzmem.h"
>
> +#define GIC_SPI_BASE 32
> +#define GIC_MAX_SPI 987 // 1019 - 32
> +#define GIC_ESPI_BASE 4096
> +#define GIC_MAX_ESPI 1024 // 5120 - 4096
> +
> +#define GIC_IRQ_TYPE_SPI 0
> +#define GIC_IRQ_TYPE_ESPI 2
> +
> static bool download_mode = IS_ENABLED(CONFIG_QCOM_SCM_DOWNLOAD_MODE_DEFAULT);
> module_param(download_mode, bool, 0);
>
> @@ -1819,6 +1827,55 @@ bool qcom_scm_is_available(void)
> }
> EXPORT_SYMBOL_GPL(qcom_scm_is_available);
>
> +static int qcom_scm_fill_irq_fwspec_params(struct irq_fwspec *fwspec, u32 virq)
> +{
> + if (WARN(virq < GIC_SPI_BASE, "Unexpected virq: %d\n", virq)) {
> + return -ENXIO;
> + } else if (virq <= (GIC_SPI_BASE + GIC_MAX_SPI)) {
> + fwspec->param_count = 3;
> + fwspec->param[0] = GIC_IRQ_TYPE_SPI;
> + fwspec->param[1] = virq - GIC_SPI_BASE;
> + fwspec->param[2] = IRQ_TYPE_EDGE_RISING;
> + } else if (WARN(virq < GIC_ESPI_BASE, "Unexpected virq: %d\n", virq)) {
> + return -ENXIO;
> + } else if (virq < (GIC_ESPI_BASE + GIC_MAX_ESPI)) {
> + fwspec->param_count = 3;
> + fwspec->param[0] = GIC_IRQ_TYPE_ESPI;
> + fwspec->param[1] = virq - GIC_ESPI_BASE;
> + fwspec->param[2] = IRQ_TYPE_EDGE_RISING;
> + } else {
> + WARN(1, "Unexpected virq: %d\n", virq);
> + return -ENXIO;
> + }
> + return 0;
> +}
> +
> +static int qcom_scm_get_waitq_irq(void)
> +{
> + int ret;
> + u32 hwirq;
> + struct qcom_scm_desc desc = {
> + .svc = QCOM_SCM_SVC_WAITQ,
> + .cmd = QCOM_SCM_WAITQ_GET_INFO,
> + .owner = ARM_SMCCC_OWNER_SIP
> + };
> + struct qcom_scm_res res;
> + struct irq_fwspec fwspec;
> +
> + ret = qcom_scm_call_atomic(__scm->dev, &desc, &res);
> + if (ret)
> + return ret;
> +
> + fwspec.fwnode = of_node_to_fwnode(__scm->dev->of_node);
> + hwirq = res.result[1] & 0xffff;
> + ret = qcom_scm_fill_irq_fwspec_params(&fwspec, hwirq);
> + if (ret)
> + return ret;
> + ret = irq_create_fwspec_mapping(&fwspec);
> +
> + return ret;
> +}
> +
> static int qcom_scm_assert_valid_wq_ctx(u32 wq_ctx)
> {
> /* FW currently only supports a single wq_ctx (zero).
> @@ -1936,7 +1993,7 @@ static int qcom_scm_probe(struct platform_device *pdev)
> /* Let all above stores be available after this */
> smp_store_release(&__scm, scm);
>
> - irq = platform_get_irq_optional(pdev, 0);
> + irq = qcom_scm_get_waitq_irq();
> if (irq < 0) {
> if (irq != -ENXIO)
> return irq;
> diff --git a/drivers/firmware/qcom/qcom_scm.h b/drivers/firmware/qcom/qcom_scm.h
> index 685b8f59e7a6..ab0f88f5f777 100644
> --- a/drivers/firmware/qcom/qcom_scm.h
> +++ b/drivers/firmware/qcom/qcom_scm.h
> @@ -143,6 +143,7 @@ struct qcom_tzmem_pool *qcom_scm_get_tzmem_pool(void);
> #define QCOM_SCM_SVC_WAITQ 0x24
> #define QCOM_SCM_WAITQ_RESUME 0x02
> #define QCOM_SCM_WAITQ_GET_WQ_CTX 0x03
> +#define QCOM_SCM_WAITQ_GET_INFO 0x04
>
> #define QCOM_SCM_SVC_GPU 0x28
> #define QCOM_SCM_SVC_GPU_INIT_REGS 0x01
> --
> 2.34.1
>
Powered by blists - more mailing lists