All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.