Re: [PATCH] edac, highbank: remove dependency on ARCH_HIGHBANK

From: Robert Richter
Date: Mon Nov 11 2013 - 05:29:16 EST


On 08.11.13 15:47:21, Borislav Petkov wrote:
> > depends on EDAC_MM_EDAC && (ARCH_MULTI_V7 || ARM64 || COMPILE_TEST)
> >
> > This should enable it with allmodconfig.
>
> You could do this but the first is already coverage enough IMO and, as I
> said previously, compiling this thing on other architectures could raise
> bugs/issues which are not necessarily such on arm/arm64, for example.

The config option was introduced for this, enable it to be built on
all archs but have an option to disable it on archs where it is not
supposed to run on. I rather prefer COMPILE_TEST. Could we agree on
this?

Thanks,

-Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/