* [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
@ 2017-12-01 12:07 Matthias Brugger
2017-12-01 12:07 ` [PATCH 2/3] arm: dts: mt7623: Update ethsys binding Matthias Brugger
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Matthias Brugger @ 2017-12-01 12:07 UTC (permalink / raw)
To: robh+dt, mark.rutland, linux
Cc: matthias.bgg, sboyd, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
The ethsys registers a reset controller, so we need to specify a
reset cell. This patch fixes the documentation.
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
---
Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
index 7aa3fa167668..6cc7840ff37a 100644
--- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
+++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
@@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 {
compatible = "mediatek,mt2701-ethsys", "syscon";
reg = <0 0x1b000000 0 0x1000>;
#clock-cells = <1>;
+ #reset-cells = <1>;
};
--
2.12.3
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/3] arm: dts: mt7623: Update ethsys binding
2017-12-01 12:07 [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Matthias Brugger
@ 2017-12-01 12:07 ` Matthias Brugger
2017-12-01 12:07 ` [PATCH 3/3] arm: dts: mt2701: Add reset-cells Matthias Brugger
` (2 subsequent siblings)
3 siblings, 0 replies; 9+ messages in thread
From: Matthias Brugger @ 2017-12-01 12:07 UTC (permalink / raw)
To: robh+dt, mark.rutland, linux
Cc: matthias.bgg, sboyd, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
The ethsys binding misses the reset-cells, this patch
adds this property.
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
---
arch/arm/boot/dts/mt7623.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index 343d3b1a4448..b750da5362f7 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -758,6 +758,7 @@
"syscon";
reg = <0 0x1b000000 0 0x1000>;
#clock-cells = <1>;
+ #reset-cells = <1>;
};
eth: ethernet@1b100000 {
--
2.12.3
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 3/3] arm: dts: mt2701: Add reset-cells
2017-12-01 12:07 [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Matthias Brugger
2017-12-01 12:07 ` [PATCH 2/3] arm: dts: mt7623: Update ethsys binding Matthias Brugger
@ 2017-12-01 12:07 ` Matthias Brugger
2017-12-14 11:17 ` Matthias Brugger
2017-12-04 21:48 ` [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Rob Herring
2017-12-14 11:10 ` Matthias Brugger
3 siblings, 1 reply; 9+ messages in thread
From: Matthias Brugger @ 2017-12-01 12:07 UTC (permalink / raw)
To: robh+dt, mark.rutland, linux
Cc: matthias.bgg, sboyd, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
The hifsys and ethsys needs the definition of the reset-cells
property. Fix this.
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
---
arch/arm/boot/dts/mt2701.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi
index 965ddfbc9953..05557fce0f1d 100644
--- a/arch/arm/boot/dts/mt2701.dtsi
+++ b/arch/arm/boot/dts/mt2701.dtsi
@@ -604,6 +604,7 @@
compatible = "mediatek,mt2701-hifsys", "syscon";
reg = <0 0x1a000000 0 0x1000>;
#clock-cells = <1>;
+ #reset-cells = <1>;
};
usb0: usb@1a1c0000 {
@@ -688,6 +689,7 @@
compatible = "mediatek,mt2701-ethsys", "syscon";
reg = <0 0x1b000000 0 0x1000>;
#clock-cells = <1>;
+ #reset-cells = <1>;
};
eth: ethernet@1b100000 {
--
2.12.3
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
2017-12-01 12:07 [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Matthias Brugger
2017-12-01 12:07 ` [PATCH 2/3] arm: dts: mt7623: Update ethsys binding Matthias Brugger
2017-12-01 12:07 ` [PATCH 3/3] arm: dts: mt2701: Add reset-cells Matthias Brugger
@ 2017-12-04 21:48 ` Rob Herring
2017-12-14 11:10 ` Matthias Brugger
3 siblings, 0 replies; 9+ messages in thread
From: Rob Herring @ 2017-12-04 21:48 UTC (permalink / raw)
To: Matthias Brugger
Cc: mark.rutland, linux, sboyd, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
On Fri, Dec 01, 2017 at 01:07:06PM +0100, Matthias Brugger wrote:
> The ethsys registers a reset controller, so we need to specify a
> reset cell. This patch fixes the documentation.
>
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> ---
> Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
> 1 file changed, 1 insertion(+)
For all 3,
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
2017-12-01 12:07 [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Matthias Brugger
` (2 preceding siblings ...)
2017-12-04 21:48 ` [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Rob Herring
@ 2017-12-14 11:10 ` Matthias Brugger
2017-12-19 1:32 ` Stephen Boyd
3 siblings, 1 reply; 9+ messages in thread
From: Matthias Brugger @ 2017-12-14 11:10 UTC (permalink / raw)
To: Stephen Boyd, Michael Turquette, linux
Cc: sboyd, sean.wang, chen.zhong, devicetree, linux-kernel,
linux-arm-kernel, linux-mediatek
Hi Stephen, Michael,
On 12/01/2017 01:07 PM, Matthias Brugger wrote:
> The ethsys registers a reset controller, so we need to specify a
> reset cell. This patch fixes the documentation.
>
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> ---
> Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> index 7aa3fa167668..6cc7840ff37a 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> @@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 {
> compatible = "mediatek,mt2701-ethsys", "syscon";
> reg = <0 0x1b000000 0 0x1000>;
> #clock-cells = <1>;
> + #reset-cells = <1>;
> };
>
Will you take this patch through the clk tree, or shall I take it through my SoC
tree?
Regards,
Matthias
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] arm: dts: mt2701: Add reset-cells
2017-12-01 12:07 ` [PATCH 3/3] arm: dts: mt2701: Add reset-cells Matthias Brugger
@ 2017-12-14 11:17 ` Matthias Brugger
0 siblings, 0 replies; 9+ messages in thread
From: Matthias Brugger @ 2017-12-14 11:17 UTC (permalink / raw)
To: robh+dt, mark.rutland, linux
Cc: sboyd, sean.wang, chen.zhong, devicetree, linux-kernel,
linux-arm-kernel, linux-mediatek
On 12/01/2017 01:07 PM, Matthias Brugger wrote:
> The hifsys and ethsys needs the definition of the reset-cells
> property. Fix this.
>
> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> ---
> arch/arm/boot/dts/mt2701.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi
> index 965ddfbc9953..05557fce0f1d 100644
> --- a/arch/arm/boot/dts/mt2701.dtsi
> +++ b/arch/arm/boot/dts/mt2701.dtsi
> @@ -604,6 +604,7 @@
> compatible = "mediatek,mt2701-hifsys", "syscon";
> reg = <0 0x1a000000 0 0x1000>;
> #clock-cells = <1>;
> + #reset-cells = <1>;
> };
>
> usb0: usb@1a1c0000 {
> @@ -688,6 +689,7 @@
> compatible = "mediatek,mt2701-ethsys", "syscon";
> reg = <0 0x1b000000 0 0x1000>;
> #clock-cells = <1>;
> + #reset-cells = <1>;
> };
>
> eth: ethernet@1b100000 {
>
2 and 3 pushed to v4.15-next/dts32
Thanks!
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
2017-12-14 11:10 ` Matthias Brugger
@ 2017-12-19 1:32 ` Stephen Boyd
2017-12-20 17:13 ` Matthias Brugger
0 siblings, 1 reply; 9+ messages in thread
From: Stephen Boyd @ 2017-12-19 1:32 UTC (permalink / raw)
To: Matthias Brugger
Cc: Michael Turquette, linux, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
On 12/14, Matthias Brugger wrote:
> Hi Stephen, Michael,
>
> On 12/01/2017 01:07 PM, Matthias Brugger wrote:
> > The ethsys registers a reset controller, so we need to specify a
> > reset cell. This patch fixes the documentation.
> >
> > Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> > ---
> > Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> > index 7aa3fa167668..6cc7840ff37a 100644
> > --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> > +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> > @@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 {
> > compatible = "mediatek,mt2701-ethsys", "syscon";
> > reg = <0 0x1b000000 0 0x1000>;
> > #clock-cells = <1>;
> > + #reset-cells = <1>;
> > };
> >
>
> Will you take this patch through the clk tree, or shall I take it through my SoC
> tree?
>
It's resets, we are clk maintainers. I'm clkfused.
You can take it, along with my
Acked-by: Stephen Boyd <sboyd@codeaurora.org>
if you like/expect conflicts.
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
2017-12-19 1:32 ` Stephen Boyd
@ 2017-12-20 17:13 ` Matthias Brugger
2017-12-20 19:26 ` Michael Turquette
0 siblings, 1 reply; 9+ messages in thread
From: Matthias Brugger @ 2017-12-20 17:13 UTC (permalink / raw)
To: Stephen Boyd
Cc: Michael Turquette, linux, sean.wang, chen.zhong, devicetree,
linux-kernel, linux-arm-kernel, linux-mediatek
On 12/19/2017 02:32 AM, Stephen Boyd wrote:
> On 12/14, Matthias Brugger wrote:
>> Hi Stephen, Michael,
>>
>> On 12/01/2017 01:07 PM, Matthias Brugger wrote:
>>> The ethsys registers a reset controller, so we need to specify a
>>> reset cell. This patch fixes the documentation.
>>>
>>> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
>>> ---
>>> Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
>>> index 7aa3fa167668..6cc7840ff37a 100644
>>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
>>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
>>> @@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 {
>>> compatible = "mediatek,mt2701-ethsys", "syscon";
>>> reg = <0 0x1b000000 0 0x1000>;
>>> #clock-cells = <1>;
>>> + #reset-cells = <1>;
>>> };
>>>
>>
>> Will you take this patch through the clk tree, or shall I take it through my SoC
>> tree?
>>
>
> It's resets, we are clk maintainers. I'm clkfused.
>
> You can take it, along with my
>
> Acked-by: Stephen Boyd <sboyd@codeaurora.org>
>
> if you like/expect conflicts.
>
These are resets in the clock IP-block. I'll take it through my branch, I don't
expect any conflicts.
Regards,
Matthias
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
2017-12-20 17:13 ` Matthias Brugger
@ 2017-12-20 19:26 ` Michael Turquette
0 siblings, 0 replies; 9+ messages in thread
From: Michael Turquette @ 2017-12-20 19:26 UTC (permalink / raw)
To: Matthias Brugger, Stephen Boyd
Cc: linux, sean.wang, chen.zhong, devicetree, linux-kernel,
linux-arm-kernel, linux-mediatek
Quoting Matthias Brugger (2017-12-20 09:13:12)
>
>
> On 12/19/2017 02:32 AM, Stephen Boyd wrote:
> > On 12/14, Matthias Brugger wrote:
> >> Hi Stephen, Michael,
> >>
> >> On 12/01/2017 01:07 PM, Matthias Brugger wrote:
> >>> The ethsys registers a reset controller, so we need to specify a
> >>> reset cell. This patch fixes the documentation.
> >>>
> >>> Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
> >>> ---
> >>> Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 +
> >>> 1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> >>> index 7aa3fa167668..6cc7840ff37a 100644
> >>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> >>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt
> >>> @@ -20,4 +20,5 @@ ethsys: clock-controller@1b000000 {
> >>> compatible = "mediatek,mt2701-ethsys", "syscon";
> >>> reg = <0 0x1b000000 0 0x1000>;
> >>> #clock-cells = <1>;
> >>> + #reset-cells = <1>;
> >>> };
> >>>
> >>
> >> Will you take this patch through the clk tree, or shall I take it through my SoC
> >> tree?
> >>
> >
> > It's resets, we are clk maintainers. I'm clkfused.
> >
> > You can take it, along with my
> >
> > Acked-by: Stephen Boyd <sboyd@codeaurora.org>
> >
> > if you like/expect conflicts.
> >
>
> These are resets in the clock IP-block. I'll take it through my branch, I don't
> expect any conflicts.
Sounds good to me.
Best regards,
Mike
>
> Regards,
> Matthias
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-12-20 19:27 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-01 12:07 [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Matthias Brugger
2017-12-01 12:07 ` [PATCH 2/3] arm: dts: mt7623: Update ethsys binding Matthias Brugger
2017-12-01 12:07 ` [PATCH 3/3] arm: dts: mt2701: Add reset-cells Matthias Brugger
2017-12-14 11:17 ` Matthias Brugger
2017-12-04 21:48 ` [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation Rob Herring
2017-12-14 11:10 ` Matthias Brugger
2017-12-19 1:32 ` Stephen Boyd
2017-12-20 17:13 ` Matthias Brugger
2017-12-20 19:26 ` Michael Turquette
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).