public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Andre Przywara <andre.przywara@arm.com>
To: u-boot@lists.denx.de
Subject: [PATCH] configs: add PineTab defconfig
Date: Sun, 7 Mar 2021 22:15:30 +0000	[thread overview]
Message-ID: <20210307221530.498c5ffb@slackpad.fritz.box> (raw)
In-Reply-To: <87eegqsny1.fsf@yucca>

On Sun, 07 Mar 2021 12:34:14 -0800
Vagrant Cascadian <vagrant@debian.org> wrote:

> On 2021-03-07, Nicolas Boulenguez wrote:
> > From: Arnaud Ferraris <arnaud.ferraris@collabora.com>
> >
> > The PineTab device-tree is already in u-boot, this commit adds the corresponding
> > defconfig, based on pinephone_defconfig.
> >
> > Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>  
> ...
> > --- /dev/null
> > +++ b/configs/pinetab_defconfig  
> ...
> > +CONFIG_BOOTDELAY=0  
> 
> Setting bootdelay to 0 it almost impossible to debug issues in a running
> u-boot.
> 
> The default of 2 seconds that distro_bootcmd uses tries to strike a
> balance between not slowing the boot down too much while still being
> reasonably able to get into a u-boot shell when something goes wrong.
> 
> Individual users or vendors can set this value as they see fit, but this
> doesn't seem like a good default for mainline u-boot, at least to me.

Yeah, we just had a similar discussion recently about the Pinephone.
I think we keep the default of 2 seconds for the mainline defconfig, by
not having any explicit entry in that file, so it reverts to the
platform default.
People can always change this in their .config, even with automated
build systems, by using a simple sed command.

Cheers,
Andre

  reply	other threads:[~2021-03-07 22:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-07 12:53 [PATCH] configs: add PineTab defconfig Nicolas Boulenguez
2021-03-07 13:58 ` Peter Robinson
2021-03-07 20:34 ` Vagrant Cascadian
2021-03-07 22:15   ` Andre Przywara [this message]
2021-03-08  0:12 ` Andre Przywara
2021-03-08  0:13   ` Icenowy Zheng
2021-03-13 15:32   ` Arnaud Ferraris
  -- strict thread matches above, loose matches on Subject: below --
2021-09-06 21:06 Arnaud Ferraris
2021-09-06 23:47 ` Andre Przywara

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=20210307221530.498c5ffb@slackpad.fritz.box \
    --to=andre.przywara@arm.com \
    --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