2020-02-17 08:20:18

by Yong Liang

Subject: [PATCH 0/1] Add watchdog device node

From: "yong.liang" <[email protected]>

Add watchdog device node

yong.liang (1):
amr64: dts: modify mt8183.dtsi

arch/arm64/boot/dts/mediatek/mt8183.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

--
2.18.0

2020-02-17 08:21:20

by Yong Liang

Subject: [PATCH 1/1] amr64: dts: modify mt8183.dtsi

From: "yong.liang" <[email protected]>

Add watchdog device node
Document base on http://lists.infradead.org/pipermail/linux-mediatek/2020-January/026415.html

Signed-off-by: yong.liang <[email protected]>
---
arch/arm64/boot/dts/mediatek/mt8183.dtsi | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
index 10b32471bc7b..8b59e0eba2eb 100644
--- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
@@ -253,6 +253,13 @@
#interrupt-cells = <2>;
};

+ watchdog: watchdog@10007000 {
+ compatible = "mediatek,mt8183-wdt",
+ "mediatek,mt6589-wdt";
+ reg = <0 0x10007000 0 0x100>;
+ #reset-cells = <1>;
+ };
+
apmixedsys: syscon@1000c000 {
compatible = "mediatek,mt8183-apmixedsys", "syscon";
reg = <0 0x1000c000 0 0x1000>;
--
2.18.0

2020-02-17 10:05:45

by Matthias Brugger

Subject: Re: [PATCH 1/1] amr64: dts: modify mt8183.dtsi

On 17/02/2020 09:19, Yong Liang wrote:
> From: "yong.liang" <[email protected]>
>
> Add watchdog device node
> Document base on http://lists.infradead.org/pipermail/linux-mediatek/2020-January/026415.html
>

Commit message shouldn't contain links to the mailinglist this is additional
information just for review and should go after '---'


> Signed-off-by: yong.liang <[email protected]>
> ---

Next time please put reference to series your patches are based on here :)

I fixed the commit message (the subject wasn't really usefull, I fixed that too)
and pushed it to v5.6-next/dts64

Thanks,
Matthias

> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index 10b32471bc7b..8b59e0eba2eb 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -253,6 +253,13 @@
> #interrupt-cells = <2>;
> };
>
> + watchdog: watchdog@10007000 {
> + compatible = "mediatek,mt8183-wdt",
> + "mediatek,mt6589-wdt";
> + reg = <0 0x10007000 0 0x100>;
> + #reset-cells = <1>;
> + };
> +
> apmixedsys: syscon@1000c000 {
> compatible = "mediatek,mt8183-apmixedsys", "syscon";
> reg = <0 0x1000c000 0 0x1000>;
>

2020-02-17 10:16:29

by Yong Liang

Subject: Re: [PATCH 1/1] amr64: dts: modify mt8183.dtsi

On Mon, 2020-02-17 at 18:05 +0800, Matthias Brugger wrote:
>
> On 17/02/2020 09:19, Yong Liang wrote:
> > From: "yong.liang" <[email protected]>
> >
> > Add watchdog device node
> > Document base on http://lists.infradead.org/pipermail/linux-mediatek/2020-January/026415.html
> >
>
> Commit message shouldn't contain links to the mailinglist this is additional
> information just for review and should go after '---'
>
>
> > Signed-off-by: yong.liang <[email protected]>
> > ---
>
> Next time please put reference to series your patches are based on here :)
>
> I fixed the commit message (the subject wasn't really usefull, I fixed that too)
> and pushed it to v5.6-next/dts64
>
> Thanks,
> Matthias

Hi Matthias:
So this patch confirmd to be merged?
Thansk you very much!
>
> > arch/arm64/boot/dts/mediatek/mt8183.dtsi | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > index 10b32471bc7b..8b59e0eba2eb 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > @@ -253,6 +253,13 @@
> > #interrupt-cells = <2>;
> > };
> >
> > + watchdog: watchdog@10007000 {
> > + compatible = "mediatek,mt8183-wdt",
> > + "mediatek,mt6589-wdt";
> > + reg = <0 0x10007000 0 0x100>;
> > + #reset-cells = <1>;
> > + };
> > +
> > apmixedsys: syscon@1000c000 {
> > compatible = "mediatek,mt8183-apmixedsys", "syscon";
> > reg = <0 0x1000c000 0 0x1000>;
> >

2020-02-17 10:19:13

by Matthias Brugger

Subject: Re: [PATCH 1/1] amr64: dts: modify mt8183.dtsi

On 17/02/2020 11:14, Yong Liang wrote:
> On Mon, 2020-02-17 at 18:05 +0800, Matthias Brugger wrote:
>>
>> On 17/02/2020 09:19, Yong Liang wrote:
>>> From: "yong.liang" <[email protected]>
>>>
>>> Add watchdog device node
>>> Document base on http://lists.infradead.org/pipermail/linux-mediatek/2020-January/026415.html
>>>
>>
>> Commit message shouldn't contain links to the mailinglist this is additional
>> information just for review and should go after '---'
>>
>>
>>> Signed-off-by: yong.liang <[email protected]>
>>> ---
>>
>> Next time please put reference to series your patches are based on here :)
>>
>> I fixed the commit message (the subject wasn't really usefull, I fixed that too)
>> and pushed it to v5.6-next/dts64
>>
>> Thanks,
>> Matthias
>
> Hi Matthias:
> So this patch confirmd to be merged?

Yes, you can find it here:
https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/commit/?h=v5.6-next/dts64&id=a39f842541038e20063fc880652153c0d93a95bf

> Thansk you very much!

Thanks to you.

Matthias

>>
>>> arch/arm64/boot/dts/mediatek/mt8183.dtsi | 7 +++++++
>>> 1 file changed, 7 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>>> index 10b32471bc7b..8b59e0eba2eb 100644
>>> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>>> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
>>> @@ -253,6 +253,13 @@
>>> #interrupt-cells = <2>;
>>> };
>>>
>>> + watchdog: watchdog@10007000 {
>>> + compatible = "mediatek,mt8183-wdt",
>>> + "mediatek,mt6589-wdt";
>>> + reg = <0 0x10007000 0 0x100>;
>>> + #reset-cells = <1>;
>>> + };
>>> +
>>> apmixedsys: syscon@1000c000 {
>>> compatible = "mediatek,mt8183-apmixedsys", "syscon";
>>> reg = <0 0x1000c000 0 0x1000>;
>>>
>