2020-09-29 03:24:31

by Crystal Guo

[permalink] [raw]
Subject: [v5,2/4] dt-binding: mediatek: mt8192: update mtk-wdt document

update mtk-wdt document for MT8192 platform

Signed-off-by: Crystal Guo <[email protected]>
Reviewed-by: Matthias Brugger <[email protected]>
Reviewed-by: Guenter Roeck <[email protected]>
---
Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
index 45eedc2c3141..e36ba60de829 100644
--- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
+++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
@@ -12,6 +12,7 @@ Required properties:
"mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629
"mediatek,mt8183-wdt": for MT8183
"mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
+ "mediatek,mt8192-wdt": for MT8192

- reg : Specifies base physical address and size of the registers.

--
2.18.0


2020-10-02 09:32:39

by Matthias Brugger

[permalink] [raw]
Subject: Re: [v5,2/4] dt-binding: mediatek: mt8192: update mtk-wdt document



On 29/09/2020 05:20, Crystal Guo wrote:
> update mtk-wdt document for MT8192 platform
>
> Signed-off-by: Crystal Guo <[email protected]>
> Reviewed-by: Matthias Brugger <[email protected]>
> Reviewed-by: Guenter Roeck <[email protected]>

You added Guenters Reviewed-by in v4 of this series, but I don't see that on
Guenter provided this tag. In the future please make sure that you don't add
tags to your patches that were not provided. This creates great confusion.

Regards,
Matthias

> ---
> Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> index 45eedc2c3141..e36ba60de829 100644
> --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> @@ -12,6 +12,7 @@ Required properties:
> "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629
> "mediatek,mt8183-wdt": for MT8183
> "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
> + "mediatek,mt8192-wdt": for MT8192
>
> - reg : Specifies base physical address and size of the registers.
>
>

2020-10-09 06:08:12

by Crystal Guo

[permalink] [raw]
Subject: Re: [v5,2/4] dt-binding: mediatek: mt8192: update mtk-wdt document

On Fri, 2020-10-02 at 17:28 +0800, Matthias Brugger wrote:
>
> On 29/09/2020 05:20, Crystal Guo wrote:
> > update mtk-wdt document for MT8192 platform
> >
> > Signed-off-by: Crystal Guo <[email protected]>
> > Reviewed-by: Matthias Brugger <[email protected]>
> > Reviewed-by: Guenter Roeck <[email protected]>
>
> You added Guenters Reviewed-by in v4 of this series, but I don't see that on
> Guenter provided this tag. In the future please make sure that you don't add
> tags to your patches that were not provided. This creates great confusion.
>
> Regards,
> Matthias
>

Hi Guenter,

Should I remove the "Reviewed-by:Guenter" tag, and re-submit this patch
for your review, or keeping the status quo and wait for Wim's comment?

Thanks
Crystal
> > ---
> > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> > index 45eedc2c3141..e36ba60de829 100644
> > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> > @@ -12,6 +12,7 @@ Required properties:
> > "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629
> > "mediatek,mt8183-wdt": for MT8183
> > "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
> > + "mediatek,mt8192-wdt": for MT8192
> >
> > - reg : Specifies base physical address and size of the registers.
> >
> >