2022-10-26 06:50:45

by Runyang Chen

[permalink] [raw]
Subject: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

From: Runyang Chen <[email protected]>

Add dt-binding documentation of watchdog for MediaTek MT8188 Soc

Signed-off-by: Runyang Chen <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
Reviewed-by: AngeloGioacchino Del Regno <[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 762c62e428ef..b900c85d4560 100644
--- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
+++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
@@ -17,6 +17,7 @@ Required properties:
"mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986
"mediatek,mt8183-wdt": for MT8183
"mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186
+ "mediatek,mt8188-wdt", "mediatek,mt6589-wdt": for MT8188
"mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
"mediatek,mt8192-wdt": for MT8192
"mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195
--
2.18.0



2022-10-26 16:20:08

by Guenter Roeck

[permalink] [raw]
Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

On Wed, Oct 26, 2022 at 02:33:25PM +0800, Runyang Chen wrote:
> From: Runyang Chen <[email protected]>
>
> Add dt-binding documentation of watchdog for MediaTek MT8188 Soc
>
> Signed-off-by: Runyang Chen <[email protected]>
> Acked-by: Krzysztof Kozlowski <[email protected]>
> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>

This conflicts with the ongoing yaml conversion of this file
which is still not accepted.

https://patchwork.kernel.org/project/linux-watchdog/patch/[email protected]/

Nevertheless, I'll apply this series to my watchdog-next branch
and assume that it will be included in the next version of the
yaml conversion patch.

For my and Wim's reference:

Reviewed-by: Guenter Roeck <[email protected]>

Thanks,
Guenter

> ---
> 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 762c62e428ef..b900c85d4560 100644
> --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
> @@ -17,6 +17,7 @@ Required properties:
> "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986
> "mediatek,mt8183-wdt": for MT8183
> "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186
> + "mediatek,mt8188-wdt", "mediatek,mt6589-wdt": for MT8188
> "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
> "mediatek,mt8192-wdt": for MT8192
> "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195

Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

Il 26/10/22 17:26, Guenter Roeck ha scritto:
> On Wed, Oct 26, 2022 at 02:33:25PM +0800, Runyang Chen wrote:
>> From: Runyang Chen <[email protected]>
>>
>> Add dt-binding documentation of watchdog for MediaTek MT8188 Soc
>>
>> Signed-off-by: Runyang Chen <[email protected]>
>> Acked-by: Krzysztof Kozlowski <[email protected]>
>> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
>
> This conflicts with the ongoing yaml conversion of this file
> which is still not accepted.
>
> https://patchwork.kernel.org/project/linux-watchdog/patch/[email protected]/
>
> Nevertheless, I'll apply this series to my watchdog-next branch
> and assume that it will be included in the next version of the
> yaml conversion patch.
>
> For my and Wim's reference:
>
> Reviewed-by: Guenter Roeck <[email protected]>
>
> Thanks,
> Guenter
>
Adding Allen to the Cc's to make him aware of that, as he took over the
mtk-wdt yaml conversion.

2022-10-27 09:59:26

by Allen-KH Cheng

[permalink] [raw]
Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

On Thu, 2022-10-27 at 09:54 +0200, AngeloGioacchino Del Regno wrote:
> Il 26/10/22 17:26, Guenter Roeck ha scritto:
> > On Wed, Oct 26, 2022 at 02:33:25PM +0800, Runyang Chen wrote:
> > > From: Runyang Chen <[email protected]>
> > >
> > > Add dt-binding documentation of watchdog for MediaTek MT8188 Soc
> > >
> > > Signed-off-by: Runyang Chen <[email protected]>
> > > Acked-by: Krzysztof Kozlowski <[email protected]>
> > > Reviewed-by: AngeloGioacchino Del Regno <
> > > [email protected]>
> >
> > This conflicts with the ongoing yaml conversion of this file
> > which is still not accepted.
> >
> >
https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-watchdog/patch/[email protected]/__;!!CTRNKA9wMg0ARbw!zT39OehD4gnyDKPYwMbLdeyGI_oNOfvWa4HIrcooL3Ax8O7-N-BjXBZAolsCOuLb39fJ7Q$
> >
> >
> > Nevertheless, I'll apply this series to my watchdog-next branch
> > and assume that it will be included in the next version of the
> > yaml conversion patch.
> >
> > For my and Wim's reference:
> >
> > Reviewed-by: Guenter Roeck <[email protected]>
> >
> > Thanks,
> > Guenter
> >
>
> Adding Allen to the Cc's to make him aware of that, as he took over
> the
> mtk-wdt yaml conversion.

Hi Guenter,

I can send the following version of yaml conversion[1] for conflicts
after you apply this series. Thanks.

[1]

https://patchwork.kernel.org/project/linux-mediatek/patch/[email protected]/

BRs,
Allen

2022-10-27 14:08:23

by Allen-KH Cheng

[permalink] [raw]
Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

On Thu, 2022-10-27 at 06:42 -0700, Guenter Roeck wrote:
> On 10/27/22 01:56, Allen-KH Cheng (程冠勳) wrote:
> > On Thu, 2022-10-27 at 09:54 +0200, AngeloGioacchino Del Regno
> > wrote:
> > > Il 26/10/22 17:26, Guenter Roeck ha scritto:
> > > > On Wed, Oct 26, 2022 at 02:33:25PM +0800, Runyang Chen wrote:
> > > > > From: Runyang Chen <[email protected]>
> > > > >
> > > > > Add dt-binding documentation of watchdog for MediaTek MT8188
> > > > > Soc
> > > > >
> > > > > Signed-off-by: Runyang Chen <[email protected]>
> > > > > Acked-by: Krzysztof Kozlowski <[email protected]
> > > > > >
> > > > > Reviewed-by: AngeloGioacchino Del Regno <
> > > > > [email protected]>
> > > >
> > > > This conflicts with the ongoing yaml conversion of this file
> > > > which is still not accepted.
> > > >
> > > >
> >
> >
https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-watchdog/patch/[email protected]/__;!!CTRNKA9wMg0ARbw!zT39OehD4gnyDKPYwMbLdeyGI_oNOfvWa4HIrcooL3Ax8O7-N-BjXBZAolsCOuLb39fJ7Q$
> > > >
> > > >
> > > > Nevertheless, I'll apply this series to my watchdog-next branch
> > > > and assume that it will be included in the next version of the
> > > > yaml conversion patch.
> > > >
> > > > For my and Wim's reference:
> > > >
> > > > Reviewed-by: Guenter Roeck <[email protected]>
> > > >
> > > > Thanks,
> > > > Guenter
> > > >
> > >
> > > Adding Allen to the Cc's to make him aware of that, as he took
> > > over
> > > the
> > > mtk-wdt yaml conversion.
> >
> > Hi Guenter,
> >
> > I can send the following version of yaml conversion[1] for
> > conflicts
> > after you apply this series. Thanks.
> >
> > [1]
> >
> >
https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/patch/[email protected]/__;!!CTRNKA9wMg0ARbw!x4FHF5pdLxzPJgGcQt0ynL6wlmvETMGvJUCO9n1hlI8-beeKNYEnONwnl5-WI4kvCV-45w$
> >
> >
>
> You did not copy the watchdog mailing list with this patch, causing
> it to get lost
> from my queue (even though I had a look at it). If there were other
> watchdog related
> patches in this version of the series, they got lost as well.
>
> Guenter
>

I apologize for missing this and will pay attention next time (v3).

BRs,
Allen

2022-10-27 15:00:33

by Guenter Roeck

[permalink] [raw]
Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

On 10/27/22 01:56, Allen-KH Cheng (程冠勳) wrote:
> On Thu, 2022-10-27 at 09:54 +0200, AngeloGioacchino Del Regno wrote:
>> Il 26/10/22 17:26, Guenter Roeck ha scritto:
>> > On Wed, Oct 26, 2022 at 02:33:25PM +0800, Runyang Chen wrote:
>> > > From: Runyang Chen <[email protected]>
>> > >
>> > > Add dt-binding documentation of watchdog for MediaTek MT8188 Soc
>> > >
>> > > Signed-off-by: Runyang Chen <[email protected]>
>> > > Acked-by: Krzysztof Kozlowski <[email protected]>
>> > > Reviewed-by: AngeloGioacchino Del Regno <
>> > > [email protected]>
>> >
>> > This conflicts with the ongoing yaml conversion of this file
>> > which is still not accepted.
>> >
>> >
> https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-watchdog/patch/[email protected]/__;!!CTRNKA9wMg0ARbw!zT39OehD4gnyDKPYwMbLdeyGI_oNOfvWa4HIrcooL3Ax8O7-N-BjXBZAolsCOuLb39fJ7Q$
>> >
>> >
>> > Nevertheless, I'll apply this series to my watchdog-next branch
>> > and assume that it will be included in the next version of the
>> > yaml conversion patch.
>> >
>> > For my and Wim's reference:
>> >
>> > Reviewed-by: Guenter Roeck <[email protected]>
>> >
>> > Thanks,
>> > Guenter
>> >
>>
>> Adding Allen to the Cc's to make him aware of that, as he took over
>> the
>> mtk-wdt yaml conversion.
>
> Hi Guenter,
>
> I can send the following version of yaml conversion[1] for conflicts
> after you apply this series. Thanks.
>
> [1]
>
> https://patchwork.kernel.org/project/linux-mediatek/patch/[email protected]/
>

You did not copy the watchdog mailing list with this patch, causing it to get lost
from my queue (even though I had a look at it). If there were other watchdog related
patches in this version of the series, they got lost as well.

Guenter


2022-10-27 15:08:18

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [RESEND v3 1/3] dt-bindings: watchdog: Add compatible for MediaTek MT8188

On 27/10/2022 09:54, Allen-KH Cheng (程冠勳) wrote:
>> You did not copy the watchdog mailing list with this patch, causing
>> it to get lost
>> from my queue (even though I had a look at it). If there were other
>> watchdog related
>> patches in this version of the series, they got lost as well.
>>
>> Guenter
>>
>
> I apologize for missing this and will pay attention next time (v3).
>

Just use get_maintainers.pl and automate your task... Automation removes
(most) mistakes.

Best regards,
Krzysztof