[PATCH] rtc: zynqmp: allow on Versal architectures
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com --- drivers/rtc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..ecdcfe1f240 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,7 +294,7 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver" - depends on ARCH_ZYNQMP + depends on ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2 help Say "yes" here to support the on chip real time clock present on Xilinx ZynqMP SoC.
base-commit: 6d41f0a39d6423c8e57e92ebbe9f8c0333a63f72
On 1/7/25 17:20, Vincent Fazio wrote:
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com
drivers/rtc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..ecdcfe1f240 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,7 +294,7 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver"
- depends on ARCH_ZYNQMP
- depends on ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2 help Say "yes" here to support the on chip real time clock present on Xilinx ZynqMP SoC.
base-commit: 6d41f0a39d6423c8e57e92ebbe9f8c0333a63f72
The patch looks good but there is missing also DM_RTC dependency. Can you please send v2 with also adding it?
Thanks, Michal
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com --- Changes in v2: - Add DM_RTC dependency (Michal) --- drivers/rtc/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..fe8cb254603 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,10 +294,10 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver" - depends on ARCH_ZYNQMP + depends on DM_RTC && (ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2) help Say "yes" here to support the on chip real time clock - present on Xilinx ZynqMP SoC. + present on Xilinx ZynqMP SoCs.
config RTC_HT1380 bool "Enable Holtek HT1380/HT1381 RTC driver"
-----Original Message----- From: Vincent Fazio vfazio@xes-inc.com Sent: Tuesday, January 14, 2025 7:31 AM To: michal.simek@amd.com; u-boot@lists.denx.de Cc: trini@konsulko.com; vfazio@gmail.com; Vincent Fazio <vfazio@xes- inc.com> Subject: [PATCH v2] rtc: zynqmp: allow on Versal architectures
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com
Changes in v2:
- Add DM_RTC dependency (Michal)
drivers/rtc/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..fe8cb254603 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,10 +294,10 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver"
- depends on ARCH_ZYNQMP
- depends on DM_RTC && (ARCH_ZYNQMP || ARCH_VERSAL ||
ARCH_VERSAL_NET || +ARCH_VERSAL2) help Say "yes" here to support the on chip real time clock
present on Xilinx ZynqMP SoC.
present on Xilinx ZynqMP SoCs.
Sorry for the noise here. I had at one point included Versal in the help text and pluralized SoCs.
I backed out the "Versal" part of that out but apparently failed to revert all of it.
I'll send a v3 to drop this change.
config RTC_HT1380 bool "Enable Holtek HT1380/HT1381 RTC driver" -- 2.34.1
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com --- Changes in v3: - Revert a typo in the help text
Changes in v2: - Add DM_RTC dependency (Michal) --- drivers/rtc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..6467f20422b 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,7 +294,7 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver" - depends on ARCH_ZYNQMP + depends on DM_RTC && (ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2) help Say "yes" here to support the on chip real time clock present on Xilinx ZynqMP SoC.
On 1/14/25 14:58, Vincent Fazio wrote:
Allow the ZyncMP RTC driver to be enabled on Versal architectures.
Also, require DM_RTC since the driver uses the RTC driver model.
Signed-off-by: Vincent Fazio vfazio@xes-inc.com
Changes in v3:
- Revert a typo in the help text
Changes in v2:
- Add DM_RTC dependency (Michal)
drivers/rtc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index 9c2d1398247..6467f20422b 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -294,7 +294,7 @@ config RTC_DAVINCI
config RTC_ZYNQMP bool "Enable ZynqMP RTC driver"
- depends on ARCH_ZYNQMP
- depends on DM_RTC && (ARCH_ZYNQMP || ARCH_VERSAL || ARCH_VERSAL_NET || ARCH_VERSAL2) help Say "yes" here to support the on chip real time clock present on Xilinx ZynqMP SoC.
Applied. M
participants (2)
-
Michal Simek -
Vincent Fazio