* [PATCH V5 1/4] dt-bindings: fsl: scu: add rtc binding
2018-12-20 8:55 [PATCH V5 0/4] Add i.MX system controller RTC driver Anson Huang
@ 2018-12-20 8:56 ` Anson Huang
2018-12-21 9:24 ` Alexandre Belloni
2018-12-20 8:56 ` [PATCH V5 2/4] rtc: add i.MX system controller RTC support Anson Huang
` (2 subsequent siblings)
3 siblings, 1 reply; 9+ messages in thread
From: Anson Huang @ 2018-12-20 8:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, shawnguo, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc
Cc: dl-linux-imx
NXP i.MX8QXP is an ARMv8 SoC with a Cortex-M4 core inside as
system controller, the system controller is in charge of system
power, clock and secure RTC etc. management, Linux kernel
has to communicate with system controller via MU (message unit)
IPC to do RTC operation, this patch adds binding doc for i.MX
system controller RTC driver.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
No change since V4.
---
Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
index c20f38e..3f3d697 100644
--- a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
+++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
@@ -106,6 +106,12 @@ Required properties for Pinctrl sub nodes:
[2] Documentation/devicetree/bindings/power/power_domain.txt
[3] Documentation/devicetree/bindings/pinctrl/fsl,imx-pinctrl.txt
+RTC bindings based on SCU Message Protocol
+------------------------------------------------------------
+
+Required properties:
+- compatible: should be "fsl,imx8qxp-sc-rtc";
+
Example (imx8qxp):
-------------
lsio_mu1: mailbox@5d1c0000 {
@@ -148,6 +154,10 @@ firmware {
compatible = "fsl,imx8qxp-scu-pd";
#power-domain-cells = <1>;
};
+
+ rtc: rtc {
+ compatible = "fsl,imx8qxp-sc-rtc";
+ };
};
};
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V5 1/4] dt-bindings: fsl: scu: add rtc binding
2018-12-20 8:56 ` [PATCH V5 1/4] dt-bindings: fsl: scu: add rtc binding Anson Huang
@ 2018-12-21 9:24 ` Alexandre Belloni
0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2018-12-21 9:24 UTC (permalink / raw)
To: Anson Huang
Cc: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
Aisheng Dong, shawnguo, s.hauer, Andy Gross, horms+renesas,
heiko, arnd, bjorn.andersson, enric.balletbo, amit.kucheria,
ezequiel, olof, devicetree, linux-kernel, linux-arm-kernel,
linux-rtc, dl-linux-imx
On 20/12/2018 08:56:07+0000, Anson Huang wrote:
> NXP i.MX8QXP is an ARMv8 SoC with a Cortex-M4 core inside as
> system controller, the system controller is in charge of system
> power, clock and secure RTC etc. management, Linux kernel
> has to communicate with system controller via MU (message unit)
> IPC to do RTC operation, this patch adds binding doc for i.MX
> system controller RTC driver.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> No change since V4.
> ---
> Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
Applied, thanks.
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH V5 2/4] rtc: add i.MX system controller RTC support
2018-12-20 8:55 [PATCH V5 0/4] Add i.MX system controller RTC driver Anson Huang
2018-12-20 8:56 ` [PATCH V5 1/4] dt-bindings: fsl: scu: add rtc binding Anson Huang
@ 2018-12-20 8:56 ` Anson Huang
2018-12-21 9:24 ` Alexandre Belloni
2018-12-20 8:56 ` [PATCH V5 3/4] defconfig: arm64: " Anson Huang
2018-12-20 8:56 ` [PATCH V5 4/4] ARM64: dts: imx: add i.MX8QXP " Anson Huang
3 siblings, 1 reply; 9+ messages in thread
From: Anson Huang @ 2018-12-20 8:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, shawnguo, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc
Cc: dl-linux-imx
i.MX8QXP is an ARMv8 SoC which has a Cortex-M4 system controller
inside, the system controller is in charge of controlling power,
clock and secure rtc etc..
This patch adds i.MX system controller RTC driver support,
Linux kernel has to communicate with system controller via MU
(message unit) IPC to set/get RTC time and other alarm functions,
since the RTC set time needs to be done in secure EL3 mode (required
by system controller firmware) and alarm functions needs to be done
with general MU IRQ handle, these depend on other components which
are NOT ready, so this patch ONLY enables the RTC time read.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
ChangeLog since V4:
- Add dependency on IMX_SCU to avoid compile error when IMX_SCU is NOT enabled;
- Use dev_err instead of pr_err;
- Remove unneccssary different errors check in imx_scu_get_handle which is already done in API.
---
drivers/rtc/Kconfig | 7 ++++
drivers/rtc/Makefile | 1 +
drivers/rtc/rtc-imx-sc.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 99 insertions(+)
create mode 100644 drivers/rtc/rtc-imx-sc.c
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index a819ef0..225b0b8 100644
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -1677,6 +1677,13 @@ config RTC_DRV_SNVS
This driver can also be built as a module, if so, the module
will be called "rtc-snvs".
+config RTC_DRV_IMX_SC
+ depends on IMX_SCU
+ tristate "NXP i.MX System Controller RTC support"
+ help
+ If you say yes here you get support for the NXP i.MX System
+ Controller RTC module.
+
config RTC_DRV_SIRFSOC
tristate "SiRFSOC RTC"
depends on ARCH_SIRF
diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile
index 290c173..f97c05e 100644
--- a/drivers/rtc/Makefile
+++ b/drivers/rtc/Makefile
@@ -75,6 +75,7 @@ obj-$(CONFIG_RTC_DRV_GOLDFISH) += rtc-goldfish.o
obj-$(CONFIG_RTC_DRV_HID_SENSOR_TIME) += rtc-hid-sensor-time.o
obj-$(CONFIG_RTC_DRV_HYM8563) += rtc-hym8563.o
obj-$(CONFIG_RTC_DRV_IMXDI) += rtc-imxdi.o
+obj-$(CONFIG_RTC_DRV_IMX_SC) += rtc-imx-sc.o
obj-$(CONFIG_RTC_DRV_ISL12022) += rtc-isl12022.o
obj-$(CONFIG_RTC_DRV_ISL12026) += rtc-isl12026.o
obj-$(CONFIG_RTC_DRV_ISL1208) += rtc-isl1208.o
diff --git a/drivers/rtc/rtc-imx-sc.c b/drivers/rtc/rtc-imx-sc.c
new file mode 100644
index 0000000..7ff0854
--- /dev/null
+++ b/drivers/rtc/rtc-imx-sc.c
@@ -0,0 +1,91 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright 2018 NXP.
+ */
+
+#include <linux/firmware/imx/sci.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/platform_device.h>
+#include <linux/rtc.h>
+
+#define IMX_SC_TIMER_FUNC_GET_RTC_SEC1970 9
+#define IMX_SC_TIMER_FUNC_SET_RTC_TIME 6
+
+static struct imx_sc_ipc *rtc_ipc_handle;
+static struct rtc_device *imx_sc_rtc;
+
+struct imx_sc_msg_timer_get_rtc_time {
+ struct imx_sc_rpc_msg hdr;
+ u32 time;
+} __packed;
+
+static int imx_sc_rtc_read_time(struct device *dev, struct rtc_time *tm)
+{
+ struct imx_sc_msg_timer_get_rtc_time msg;
+ struct imx_sc_rpc_msg *hdr = &msg.hdr;
+ int ret;
+
+ hdr->ver = IMX_SC_RPC_VERSION;
+ hdr->svc = IMX_SC_RPC_SVC_TIMER;
+ hdr->func = IMX_SC_TIMER_FUNC_GET_RTC_SEC1970;
+ hdr->size = 1;
+
+ ret = imx_scu_call_rpc(rtc_ipc_handle, &msg, true);
+ if (ret) {
+ dev_err(dev, "read rtc time failed, ret %d\n", ret);
+ return ret;
+ }
+
+ rtc_time_to_tm(msg.time, tm);
+
+ return 0;
+}
+
+static const struct rtc_class_ops imx_sc_rtc_ops = {
+ .read_time = imx_sc_rtc_read_time,
+};
+
+static int imx_sc_rtc_probe(struct platform_device *pdev)
+{
+ int ret;
+
+ ret = imx_scu_get_handle(&rtc_ipc_handle);
+ if (ret)
+ return ret;
+
+ imx_sc_rtc = devm_rtc_allocate_device(&pdev->dev);
+ if (IS_ERR(imx_sc_rtc))
+ return PTR_ERR(imx_sc_rtc);
+
+ imx_sc_rtc->ops = &imx_sc_rtc_ops;
+ imx_sc_rtc->range_min = 0;
+ imx_sc_rtc->range_max = U32_MAX;
+
+ ret = rtc_register_device(imx_sc_rtc);
+ if (ret) {
+ dev_err(&pdev->dev, "failed to register rtc: %d\n", ret);
+ return ret;
+ }
+
+ return 0;
+}
+
+static const struct of_device_id imx_sc_dt_ids[] = {
+ { .compatible = "fsl,imx8qxp-sc-rtc", },
+ {}
+};
+MODULE_DEVICE_TABLE(of, imx_sc_dt_ids);
+
+static struct platform_driver imx_sc_rtc_driver = {
+ .driver = {
+ .name = "imx-sc-rtc",
+ .of_match_table = imx_sc_dt_ids,
+ },
+ .probe = imx_sc_rtc_probe,
+};
+module_platform_driver(imx_sc_rtc_driver);
+
+MODULE_AUTHOR("Anson Huang <Anson.Huang@nxp.com>");
+MODULE_DESCRIPTION("NXP i.MX System Controller RTC Driver");
+MODULE_LICENSE("GPL");
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V5 2/4] rtc: add i.MX system controller RTC support
2018-12-20 8:56 ` [PATCH V5 2/4] rtc: add i.MX system controller RTC support Anson Huang
@ 2018-12-21 9:24 ` Alexandre Belloni
0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2018-12-21 9:24 UTC (permalink / raw)
To: Anson Huang
Cc: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
Aisheng Dong, shawnguo, s.hauer, Andy Gross, horms+renesas,
heiko, arnd, bjorn.andersson, enric.balletbo, amit.kucheria,
ezequiel, olof, devicetree, linux-kernel, linux-arm-kernel,
linux-rtc, dl-linux-imx
On 20/12/2018 08:56:16+0000, Anson Huang wrote:
> i.MX8QXP is an ARMv8 SoC which has a Cortex-M4 system controller
> inside, the system controller is in charge of controlling power,
> clock and secure rtc etc..
>
> This patch adds i.MX system controller RTC driver support,
> Linux kernel has to communicate with system controller via MU
> (message unit) IPC to set/get RTC time and other alarm functions,
> since the RTC set time needs to be done in secure EL3 mode (required
> by system controller firmware) and alarm functions needs to be done
> with general MU IRQ handle, these depend on other components which
> are NOT ready, so this patch ONLY enables the RTC time read.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
> ChangeLog since V4:
> - Add dependency on IMX_SCU to avoid compile error when IMX_SCU is NOT enabled;
> - Use dev_err instead of pr_err;
> - Remove unneccssary different errors check in imx_scu_get_handle which is already done in API.
> ---
> drivers/rtc/Kconfig | 7 ++++
> drivers/rtc/Makefile | 1 +
> drivers/rtc/rtc-imx-sc.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 99 insertions(+)
> create mode 100644 drivers/rtc/rtc-imx-sc.c
>
Applied, thanks.
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH V5 3/4] defconfig: arm64: add i.MX system controller RTC support
2018-12-20 8:55 [PATCH V5 0/4] Add i.MX system controller RTC driver Anson Huang
2018-12-20 8:56 ` [PATCH V5 1/4] dt-bindings: fsl: scu: add rtc binding Anson Huang
2018-12-20 8:56 ` [PATCH V5 2/4] rtc: add i.MX system controller RTC support Anson Huang
@ 2018-12-20 8:56 ` Anson Huang
2019-01-14 1:43 ` Shawn Guo
2018-12-20 8:56 ` [PATCH V5 4/4] ARM64: dts: imx: add i.MX8QXP " Anson Huang
3 siblings, 1 reply; 9+ messages in thread
From: Anson Huang @ 2018-12-20 8:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, shawnguo, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc
Cc: dl-linux-imx
This patch enables CONFIG_RTC_DRV_IMX_SC as module by default.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
No change since V4.
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index d3aa3f7..abd80c5 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -572,6 +572,7 @@ CONFIG_RTC_DRV_PL031=y
CONFIG_RTC_DRV_SUN6I=y
CONFIG_RTC_DRV_ARMADA38X=y
CONFIG_RTC_DRV_TEGRA=y
+CONFIG_RTC_DRV_IMX_SC=m
CONFIG_RTC_DRV_XGENE=y
CONFIG_DMADEVICES=y
CONFIG_DMA_BCM2835=m
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V5 3/4] defconfig: arm64: add i.MX system controller RTC support
2018-12-20 8:56 ` [PATCH V5 3/4] defconfig: arm64: " Anson Huang
@ 2019-01-14 1:43 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2019-01-14 1:43 UTC (permalink / raw)
To: Anson Huang
Cc: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc, dl-linux-imx
On Thu, Dec 20, 2018 at 08:56:25AM +0000, Anson Huang wrote:
> This patch enables CONFIG_RTC_DRV_IMX_SC as module by default.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH V5 4/4] ARM64: dts: imx: add i.MX8QXP system controller RTC support
2018-12-20 8:55 [PATCH V5 0/4] Add i.MX system controller RTC driver Anson Huang
` (2 preceding siblings ...)
2018-12-20 8:56 ` [PATCH V5 3/4] defconfig: arm64: " Anson Huang
@ 2018-12-20 8:56 ` Anson Huang
2019-01-14 1:50 ` Shawn Guo
3 siblings, 1 reply; 9+ messages in thread
From: Anson Huang @ 2018-12-20 8:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, shawnguo, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc
Cc: dl-linux-imx
Add i.MX8QXP system controller RTC support.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
No change since V4.
---
arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
index da99b6f..9b08688 100644
--- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
@@ -83,6 +83,10 @@
compatible = "fsl,imx8qxp-scu-pd";
#power-domain-cells = <1>;
};
+
+ rtc: rtc {
+ compatible = "fsl,imx8qxp-sc-rtc";
+ };
};
adma_subsys: bus@59000000 {
--
2.7.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V5 4/4] ARM64: dts: imx: add i.MX8QXP system controller RTC support
2018-12-20 8:56 ` [PATCH V5 4/4] ARM64: dts: imx: add i.MX8QXP " Anson Huang
@ 2019-01-14 1:50 ` Shawn Guo
0 siblings, 0 replies; 9+ messages in thread
From: Shawn Guo @ 2019-01-14 1:50 UTC (permalink / raw)
To: Anson Huang
Cc: robh+dt, mark.rutland, catalin.marinas, will.deacon, a.zummo,
alexandre.belloni, Aisheng Dong, s.hauer, Andy Gross,
horms+renesas, heiko, arnd, bjorn.andersson, enric.balletbo,
amit.kucheria, ezequiel, olof, devicetree, linux-kernel,
linux-arm-kernel, linux-rtc, dl-linux-imx
On Thu, Dec 20, 2018 at 08:56:33AM +0000, Anson Huang wrote:
> Add i.MX8QXP system controller RTC support.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Different from arm32 dts patches, we use prefix 'arm64: ...' for arm64
ones. I fixed it up and applied the patch.
Shawn
^ permalink raw reply [flat|nested] 9+ messages in thread