From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 9CC712C009C for ; Wed, 3 Jul 2013 08:01:50 +1000 (EST) Message-ID: <1372802478.4122.24.camel@pasglop> Subject: Re: [PATCH] powerpc/powernv: Mark pnv_pci_init_ioda2_phb() as __init From: Benjamin Herrenschmidt To: Bjorn Helgaas Date: Wed, 03 Jul 2013 08:01:18 +1000 In-Reply-To: <20130702182003.28238.61942.stgit@bhelgaas-glaptop> References: <20130702182003.28238.61942.stgit@bhelgaas-glaptop> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: linuxppc-dev@lists.ozlabs.org, Paul Mackerras List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2013-07-02 at 12:20 -0600, Bjorn Helgaas wrote: > Mark pnv_pci_init_ioda2_phb() as __init. It is called only from an > init function (pnv_pci_init()), and it calls an init function > (pnv_pci_init_ioda_phb()): > > pnv_pci_init # init > pnv_pci_init_ioda2_phb # non-init > pnv_pci_init_ioda_phb # init > > This should fix a section mismatch warning. Thanks. I should look at those warnings more often... Cheers, Ben. > Signed-off-by: Bjorn Helgaas > --- > arch/powerpc/platforms/powernv/pci-ioda.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c > index 9c9d15e..dfbb03d 100644 > --- a/arch/powerpc/platforms/powernv/pci-ioda.c > +++ b/arch/powerpc/platforms/powernv/pci-ioda.c > @@ -1210,7 +1210,7 @@ void __init pnv_pci_init_ioda_phb(struct device_node *np, int ioda_type) > opal_pci_set_pe(phb_id, 0, 0, 7, 1, 1 , OPAL_MAP_PE); > } > > -void pnv_pci_init_ioda2_phb(struct device_node *np) > +void __init pnv_pci_init_ioda2_phb(struct device_node *np) > { > pnv_pci_init_ioda_phb(np, PNV_PHB_IODA2); > }