From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 9 Oct 2014 00:48:14 +0200 Subject: [U-Boot] [PATCH] ARM: mxs: tools: Add support for verbose and silent boot progress flags In-Reply-To: References: <1412801378-9032-1-git-send-email-lexszero@gmail.com> <201410090010.10999.marex@denx.de> Message-ID: <201410090048.14131.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thursday, October 09, 2014 at 12:29:10 AM, Alexey Ignatov wrote: > 2014-10-09 2:10 GMT+04:00 Marek Vasut : > > On Wednesday, October 08, 2014 at 10:49:38 PM, Alexey Ignatov wrote: > >> mkimage -T mxs now support new flags in config file: > >> NODISPLAYPROGRESS - makes boot silent > > > > Please make this the other way -- that is, add flag to make the display > > not- silent. What does this mean anyway? > > Default behaviour of current MXS booting is to print HTLLCHTLLC > characters when before usual uboot output - it is a feature to debug > early boot stages, signatures checking, etc. Negation here to not > break default behaviour That's OK, you can just patch the arch/arm/cpu/arm926ejs/mxs/mxsimage*cfg to keep the behavior intact. > >> VERBOSEPROGRESS - makes boot progress display more verbose > > > > And how is the above different from this new flag? > > I'm not really sure what this does (I haven't seen any differences > with this flag on and off), but > http://www.rockbox.org/wiki/SbFileFormat mentions existence of this > bit. Unfortunately, Freescale documentation is not full and a bit hard > to search and use :( Do we need this at all then ? > > Please document those flags in doc/README.mxsimage too. > > The code looks mostly OK, thanks ! > > Thank you for comments, I'll add documentation, fix DISPLAYPROGRESS > and post a v2 of the patch. Thanks! Best regards, Marek Vasut