Re: [PATCH v3 1/6] clk: mediatek: Refine the makefile to support multiple clock drivers

[Yingjoe Chen]

On Tue, 2016-01-12 at 16:31 +0800, James Liao wrote:
> Add a Kconfig to define clock configuration for each SoC, and
> modify the Makefile to build drivers that only selected in config.
> 
> Signed-off-by: Shunli Wang <shunli.wang@xxxxxxxxxxxx>
> Signed-off-by: James Liao <jamesjj.liao@xxxxxxxxxxxx>
> ---
>  drivers/clk/Kconfig           |  1 +
>  drivers/clk/mediatek/Kconfig  | 23 +++++++++++++++++++++++
>  drivers/clk/mediatek/Makefile |  6 +++---
>  3 files changed, 27 insertions(+), 3 deletions(-)
>  create mode 100644 drivers/clk/mediatek/Kconfig
> 
> diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
> index c3e3a02..b7a37dc 100644
> --- a/drivers/clk/Kconfig
> +++ b/drivers/clk/Kconfig
> @@ -198,3 +198,4 @@ source "drivers/clk/mvebu/Kconfig"
>  
>  source "drivers/clk/samsung/Kconfig"
>  source "drivers/clk/tegra/Kconfig"
> +source "drivers/clk/mediatek/Kconfig"


Hi James,

drivers/clk/mediatek/Kconfig add user selectable options, menuconfig
will list them outside of "Common Clock Framework" sub-menu if you
source the file here. Kconfig for samsung & tegra doesn't have any, so
it is OK for them to stay here.

Please move it inside the menu, also it seems the source lines are
sorted now, so let's keep them sorted.

Joe.C