On Sat, 2013-09-14 at 06:26 +0800, Stephen Warren wrote: > On 09/12/2013 05:24 AM, Bill Huang wrote: > > Re-model Tegra cpufreq driver to support all Tegra series of SoCs. > > > > * Make tegra-cpufreq.c a generic Tegra cpufreq driver. > > * Move Tegra20 specific codes into tegra20-cpufreq.c. > > * Bind Tegra cpufreq dirver with a fake device so defer probe would work > > when we're going to get regulator in the driver to support voltage > > scaling (DVFS). > > > diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm > > > @@ -232,6 +232,12 @@ config ARM_TEGRA_CPUFREQ > > bool "TEGRA CPUFreq support" > > depends on ARCH_TEGRA > > select CPU_FREQ_TABLE > > + > > +config ARM_TEGRA20_CPUFREQ > > + bool "NVIDIA TEGRA20" > > + depends on ARM_TEGRA_CPUFREQ && ARCH_TEGRA_2x_SOC > > default y > > help > > - This adds the CPUFreq driver support for TEGRA SOCs. > > + This adds the CPUFreq driver for NVIDIA TEGRA20 SoC. > > + > > + If in doubt, say N. > > This patch removes the following from "config ARM_TEGRA_CPUFREQ": > > default y > help > This adds the CPUFreq driver support for TEGRA SOCs. > > I think that option should be left unchanged, such that this patch > *just* adds the new option without changing the existing one. If you do > that, then you can completely drop patch 4/4. Thanks, I'll change and rebase in next version after Viresh's cleanup patch series is merged. > > Other than that, this series looks fine. -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html