linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Timur Tabi <timur@freescale.com>
To: Josh Boyer <jwboyer@gmail.com>
Cc: kumar.gala@freescale.com, linux-watchdog@vger.kernel.org,
	linuxppc-dev@ozlabs.org
Subject: Re: [PATCH 1/2] powerpc: export ppc_tb_freq so that modules can reference it
Date: Fri, 17 Sep 2010 20:20:31 -0500	[thread overview]
Message-ID: <AANLkTinidrifo3ry7o=DPKF96f3o4+_2zt7rsn=Zvh9D@mail.gmail.com> (raw)
In-Reply-To: <AANLkTikmfd=2N3GE1zpTqQ59583EEe1PDXggwHBUAj_5@mail.gmail.com>

On Fri, Sep 17, 2010 at 7:38 PM, Josh Boyer <jwboyer@gmail.com> wrote:

>> =A0unsigned long ppc_proc_freq;
>> =A0EXPORT_SYMBOL(ppc_proc_freq);
>> =A0unsigned long ppc_tb_freq;
>> +EXPORT_SYMBOL(ppc_tb_freq);
>
> EXPORT_SYMBOL_GPL probably, no?

I don't see any reason to limit it to GPL drivers.  Not only that, but
then we'll have this:

EXPORT_SYMBOL(ppc_proc_freq);
EXPORT_SYMBOL_GPL(ppc_tb_freq);

That just looks dumb.


--=20
Timur Tabi
Linux kernel developer at Freescale

  reply	other threads:[~2010-09-18  1:21 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-17 22:53 [PATCH 1/2] powerpc: export ppc_tb_freq so that modules can reference it Timur Tabi
2010-09-17 22:53 ` [PATCH 2/2] powerpc/watchdog: allow the e500 watchdog driver to be compiled as a module Timur Tabi
2010-09-18  0:37   ` Josh Boyer
2010-09-20 15:51     ` Timur Tabi
2010-09-20 19:30       ` Josh Boyer
2010-09-20 19:53         ` Timur Tabi
2010-09-18  0:38 ` [PATCH 1/2] powerpc: export ppc_tb_freq so that modules can reference it Josh Boyer
2010-09-18  1:20   ` Timur Tabi [this message]
2010-09-18  3:14     ` Benjamin Herrenschmidt
2010-09-18 14:36       ` Tabi Timur-B04825
2010-09-18 15:34         ` Kumar Gala
2010-09-18 15:52           ` Vitaly Wool
2010-09-18 16:56           ` Josh Boyer
2010-09-18 17:36             ` Tabi Timur-B04825
2010-09-18 17:46               ` Josh Boyer
2010-09-18 17:55                 ` Tabi Timur-B04825
2010-09-18 18:22                   ` Josh Boyer
2010-09-20 18:51                     ` Scott Wood
     [not found]                     ` <20100920135136.04ceb772__36164.799918379$1285008751$gmane$org@udp111988uds.am.freescale.net>
2010-09-21 12:34                       ` Detlev Zundel
2010-09-18 18:36             ` Kumar Gala
2010-09-19  2:42           ` Benjamin Herrenschmidt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='AANLkTinidrifo3ry7o=DPKF96f3o4+_2zt7rsn=Zvh9D@mail.gmail.com' \
    --to=timur@freescale.com \
    --cc=jwboyer@gmail.com \
    --cc=kumar.gala@freescale.com \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).