Re: [PATCH] sh: Fix build error caused by missing dependencies

2014-09-02 Thread Geert Uytterhoeven
Hi Pranith,

On Tue, Sep 2, 2014 at 4:39 PM, Pranith Kumar  wrote:
> Fix a sh4-randconfig build failure which has the following splat:
>
> arch/sh/mm/cache-sh4.c:99:17: error: 'cached_to_uncached' undeclared (first 
> use in this function)
> arch/sh/mm/cache-sh4.c:192:2: error: implicit declaration of function 
> 'cpu_context' [-Werror=implicit-function-declaration]
>
> These errors are caused by missing dependencies on UNCACHED_MAPPING and MMU.
>
> Reported-by: Geert Uytterhoeven 
> Signed-off-by: Pranith Kumar 
> ---
>  arch/sh/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/sh/Kconfig b/arch/sh/Kconfig
> index 244fb4c..619d5eb 100644
> --- a/arch/sh/Kconfig
> +++ b/arch/sh/Kconfig
> @@ -194,6 +194,8 @@ config CPU_SH4
> select SH_INTC
> select SYS_SUPPORTS_SH_TMU
> select SYS_SUPPORTS_HUGETLBFS if MMU
> +   select UNCACHED_MAPPING
> +   select MMU

Selecting MMU is never the correct solution, as it's a platform feature.
Making something depend on MMU is something different, but I doubt
SH4 can only work with a MMU, cfr. the "if MMU" in the line above.

Back to the drawing board...

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] sh: Fix build error caused by missing dependencies

2014-09-02 Thread Geert Uytterhoeven
Hi Pranith,

On Tue, Sep 2, 2014 at 4:39 PM, Pranith Kumar bobby.pr...@gmail.com wrote:
 Fix a sh4-randconfig build failure which has the following splat:

 arch/sh/mm/cache-sh4.c:99:17: error: 'cached_to_uncached' undeclared (first 
 use in this function)
 arch/sh/mm/cache-sh4.c:192:2: error: implicit declaration of function 
 'cpu_context' [-Werror=implicit-function-declaration]

 These errors are caused by missing dependencies on UNCACHED_MAPPING and MMU.

 Reported-by: Geert Uytterhoeven ge...@linux-m68k.org
 Signed-off-by: Pranith Kumar bobby.pr...@gmail.com
 ---
  arch/sh/Kconfig | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/arch/sh/Kconfig b/arch/sh/Kconfig
 index 244fb4c..619d5eb 100644
 --- a/arch/sh/Kconfig
 +++ b/arch/sh/Kconfig
 @@ -194,6 +194,8 @@ config CPU_SH4
 select SH_INTC
 select SYS_SUPPORTS_SH_TMU
 select SYS_SUPPORTS_HUGETLBFS if MMU
 +   select UNCACHED_MAPPING
 +   select MMU

Selecting MMU is never the correct solution, as it's a platform feature.
Making something depend on MMU is something different, but I doubt
SH4 can only work with a MMU, cfr. the if MMU in the line above.

Back to the drawing board...

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/