From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted CONFIG_BOOTDELAY
Date: Mon, 13 Jun 2016 10:21:06 +0200 [thread overview]
Message-ID: <575E6CF2.8020308@denx.de> (raw)
In-Reply-To: <5dd792c10ff24e86901ac691e13d1f05@SOC-EX01V.e01.socionext.com>
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
>> <yamada.masahiro@socionext.com>:
>>>> Hi Tom,
>>>>
>>>> 2016-06-13 1:50 GMT+09:00 Tom Rini <trini@konsulko.com>:
>>>>> 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
next prev parent reply other threads:[~2016-06-13 8:21 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-11 9:44 [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted CONFIG_BOOTDELAY Masahiro Yamada
2016-06-11 9:44 ` [U-Boot] [PATCH v2 1/5] ARM: stm32: remove unused CONFIG_AUTOBOOT Masahiro Yamada
2016-06-12 22:55 ` Masahiro Yamada
2016-06-11 9:44 ` [U-Boot] [PATCH v2 2/5] autoboot: follow-up cleanup after CONFIG_BOOTDELAY moves Masahiro Yamada
2016-06-12 22:56 ` Masahiro Yamada
2016-06-11 9:44 ` [U-Boot] [PATCH v2 3/5] tools: fix define2mk.sed to not add quotes around negative integers Masahiro Yamada
2016-06-12 22:54 ` Masahiro Yamada
2016-06-11 9:44 ` [U-Boot] [PATCH v2 4/5] autoboot: fix a bunch of misconversion of CONFIG_BOOTDELAY Masahiro Yamada
2016-06-11 11:31 ` Hans de Goede
2016-06-11 12:43 ` Tom Rini
2016-06-11 14:38 ` Hans de Goede
2016-06-11 13:46 ` Masahiro Yamada
2016-06-11 14:54 ` Hans de Goede
2016-06-13 8:02 ` Stefano Babic
2016-06-13 8:17 ` Maxime Ripard
2016-06-19 14:10 ` Tom Rini
2016-06-11 9:44 ` [U-Boot] [PATCH v2 5/5] autoboot: add CONFIG_AUTOBOOT to allow to not compile autoboot.c Masahiro Yamada
2016-06-12 16:50 ` [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted CONFIG_BOOTDELAY Tom Rini
2016-06-12 21:07 ` Masahiro Yamada
2016-06-12 23:15 ` Masahiro Yamada
2016-06-13 5:42 ` Heiko Schocher
2016-06-13 7:53 ` yamada.masahiro at socionext.com
2016-06-13 8:21 ` Heiko Schocher [this message]
2016-06-13 12:47 ` Tom Rini
2016-06-13 13:07 ` Wolfgang Denk
2016-06-13 13:51 ` Hans de Goede
2016-06-13 13:58 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=575E6CF2.8020308@denx.de \
--to=hs@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox