All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH v2 13/20] sandbox: Add OS dependent layer
Date: Mon, 26 Sep 2011 17:19:07 -0400	[thread overview]
Message-ID: <201109261719.09127.vapier@gentoo.org> (raw)
In-Reply-To: <CAPnjgZ3vEz2STXJ0YtOoo7TjMhnR_=-YGHK_WJbFWaSLEGo3kA@mail.gmail.com>

On Monday, September 26, 2011 17:04:49 Simon Glass wrote:
> On Sun, Sep 25, 2011 at 10:16 PM, Mike Frysinger wrote:
> > On Friday, September 23, 2011 12:22:15 Simon Glass wrote:
> >> --- a/arch/sandbox/cpu/sandbox/Makefile
> >> +++ b/arch/sandbox/cpu/sandbox/Makefile
> >> 
> >> +# I want to do this, but it doesn't seem to work
> >> +CFLAGS_arch/sandbox/cpu/sandbox/os.o += -I/usr/include
> > 
> > seems to work for me ...
> > CFLAGS_arch/blackfin/lib/board.o = -ffoo
> 
> That is not a -I flag, though. If you try -I I think you will get a
> failure. Looking a bit closer it is because the dependency stuff uses
> CPPFLAGS which doesn't support per file/dir flags.

OK, if it's the dependency step, that's a different story.  untested patch:

--- a/config.mk
+++ b/config.mk
@@ -277,6 +277,8 @@
 BCURDIR = $(subst $(SRCTREE)/,,$(CURDIR:$(obj)%=%))
 ALL_AFLAGS = $(AFLAGS) $(AFLAGS_$(BCURDIR)/$(@F)) $(AFLAGS_$(BCURDIR))
 ALL_CFLAGS = $(CFLAGS) $(CFLAGS_$(BCURDIR)/$(@F)) $(CFLAGS_$(BCURDIR))
+EXTRA_CPPFLAGS = $(CPPFLAGS_$(BCURDIR)/$(@F)) $(CPPFLAGS_$(BCURDIR))
+ALL_CFLAGS += $(EXTRA_CPPFLAGS)
 $(obj)%.s:	%.S
 	$(CPP) $(ALL_AFLAGS) -o $@ $<
 $(obj)%.o:	%.S
--- a/rules.mk
+++ b/rules.mk
@@ -30,7 +30,7 @@
 		@touch $@
 		@for f in $(SRCS); do \
 			g=`basename $$f | sed -e 's/\(.*\)\.[[:alnum:]_]/\1.o/'`; \
-			$(CC) -M $(CPPFLAGS) -MQ $(obj)$$g $$f >> $@ ; \
+			$(CC) -M $(CPPFLAGS) $(EXTRA_CPPFLAGS) -MQ $(obj)$$g $$f >> $@ ; \
 		done
 		@for f in $(HOSTSRCS); do \
 			g=`basename $$f | sed -e 's/\(.*\)\.[[:alnum:]_]/\1.o/'`; \

-mike

  reply	other threads:[~2011-09-26 21:19 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-23 16:22 [U-Boot] [RFC PATCH v2 0/20] New 'sandbox' test architecture for U-Boot Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 01/20] sandbox: Add architecture header files Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 02/20] Fix use of int as pointer in image.c Simon Glass
2011-09-26  5:00   ` Mike Frysinger
2011-09-26 16:51     ` Simon Glass
2011-10-03  5:14       ` Mike Frysinger
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 03/20] sandbox: Add architecture image support Simon Glass
2011-09-26  5:01   ` Mike Frysinger
2011-09-26 19:39     ` Simon Glass
2011-09-26 20:19       ` Mike Frysinger
2011-09-26 20:24         ` Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 04/20] sandbox: Add compiler defines to support a 64-bit x86_64 platform Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 05/20] sandbox: Add cpu files Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 06/20] sandbox: Add architecture lib files Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 07/20] sandbox: Add sandbox board Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 08/20] sandbox: Add board info for architecture Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 09/20] sandbox: Add bootm support Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 10/20] sandbox: Disable built-in malloc Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 11/20] sandbox: Disable standalone/API support Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 12/20] sandbox: Force command sections to be 4-byte aligned Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 13/20] sandbox: Add OS dependent layer Simon Glass
2011-09-26  5:16   ` Mike Frysinger
2011-09-26 21:04     ` Simon Glass
2011-09-26 21:19       ` Mike Frysinger [this message]
2011-09-26 22:03         ` Simon Glass
2011-09-26 23:53           ` Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 14/20] sandbox: Add board_init() Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 15/20] sandbox: Add main program Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 16/20] sandbox: Add serial uart Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 17/20] sandbox: Add basic config file Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 18/20] Remove unused variable warnings in cmd_mem, cmd_nvedit Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 19/20] Use uintptr_t for 32/64-bit compatibility Simon Glass
2011-09-23 16:22 ` [U-Boot] [RFC PATCH v2 20/20] sandbox: Makefile changes to build sandbox architecture Simon Glass
2011-09-26  4:59 ` [U-Boot] [RFC PATCH v2 0/20] New 'sandbox' test architecture for U-Boot Mike Frysinger
2011-09-27  0:07   ` Simon Glass
2011-10-03  5:16     ` Mike Frysinger

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=201109261719.09127.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.