linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: mcuos.com@gmail.com (Wan ZongShun)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] NUC900: Add extern header file support for some driver
Date: Tue, 5 Jan 2010 10:43:09 +0800	[thread overview]
Message-ID: <e68bb3471001041843s57085e98v324f977e6ae9f9c1@mail.gmail.com> (raw)
In-Reply-To: <ae11f36b1001041826m2dea49dfvc0511aa9e1783ba9@mail.gmail.com>

2010/1/5 ?? <rurality.linux@gmail.com>:
>
>
> 2010/1/5 Russell King - ARM Linux <linux@arm.linux.org.uk>
>>
>> On Sat, Jan 02, 2010 at 11:44:45PM +0800, Wan ZongShun wrote:
>> > Dear Wang Qiang,
>> >
>> > This patch provides a API for your LCD driver or other drivers, if you
>> > want to use those APIs, please include corresponding header file.
>> >
>> > Signed-off-by: Wan ZongShun <mcuos.com@gmail.com>
>> > ---
>> > ?arch/arm/mach-w90x900/include/mach/clkdev.h | ? ?3 +++
>> > ?arch/arm/mach-w90x900/include/mach/mfp.h ? ?| ? 23
>> > +++++++++++++++++++++++
>> > ?2 files changed, 26 insertions(+), 0 deletions(-)
>> > ?create mode 100644 arch/arm/mach-w90x900/include/mach/mfp.h
>> >
>> > diff --git a/arch/arm/mach-w90x900/include/mach/clkdev.h
>> > b/arch/arm/mach-w90x900/include/mach/clkdev.h
>> > index 04b37a8..b35f4bc 100644
>> > --- a/arch/arm/mach-w90x900/include/mach/clkdev.h
>> > +++ b/arch/arm/mach-w90x900/include/mach/clkdev.h
>> > @@ -4,4 +4,7 @@
>> > ?#define __clk_get(clk) ({ 1; })
>> > ?#define __clk_put(clk) do { } while (0)
>> >
>> > +extern void nuc900_driver_clksrc_div(struct device *dev, unsigned char
>> > *src,
>> > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned char
>> > divider);
>> > +
>>
>> I'd prefer it if this header didn't get used for non-clkdev<->plat code
>> stuff.
>
> So, you mean I can use this header in the LCD driver of?nuc900 plat?
>

Hmmm, I don't think so.

I think it is better to implement other API as Ben Dooks mentioned,
but if you want to use
this API provided by me, we have to add the extern to other header
file rather than the specific clkdev header file.

So, please wait me for new patch.

-- 
*linux-arm-kernel mailing list
mail addr:linux-arm-kernel at lists.infradead.org
you can subscribe by:
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

* linux-arm-NUC900 mailing list
mail addr:NUC900 at googlegroups.com
main web: https://groups.google.com/group/NUC900
you can subscribe it by sending me mail:
mcuos.com at gmail.com

  reply	other threads:[~2010-01-05  2:43 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-02 15:44 [PATCH] NUC900: Add extern header file support for some driver Wan ZongShun
     [not found] ` <4B3F7EC3.7060707@ru.mvista.com>
2010-01-03  2:19   ` Wan ZongShun
2010-01-04 16:42 ` Ben Dooks
2010-01-04 16:58 ` Russell King - ARM Linux
2010-01-05  2:26   ` 王强
2010-01-05  2:43     ` Wan ZongShun [this message]
2010-01-05  3:02       ` Wang Qiang

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=e68bb3471001041843s57085e98v324f977e6ae9f9c1@mail.gmail.com \
    --to=mcuos.com@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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).