[PATCH v4 0/3] some fixups for MediaTek cpufreq driver

2017-07-10 Thread sean.wang 
From: Sean Wang 

Changes since v3:
- remove the redundant line depends on ARM || ARM64 || COMPILE_TEST
since the whole Kconfig.arm file depends on ARM || ARM64.
- renaming from mtk-cpufreq.c to mediatek-cpufreq.c avoiding
collide and cause confusion at some point in time.

Changes since v2:
- correct the typo in the binding document

Changes since v1:
- drop those patches already accepted
- refine the commit messages and Kconfig dependency
- Kconfig menu entry and file name itself are updated with more
generic name to drop "MT8173" since this driver actually supports
all MediaTek SoCs.
- generate patchset again with git format-patch -C -M --thread=shallow
- fix binding examples by replacing '@' with '-' as the OPP nodes will
never have a "reg" property.

Hi,

The purpose of the series is
- (patch 1 to 3) to fix up current Mediatek cpufreq driver
can't work with the latest tree since one required CPU clock muxer missing
would cause the driver getting the resource fails when driver probe gets
called.
- (patch 4) to enable cpufreq feature on MT2701/MT7623 platform.
- (patch 5 to 6) to update the binding document to reflect
latest driver logic and add more examples guiding people how to apply for
Mediatek cpufreq driver.

Sean Wang (3):
  cpufreq: mediatek: Add support of cpufreq to MT2701/MT7623 SoC
  dt-bindings: cpufreq: move MediaTek cpufreq dt-bindings document to
proper place
  dt-bindings: cpufreq: enhance MediaTek cpufreq dt-binding document

 .../devicetree/bindings/clock/mt8173-cpu-dvfs.txt  |  83 ---
 .../bindings/cpufreq/cpufreq-mediatek.txt  | 247 +
 drivers/cpufreq/Kconfig.arm|   7 +-
 drivers/cpufreq/Makefile   |   2 +-
 .../{mt8173-cpufreq.c => mediatek-cpufreq.c}   |   2 +
 5 files changed, 253 insertions(+), 88 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/clock/mt8173-cpu-dvfs.txt
 create mode 100644 
Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt
 rename drivers/cpufreq/{mt8173-cpufreq.c => mediatek-cpufreq.c} (99%)

-- 
2.7.4

Re: [PATCH v4 0/3] some fixups for MediaTek cpufreq driver

2017-07-16 Thread Sean Wang 
Hi, Viresh

I missed to add Acks from Rob for patch 2 and 3 since we sent out almost
at the same time. Do I need to resend again for this or the series is
okay for you?

Sean


On Mon, 2017-07-10 at 22:23 +0800, sean.w...@mediatek.com wrote:
> From: Sean Wang 
> 
> Changes since v3:
> - remove the redundant line depends on ARM || ARM64 || COMPILE_TEST
> since the whole Kconfig.arm file depends on ARM || ARM64.
> - renaming from mtk-cpufreq.c to mediatek-cpufreq.c avoiding
> collide and cause confusion at some point in time.
> 
> Changes since v2:
> - correct the typo in the binding document
> 
> Changes since v1:
> - drop those patches already accepted
> - refine the commit messages and Kconfig dependency
> - Kconfig menu entry and file name itself are updated with more
> generic name to drop "MT8173" since this driver actually supports
> all MediaTek SoCs.
> - generate patchset again with git format-patch -C -M --thread=shallow
> - fix binding examples by replacing '@' with '-' as the OPP nodes will
> never have a "reg" property.
> 
> Hi,
> 
> The purpose of the series is
> - (patch 1 to 3) to fix up current Mediatek cpufreq driver
> can't work with the latest tree since one required CPU clock muxer missing
> would cause the driver getting the resource fails when driver probe gets
> called.
> - (patch 4) to enable cpufreq feature on MT2701/MT7623 platform.
> - (patch 5 to 6) to update the binding document to reflect
> latest driver logic and add more examples guiding people how to apply for
> Mediatek cpufreq driver.
> 
> Sean Wang (3):
>   cpufreq: mediatek: Add support of cpufreq to MT2701/MT7623 SoC
>   dt-bindings: cpufreq: move MediaTek cpufreq dt-bindings document to
> proper place
>   dt-bindings: cpufreq: enhance MediaTek cpufreq dt-binding document
> 
>  .../devicetree/bindings/clock/mt8173-cpu-dvfs.txt  |  83 ---
>  .../bindings/cpufreq/cpufreq-mediatek.txt  | 247 
> +
>  drivers/cpufreq/Kconfig.arm|   7 +-
>  drivers/cpufreq/Makefile   |   2 +-
>  .../{mt8173-cpufreq.c => mediatek-cpufreq.c}   |   2 +
>  5 files changed, 253 insertions(+), 88 deletions(-)
>  delete mode 100644 
> Documentation/devicetree/bindings/clock/mt8173-cpu-dvfs.txt
>  create mode 100644 
> Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt
>  rename drivers/cpufreq/{mt8173-cpufreq.c => mediatek-cpufreq.c} (99%)
>

Re: [PATCH v4 0/3] some fixups for MediaTek cpufreq driver

2017-07-17 Thread Viresh Kumar 
On 17-07-17, 14:16, Sean Wang wrote:
> Hi, Viresh
> 
> I missed to add Acks from Rob for patch 2 and 3 since we sent out almost
> at the same time. Do I need to resend again for this or the series is
> okay for you?

Looks like you missed including Rafael for this series (He is the one
who applies these patches).

I will let him answer for this.

-- 
viresh

Re: [PATCH v4 0/3] some fixups for MediaTek cpufreq driver

2017-07-17 Thread Rafael J. Wysocki 
On Monday, July 17, 2017 01:05:43 PM Viresh Kumar wrote:
> On 17-07-17, 14:16, Sean Wang wrote:
> > Hi, Viresh
> > 
> > I missed to add Acks from Rob for patch 2 and 3 since we sent out almost
> > at the same time. Do I need to resend again for this or the series is
> > okay for you?
> 
> Looks like you missed including Rafael for this series (He is the one
> who applies these patches).
> 
> I will let him answer for this.
> 
> 

Please respin with all of the applicable tags.

I haven't been following all of the discussions on this that closely.

Thanks,
Rafael