[U-Boot] [RFC PATCH v2 13/20] sandbox: Add OS dependent layer

[Mike Frysinger]

On Friday, September 23, 2011 12:22:15 Simon Glass wrote:
>  arch/sandbox/cpu/sandbox/Makefile |    7 ++++-
>  arch/sandbox/cpu/sandbox/os.c     |   49

you don't need to follow the "SoC" convention.  arch/sandbox/cpu/*.c should 
work fine ...

> --- 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

if you edit config.mk like so, does that help you figure out what is wrong ?

--- a/config.mk
+++ b/config.mk
@@ -280,6 +280,8 @@
 $(obj)%.s:	%.S
 	$(CPP) $(ALL_AFLAGS) -o $@ $<
 $(obj)%.o:	%.S
+	echo $(BCURDIR)/$(@F)
+	echo $(ALL_CFLAGS)
 	$(CC)  $(ALL_AFLAGS) -o $@ $< -c
 $(obj)%.o:	%.c
 	$(CC)  $(ALL_CFLAGS) -o $@ $< -c

> --- /dev/null
> +++ b/arch/sandbox/cpu/sandbox/os.c
> 
> +int os_open(const char *pathname, int flags)
> +{
> +	return open(pathname, flags);
> +}

i guess since u-boot can't really create files, we don't need to support the 
3rd arg (mode) ... what about creating file-backed flash devices on the fly ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110926/0de28773/attachment.pgp