From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nicolas Ferre <nicolas.ferre@atmel.com>
Subject: Re: [PATCH] net: cadence: Add architecture dependencies
Date: Mon, 14 Apr 2014 14:59:53 +0200
Message-ID: <534BDBC9.8000509@atmel.com>
References: <20140414141151.5ee5434d@endymion.delvare>
Mime-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: Grant Likely <grant.likely@linaro.org>,
	Rob Herring <robh+dt@kernel.org>
To: Jean Delvare <jdelvare@suse.de>, <netdev@vger.kernel.org>
Return-path: <netdev-owner@vger.kernel.org>
Received: from eusmtp01.atmel.com ([212.144.249.242]:64617 "EHLO
	eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751519AbaDNM7y (ORCPT
	<rfc822;netdev@vger.kernel.org>); Mon, 14 Apr 2014 08:59:54 -0400
In-Reply-To: <20140414141151.5ee5434d@endymion.delvare>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

On 14/04/2014 14:11, Jean Delvare :
> The Cadence ethernet chipsets are only used on specific ARM
> architectures. Add Kconfig dependencies so that drivers for these
> chipsets are only buildable on the relevant architectures.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> I'm not 100% certain I got the dependencies for MACB right, please let
> me know if I forgot something (or it can always be added later.)
> 
>  drivers/net/ethernet/cadence/Kconfig |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> --- linux-3.15-rc1.orig/drivers/net/ethernet/cadence/Kconfig	2014-03-31 05:40:15.000000000 +0200
> +++ linux-3.15-rc1/drivers/net/ethernet/cadence/Kconfig	2014-04-14 13:38:09.308168512 +0200
> @@ -4,7 +4,7 @@
>  
>  config NET_CADENCE
>  	bool "Cadence devices"
> -	depends on HAS_IOMEM
> +	depends on HAS_IOMEM && (ARM || COMPILE_TEST)

This one is wrong: PLATFORM_AT32AP is not an ARM platform but an AVR32 one.

>  	default y
>  	---help---
>  	  If you have a network (Ethernet) card belonging to this class, say Y.
> @@ -22,7 +22,7 @@ if NET_CADENCE
>  
>  config ARM_AT91_ETHER
>  	tristate "AT91RM9200 Ethernet support"
> -	depends on HAS_DMA
> +	depends on HAS_DMA && (ARCH_AT91RM9200 || COMPILE_TEST)
>  	select MACB
>  	---help---
>  	  If you wish to compile a kernel for the AT91RM9200 and enable
> @@ -30,7 +30,7 @@ config ARM_AT91_ETHER
>  
>  config MACB
>  	tristate "Cadence MACB/GEM support"
> -	depends on HAS_DMA
> +	depends on HAS_DMA && (PLATFORM_AT32AP || ARCH_AT91 || ARCH_PICOXCELL || ARCH_ZYNQ || COMPILE_TEST)
>  	select PHYLIB
>  	---help---
>  	  The Cadence MACB ethernet interface is found on many Atmel AT32 and
> 
> 

Well, do we really need the adding of these dependencies? I though that
we were moving toward the decreasing of dependency checks in Kconfig...

Bye,
-- 
Nicolas Ferre