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

[Masahiro Yamada]

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.


-- 
Best Regards
Masahiro Yamada