From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andre Przywara <andre.przywara@arm.com>
Date: Sun, 28 Feb 2021 15:22:19 +0000
Subject: [PATCH 2/4] pinephone_defconfig: reduce boot delay
In-Reply-To: <20210225205838.n2rivse6ab2pp3mj@gilmour>
References: <20210220121416.15215-1-nicolas@debian.org>
 <c2647bab-b628-4a6f-a5c5-c966dab70f52@arm.com>
 <20210225205838.n2rivse6ab2pp3mj@gilmour>
Message-ID: <20210228152219.2fa075f7@slackpad.fritz.box>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thu, 25 Feb 2021 21:58:38 +0100
Maxime Ripard <maxime@cerno.tech> wrote:

Hi,

> On Thu, Feb 25, 2021 at 05:02:40PM +0000, Andr? Przywara wrote:
> > On 20/02/2021 12:14, Nicolas Boulenguez wrote:  
> > > From: Arnaud Ferraris <arnaud.ferraris@collabora.com>
> > > 
> > > On a cellular phone, the vast majority of users can be expected to
> > > have no serial console connected and prefer a short boot.  
> > 
> > It's a bit tricky to break in with a delay of 0, but indeed most users
> > won't care, so looks fine to me.
> > 
> > But it's missing a Signed-off-by: line.  
> 
> I'm not sure we should start accepting custom boards changes like it's
> done here, this will only lead to inconsistencies between boards in the
> long run.

I am a bit undecided, I see merits in both arguments.
 
> And a defconfig is really easy to change anyway, even when it's
> integrated in build systems

Nicolas, could you live with:

$ sed -i s/^CONFIG_BOOTDELAY=2/CONFIG_BOOTDELAY=0/ .config

in your build system? Or is the U-Boot build generic?

Cheers,
Andre