mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: hacklu <embedway.linux@gmail.com>
Cc: Am?rico Wang <xiyou.wangcong@gmail.com>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: Re: how about KBUILD_MODNAME
Date: Tue, 3 Aug 2010 12:00:07 +0200	[thread overview]
Message-ID: <20100803100006.GA19424@merkur.ravnborg.org> (raw)
In-Reply-To: <201008031735377545190@gmail.com>

On Tue, Aug 03, 2010 at 05:35:40PM +0800, hacklu wrote:
> 
> I am compiling the boardcom's driver now.
> the driver had defined EXTRA_CFLAGS = -D"KBUILD_STR(s)=\#s" $(basename_flags) $(modname_flags) 
> in the a.makefile.
> and the a.makefile was included by b.makefile.
> when in the b.makefile,
> there is this command.
>  $(CC) $(DEPS_OPT) $(CFLAGS) $(EXTRA_CFLAGS) -o $@ -c $< ;) 
> 
> I had add "echo $(EXTRA_CFLAGS)" before this line;but returns NULL;
> 
> so I suspicion the $(EXTRA_CFLAGS) hasn't been work.
> 
> I aslo try to add "export EXTRA_CFLAGS" in a.makefile,but it doesn't work either.
> 
> the problem has puzzled me for some days.
> 
> help me please~~


It looks like the driver uses the old 2.4 way to define a Makefile.
This will not work with 2.6.

What you need to do is to define a Makefile (*) like you would
if the driver was included in the kernel and then build
it using the following command:

make -C path/to/kernel/ M=`pwd`

The kernel shall be built before you do this as the method uses
information from the buitl kernel.

(*) The Makefile may not refer to new CONFIG_ symbols.
So it likely looks like this:

obj-m := foo.o bar.o

Do not be suprised that the Makefile becomes much much simpler - this is expected.

	Sam

  reply	other threads:[~2010-08-03 10:00 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <201008031037022524247@gmail.com>
2010-08-03  7:11 ` hacklu
2010-08-03  8:36   ` Américo Wang
2010-08-03  9:35   ` hacklu
2010-08-03 10:00     ` Sam Ravnborg [this message]
2010-08-03 10:12     ` Américo Wang
2010-08-03  9:52   ` Boaz Harrosh
2010-08-03  9:56     ` Boaz Harrosh
2010-08-03 10:15       ` Américo Wang
2010-08-03 10:24         ` Boaz Harrosh
2010-08-03 11:02           ` Sam Ravnborg

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=20100803100006.GA19424@merkur.ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=embedway.linux@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=xiyou.wangcong@gmail.com \
    /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

all inboxes | Powered by JetHome®