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 ESMTP id D7C80DDE3C for ; Thu, 18 Oct 2007 01:29:39 +1000 (EST) In-Reply-To: <24BADA7A-1716-43B1-BC92-F4DB7ED0D331@kernel.crashing.org> References: <989B956029373F45A0B8AF0297081890B768A2@zch01exm26.fsl.freescale.net> <24BADA7A-1716-43B1-BC92-F4DB7ED0D331@kernel.crashing.org> Mime-Version: 1.0 (Apple Message framework v752.2) Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: From: Kumar Gala Subject: Re: [PATCH] qe_lib: export symbols to fix compile error when QE drivers compile as modules Date: Wed, 17 Oct 2007 10:30:08 -0500 To: Li Yang-r58472 Cc: linuxppc-dev list , Timur Tabi List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On May 25, 2007, at 7:15 AM, Kumar Gala wrote: > > On May 24, 2007, at 9:45 PM, Li Yang-r58472 wrote: > >> >>> -----Original Message----- >>> From: Kumar Gala [mailto:galak@kernel.crashing.org] >>> Sent: Thursday, May 24, 2007 7:44 PM >>> To: Li Yang-r58472 >>> Cc: Paul; linuxppc-dev@ozlabs.org >>> Subject: Re: [PATCH] qe_lib: export symbols to fix compile error >>> when >> QE drivers >>> compile as modules >>> >>> >>> On May 24, 2007, at 4:14 AM, Li Yang wrote: >>> >>>> Export symbols of qe_lib to be used by QE drivers. >>>> >>>> Signed-off-by: Li Yang >>>> --- >>>> arch/powerpc/sysdev/qe_lib/ucc.c | 7 +++++++ >>>> arch/powerpc/sysdev/qe_lib/ucc_fast.c | 8 ++++++++ >>>> arch/powerpc/sysdev/qe_lib/ucc_slow.c | 12 ++++++++++-- >>>> 3 files changed, 25 insertions(+), 2 deletions(-) >>> >>> A number of the EXPORT_SYMBOL() are not used by any in kernel >>> drivers >>> and thus should not be added. See comments below. Also, should >>> these be EXPORT_SYMBOL_GPL(). >> >> Some of them are not used by in kernel driver, but they are used by >> some >> off kernel drivers and driver in development. Shouldn't we export >> them? > > Nope, its been a pretty established rule that we don't export symbols > for drivers not in the kernel. These drivers should really be > submitted for acceptance into the kernel and than we can export the > symbols they need. > > While I can see a need for the *slow* symbols to be exported until > there is a driver needing them we shouldn't export the symbols. > >> I know Linux is again non-GPL modules, but Freescale won't be forcing >> customers to use GPL as long as Linux doesn't ban non-GPL module >> completely. > > That's fine. > >>>> static void get_cmxucr_reg(int ucc_num, volatile u32 ** p_cmxucr, >>>> u8 * reg_num, >>>> u8 * shift) >>>> @@ -122,6 +126,7 @@ static void get_cmxucr_reg(int ucc_num, >>>> volatile u32 ** p_cmxucr, u8 * reg_num, >>>> break; >>>> } >>>> } >>>> +EXPORT_SYMBOL(get_cmxucr_reg); >>> >>> remove, you shouldn't be exporting something marked static. >> >> Good catch. >> >> Thanks, >> Leo What happened with this? - k