Re: [PATCH] clk: mediatek: Fix MT2701 dependencies

Yingjoe Chen
2017-01-11T14:05:26Z

On Wed, 2017-01-11 at 13:41 +0100, Jean Delvare wrote:
> Hi James,
>
> On Wed, 11 Jan 2017 09:56:28 +0800, James Liao wrote:
> > On Tue, 2017-01-10 at 13:03 +0100, Jean Delvare wrote:
> > > What about MT8135 and MT8173, are they 32-bit SoCs as well?
> >
> > MT8135 is a 32-bit SoC and MT8173 is a 64-bit SoC.
>
> OK, so would the following change make sense?
>
> ---
> drivers/clk/mediatek/Kconfig | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> --- linux-4.10-rc2.orig/drivers/clk/mediatek/Kconfig 2017-01-09 11:17:37.542344083 +0100
> +++ linux-4.10-rc2/drivers/clk/mediatek/Kconfig 2017-01-11 13:40:21.965890913 +0100
> @@ -52,16 +52,16 @@ config COMMON_CLK_MT2701_BDPSYS
>
> config COMMON_CLK_MT8135
> bool "Clock driver for Mediatek MT8135"
> - depends on ARCH_MEDIATEK || COMPILE_TEST
> + depends on (ARCH_MEDIATEK && !ARM64) || COMPILE_TEST

Why not just check for ARM?

+ depends on (ARCH_MEDIATEK && ARM) || COMPILE_TEST

Joe.C