Re: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
On Fri, Apr 12, 2019 at 10:41:16AM +0200, Alexandre Belloni wrote: > On 12/04/2019 02:32:54+, Anson Huang wrote: > > > > As i.MX SCU general irq function is picked up by Shawn, could > > > > you > > > > please also pick up below i.MX SC RTC alarm support patch ? > > > > https://patchwork.kernel.org/patch/10890525/ > > > > > > > > > > No, it can't go through Alexandre's tree due to dependency issue. > > > > > > Shawn, > > > Do you think you can pick it up? Alexandre already gave the Ack. > > > > > > AFAIK there's no other SCU RTC patches since v5.1-rc1 in Alexandre's tree. > > > > Yes, need to make sure the dependency is handled, the RTC alarm patch MUST > > be picked up > > ONLY when scu irq patch is ready, NOT sure how to handle this case for such > > scenario? > > > > Either it goes through arm-soc or if it can wait one release, I pick it > up for 5.3. The other solution involves immutable branch but I don't > think this is necessary. I send other subsystem patch through arm-soc tree only when it's absolutely necessary. This is not anything either critical or urgent, so I'm not going to take it. Wait one release, I'd say. Shawn
Re: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
On 12/04/2019 02:32:54+, Anson Huang wrote: > > > As i.MX SCU general irq function is picked up by Shawn, could you > > > please also pick up below i.MX SC RTC alarm support patch ? > > > https://patchwork.kernel.org/patch/10890525/ > > > > > > > No, it can't go through Alexandre's tree due to dependency issue. > > > > Shawn, > > Do you think you can pick it up? Alexandre already gave the Ack. > > > > AFAIK there's no other SCU RTC patches since v5.1-rc1 in Alexandre's tree. > > Yes, need to make sure the dependency is handled, the RTC alarm patch MUST be > picked up > ONLY when scu irq patch is ready, NOT sure how to handle this case for such > scenario? > Either it goes through arm-soc or if it can wait one release, I pick it up for 5.3. The other solution involves immutable branch but I don't think this is necessary. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
RE: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
Best Regards! Anson Huang > -Original Message- > From: Aisheng Dong > Sent: 2019年4月12日 10:26 > To: Anson Huang ; Shawn Guo > ; alexandre.bell...@bootlin.com > Cc: robh...@kernel.org; mark.rutl...@arm.com; s.ha...@pengutronix.de; > ker...@pengutronix.de; feste...@gmail.com; a.zu...@towertech.it; > alexandre.bell...@bootlin.com; ulf.hans...@linaro.org; sb...@kernel.org; > Peng Fan ; Daniel Baluta ; > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm- > ker...@lists.infradead.org; linux-...@vger.kernel.org; dl-linux-imx i...@nxp.com> > Subject: RE: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general > irq function > > > From: Anson Huang > > Sent: Friday, April 12, 2019 9:06 AM > > > > Hello, Alexandre > > As i.MX SCU general irq function is picked up by Shawn, could you > > please also pick up below i.MX SC RTC alarm support patch ? > > https://patchwork.kernel.org/patch/10890525/ > > > > No, it can't go through Alexandre's tree due to dependency issue. > > Shawn, > Do you think you can pick it up? Alexandre already gave the Ack. > > AFAIK there's no other SCU RTC patches since v5.1-rc1 in Alexandre's tree. Yes, need to make sure the dependency is handled, the RTC alarm patch MUST be picked up ONLY when scu irq patch is ready, NOT sure how to handle this case for such scenario? Anson. > > Regards > Dong Aisheng > > > > On Tue, Apr 09, 2019 at 04:59:55AM +, Anson Huang wrote: > > > > The System Controller Firmware (SCFW) controls RTC, thermal and > > > > WDOG etc., these resources' interrupt function are managed by SCU. > > > > When any IRQ pending, SCU will notify Linux via MU general > > > > interrupt channel #3, and Linux kernel needs to call SCU APIs to > > > > get IRQ status and notify each module to handle the interrupt. > > > > > > > > Since there is no data transmission for SCU IRQ notification, so > > > > doorbell mode is used for this MU channel, and SCU driver will use > > > > notifier mechanism to broadcast to every module which registers > > > > the SCU block notifier. > > > > > > > > Signed-off-by: Anson Huang > > > > Reviewed-by: Dong Aisheng > > > > > > Applied, thanks.
RE: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
> From: Anson Huang > Sent: Friday, April 12, 2019 9:06 AM > > Hello, Alexandre > As i.MX SCU general irq function is picked up by Shawn, could you please > also pick up below i.MX SC RTC alarm support patch ? > https://patchwork.kernel.org/patch/10890525/ > No, it can't go through Alexandre's tree due to dependency issue. Shawn, Do you think you can pick it up? Alexandre already gave the Ack. AFAIK there's no other SCU RTC patches since v5.1-rc1 in Alexandre's tree. Regards Dong Aisheng > > On Tue, Apr 09, 2019 at 04:59:55AM +, Anson Huang wrote: > > > The System Controller Firmware (SCFW) controls RTC, thermal and WDOG > > > etc., these resources' interrupt function are managed by SCU. When > > > any IRQ pending, SCU will notify Linux via MU general interrupt > > > channel #3, and Linux kernel needs to call SCU APIs to get IRQ > > > status and notify each module to handle the interrupt. > > > > > > Since there is no data transmission for SCU IRQ notification, so > > > doorbell mode is used for this MU channel, and SCU driver will use > > > notifier mechanism to broadcast to every module which registers the > > > SCU block notifier. > > > > > > Signed-off-by: Anson Huang > > > Reviewed-by: Dong Aisheng > > > > Applied, thanks.
RE: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
Hello, Alexandre As i.MX SCU general irq function is picked up by Shawn, could you please also pick up below i.MX SC RTC alarm support patch ? https://patchwork.kernel.org/patch/10890525/ Best Regards! Anson Huang > -Original Message- > From: Shawn Guo [mailto:shawn...@kernel.org] > Sent: 2019年4月11日 15:33 > To: Anson Huang > Cc: robh...@kernel.org; mark.rutl...@arm.com; s.ha...@pengutronix.de; > ker...@pengutronix.de; feste...@gmail.com; a.zu...@towertech.it; > alexandre.bell...@bootlin.com; Aisheng Dong ; > ulf.hans...@linaro.org; sb...@kernel.org; Peng Fan ; > Daniel Baluta ; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; linux-arm-ker...@lists.infradead.org; linux- > r...@vger.kernel.org; dl-linux-imx > Subject: [EXT] Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq > function > > WARNING: This email was created outside of NXP. DO NOT CLICK links or > attachments unless you recognize the sender and know the content is safe. > > > > On Tue, Apr 09, 2019 at 04:59:55AM +, Anson Huang wrote: > > The System Controller Firmware (SCFW) controls RTC, thermal and WDOG > > etc., these resources' interrupt function are managed by SCU. When any > > IRQ pending, SCU will notify Linux via MU general interrupt channel > > #3, and Linux kernel needs to call SCU APIs to get IRQ status and > > notify each module to handle the interrupt. > > > > Since there is no data transmission for SCU IRQ notification, so > > doorbell mode is used for this MU channel, and SCU driver will use > > notifier mechanism to broadcast to every module which registers the > > SCU block notifier. > > > > Signed-off-by: Anson Huang > > Reviewed-by: Dong Aisheng > > Applied, thanks.
Re: [PATCH V7 2/4] firmware: imx: enable imx scu general irq function
On Tue, Apr 09, 2019 at 04:59:55AM +, Anson Huang wrote: > The System Controller Firmware (SCFW) controls RTC, thermal > and WDOG etc., these resources' interrupt function are managed > by SCU. When any IRQ pending, SCU will notify Linux via MU general > interrupt channel #3, and Linux kernel needs to call SCU APIs > to get IRQ status and notify each module to handle the interrupt. > > Since there is no data transmission for SCU IRQ notification, so > doorbell mode is used for this MU channel, and SCU driver will > use notifier mechanism to broadcast to every module which registers > the SCU block notifier. > > Signed-off-by: Anson Huang > Reviewed-by: Dong Aisheng Applied, thanks.
[PATCH V7 2/4] firmware: imx: enable imx scu general irq function
The System Controller Firmware (SCFW) controls RTC, thermal
and WDOG etc., these resources' interrupt function are managed
by SCU. When any IRQ pending, SCU will notify Linux via MU general
interrupt channel #3, and Linux kernel needs to call SCU APIs
to get IRQ status and notify each module to handle the interrupt.
Since there is no data transmission for SCU IRQ notification, so
doorbell mode is used for this MU channel, and SCU driver will
use notifier mechanism to broadcast to every module which registers
the SCU block notifier.
Signed-off-by: Anson Huang
Reviewed-by: Dong Aisheng
---
Changes since V6:
- improve imx_scu_irq_enable() function name to
imx_scu_irq_group_enable();
- make imx_scu_irq_group_enable() function return int;
---
drivers/firmware/imx/Makefile | 2 +-
drivers/firmware/imx/imx-scu-irq.c | 168 +
drivers/firmware/imx/imx-scu.c | 6 ++
include/linux/firmware/imx/sci.h | 5 ++
4 files changed, 180 insertions(+), 1 deletion(-)
create mode 100644 drivers/firmware/imx/imx-scu-irq.c
diff --git a/drivers/firmware/imx/Makefile b/drivers/firmware/imx/Makefile
index 1b2e15b..802c4ad 100644
--- a/drivers/firmware/imx/Makefile
+++ b/drivers/firmware/imx/Makefile
@@ -1,3 +1,3 @@
# SPDX-License-Identifier: GPL-2.0
-obj-$(CONFIG_IMX_SCU) += imx-scu.o misc.o
+obj-$(CONFIG_IMX_SCU) += imx-scu.o misc.o imx-scu-irq.o
obj-$(CONFIG_IMX_SCU_PD) += scu-pd.o
diff --git a/drivers/firmware/imx/imx-scu-irq.c
b/drivers/firmware/imx/imx-scu-irq.c
new file mode 100644
index 000..043833a
--- /dev/null
+++ b/drivers/firmware/imx/imx-scu-irq.c
@@ -0,0 +1,168 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright 2019 NXP
+ *
+ * Implementation of the SCU IRQ functions using MU.
+ *
+ */
+
+#include
+#include
+#include
+
+#define IMX_SC_IRQ_FUNC_ENABLE 1
+#define IMX_SC_IRQ_FUNC_STATUS 2
+#define IMX_SC_IRQ_NUM_GROUP 4
+
+static u32 mu_resource_id;
+
+struct imx_sc_msg_irq_get_status {
+ struct imx_sc_rpc_msg hdr;
+ union {
+ struct {
+ u16 resource;
+ u8 group;
+ u8 reserved;
+ } __packed req;
+ struct {
+ u32 status;
+ } resp;
+ } data;
+};
+
+struct imx_sc_msg_irq_enable {
+ struct imx_sc_rpc_msg hdr;
+ u32 mask;
+ u16 resource;
+ u8 group;
+ u8 enable;
+} __packed;
+
+static struct imx_sc_ipc *imx_sc_irq_ipc_handle;
+static struct work_struct imx_sc_irq_work;
+static ATOMIC_NOTIFIER_HEAD(imx_scu_irq_notifier_chain);
+
+int imx_scu_irq_register_notifier(struct notifier_block *nb)
+{
+ return atomic_notifier_chain_register(
+ _scu_irq_notifier_chain, nb);
+}
+EXPORT_SYMBOL(imx_scu_irq_register_notifier);
+
+int imx_scu_irq_unregister_notifier(struct notifier_block *nb)
+{
+ return atomic_notifier_chain_unregister(
+ _scu_irq_notifier_chain, nb);
+}
+EXPORT_SYMBOL(imx_scu_irq_unregister_notifier);
+
+static int imx_scu_irq_notifier_call_chain(unsigned long status, u8 *group)
+{
+ return atomic_notifier_call_chain(_scu_irq_notifier_chain,
+ status, (void *)group);
+}
+
+static void imx_scu_irq_work_handler(struct work_struct *work)
+{
+ struct imx_sc_msg_irq_get_status msg;
+ struct imx_sc_rpc_msg *hdr =
+ u32 irq_status;
+ int ret;
+ u8 i;
+
+ for (i = 0; i < IMX_SC_IRQ_NUM_GROUP; i++) {
+ hdr->ver = IMX_SC_RPC_VERSION;
+ hdr->svc = IMX_SC_RPC_SVC_IRQ;
+ hdr->func = IMX_SC_IRQ_FUNC_STATUS;
+ hdr->size = 2;
+
+ msg.data.req.resource = mu_resource_id;
+ msg.data.req.group = i;
+
+ ret = imx_scu_call_rpc(imx_sc_irq_ipc_handle, , true);
+ if (ret) {
+ pr_err("get irq group %d status failed, ret %d\n",
+ i, ret);
+ return;
+ }
+
+ irq_status = msg.data.resp.status;
+ if (!irq_status)
+ continue;
+
+ imx_scu_irq_notifier_call_chain(irq_status, );
+ }
+}
+
+int imx_scu_irq_group_enable(u8 group, u32 mask, u8 enable)
+{
+ struct imx_sc_msg_irq_enable msg;
+ struct imx_sc_rpc_msg *hdr =
+ int ret;
+
+ hdr->ver = IMX_SC_RPC_VERSION;
+ hdr->svc = IMX_SC_RPC_SVC_IRQ;
+ hdr->func = IMX_SC_IRQ_FUNC_ENABLE;
+ hdr->size = 3;
+
+ msg.resource = mu_resource_id;
+ msg.group = group;
+ msg.mask = mask;
+ msg.enable = enable;
+
+ ret = imx_scu_call_rpc(imx_sc_irq_ipc_handle, , true);
+ if (ret)
+ pr_err("enable irq failed, group %d, mask %d, ret %d\n",
+ group, mask, ret);
+
+ return ret;
+}
+EXPORT_SYMBOL(imx_scu_irq_group_enable);
+
+static void
