[PATCH v3 3/3] clk: keystone: Build Keystone clock drivers

[santosh.shilimkar@ti.com (Santosh Shilimkar)]

On Monday 07 October 2013 09:28 PM, Mike Turquette wrote:
> Quoting Santosh Shilimkar (2013-09-25 18:18:15)
>> Now build the keystone common clock drivers. The build is made
>> conditional based on COMMON_CLK_KEYSTONE
>>
>> Cc: Mike Turquette <mturquette@linaro.org>
>>
>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>> ---
>>  drivers/clk/Kconfig           |    7 +++++++
>>  drivers/clk/Makefile          |    1 +
>>  drivers/clk/keystone/Makefile |    1 +
>>  3 files changed, 9 insertions(+)
>>  create mode 100644 drivers/clk/keystone/Makefile
>>
>> diff --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