[PATCH v3 3/3] clk: keystone: Build Keystone clock drivers
From: Santosh Shilimkar <hidden>
Date: 2013-10-08 13:05:23
On Monday 07 October 2013 09:28 PM, Mike Turquette wrote:
Quoting Santosh Shilimkar (2013-09-25 18:18:15)quoted
Now build the keystone common clock drivers. The build is made conditional based on COMMON_CLK_KEYSTONE Cc: Mike Turquette <redacted> Signed-off-by: Santosh Shilimkar <redacted> --- drivers/clk/Kconfig | 7 +++++++ drivers/clk/Makefile | 1 + drivers/clk/keystone/Makefile | 1 + 3 files changed, 9 insertions(+) create mode 100644 drivers/clk/keystone/Makefilediff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig index 279407a..10f85cc 100644 --- a/drivers/clk/Kconfig +++ b/drivers/clk/Kconfig@@ -93,6 +93,13 @@ config CLK_PPC_CORENET This adds the clock driver support for Freescale PowerPC corenet platforms using common clock framework. +config COMMON_CLK_KEYSTONE + tristate "Clock drivers for Keystone based SOCs" + depends on ARCH_KEYSTONE && OF + ---help--- + Supports clock drivers for Keystone based SOCs. These SOCs have local + a power sleep control module that gate the clock to the IPs and PLLs. + endmenu source "drivers/clk/mvebu/Kconfig"diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile index 7b11106..2a44d533 100644 --- a/drivers/clk/Makefile +++ b/drivers/clk/Makefile@@ -32,6 +32,7 @@ obj-$(CONFIG_ARCH_VT8500) += clk-vt8500.o obj-$(CONFIG_ARCH_ZYNQ) += zynq/ obj-$(CONFIG_ARCH_TEGRA) += tegra/ obj-$(CONFIG_PLAT_SAMSUNG) += samsung/ +obj-$(CONFIG_COMMON_CLK_KEYSTONE) += keystone/Thanks for respinning these patches Santosh. They look good and I've taken them into clk-next for 3.13.
Great.
My only question: why CONFIG_COMMON_CLK_KEYSTONE and not just CONFIG_ARCH_KEYSTONE above? This is not a blocking issue and can be revisited later.
The clock Kconfig had few arch using directly ARCH defines as few creating arch CCF. I gone ahead with CCF bunch. Probably we can just make that consistent for all other arch in future as you said. Thanks a lot Mike. Regards, Santosh