* [PATCH v3 1/4] ARM: dts: ti: am57xx-idk-common: Remove unused ext-clk-src property
2026-08-04 17:05 [PATCH v3 0/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema and cleanups Bhargav Joshi
@ 2026-08-04 17:05 ` Bhargav Joshi
2026-08-04 17:13 ` sashiko-bot
2026-08-04 17:05 ` [PATCH v3 2/4] ARM: dts: ti: am335x-shc: Remove unused ti,no-init property Bhargav Joshi
` (2 subsequent siblings)
3 siblings, 1 reply; 9+ messages in thread
From: Bhargav Joshi @ 2026-08-04 17:05 UTC (permalink / raw)
To: Aaro Koskinen, Andreas Kemnade, Kevin Hilman, Roger Quadros,
Tony Lindgren, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Alexandre Belloni, Keerthy, Afzal Mohammed
Cc: linux-omap, devicetree, linux-kernel, linux-rtc, goledhruva,
m-chawdhry, daniel.baluta, simona.toaca, j.bhargav.u
The 'ext-clk-src' property is a legacy, undocumented boolean property
that is entirely unhandled by the rtc-omap driver. The driver instead
determines the clock source natively by checking the presence of the
"ext-clk" clock via devm_clk_get().
Remove this unnecessary property.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi b/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi
index 43e3623f079c..ef4bd554e92b 100644
--- a/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am57xx-idk-common.dtsi
@@ -445,7 +445,6 @@ &uart3 {
&rtc {
status = "okay";
- ext-clk-src;
};
&mac_sw {
--
2.55.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 2/4] ARM: dts: ti: am335x-shc: Remove unused ti,no-init property
2026-08-04 17:05 [PATCH v3 0/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema and cleanups Bhargav Joshi
2026-08-04 17:05 ` [PATCH v3 1/4] ARM: dts: ti: am57xx-idk-common: Remove unused ext-clk-src property Bhargav Joshi
@ 2026-08-04 17:05 ` Bhargav Joshi
2026-08-04 17:12 ` sashiko-bot
2026-08-04 17:05 ` [PATCH v3 3/4] ARM: dts: omap: dra7: add da830 fallback to am3352 rtc compatible string Bhargav Joshi
2026-08-04 17:05 ` [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema Bhargav Joshi
3 siblings, 1 reply; 9+ messages in thread
From: Bhargav Joshi @ 2026-08-04 17:05 UTC (permalink / raw)
To: Aaro Koskinen, Andreas Kemnade, Kevin Hilman, Roger Quadros,
Tony Lindgren, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Alexandre Belloni, Keerthy, Afzal Mohammed
Cc: linux-omap, devicetree, linux-kernel, linux-rtc, goledhruva,
m-chawdhry, daniel.baluta, simona.toaca, j.bhargav.u
The 'ti,no-init' property is a legacy, undocumented boolean property
that is entirely unhandled by the rtc-omap driver.
Remove this unnecessary property from dts.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
arch/arm/boot/dts/ti/omap/am335x-shc.dts | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm/boot/dts/ti/omap/am335x-shc.dts b/arch/arm/boot/dts/ti/omap/am335x-shc.dts
index 597482822608..2b946823928f 100644
--- a/arch/arm/boot/dts/ti/omap/am335x-shc.dts
+++ b/arch/arm/boot/dts/ti/omap/am335x-shc.dts
@@ -241,10 +241,6 @@ &mmc3 {
status = "okay";
};
-&rtc {
- ti,no-init;
-};
-
&sham {
status = "okay";
};
--
2.55.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 3/4] ARM: dts: omap: dra7: add da830 fallback to am3352 rtc compatible string
2026-08-04 17:05 [PATCH v3 0/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema and cleanups Bhargav Joshi
2026-08-04 17:05 ` [PATCH v3 1/4] ARM: dts: ti: am57xx-idk-common: Remove unused ext-clk-src property Bhargav Joshi
2026-08-04 17:05 ` [PATCH v3 2/4] ARM: dts: ti: am335x-shc: Remove unused ti,no-init property Bhargav Joshi
@ 2026-08-04 17:05 ` Bhargav Joshi
2026-08-04 17:16 ` sashiko-bot
2026-08-04 17:05 ` [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema Bhargav Joshi
3 siblings, 1 reply; 9+ messages in thread
From: Bhargav Joshi @ 2026-08-04 17:05 UTC (permalink / raw)
To: Aaro Koskinen, Andreas Kemnade, Kevin Hilman, Roger Quadros,
Tony Lindgren, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Alexandre Belloni, Keerthy, Afzal Mohammed
Cc: linux-omap, devicetree, linux-kernel, linux-rtc, goledhruva,
m-chawdhry, daniel.baluta, simona.toaca, j.bhargav.u
The AM3352 RTC is compatible with the DA830 RTC. Update the compatible
string to include the ti,da830-rtc fallback to align with the device
tree bindings and other board files.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
arch/arm/boot/dts/ti/omap/dra7-l4.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/ti/omap/dra7-l4.dtsi b/arch/arm/boot/dts/ti/omap/dra7-l4.dtsi
index c8d325b0f57b..9e9609cf3188 100644
--- a/arch/arm/boot/dts/ti/omap/dra7-l4.dtsi
+++ b/arch/arm/boot/dts/ti/omap/dra7-l4.dtsi
@@ -3556,7 +3556,7 @@ rtctarget: target-module@38000 { /* 0x48838000, ap 29 12.0 */
ranges = <0x0 0x38000 0x1000>;
rtc: rtc@0 {
- compatible = "ti,am3352-rtc";
+ compatible = "ti,am3352-rtc", "ti,da830-rtc";
reg = <0x0 0x100>;
interrupts = <GIC_SPI 217 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 217 IRQ_TYPE_LEVEL_HIGH>;
--
2.55.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema
2026-08-04 17:05 [PATCH v3 0/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema and cleanups Bhargav Joshi
` (2 preceding siblings ...)
2026-08-04 17:05 ` [PATCH v3 3/4] ARM: dts: omap: dra7: add da830 fallback to am3352 rtc compatible string Bhargav Joshi
@ 2026-08-04 17:05 ` Bhargav Joshi
2026-08-04 17:30 ` sashiko-bot
3 siblings, 1 reply; 9+ messages in thread
From: Bhargav Joshi @ 2026-08-04 17:05 UTC (permalink / raw)
To: Aaro Koskinen, Andreas Kemnade, Kevin Hilman, Roger Quadros,
Tony Lindgren, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Alexandre Belloni, Keerthy, Afzal Mohammed
Cc: linux-omap, devicetree, linux-kernel, linux-rtc, goledhruva,
m-chawdhry, daniel.baluta, simona.toaca, j.bhargav.u
Convert the Texas Instruments OMAP Real Time Clock (RTC) binding from
the legacy text format to the DT schema. Mark 'ti,hwmods' as deprecated
as it is no longer used, it is kept to support legacy boards.
Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
Documentation/devicetree/bindings/rtc/rtc-omap.txt | 53 --------
.../devicetree/bindings/rtc/ti,omap-rtc.yaml | 140 +++++++++++++++++++++
2 files changed, 140 insertions(+), 53 deletions(-)
diff --git a/Documentation/devicetree/bindings/rtc/rtc-omap.txt b/Documentation/devicetree/bindings/rtc/rtc-omap.txt
deleted file mode 100644
index 062ebb14cecf..000000000000
--- a/Documentation/devicetree/bindings/rtc/rtc-omap.txt
+++ /dev/null
@@ -1,53 +0,0 @@
-TI Real Time Clock
-
-Required properties:
-- compatible:
- - "ti,da830-rtc" - for RTC IP used similar to that on DA8xx SoC family.
- - "ti,am3352-rtc" - for RTC IP used similar to that on AM335x SoC family.
- This RTC IP has special WAKE-EN Register to enable
- Wakeup generation for event Alarm. It can also be
- used to control an external PMIC via the
- pmic_power_en pin.
- - "ti,am4372-rtc" - for RTC IP used similar to that on AM437X SoC family.
-- reg: Address range of rtc register set
-- interrupts: rtc timer, alarm interrupts in order
-
-Optional properties:
-- system-power-controller: whether the rtc is controlling the system power
- through pmic_power_en
-- clocks: Any internal or external clocks feeding in to rtc
-- clock-names: Corresponding names of the clocks
-- pinctrl-0: a phandle pointing to the pin settings for the device
-- pinctrl-names: should be "default"
-
-Optional subnodes:
-- generic pinctrl node
-
-Required pinctrl subnodes properties:
-- pins - Names of ext_wakeup pins to configure
-
-Optional pinctrl subnodes properties:
-- input-enable - Enables ext_wakeup
-- ti,active-high - Set input active high (by default active low)
-
-Example:
-
-rtc@1c23000 {
- compatible = "ti,da830-rtc";
- reg = <0x23000 0x1000>;
- interrupts = <19
- 19>;
- interrupt-parent = <&intc>;
- system-power-controller;
- clocks = <&clk_32k_rtc>, <&clk_32768_ck>;
- clock-names = "ext-clk", "int-clk";
-
- pinctrl-0 = <&ext_wakeup>;
- pinctrl-names = "default";
-
- ext_wakeup: ext-wakeup {
- pins = "ext_wakeup0";
- input-enable;
- ti,active-high;
- };
-};
diff --git a/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml b/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml
new file mode 100644
index 000000000000..02b3c23cf435
--- /dev/null
+++ b/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml
@@ -0,0 +1,140 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/rtc/ti,omap-rtc.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments OMAP Real Time Clock
+
+maintainers:
+ - Keerthy <j-keerthy@ti.com>
+ - Afzal Mohammed <afzal@ti.com>
+
+description:
+ The OMAP RTC is a year/month/day/hours/minutes/seconds BCD clock with
+ century-range alarm matching, driven by the 32kHz clock.
+
+properties:
+ compatible:
+ oneOf:
+ - const: ti,da830-rtc
+ - items:
+ - const: ti,am3352-rtc
+ - const: ti,da830-rtc
+ - items:
+ - const: ti,am4372-rtc
+ - const: ti,am3352-rtc
+ - const: ti,da830-rtc
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ minItems: 2
+ maxItems: 2
+
+ system-power-controller:
+ type: boolean
+ description:
+ Indicates that this RTC controls system power via the pmic_power_en pin.
+
+ clocks:
+ minItems: 1
+ maxItems: 2
+
+ clock-names:
+ minItems: 1
+ items:
+ - enum: [ext-clk, int-clk]
+ - const: int-clk
+
+ pinctrl-0:
+ description:
+ Phandle to pin configuration for the external wakeup pins.
+
+ pinctrl-names:
+ minItems: 1
+
+ ti,hwmods:
+ $ref: /schemas/types.yaml#/definitions/string
+ description:
+ Name of the hwmod associated with the RTC.
+ const: rtc
+ deprecated: true
+
+patternProperties:
+ "^ext-wakeup(-[0-9]+)?$":
+ type: object
+
+ description:
+ Child node describing external wakeup pin configuration.
+
+ properties:
+ pins:
+ pattern: '^ext_wakeup[0-3]$'
+
+ input-enable:
+ type: boolean
+ description:
+ Enables the external wakeup input on the selected pin.
+
+ ti,active-high:
+ type: boolean
+ description:
+ Sets the wakeup input polarity to active high. By default the
+ input is active low.
+
+ required:
+ - pins
+
+ additionalProperties: false
+
+required:
+ - compatible
+ - reg
+ - interrupts
+
+allOf:
+ - $ref: rtc.yaml#
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: ti,am3352-rtc
+ then:
+ properties:
+ system-power-controller: false
+ patternProperties:
+ "^ext-wakeup(-[0-9]+)?$": false
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ rtc@23000 {
+ compatible = "ti,da830-rtc";
+ reg = <0x23000 0x1000>;
+ interrupts = <19>, <19>;
+ clocks = <&clk_32768_ck>;
+ clock-names = "int-clk";
+ };
+
+ - |
+ rtc@0 {
+ compatible = "ti,am3352-rtc", "ti,da830-rtc";
+ reg = <0x0 0x1000>;
+ interrupts = <75>, <76>;
+ system-power-controller;
+ clocks = <&clk_32k_rtc>, <&clk_32768_ck>;
+ clock-names = "ext-clk", "int-clk";
+
+ pinctrl-0 = <&ext_wakeup>;
+ pinctrl-names = "default";
+
+ ext-wakeup {
+ pins = "ext_wakeup0";
+ input-enable;
+ ti,active-high;
+ };
+ };
--
2.55.0
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema
2026-08-04 17:05 ` [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema Bhargav Joshi
@ 2026-08-04 17:30 ` sashiko-bot
0 siblings, 0 replies; 9+ messages in thread
From: sashiko-bot @ 2026-08-04 17:30 UTC (permalink / raw)
To: Bhargav Joshi; +Cc: conor+dt, linux-rtc, devicetree, Alexandre Belloni, robh
Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Low] The `pattern` constraint on `pins` is bypassed because it is applied directly to an array property, rather than its items.
- [Low] The example contains an undefined label `ext_wakeup`, causing a dtc compilation error.
--
--- Patch [4]: [PATCH v3 4/4] dt-bindings: rtc: ti,omap-rtc: Convert to DT schema ---
commit 37b8c1cde7af9ea6b7fed4b7f431ec999b775ef6
Author: Bhargav Joshi <j.bhargav.u@gmail.com>
dt-bindings: rtc: ti,omap-rtc: Convert to DT schema
Convert the Texas Instruments OMAP Real Time Clock (RTC) binding from
the legacy text format to the DT schema. Mark 'ti,hwmods' as deprecated
as it is no longer used, it is kept to support legacy boards.
> diff --git a/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml b/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml
> new file mode 100644
> index 0000000000000..02b3c23cf4350
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/ti,omap-rtc.yaml
[ ... ]
> +patternProperties:
> + "^ext-wakeup(-[0-9]+)?$":
> + type: object
> +
> + description:
> + Child node describing external wakeup pin configuration.
> +
> + properties:
> + pins:
> + pattern: '^ext_wakeup[0-3]$'
[Severity: Low]
Does this pattern constraint actually apply? Because pins is a string-array,
the pattern keyword directly on the property is ignored by the JSON schema
validator. It needs to be nested under items: { pattern: '^ext_wakeup[0-3]$' }
to validate the array elements.
[ ... ]
> + - |
> + rtc@0 {
> + compatible = "ti,am3352-rtc", "ti,da830-rtc";
> + reg = <0x0 0x1000>;
> + interrupts = <75>, <76>;
> + system-power-controller;
> + clocks = <&clk_32k_rtc>, <&clk_32768_ck>;
> + clock-names = "ext-clk", "int-clk";
> +
> + pinctrl-0 = <&ext_wakeup>;
> + pinctrl-names = "default";
> +
> + ext-wakeup {
> + pins = "ext_wakeup0";
[Severity: Low]
Will this example compile with dtc? The pinctrl-0 property references
<&ext_wakeup>, but the ext-wakeup subnode here is missing the ext_wakeup:
label that was present in the legacy text binding, which will cause a dtc
compilation error.
> + input-enable;
> + ti,active-high;
> + };
> + };
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260804-ti-omap-rtc-v3-0-ba3bbd8af570@gmail.com?part=4
^ permalink raw reply [flat|nested] 9+ messages in thread