Re: [PATCH v6 0/4] Add Mediatek MT8173 cpufreq driver

On 09-07-15, 18:27, Pi-Cheng Chen wrote:

MT8173 is a ARMv8 based SoC with 2 clusters. All CPUs in a single cluster share the same power and clock domain. This series tries to add cpufreq support for MT8173 SoC.

changes in v6:

• Move clock and regulator consumer properties document to the device tree bindings documents of MT8173 CPU DVFS clock driver
• Add change log to describe what is implemented in the MT8173 cpufreq driver
• Add missed rcu_read_unlock() in the error path
• Move of_init_opp_table() call to make sure all required hardware resources are already there before it is called
• Add comments to describe why both platform driver and deivce registration codes are put in the initcall function
• Use the term "voltage tracking" instead of "voltage trace" according to an internal SoC document

I think Mike would again need to comment or Ack 1/4 as we have moved the stuff to another file.

But for [1,3-4]:

Acked-by: Viresh Kumar viresh.kumar@linaro.org

Thanks for your hardwork :)