linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
@ 2016-01-28  8:58 James Liao
  2016-01-28 11:27 ` Arnd Bergmann
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: James Liao @ 2016-01-28  8:58 UTC (permalink / raw)
  To: linux-arm-kernel

Move all vendor's Kconfig into CCF menu section to prevent
new drivers putting their Kconfig files in a wrong place.

Some Kconigs need to modify at the same time to avoid build
warnings.

Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
---
This patch is based on v4.5-rc1.

changes since v1:
- Modify s3c24xx and samsung's Kconfig to avoid build warngings.

 arch/arm/mach-s3c24xx/Kconfig | 1 +
 drivers/clk/Kconfig           | 8 +++-----
 drivers/clk/samsung/Kconfig   | 1 -
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/arch/arm/mach-s3c24xx/Kconfig b/arch/arm/mach-s3c24xx/Kconfig
index ef68ecb..f02495f 100644
--- a/arch/arm/mach-s3c24xx/Kconfig
+++ b/arch/arm/mach-s3c24xx/Kconfig
@@ -15,6 +15,7 @@ config PLAT_S3C24XX
 	select NO_IOPORT_MAP
 	select S3C_DEV_NAND
 	select IRQ_DOMAIN
+	select COMMON_CLK
 	help
 	  Base platform code for any Samsung S3C24XX device
 
diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
index eca8e01..de707b2 100644
--- a/drivers/clk/Kconfig
+++ b/drivers/clk/Kconfig
@@ -202,11 +202,9 @@ config COMMON_CLK_CDCE706
 
 source "drivers/clk/bcm/Kconfig"
 source "drivers/clk/hisilicon/Kconfig"
-source "drivers/clk/qcom/Kconfig"
-
-endmenu
-
 source "drivers/clk/mvebu/Kconfig"
-
+source "drivers/clk/qcom/Kconfig"
 source "drivers/clk/samsung/Kconfig"
 source "drivers/clk/tegra/Kconfig"
+
+endmenu
diff --git a/drivers/clk/samsung/Kconfig b/drivers/clk/samsung/Kconfig
index 84196ec..b3fe5cb 100644
--- a/drivers/clk/samsung/Kconfig
+++ b/drivers/clk/samsung/Kconfig
@@ -1,6 +1,5 @@
 config COMMON_CLK_SAMSUNG
 	bool
-	select COMMON_CLK
 
 config S3C2410_COMMON_CLK
 	bool
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
  2016-01-28  8:58 [PATCH v2] clk: Move vendor's Kconfig into CCF menu section James Liao
@ 2016-01-28 11:27 ` Arnd Bergmann
  2016-01-28 12:57 ` Sylwester Nawrocki
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2016-01-28 11:27 UTC (permalink / raw)
  To: linux-arm-kernel

On Thursday 28 January 2016 16:58:57 James Liao wrote:
> Move all vendor's Kconfig into CCF menu section to prevent
> new drivers putting their Kconfig files in a wrong place.
> 
> Some Kconigs need to modify at the same time to avoid build
> warnings.
> 
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> ---
> This patch is based on v4.5-rc1.
> 
> changes since v1:
> - Modify s3c24xx and samsung's Kconfig to avoid build warngings.
> 
> 

Acked-by: Arnd Bergmann <arnd@arndb.de>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
  2016-01-28  8:58 [PATCH v2] clk: Move vendor's Kconfig into CCF menu section James Liao
  2016-01-28 11:27 ` Arnd Bergmann
@ 2016-01-28 12:57 ` Sylwester Nawrocki
  2016-01-30  0:30 ` Stephen Boyd
  2016-02-16  4:59 ` Krzysztof Kozlowski
  3 siblings, 0 replies; 6+ messages in thread
From: Sylwester Nawrocki @ 2016-01-28 12:57 UTC (permalink / raw)
  To: linux-arm-kernel

On 28/01/16 09:58, James Liao wrote:
> Move all vendor's Kconfig into CCF menu section to prevent
> new drivers putting their Kconfig files in a wrong place.
> 
> Some Kconigs need to modify at the same time to avoid build

"Some Kconfigs need to be modified..." ?

> warnings.
> 
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> ---
> This patch is based on v4.5-rc1.
> 
> changes since v1:
> - Modify s3c24xx and samsung's Kconfig to avoid build warngings.
> 
>  arch/arm/mach-s3c24xx/Kconfig | 1 +
>  drivers/clk/Kconfig           | 8 +++-----
>  drivers/clk/samsung/Kconfig   | 1 -
>  3 files changed, 4 insertions(+), 6 deletions(-)

For the samsung code changes:

Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

-- 
Thanks,
Sylwester

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
  2016-01-28  8:58 [PATCH v2] clk: Move vendor's Kconfig into CCF menu section James Liao
  2016-01-28 11:27 ` Arnd Bergmann
  2016-01-28 12:57 ` Sylwester Nawrocki
@ 2016-01-30  0:30 ` Stephen Boyd
  2016-02-16  4:59 ` Krzysztof Kozlowski
  3 siblings, 0 replies; 6+ messages in thread
From: Stephen Boyd @ 2016-01-30  0:30 UTC (permalink / raw)
  To: linux-arm-kernel

On 01/28, James Liao wrote:
> Move all vendor's Kconfig into CCF menu section to prevent
> new drivers putting their Kconfig files in a wrong place.
> 
> Some Kconigs need to modify at the same time to avoid build
> warnings.
> 
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> ---

Applied to clk-next

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
  2016-01-28  8:58 [PATCH v2] clk: Move vendor's Kconfig into CCF menu section James Liao
                   ` (2 preceding siblings ...)
  2016-01-30  0:30 ` Stephen Boyd
@ 2016-02-16  4:59 ` Krzysztof Kozlowski
  2016-02-16  5:45   ` James Liao
  3 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-16  4:59 UTC (permalink / raw)
  To: linux-arm-kernel

2016-01-28 17:58 GMT+09:00 James Liao <jamesjj.liao@mediatek.com>:
> Move all vendor's Kconfig into CCF menu section to prevent
> new drivers putting their Kconfig files in a wrong place.
>
> Some Kconigs need to modify at the same time to avoid build
> warnings.
>
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> ---
> This patch is based on v4.5-rc1.
>
> changes since v1:
> - Modify s3c24xx and samsung's Kconfig to avoid build warngings.
>
>  arch/arm/mach-s3c24xx/Kconfig | 1 +
>  drivers/clk/Kconfig           | 8 +++-----
>  drivers/clk/samsung/Kconfig   | 1 -
>  3 files changed, 4 insertions(+), 6 deletions(-)
>

I wish you had used scripts/get_maintainer.pl to get list of people to CC...
Kukjin Kim <kgene@kernel.org> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
Krzysztof Kozlowski <k.kozlowski@samsung.com> (maintainer:ARM/SAMSUNG
EXYNOS ARM ARCHITECTURES)
linux-samsung-soc at vger.kernel.org (moderated list:ARM/SAMSUNG EXYNOS
ARM ARCHITECTURES)

Beside the fact that it is just nice and welcomed... it would help me
to avoid some of duplicated work...

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] clk: Move vendor's Kconfig into CCF menu section
  2016-02-16  4:59 ` Krzysztof Kozlowski
@ 2016-02-16  5:45   ` James Liao
  0 siblings, 0 replies; 6+ messages in thread
From: James Liao @ 2016-02-16  5:45 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Krzysztof,

On Tue, 2016-02-16 at 13:59 +0900, Krzysztof Kozlowski wrote:
> 2016-01-28 17:58 GMT+09:00 James Liao <jamesjj.liao@mediatek.com>:
> > Move all vendor's Kconfig into CCF menu section to prevent
> > new drivers putting their Kconfig files in a wrong place.
> >
> > Some Kconigs need to modify at the same time to avoid build
> > warnings.
> >
> > Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> > ---
> > This patch is based on v4.5-rc1.
> >
> > changes since v1:
> > - Modify s3c24xx and samsung's Kconfig to avoid build warngings.
> >
> >  arch/arm/mach-s3c24xx/Kconfig | 1 +
> >  drivers/clk/Kconfig           | 8 +++-----
> >  drivers/clk/samsung/Kconfig   | 1 -
> >  3 files changed, 4 insertions(+), 6 deletions(-)
> >
> 
> I wish you had used scripts/get_maintainer.pl to get list of people to CC...
> Kukjin Kim <kgene@kernel.org> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> Krzysztof Kozlowski <k.kozlowski@samsung.com> (maintainer:ARM/SAMSUNG
> EXYNOS ARM ARCHITECTURES)
> linux-samsung-soc at vger.kernel.org (moderated list:ARM/SAMSUNG EXYNOS
> ARM ARCHITECTURES)
> 
> Beside the fact that it is just nice and welcomed... it would help me
> to avoid some of duplicated work...

I'm sorry about that. I modified mach-s3c24xx/Kconfig and
samsung/Kconfig in patch v2 but I reused the TO and CC list from patch
v1, so this patch missed Samsung's maintainers.


Best regards,

James

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-02-16  5:45 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-28  8:58 [PATCH v2] clk: Move vendor's Kconfig into CCF menu section James Liao
2016-01-28 11:27 ` Arnd Bergmann
2016-01-28 12:57 ` Sylwester Nawrocki
2016-01-30  0:30 ` Stephen Boyd
2016-02-16  4:59 ` Krzysztof Kozlowski
2016-02-16  5:45   ` James Liao

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).