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

[Simon Glass]

Hi Mike,

On Sun, Sep 25, 2011 at 10:16 PM, Mike Frysinger <vapier at 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