Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755464Ab3FKUqB (ORCPT ); Tue, 11 Jun 2013 16:46:01 -0400 Received: from mail-lb0-f177.google.com ([209.85.217.177]:49014 "EHLO mail-lb0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754695Ab3FKUp7 (ORCPT ); Tue, 11 Jun 2013 16:45:59 -0400 Message-ID: <51B78C82.7020709@cogentembedded.com> Date: Wed, 12 Jun 2013 00:45:54 +0400 From: Sergei Shtylyov Organization: Cogent Embedded User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130509 Thunderbird/17.0.6 MIME-Version: 1.0 To: Paul Gortmaker CC: Markos Chandras , Markos Chandras , linux-kernel@vger.kernel.org, "David S. Miller" , netdev@vger.kernel.org Subject: Re: [PATCH] drivers/net/ethernet/3com: Drop EISA dependency from VORTEX References: <1370962642-12097-1-git-send-email-markos.chandras@imgtec.com> <51B75088.8050800@windriver.com> <51B78594.5050903@windriver.com> In-Reply-To: <51B78594.5050903@windriver.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2085 Lines: 56 Hello. On 06/12/2013 12:16 AM, Paul Gortmaker wrote: >>>> The Vortex driver depends on PCI with optional support for EISA. >>>> This fixes build problems when EISA is selected but not PCI. >>>> drivers/net/ethernet/3com/3c59x.c:1031:2: error: implicit declaration of >>>> function 'pci_iomap' [-Werror=implicit-function-declaration] >>>> drivers/net/ethernet/3com/3c59x.c:1044:3: error: implicit declaration of >>>> function 'pci_iounmap' [-Werror=implicit-function-declaration] >>>> Cc: David S. Miller >>>> Cc: Paul Gortmaker >>>> Cc: netdev@vger.kernel.org >>>> Signed-off-by: Markos Chandras >>>> --- >>>> drivers/net/ethernet/3com/Kconfig | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> diff --git a/drivers/net/ethernet/3com/Kconfig b/drivers/net/ethernet/3com/Kconfig >>>> index 1c71c76..5c67f44 100644 >>>> --- a/drivers/net/ethernet/3com/Kconfig >>>> +++ b/drivers/net/ethernet/3com/Kconfig >>>> @@ -66,7 +66,7 @@ config PCMCIA_3C589 >>>> >>>> config VORTEX >>>> tristate "3c590/3c900 series (592/595/597) \"Vortex/Boomerang\" support" >>>> - depends on (PCI || EISA) && HAS_IOPORT >>>> + depends on PCI && HAS_IOPORT >>> How can this fix it? The way I read your change is that you >>> have now made it impossible to select for EISA=y and PCI=n, >>> thus breaking EISA only configurations who would find their >>> driver support gone when they run "make oldconfig". [...] >> Hi Paul, >> The driver does not seem to build if you only have EISA=y and PCI=n. >> Does this driver really support EISA only > Yes. Not that anyone really probably cares anymore. EISA is dead. We have the user of 3Com EISA cards on this list and I've fixed EISA specific bug in this driver not long ago. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/