[PATCH] arm64: dts: amlogic: meson-sm1-odroid-hc4: wire the RTC alarm interrupt

[PATCH] arm64: dts: amlogic: meson-sm1-odroid-hc4: wire the RTC alarm interrupt

[Igor Velkov via B4 Relay]

From: Igor Velkov <iav@iav.lv>

The PCF8563 on the OLED/RTC add-on board drives its interrupt output
to GPIOX_5, pin 4 of the P1 header, but the rtc node does not describe
it. The alarm interrupt is therefore never delivered and cannot wake
the board.

Describe the line as a level-low interrupt of the GPIO interrupt
controller.

Tested on ODROID-HC4: the alarm interrupt fires and wakes the board
from s2idle.

Assisted-by: LLM
Signed-off-by: Igor Velkov <iav@iav.lv>
---
This needs commit 5831f5fef100 ("rtc: pcf8563: pass the driver data to
the alarm interrupt handler") in the rtc tree: without it the first
alarm oopses. Waking from s2idle also needs commit 5b6e2a46b80d
("irqchip/meson-gpio: Allow the GPIO interrupts to wake the system")
in tip irq/drivers.
---
 arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
index 5f15c2299509..25a8a60e152c 100644
--- a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
@@ -105,6 +105,8 @@ rtc: rtc@51 {
 		status = "okay";
 		compatible = "nxp,pcf8563";
 		reg = <0x51>;
+		interrupt-parent = <&gpio_intc>;
+		interrupts = <IRQID_GPIOX_5 IRQ_TYPE_LEVEL_LOW>;
 		wakeup-source;
 	};
 };

---
base-commit: b44e2be772f866985cafdf90355285069ea68495
change-id: 20261020-hc4-rtc-alarm-irq-b2b8c68d54f5

Best regards,
-- 
Igor Velkov <iav@iav.lv>

Re: [PATCH] arm64: dts: amlogic: meson-sm1-odroid-hc4: wire the RTC alarm interrupt

[Neil Armstrong]

On 10/19/26 23:52, Igor Velkov via B4 Relay wrote:
> From: Igor Velkov <iav@iav.lv>
> 
> The PCF8563 on the OLED/RTC add-on board drives its interrupt output
> to GPIOX_5, pin 4 of the P1 header, but the rtc node does not describe
> it. The alarm interrupt is therefore never delivered and cannot wake
> the board.
> 
> Describe the line as a level-low interrupt of the GPIO interrupt
> controller.
> 
> Tested on ODROID-HC4: the alarm interrupt fires and wakes the board
> from s2idle.
> 
> Assisted-by: LLM
> Signed-off-by: Igor Velkov <iav@iav.lv>
> ---
> This needs commit 5831f5fef100 ("rtc: pcf8563: pass the driver data to
> the alarm interrupt handler") in the rtc tree: without it the first
> alarm oopses. Waking from s2idle also needs commit 5b6e2a46b80d
> ("irqchip/meson-gpio: Allow the GPIO interrupts to wake the system")
> in tip irq/drivers.
> ---
>   arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
> index 5f15c2299509..25a8a60e152c 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dts
> @@ -105,6 +105,8 @@ rtc: rtc@51 {
>   		status = "okay";
>   		compatible = "nxp,pcf8563";
>   		reg = <0x51>;
> +		interrupt-parent = <&gpio_intc>;
> +		interrupts = <IRQID_GPIOX_5 IRQ_TYPE_LEVEL_LOW>;
>   		wakeup-source;
>   	};
>   };
> 
> ---
> base-commit: b44e2be772f866985cafdf90355285069ea68495
> change-id: 20261020-hc4-rtc-alarm-irq-b2b8c68d54f5
> 
> Best regards,

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>

Thanks,
Neil