From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Mon, 13 Jun 2016 10:21:06 +0200 Subject: [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted CONFIG_BOOTDELAY In-Reply-To: <5dd792c10ff24e86901ac691e13d1f05@SOC-EX01V.e01.socionext.com> References: <1465638252-13196-1-git-send-email-yamada.masahiro@socionext.com> <20160612165058.GP11619@bill-the-cat> <575E47B6.8070008@denx.de> <5dd792c10ff24e86901ac691e13d1f05@SOC-EX01V.e01.socionext.com> Message-ID: <575E6CF2.8020308@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 Hello Masahiro, Am 13.06.2016 um 09:53 schrieb yamada.masahiro at socionext.com: > Hi Heiko. > >> Hmm... didn;t you had a script, with which you checked all board binaries >> if they are the same before and after one patch? That could be a solution >> for doing such Kconfig moves ... > > When I converted lots of Makefiles for Kbuild migration more than 2 years ago, > I hacked include/timestamp.h, include/version.h, and MAKEALL > to show MD5SUM for each board. You can generate a reproducible build with settings SOURCE_DATE_EPOCH see README, so the hacks in include/timestamp.h and include/version.h are not longer necessary. > I noted a small tip in the page 9 of the following slide: > http://www.denx.de/wiki/pub/U-Boot/MiniSummitELCE2014/uboot2014_kconfig.pdf > > But, MAKEALL will be removed sooner or later. > > > Instead, I found an interesting option in Buildman. > > -K, --show-config Show configuration changes in summary (both board > config files and Kconfig) > > > I tried it, but the output seems too noisy to check the diff... > (unless I am missing something.) I do not know, if this is possible with buildman. Maybe it is possible to add a buildman option to add printing a md5sum for the resulting images? bye, Heiko > > > >> -----Original Message----- >> From: Heiko Schocher [mailto:hs at denx.de] >> Sent: Monday, June 13, 2016 2:42 PM >> To: Yamada, Masahiro/?? ?? >> Cc: Tom Rini; Steve Rae; Lucas Stach; Dimitar Penev; Viresh Kumar; Lars >> Poeschel; Albert ARIBAUD (3ADEV); U-Boot Mailing List; Magnus Lilja; Soeren >> Moch; Przemyslaw Marczak; Eric B?nard; Valentin Yakovenkov; Wolfgang >> Wegner; Stefan Roese; Mark Jonas; Richard Hu; Chander Kashyap; Valentin >> Longchamp; Akshay Saraswat; Stephen Warren; Otavio Salvador; Angelo >> Dureghello; Peter Meerwald; Thomas Lange; Thomas Abraham; Holger Brunck; >> Ash Charles; Ian Campbell; Lauri Hintsala; Markus Niebel; Stefan Agner; >> Sergei Poselenov; Tim Kryger; Thomas Weber; Gilles Gameiro; Robert Baldyga; >> Phil Han; Lucile Quirion; Lukasz Dalek; Adrian Alonso; Fabio Estevam; Sergey >> Kostanbaev; Christophe Ricard; Zoltan Herpai; Nobuhiro Iwamatsu; Oleksandr >> Zhadan and Michael Durrant; Vladimir Zapolskiy; Aleksei Mamlin; Po Liu; >> Vipin Kumar; Gong Qianyu; Priit Laes; huang lin; Marcel Ziswiler; Joe >> Hershberger; Werner Pfister; Felipe Balbi; Kamil Lulko; Marcus Cooper; Tom >> Warren; Lothar Felten; I-SYST Micromodule; Enric Balletbo i Serra; Jelle >> de Jong; M.Hasewinkel (MHA); Alban Bedel; Daniel Allred; TsiChung Liew >> Subject: Re: [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted >> CONFIG_BOOTDELAY >> >> Hello Masahiro, >> >> Am 13.06.2016 um 01:15 schrieb Masahiro Yamada: >>> Hi Tom, >>> >>> >>> 2016-06-13 6:07 GMT+09:00 Masahiro Yamada >> : >>>> Hi Tom, >>>> >>>> 2016-06-13 1:50 GMT+09:00 Tom Rini : >>>>> On Sat, Jun 11, 2016 at 06:44:07PM +0900, Masahiro Yamada wrote: >>>>> >>>>>> >>>>>> Masahiro Yamada (5): >>>>>> ARM: stm32: remove unused CONFIG_AUTOBOOT >>>>>> autoboot: follow-up cleanup after CONFIG_BOOTDELAY moves >>>>>> tools: fix define2mk.sed to not add quotes around negative integers >>>>>> autoboot: fix a bunch of misconversion of CONFIG_BOOTDELAY >>>>>> autoboot: add CONFIG_AUTOBOOT to allow to not compile autoboot.c >>>>> >>>>> Please put together a PR with everything except the defconfig >>>>> changes, along with the other moveconfig.py related fixes for me and >>>>> I'll go unbreak everything (along with Hans' patch). Thanks! >>>> >>>> >>>> Please let me confirm the ones I should include in my PR. >>>> >>>> moveconfig patches and these three? >>>> >>>>>> ARM: stm32: remove unused CONFIG_AUTOBOOT >>>>>> autoboot: follow-up cleanup after CONFIG_BOOTDELAY moves >>>>>> tools: fix define2mk.sed to not add quotes around negative >>>>>> integers >>>> >>> >>> Done. >>> >>> >>> BTW, would you consider changing the default of BOOTDELAY from 0 to 3 >>> ? >> >> Damn, Tom suggested me to change it to 2 .... and in my rebase to >> 2016.05-rc1 accidently I used the wrong value >> >> I am sorry for all the trouble ... >> >> Hmm... didn;t you had a script, with which you checked all board binaries >> if they are the same before and after one patch? That could be a solution >> for doing such Kconfig moves ... >> >> bye, >> Heiko >> -- >> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk >> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany