Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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> >>> --- > >>> 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@1b00 { > >>> compatible = "mediatek,mt2701-ethsys", "syscon"; > >>> reg = <0 0x1b00 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 > > > > 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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 > >>> --- > >>> 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@1b00 { > >>> compatible = "mediatek,mt2701-ethsys", "syscon"; > >>> reg = <0 0x1b00 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 > > > > 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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>>> --- >>> 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@1b00 { >>> compatible = "mediatek,mt2701-ethsys", "syscon"; >>> reg = <0 0x1b00 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 > > 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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 >>> --- >>> 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@1b00 { >>> compatible = "mediatek,mt2701-ethsys", "syscon"; >>> reg = <0 0x1b00 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 > > 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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> > --- > > 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@1b00 { > > compatible = "mediatek,mt2701-ethsys", "syscon"; > > reg = <0 0x1b00 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 if you like/expect conflicts. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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 > > --- > > 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@1b00 { > > compatible = "mediatek,mt2701-ethsys", "syscon"; > > reg = <0 0x1b00 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 if you like/expect conflicts. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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> --- > 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@1b00 { > compatible = "mediatek,mt2701-ethsys", "syscon"; > reg = <0 0x1b00 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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 > --- > 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@1b00 { > compatible = "mediatek,mt2701-ethsys", "syscon"; > reg = <0 0x1b00 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
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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> --- > Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 + > 1 file changed, 1 insertion(+) For all 3, Reviewed-by: Rob Herring
Re: [PATCH 1/3] dt-bindings: ARM: Mediatek: Fix ethsys documentation
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 > --- > Documentation/devicetree/bindings/arm/mediatek/mediatek,ethsys.txt | 1 + > 1 file changed, 1 insertion(+) For all 3, Reviewed-by: Rob Herring