U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/3] cmd: booti: move CONFIG_CMD_BOOTI to Kconfig
Date: Sun, 31 Jul 2016 13:23:15 -0400	[thread overview]
Message-ID: <20160731172315.GG17970@bill-the-cat> (raw)
In-Reply-To: <CAK7LNARazLKXP6GM4rwf6yroVnXT1C9jK=ejmhfwjR2O-0nosQ@mail.gmail.com>

On Mon, Aug 01, 2016 at 01:27:45AM +0900, Masahiro Yamada wrote:
> 2016-07-25 22:42 GMT+09:00 Michal Simek <michal.simek@xilinx.com>:
> > On 25.7.2016 15:06, Masahiro Yamada wrote:
> >> This command is used to boot ARM64 Linux.
> >>
> >> I made DISTRO_DEFAULTS select this option for ARM64 to respect
> >> include/config_distro_defaults.h.
> >>
> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> >> ---
> >>
> >>  Kconfig                                          | 1 +
> >>  cmd/Kconfig                                      | 6 ++++++
> >>  configs/dragonboard410c_defconfig                | 1 +
> >>  configs/e2220-1170_defconfig                     | 1 +
> >>  configs/hikey_defconfig                          | 1 +
> >>  configs/odroid-c2_defconfig                      | 1 +
> >>  configs/p2371-0000_defconfig                     | 1 +
> >>  configs/p2371-2180_defconfig                     | 1 +
> >>  configs/p2571_defconfig                          | 1 +
> >>  configs/p2771-0000-a02_defconfig                 | 1 +
> >>  configs/p2771-0000-b00_defconfig                 | 1 +
> >>  configs/rpi_3_defconfig                          | 1 +
> >>  configs/s32v234evb_defconfig                     | 1 +
> >>  configs/uniphier_ld11_defconfig                  | 1 +
> >>  configs/uniphier_ld20_defconfig                  | 1 +
> >>  configs/vexpress_aemv8a_dram_defconfig           | 1 +
> >>  configs/vexpress_aemv8a_juno_defconfig           | 1 +
> >>  configs/vexpress_aemv8a_semi_defconfig           | 1 +
> >>  configs/xilinx_zynqmp_ep_defconfig               | 1 +
> >>  configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
> >>  configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 1 +
> >>  configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 1 +
> >>  configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 1 +
> >>  configs/xilinx_zynqmp_zcu102_defconfig           | 1 +
> >>  configs/xilinx_zynqmp_zcu102_revB_defconfig      | 1 +
> >>  include/config_distro_defaults.h                 | 3 ---
> >>  include/configs/uniphier.h                       | 1 -
> >>  include/configs/vexpress_aemv8a.h                | 1 -
> >>  include/configs/xilinx_zynqmp.h                  | 1 -
> >>  29 files changed, 30 insertions(+), 6 deletions(-)
> >>
> >> diff --git a/Kconfig b/Kconfig
> >> index 0d6afd2..335efcd 100644
> >> --- a/Kconfig
> >> +++ b/Kconfig
> >> @@ -59,6 +59,7 @@ config DISTRO_DEFAULTS
> >>       default n
> >>       select CMD_BOOTM
> >>       select CMD_BOOTZ
> >> +     select CMD_BOOTI if ARM64
> >>       select CMD_DHCP
> >>       select CMD_EXT2
> >>       select CMD_EXT4
> >> diff --git a/cmd/Kconfig b/cmd/Kconfig
> >> index 171114d..f7c609d 100644
> >> --- a/cmd/Kconfig
> >> +++ b/cmd/Kconfig
> >> @@ -166,6 +166,12 @@ config CMD_BOOTZ
> >>       help
> >>         Boot the Linux zImage
> >>
> >> +config CMD_BOOTI
> >> +     bool "booti"
> >> +     depends on CMD_BOOTM && ARM64
> >> +     help
> >> +       Boot the Linux Image for ARM64
> >
> > I would even add here default y which can shorten defconfigs and you can
> > disable it if you don't want it.
> > What do you think?
> 
> 
> Actually "default y" will give us shorter defconfigs.
> (but not shorter than I have expected.)
> 
> I am OK with flipping the default if Tom requests v2.

Yes please, make it default on ARM64.  And similarly bootz on the
architectures that support it (which is just ARM and SANDBOX).

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160731/a8e81f41/attachment.sig>

  reply	other threads:[~2016-07-31 17:23 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-25 13:06 [U-Boot] [PATCH 0/3] Some Kconfig move/fixups Masahiro Yamada
2016-07-25 13:06 ` [U-Boot] [PATCH 1/3] cmd: bootz: make CMD_BOOTZ depend on CMD_BOOTM Masahiro Yamada
2016-07-25 13:17   ` Robert P. J. Day
2016-07-26  4:15     ` Masahiro Yamada
2016-07-25 13:06 ` [U-Boot] [PATCH 2/3] cmd: booti: move CONFIG_CMD_BOOTI to Kconfig Masahiro Yamada
2016-07-25 13:42   ` Michal Simek
2016-07-25 14:15     ` Ryan Harkin
2016-07-31 16:27     ` Masahiro Yamada
2016-07-31 17:23       ` Tom Rini [this message]
2016-07-25 13:06 ` [U-Boot] [PATCH 3/3] arm64: thunderx_88xx_defconfig: remove unneeded CONFIG_SYS_EXTRA_OPTIONS Masahiro Yamada
2016-07-25 14:56   ` Tom Rini
2016-07-26  2:33   ` [U-Boot] [U-Boot, " Tom Rini
2016-07-25 14:55 ` [U-Boot] [PATCH 0/3] Some Kconfig move/fixups Tom Rini
2016-07-31 16:24   ` Masahiro Yamada
2016-07-31 17:21     ` Tom Rini
2016-08-01  2:26       ` Masahiro Yamada

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=20160731172315.GG17970@bill-the-cat \
    --to=trini@konsulko.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