Hi Mike,
On Sun, Sep 25, 2011 at 10:16 PM, Mike Frysinger <vap...@gentoo.org> wrote:
> 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 ...
OK, will do.
>
>> --- 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.
>
> 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
>
Who would ever want that? :-) Even if we do it's not clear that we
want to set the mode, so would prefer to leave this bit for now.
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
