On 24/01/18 12:45, Peter De Schrijver wrote: > Tegra210 has a DFLL as well and can share the majority of the code with > the Tegra124 implementation. So build the same code for both platforms. > > Signed-off-by: Peter De Schrijver <pdeschrijver@xxxxxxxxxx> > --- > drivers/clk/tegra/Kconfig | 5 +++++ > drivers/clk/tegra/Makefile | 2 +- > 2 files changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/clk/tegra/Kconfig b/drivers/clk/tegra/Kconfig > index 7ddacae..57902ab 100644 > --- a/drivers/clk/tegra/Kconfig > +++ b/drivers/clk/tegra/Kconfig > @@ -5,3 +5,8 @@ config TEGRA_CLK_EMC > config CLK_TEGRA_BPMP > def_bool y > depends on TEGRA_BPMP > + > +config TEGRA_CLK_DFLL > + depends on (ARCH_TEGRA_124_SOC || ARCH_TEGRA_210_SOC) > + select PM_OPP > + def_bool y Being picky, the description does not mention adding the dependency on PM_OPP. However, I do see that the driver does use this and so seems valid. Maybe update the description to clarify that this is needed. BTW, I see that for ARM this is already selected by ARCH_TEGRA. > diff --git a/drivers/clk/tegra/Makefile b/drivers/clk/tegra/Makefile > index b716923..fc7f4b4 100644 > --- a/drivers/clk/tegra/Makefile > +++ b/drivers/clk/tegra/Makefile > @@ -19,7 +19,7 @@ obj-$(CONFIG_ARCH_TEGRA_2x_SOC) += clk-tegra20.o > obj-$(CONFIG_ARCH_TEGRA_3x_SOC) += clk-tegra30.o > obj-$(CONFIG_ARCH_TEGRA_114_SOC) += clk-tegra114.o > obj-$(CONFIG_ARCH_TEGRA_124_SOC) += clk-tegra124.o > -obj-$(CONFIG_ARCH_TEGRA_124_SOC) += clk-tegra124-dfll-fcpu.o > +obj-$(CONFIG_TEGRA_CLK_DFLL) += clk-tegra124-dfll-fcpu.o > obj-$(CONFIG_ARCH_TEGRA_132_SOC) += clk-tegra124.o > obj-y += cvb.o > obj-$(CONFIG_ARCH_TEGRA_210_SOC) += clk-tegra210.o > Cheers Jon -- nvpublic -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
