From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <linux-wireless-owner@vger.kernel.org>
Received: from mail.windriver.com ([147.11.1.11]:32985 "EHLO
	mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1754510Ab2CZBVW (ORCPT
	<rfc822;linux-wireless@vger.kernel.org>);
	Sun, 25 Mar 2012 21:21:22 -0400
Date: Sun, 25 Mar 2012 21:20:38 -0400
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Julian Calaby <julian.calaby@gmail.com>
CC: <zajec5@gmail.com>, <linux-wireless@vger.kernel.org>,
	Hauke Mehrtens <hauke@hauke-m.de>,
	"John W. Linville" <linville@tuxdriver.com>
Subject: Re: [PATCH] bcma: fix build error on MIPS; implicit
 pcibios_enable_device
Message-ID: <20120326012038.GA7722@windriver.com> (sfid-20120326_032129_182193_C77F5F85)
References: <1332720175-7659-1-git-send-email-paul.gortmaker@windriver.com>
 <CAGRGNgVBwKppzMa7HZeWm+ZW1a7u0JKwxxFS2xCRv=bpxXnk+Q@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
In-Reply-To: <CAGRGNgVBwKppzMa7HZeWm+ZW1a7u0JKwxxFS2xCRv=bpxXnk+Q@mail.gmail.com>
Sender: linux-wireless-owner@vger.kernel.org
List-ID: <linux-wireless.vger.kernel.org>

[Re: [PATCH] bcma: fix build error on MIPS; implicit pcibios_enable_device] On 26/03/2012 (Mon 11:13) Julian Calaby wrote:

> Hi Paul,
> 
> On Mon, Mar 26, 2012 at 11:02, Paul Gortmaker
> <paul.gortmaker@windriver.com> wrote:
> > The following is seen during allmodconfig builds for MIPS:
> >
> > drivers/bcma/driver_pci_host.c:518:2: error: implicit declaration
> > of function 'pcibios_enable_device' [-Werror=implicit-function-declaration]
> > cc1: some warnings being treated as errors
> > make[3]: *** [drivers/bcma/driver_pci_host.o] Error 1
> >
> > Most likey introduced by commit 49dc9577155576b10ff79f0c1486c816b01f58bf
> >
> >    "bcma: add PCIe host controller"
> >
> > Add the header instead of implicitly assuming it will be present.
> > Sounds like a good idea, but that alone doesn't fix anything.
> >
> > The real problem is that the Kconfig has settings related to whether
> > PCI is possible, i.e.
> >
> >  config BCMA_HOST_PCI_POSSIBLE
> >        bool
> >        depends on BCMA && PCI = y
> >        default y
> >
> >  config BCMA_HOST_PCI
> >        bool "Support for BCMA on PCI-host bus"
> >        depends on BCMA_HOST_PCI_POSSIBLE
> >
> > ...but what is missing is that BCMA_DRIVER_PCI_HOSTMODE doesn't
> > have any dependencies on the above.  Add one.
> >
> > CC: Hauke Mehrtens <hauke@hauke-m.de>
> > CC: John W. Linville <linville@tuxdriver.com>
> > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
> >
> > diff --git a/drivers/bcma/Kconfig b/drivers/bcma/Kconfig
> > index c1172da..fb7c80f 100644
> > --- a/drivers/bcma/Kconfig
> > +++ b/drivers/bcma/Kconfig
> > @@ -29,7 +29,7 @@ config BCMA_HOST_PCI
> >
> >  config BCMA_DRIVER_PCI_HOSTMODE
> >        bool "Driver for PCI core working in hostmode"
> > -       depends on BCMA && MIPS
> > +       depends on BCMA && MIPS && BCMA_HOST_PCI
> 
> I don't know the hardware, but I'm sure there'd be situations
> (probably for embedded devices) where hostmode would be wanted without
> the PCI host support - so shouldn't this depend on
> BCMA_HOST_PCI_POSSIBLE?

My reading of things is that BCMA_HOST_PCI is equivalent to, or
at most a subset of BCMA_HOST_PCI_POSSIBLE.  But if the maintainers
want to tweak things accordingly, then great -- I'm just reporting
what I see in build failures.

Thanks,
Paul.
--

> 
> Thanks,
> 
> -- 
> Julian Calaby
> 
> Email: julian.calaby@gmail.com
> Profile: http://www.google.com/profiles/julian.calaby/
> .Plan: http://sites.google.com/site/juliancalaby/