Hi Nícolas, On Fri, 2022-11-04 at 10:32 -0400, Nícolas F. R. A. Prado wrote: > On Fri, Nov 04, 2022 at 10:06:58AM +0800, Allen-KH Cheng wrote: > > The proper name is 'watchdog', not 'toprgu'. > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@xxxxxxxxxxxx> > > Reviewed-by: AngeloGioacchino Del Regno < > > angelogioacchino.delregno@xxxxxxxxxxxxx> > > Reviewed-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx> > > --- > > 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 { > > I think you missed my comment about renaming the label as well on the > previous > version. > Yep, I may have misunderstood. I will remove the label. Thanks, Allen > > compatible = "mediatek,mt8516-wdt", > > "mediatek,mt6589-wdt"; > > reg = <0 0x10007000 0 0x1000>; > > -- > > 2.18.0 > >