Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-10-04 Thread Hector Yuan
On Fri, 2020-09-25 at 15:25 +0800, Hector Yuan wrote: > Hi, Rob sir: > > Yes, my patch follows > Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.txt way to > define frequency domain. > Is it OK to you if I use the same way to do? > And if there exist any schema problem, please kindly let

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-25 Thread Hector Yuan
Hi, Rob sir: Yes, my patch follows Documentation/devicetree/bindings/cpufreq/cpufreq-qcom-hw.txt way to define frequency domain. Is it OK to you if I use the same way to do? And if there exist any schema problem, please kindly let me know how to fix it. My patch reference Documentation/devicetree

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-24 Thread Viresh Kumar
On 25-09-20, 10:27, Hector Yuan wrote: > Hi, Viresh & Rob Sir: > > I will change frequency domain to below and define it in cpufreq_hw > schema rather than cpu node. > > mediatek,freq-domain-0 = <&cpu0>, <&cpu1>; I think it would be better to do it the standard way we have done it elsewhere. i.e

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-24 Thread Hector Yuan
Hi, Viresh & Rob Sir: I will change frequency domain to below and define it in cpufreq_hw schema rather than cpu node. mediatek,freq-domain-0 = <&cpu0>, <&cpu1>; Is it OK to you? Or you think OPP define is a better solution? opp-table-0 { compatible = "mediatek,hw-operating-points", "operat

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-23 Thread Hector Yuan
On Wed, 2020-09-23 at 21:10 +0800, Hector Yuan wrote: > On Tue, 2020-09-22 at 14:28 -0600, Rob Herring wrote: > > On Thu, Sep 10, 2020 at 12:31:02PM +0800, Hector Yuan wrote: > > > From: "Hector.Yuan" > > > > > > Add devicetree bindings for MediaTek HW driver. > > > > > > Signed-off-by: Hector.Y

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-23 Thread Hector Yuan
On Tue, 2020-09-22 at 14:28 -0600, Rob Herring wrote: > On Thu, Sep 10, 2020 at 12:31:02PM +0800, Hector Yuan wrote: > > From: "Hector.Yuan" > > > > Add devicetree bindings for MediaTek HW driver. > > > > Signed-off-by: Hector.Yuan > > --- > > .../bindings/cpufreq/cpufreq-mediatek-hw.yaml

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-22 Thread Rob Herring
On Thu, Sep 10, 2020 at 12:31:02PM +0800, Hector Yuan wrote: > From: "Hector.Yuan" > > Add devicetree bindings for MediaTek HW driver. > > Signed-off-by: Hector.Yuan > --- > .../bindings/cpufreq/cpufreq-mediatek-hw.yaml | 141 > > 1 file changed, 141 insertions(+) >

Re: [PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-20 Thread Hector Yuan
Hi, Rob sir: Sorry to bother you, may I have your review comment for the binding part? Appreciated. On Thu, 2020-09-10 at 12:31 +0800, Hector Yuan wrote: > From: "Hector.Yuan" > > Add devicetree bindings for MediaTek HW driver. > > Signed-off-by: Hector.Yuan > --- > .../bindings/cpufreq/cpuf

[PATCH v7 2/2] dt-bindings: cpufreq: add bindings for MediaTek cpufreq HW

2020-09-09 Thread Hector Yuan
From: "Hector.Yuan" Add devicetree bindings for MediaTek HW driver. Signed-off-by: Hector.Yuan --- .../bindings/cpufreq/cpufreq-mediatek-hw.yaml | 141 1 file changed, 141 insertions(+) create mode 100644 Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek-h