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 <glaro...@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 <glaro...@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 <mkorpersh...@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


Reply via email to