Re: [PATCH] clk: Move vendor's Kconfig into CCF menu section
From: Arnd Bergmann <arnd@arndb.de>
Date: 2016-01-27 10:33:33
Also in:
linux-arm-kernel, linux-clk, linux-mediatek, lkml
From: Arnd Bergmann <arnd@arndb.de>
Date: 2016-01-27 10:33:33
Also in:
linux-arm-kernel, linux-clk, linux-mediatek, lkml
On Wednesday 27 January 2016 17:56:18 James Liao wrote:
quoted
I think you should include this change in your patch, or as a preparation. All other samsung platforms already select COMMON_CLK through ARCH_MULTIPLATFORM, and all SoCs that are part of ARCH_S3C24XX individually select a symbol that selects COMMON_CLK.Oops, I didn't notice there are warnings after moving these Kconfigs into menu section. The original motivation of this patch is preventing new clk drivers include their Kconfig in a wrong place, so it's not critical. Do you think we should apply this patch (with your changes below) or drop this patch directly?
I think your patch is good, we just need to fix S3C24xx, which is doing things in a strange way for historic reasons. Arnd