2022-11-01 09:09:22

by Allen-KH Cheng

Subject: [PATCH v3 4/7] arm64: dts: mediatek: mt8516: Fix the watchdog node name

The proper name is 'watchdog', not 'toprgu'.

Signed-off-by: Allen-KH Cheng <[email protected]>
Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
---
arch/arm64/boot/dts/mediatek/mt8516.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
index d1b67c82d761..fff59dc1b4d7 100644
--- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
@@ -202,7 +202,7 @@
#clock-cells = <1>;
};

- toprgu: toprgu@10007000 {
+ toprgu: watchdog@10007000 {
compatible = "mediatek,mt8516-wdt",
"mediatek,mt6589-wdt";
reg = <0 0x10007000 0 0x1000>;
--
2.18.0

2022-11-02 21:18:23

by Nícolas F. R. A. Prado

Subject: Re: [PATCH v3 4/7] arm64: dts: mediatek: mt8516: Fix the watchdog node name

On Tue, Nov 01, 2022 at 05:01:13PM +0800, Allen-KH Cheng wrote:
> The proper name is 'watchdog', not 'toprgu'.
>
> Signed-off-by: Allen-KH Cheng <[email protected]>
> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
> ---
> arch/arm64/boot/dts/mediatek/mt8516.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> index d1b67c82d761..fff59dc1b4d7 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> @@ -202,7 +202,7 @@
> #clock-cells = <1>;
> };
>
> - toprgu: toprgu@10007000 {
> + toprgu: watchdog@10007000 {

Rename the label to watchdog as well. It's not used elsewhere yet anyway, so
won't add to the diff.

Otherwise,

Reviewed-by: N?colas F. R. A. Prado <[email protected]>

Thanks,
N?colas