[U-Boot] U-Boot build report @ Sat Mar 31 05:30:15 CEST 2012

[Marek Vasut <marex@denx.de>]

Dear Anatolij Gustschin,

> On Sat, 31 Mar 2012 15:43:03 +0200
> Anatolij Gustschin <agust@denx.de> wrote:
> ...
> 
> > > arm/seaboard                            FAILED FOR: eldk4.2 eldk5.1
> > > arm/ventana                             FAILED FOR: eldk4.2 eldk5.1
> > 
> > These boards actually build without errors and warnings, but
> > errors/warnings status is reported by MAKEALL because of the output:
> > 
> > DTC: dts->dtb  on file "dt.dtb.tmp"
> > 
> > This could be fixed by
> > 
> > diff --git a/dts/Makefile b/dts/Makefile
> > index 5792afd..d19a622 100644
> > --- a/dts/Makefile
> > +++ b/dts/Makefile
> > @@ -47,7 +47,7 @@ DT_BIN	:= $(obj)dt.dtb
> > 
> >  $(DT_BIN): $(TOPDIR)/board/$(VENDOR)/dts/$(DEVICE_TREE).dts
> >  
> >  	cat $< | $(CPP) -P $(DTS_CPPFLAGS) - >$@.tmp
> > 
> > -	$(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp
> > +	$(DTC) -R 4 -p 0x1000 -O dtb -o ${DT_BIN} $@.tmp 2> /dev/null
> > 
> >  	rm $@.tmp
> >  
> >  process_lds = \
> 
> Thinking more about this I see that it is wrong. dtc could die
> for other reasons and it should be handled as error.
> We should fix dtc instead not to output this info to stderr.
> Currently it does:
> 
> fprintf(stderr, "DTC: %s->%s  on file \"%s\"\n",
>                 inform, outform, arg);

Agreed! Doing stderr output of info text is plain wrong :( Will you send a patch 
mainline please?

> Thanks,
> 
> Anatolij

Best regards,
Marek Vasut