From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org ([63.228.1.57]:41231 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755891Ab2FZCu6 (ORCPT ); Mon, 25 Jun 2012 22:50:58 -0400 Message-ID: <1340679045.3732.3.camel@pasglop> Subject: Re: [PATCH V3 1/2] PCI: retrieve host bridge by PCI bus From: Benjamin Herrenschmidt To: Gavin Shan Cc: Yinghai Lu , linux-pci@vger.kernel.org, linuxppc-dev@ozlabs.org, bhelgaas@google.com Date: Tue, 26 Jun 2012 12:50:45 +1000 In-Reply-To: <20120626003020.GA6307@shangw> References: <1340593821-19011-1-git-send-email-shangw@linux.vnet.ibm.com> <20120626003020.GA6307@shangw> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org List-ID: On Tue, 2012-06-26 at 08:30 +0800, Gavin Shan wrote: > >> +EXPORT_SYMBOL(pci_bus_host_bridge); > > Yinghai, thanks for your time on this :-) > > > > >why do you need to export it? > > > > The reason is that we have introduced extra fields to "struct > pci_host_bridge" > in [PATCH 2/2] and platform want to access those extra fields. But the platform code is built-in, not modular, so it doesn't need EXPORT_SYMBOL (additionally even if we wanted to export it we probably also want it _GPL only, so EXPORT_SYMBOL_GPL, but I don't think we need to). Cheers, Ben.