Re: [PATCH 2/2] arm64: dts: qcom: ipq5018: add WDT

From: Krzysztof Kozlowski
Date: Tue Aug 15 2023 - 10:41:34 EST


On 15/08/2023 16:17, Robert Marko wrote:
> Add the required DT node for WDT operation.
>
> Signed-off-by: Robert Marko <robimarko@xxxxxxxxx>
> ---
> arch/arm64/boot/dts/qcom/ipq5018.dtsi | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq5018.dtsi b/arch/arm64/boot/dts/qcom/ipq5018.dtsi
> index 3285c86824cf..168322bfb11c 100644
> --- a/arch/arm64/boot/dts/qcom/ipq5018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq5018.dtsi
> @@ -182,6 +182,13 @@ v2m1: v2m@1000 {
> };
> };
>
> + watchdog: watchdog@b017000 {
> + compatible = "qcom,apss-wdt-ipq5018", "qcom,kpss-wdt";
> + interrupts = <GIC_SPI 3 IRQ_TYPE_EDGE_RISING>;

I think all GIC_SPI interrupts are level high.

> + reg = <0x0b017000 0x40>;

Keep the reg as second property.

> + clocks = <&sleep_clk>;
> + };

Best regards,
Krzysztof