[U-Boot] [PATCH 2/3] cmd: booti: move CONFIG_CMD_BOOTI to Kconfig

[Tom Rini]

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 at 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 at 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>