[PATCH v2 3/5] configs: khadas-vim3l_android{_ab}: move on bootmeth android

[Guillaume LA ROQUE]

Hi,

Le 19/11/2024 à 11:35, Mattijs Korpershoek a écrit :
> Hi Guillaume,
>
> Thank you for the patch.
>
> On jeu., nov. 14, 2024 at 22:29, Guillaume La Roque <glaroque at baylibre.com> wrote:
>
>> Actually khadas vim3l use distro command to boot android image.
>> Move on new bootmeth android for A/B and non-A/B vim3l android.
>>
>> Signed-off-by: Guillaume La Roque <glaroque at baylibre.com>
>> ---
>>   configs/khadas-vim3l_android_ab_defconfig |  7 ++++++-
>>   configs/khadas-vim3l_android_defconfig    |  7 ++++++-
>>   include/configs/khadas-vim3l_android.h    | 26 ++++++++++++++++++++++----
>>   include/configs/meson64_android.h         |  3 ---
>>   4 files changed, 34 insertions(+), 9 deletions(-)
> [...]
>
>>   CONFIG_CMD_GPIO=y
>>   CONFIG_CMD_GPT=y
>>   CONFIG_CMD_I2C=y
>> diff --git a/include/configs/khadas-vim3l_android.h b/include/configs/khadas-vim3l_android.h
>> index f39a3782d663..31068c59d5ed 100644
>> --- a/include/configs/khadas-vim3l_android.h
>> +++ b/include/configs/khadas-vim3l_android.h
>> @@ -41,10 +41,28 @@
>>   	"name=rootfs,size=-,uuid=" ROOT_UUID
>>   #endif
>>   
>> -#define EXTRA_ANDROID_ENV_SETTINGS \
>> -	"board=vim3l\0" \
>> -	"board_name=vim3l\0" \
>> +#define CFG_EXTRA_ENV_SETTINGS                                    \
>> +	"board=vim3l\0"                                               \
>> +	"board_name=vim3l\0"                                          \
>> +	"bootmeths=android\0"                                         \
>> +	"bootcmd=bootflow scan\0"                                     \
>> +	"adtb_idx=2\0"                                                \
>> +	"partitions=" PARTS_DEFAULT "\0"                              \
>> +	"mmcdev=2\0"                                                  \
>> +	"fastboot_raw_partition_bootloader=0x1 0xfff mmcpart 1\0"     \
>> +	"fastboot_raw_partition_bootenv=0x0 0xfff mmcpart 2\0"        \
>> +	"gpio_recovery=88\0"                                          \
> gpio_recovery is not used anywhere. Can we please drop this?
yes i forgot this part of gpio recovery not needed anymore i will do in v3
>
> With that addressed, please add:
>
> Reviewed-by: Mattijs Korpershoek <mkorpershoek at baylibre.com>
>
>> +	"stdin=" STDIN_CFG "\0"                                       \
>> +	"stdout=" STDOUT_CFG "\0"                                     \
>> +	"stderr=" STDOUT_CFG "\0"                                     \
>> +	"dtboaddr=0x08200000\0"                                       \
>> +	"loadaddr=0x01080000\0"                                       \
>> +	"fdt_addr_r=0x01000000\0"                                     \
>> +	"scriptaddr=0x08000000\0"                                     \
>> +	"kernel_addr_r=0x01080000\0"                                  \
>> +	"pxefile_addr_r=0x01080000\0"                                 \
>> +	"ramdisk_addr_r=0x13000000\0"                                 \
>>   
>> -#include <configs/meson64_android.h>
>> +#include <configs/meson64.h>
>>   
>>   #endif /* __CONFIG_H */
>> diff --git a/include/configs/meson64_android.h b/include/configs/meson64_android.h
>> index 77364bbf9cf0..37ef8b8f7a7e 100644
>> --- a/include/configs/meson64_android.h
>> +++ b/include/configs/meson64_android.h
>> @@ -104,9 +104,6 @@
>>   	"elif test $board_name = sei610; then " \
>>   		"echo \"  Reading DTB for sei610...\"; " \
>>   		"setenv dtb_index 1;" \
>> -	"elif test $board_name = vim3l; then " \
>> -		"echo \"  Reading DTB for vim3l...\"; " \
>> -		"setenv dtb_index 2;" \
>>   	"elif test $board_name = vim3; then " \
>>   		"echo \"  Reading DTB for vim3...\"; " \
>>   		"setenv dtb_index 3;" \
>>
>> -- 
>> 2.34.1