From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Sat, 31 Mar 2012 17:53:18 +0200
Subject: [U-Boot] U-Boot build report @ Sat Mar 31 05:30:15 CEST 2012
In-Reply-To: <CAPnjgZ0oqNs_7D=dq4MP4mo3bHHk66Z1EipRkrtLCxQUv+rWow@mail.gmail.com>
References: <201203311459.06597.marex@denx.de>
	<201203311612.48402.marex@denx.de>
	<CAPnjgZ0oqNs_7D=dq4MP4mo3bHHk66Z1EipRkrtLCxQUv+rWow@mail.gmail.com>
Message-ID: <201203311753.18954.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Simon Glass,

> Hi,
> 
> On Mar 31, 2012 7:13 AM, "Marek Vasut" <marex@denx.de> wrote:
> > 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?
> 
> I will do that if you like, it has bugged me for a while.

Thank you very much, I owe you :)

> Regards,
> Simon
> 
> > > Thanks,
> > > 
> > > Anatolij
> > 
> > Best regards,
> > Marek Vasut

Best regards,
Marek Vasut