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

[Borislav Petkov]

On Tue, Nov 05, 2013 at 09:25:52PM -0600, Rob Herring wrote:
> From: Rob Herring <rob.herring@xxxxxxxxxxx>
> 
> In order to enable on arm64 and improve the build coverage, remove the
> dependency on ARCH_HIGHBANK.
> 
> Signed-off-by: Rob Herring <rob.herring@xxxxxxxxxxx>
> Cc: Doug Thompson <dougthompson@xxxxxxxxxxxx>
> Cc: Robert Richter <rric@xxxxxxxxxx>
> Cc: linux-edac@xxxxxxxxxxxxxxx
> ---
>  drivers/edac/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
> index 878f090..fc8aaaf 100644
> --- a/drivers/edac/Kconfig
> +++ b/drivers/edac/Kconfig
> @@ -328,14 +328,14 @@ config EDAC_TILE
>  
>  config EDAC_HIGHBANK_MC
>  	tristate "Highbank Memory Controller"
> -	depends on EDAC_MM_EDAC && ARCH_HIGHBANK
> +	depends on EDAC_MM_EDAC

Please, no, this'll get enabled on other arches where this driver is
simply N/A. It should only depend on the ARM arches it supports.

-- 
Regards/Gruss,
Boris.
--
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/