Urgent fixes for misconverted CONFIG_BOOTDELAY

Heiko Schocher u-boot 2 versions Jun 13, 2016 New

Get series with b4

b4 am [email protected]

Versions newest first · linked by author and title

v2 Latest posting 2/2 Jun 13, 20160 of 5 patches 0 repliesNew
v2 posting 1/2 Jun 13, 20160 of 5 patches 0 repliesSuperseded

Patches in v2 posting 2/2 No applies recorded yet

None of the 5 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [U-Boot] [PATCH v2 0/5] Urgent fixes for misconverted CONFIG_BOOTDELAY

Hello Masahiro,

Am 13.06.2016 um 09:53 schrieb yamada.masahiro at socionext.com:

Quoted 9 lines

Hi Heiko.
Quoted 3 lines
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.

Quoted 6 lines

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.

Quoted 8 lines

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

Quoted 81 lines

Quoted 81 lines
-----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:
Quoted 4 lines
Hi Tom,
2016-06-13 6:07 GMT+09:00 Masahiro Yamada
<[email protected]>:
Quoted 28 lines
Quoted 13 lines
Hi Tom,
2016-06-13 1:50 GMT+09:00 Tom Rini <[email protected]>:
Quoted 10 lines · Masahiro Yamada
Quoted 8 lines
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
Quoted 14 lines
Quoted 3 lines
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?
Quoted 4 lines
Quoted 4 lines
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
Quoted 5 lines
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

Signature · 2 lines

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Read the cover letter in its thread