From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Gortmaker Subject: Re: [PATCH] drivers/net/ethernet/3com: Drop EISA dependency from VORTEX Date: Tue, 11 Jun 2013 12:30:00 -0400 Message-ID: <51B75088.8050800@windriver.com> References: <1370962642-12097-1-git-send-email-markos.chandras@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: , "David S. Miller" , To: Markos Chandras Return-path: In-Reply-To: <1370962642-12097-1-git-send-email-markos.chandras@imgtec.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 13-06-11 10:57 AM, Markos Chandras 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". P. -- > select NET_CORE > select MII > ---help--- >