From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5B9EFC433EF for ; Mon, 22 Nov 2021 09:53:34 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 88B8882BCD; Mon, 22 Nov 2021 10:53:31 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=baylibre-com.20210112.gappssmtp.com header.i=@baylibre-com.20210112.gappssmtp.com header.b="eyFfMMro"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 9D15C82EBB; Mon, 22 Nov 2021 10:53:30 +0100 (CET) Received: from mail-wr1-x434.google.com (mail-wr1-x434.google.com [IPv6:2a00:1450:4864:20::434]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 5009E812C1 for ; Mon, 22 Nov 2021 10:53:26 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=mkorpershoek@baylibre.com Received: by mail-wr1-x434.google.com with SMTP id b12so31559806wrh.4 for ; Mon, 22 Nov 2021 01:53:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:in-reply-to:references:date:message-id :mime-version; bh=I3saxV4638N+NrGgsqalOborrT90guoFu3hmWmH4qSk=; b=eyFfMMrovrZl4AxI70hF4LgdDiiB/mO8pHccIYkZ9W+jhB8LSzFqTk/IF5o0g0PrbP rwnq4pOIMqjeEIl/diT2qGG9GA45cZhoG1mg1AurXpV71xi0r6kRXlpbWc7LQ79u6tQb /WwirrLUGK1QfkgN2n1L+v3CSqbG+FQ/kBZjiRUNW3fQSdaJBCxPQydgN2acbfLRwVMW eLWvvvSQ+r2Pc0H7L+E1OWZFLCANdh2xdGDSbSO76sOvHlZV8rWf6AoGZnKZeDPKb+ea Cfb6+s8nl+oTFd90VgBbI5IJJEZReo5HqsEF7I3yKgL+ZJMSJt6ZL8zDPOHDcPWWwYjw tT/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=I3saxV4638N+NrGgsqalOborrT90guoFu3hmWmH4qSk=; b=bSCfhIPHBsnqPue4LfZG/Z9OxF6bK6BsOscScrFgvVI4bWv7d7ClZi5NqJQjXjmZwD /Cv3SdJjRooUYDcGtwt1+2QBeKc5FvwyFigWUjC7VDrrnBHa0BaxYXeZCl3XwR5DYMJe 5FrLzpgtCJNyn8u8PrT4QHbgD1r2gR0aK3+gKtUQuonWsXcxtwXCyx/05v6ms8vqq6Tm MVsUav5AP4j+a9JMHEh62kRpqEWLTRwz9uOKz1WupUupt4M41gx8pHuD3mwkwm9GHyVa 4/lr8FJkpv4xFtI2JPw6+R/2jmJdYdS20kvk0ySXhT6q83/EryQMRb97+CJOxddzrx7+ O/SA== X-Gm-Message-State: AOAM531M2NlcVv87Sdv2LyjdtEhAOTQkaD2Y6WLbVqM3Kwe4Ro/Se05j R0KtWalu3gvIwNfQqA67nsl2KA== X-Google-Smtp-Source: ABdhPJzWm+qOL3g2nWHdnlJ0Q2TLE4G+2tnE12XOP4lXtyOegjHQm/XkwuuULt8IAjYOYwP4D5xYeQ== X-Received: by 2002:a5d:59aa:: with SMTP id p10mr35691532wrr.210.1637574805762; Mon, 22 Nov 2021 01:53:25 -0800 (PST) Received: from localhost ([2a01:cb19:826e:8e00:1eb7:3acb:81f4:cb1c]) by smtp.gmail.com with ESMTPSA id u15sm10130996wmq.13.2021.11.22.01.53.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Nov 2021 01:53:25 -0800 (PST) From: Mattijs Korpershoek To: Neil Armstrong Cc: u-boot-amlogic@groups.io, u-boot@lists.denx.de, Guillaume La Roque Subject: Re: [PATCH 3/5] boards: meson: add khadas vim3{l} android support In-Reply-To: References: <20211119100829.843139-1-mkorpershoek@baylibre.com> <20211119100829.843139-4-mkorpershoek@baylibre.com> Date: Mon, 22 Nov 2021 10:53:23 +0100 Message-ID: <87bl2cecto.fsf@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.37 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean Hi Neil, Thank you for your review. Neil Armstrong writes: > Hi, > > On 19/11/2021 11:08, Mattijs Korpershoek wrote: >> Add all the necessary bits to flash and boot Android for both Khadas >> VIM3 and VIM3L boards. >> >> For Android instructions, refer to [1] >> >> [1] https://source.android.com/setup/build/devices#vim3_and_vim3l_boards >> Signed-off-by: Guillaume La Roque >> Signed-off-by: Mattijs Korpershoek >> --- >> configs/khadas-vim3_android_defconfig | 16 ++++++++++-- >> configs/khadas-vim3l_android_defconfig | 16 ++++++++++-- >> include/configs/khadas-vim3_android.h | 34 ++++++++++++++++++++++++++ >> include/configs/khadas-vim3l_android.h | 34 ++++++++++++++++++++++++++ >> include/configs/meson64_android.h | 12 +++++++++ >> 5 files changed, 108 insertions(+), 4 deletions(-) >> create mode 100644 include/configs/khadas-vim3_android.h >> create mode 100644 include/configs/khadas-vim3l_android.h >> >> diff --git a/configs/khadas-vim3_android_defconfig b/configs/khadas-vim3_android_defconfig >> index a225a564b2..9305a54b1b 100644 >> --- a/configs/khadas-vim3_android_defconfig >> +++ b/configs/khadas-vim3_android_defconfig >> @@ -1,9 +1,10 @@ >> CONFIG_ARM=y >> CONFIG_SYS_BOARD="vim3" >> +CONFIG_SYS_CONFIG_NAME="khadas-vim3_android" >> CONFIG_ARCH_MESON=y >> CONFIG_SYS_TEXT_BASE=0x01000000 >> CONFIG_NR_DRAM_BANKS=1 >> -CONFIG_ENV_SIZE=0x2000 >> +CONFIG_ENV_SIZE=0x10000 >> CONFIG_DM_GPIO=y >> CONFIG_DEFAULT_DEVICE_TREE="meson-g12b-a311d-khadas-vim3" >> CONFIG_MESON_G12A=y >> @@ -15,9 +16,14 @@ CONFIG_SYS_LOAD_ADDR=0x1000000 >> CONFIG_OF_BOARD_SETUP=y >> # CONFIG_DISPLAY_CPUINFO is not set >> CONFIG_MISC_INIT_R=y >> +CONFIG_AVB_VERIFY=y >> # CONFIG_CMD_BDI is not set >> +CONFIG_CMD_ADTIMG=y >> +CONFIG_CMD_ABOOTIMG=y >> # CONFIG_CMD_IMI is not set >> +CONFIG_CMD_BCB=y >> CONFIG_CMD_GPIO=y >> +CONFIG_CMD_GPT=y >> CONFIG_CMD_I2C=y >> # CONFIG_CMD_LOADS is not set >> CONFIG_CMD_MMC=y >> @@ -28,6 +34,7 @@ CONFIG_CMD_USB=y >> CONFIG_CMD_USB_MASS_STORAGE=y >> # CONFIG_CMD_SETEXPR is not set >> CONFIG_CMD_REGULATOR=y >> +CONFIG_CMD_AVB=y >> CONFIG_OF_CONTROL=y >> CONFIG_SYS_RELOC_GD_ENV_ADDR=y >> CONFIG_NET_RANDOM_ETHADDR=y >> @@ -35,6 +42,11 @@ CONFIG_ADC=y >> CONFIG_SARADC_MESON=y >> CONFIG_BUTTON=y >> CONFIG_BUTTON_ADC=y >> +CONFIG_USB_FUNCTION_FASTBOOT=y >> +CONFIG_FASTBOOT_BUF_ADDR=0x6000000 >> +CONFIG_FASTBOOT_FLASH=y >> +CONFIG_FASTBOOT_FLASH_MMC_DEV=2 >> +CONFIG_FASTBOOT_CMD_OEM_FORMAT=y >> CONFIG_DM_I2C=y >> CONFIG_SYS_I2C_MESON=y >> CONFIG_MMC_MESON_GX=y >> @@ -79,7 +91,6 @@ CONFIG_USB_GADGET_VENDOR_NUM=0x1b8e >> CONFIG_USB_GADGET_PRODUCT_NUM=0xfada >> CONFIG_USB_GADGET_DWC2_OTG=y >> CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8=y >> -CONFIG_USB_GADGET_DOWNLOAD=y >> CONFIG_DM_VIDEO=y >> # CONFIG_VIDEO_BPP8 is not set >> # CONFIG_VIDEO_BPP16 is not set >> @@ -92,4 +103,5 @@ CONFIG_VIDEO_BMP_RLE8=y >> CONFIG_BMP_16BPP=y >> CONFIG_BMP_24BPP=y >> CONFIG_BMP_32BPP=y >> +CONFIG_LIBAVB=y >> CONFIG_OF_LIBFDT_OVERLAY=y >> diff --git a/configs/khadas-vim3l_android_defconfig b/configs/khadas-vim3l_android_defconfig >> index 9d94c31891..5eed79b263 100644 >> --- a/configs/khadas-vim3l_android_defconfig >> +++ b/configs/khadas-vim3l_android_defconfig >> @@ -1,9 +1,10 @@ >> CONFIG_ARM=y >> CONFIG_SYS_BOARD="vim3" >> +CONFIG_SYS_CONFIG_NAME="khadas-vim3l_android" >> CONFIG_ARCH_MESON=y >> CONFIG_SYS_TEXT_BASE=0x01000000 >> CONFIG_NR_DRAM_BANKS=1 >> -CONFIG_ENV_SIZE=0x2000 >> +CONFIG_ENV_SIZE=0x10000 >> CONFIG_DM_GPIO=y >> CONFIG_DEFAULT_DEVICE_TREE="meson-sm1-khadas-vim3l" >> CONFIG_MESON_G12A=y >> @@ -15,9 +16,14 @@ CONFIG_SYS_LOAD_ADDR=0x1000000 >> CONFIG_OF_BOARD_SETUP=y >> # CONFIG_DISPLAY_CPUINFO is not set >> CONFIG_MISC_INIT_R=y >> +CONFIG_AVB_VERIFY=y >> # CONFIG_CMD_BDI is not set >> +CONFIG_CMD_ADTIMG=y >> +CONFIG_CMD_ABOOTIMG=y >> # CONFIG_CMD_IMI is not set >> +CONFIG_CMD_BCB=y >> CONFIG_CMD_GPIO=y >> +CONFIG_CMD_GPT=y >> CONFIG_CMD_I2C=y >> # CONFIG_CMD_LOADS is not set >> CONFIG_CMD_MMC=y >> @@ -28,6 +34,7 @@ CONFIG_CMD_USB=y >> CONFIG_CMD_USB_MASS_STORAGE=y >> # CONFIG_CMD_SETEXPR is not set >> CONFIG_CMD_REGULATOR=y >> +CONFIG_CMD_AVB=y >> CONFIG_OF_CONTROL=y >> CONFIG_SYS_RELOC_GD_ENV_ADDR=y >> CONFIG_NET_RANDOM_ETHADDR=y >> @@ -35,6 +42,11 @@ CONFIG_ADC=y >> CONFIG_SARADC_MESON=y >> CONFIG_BUTTON=y >> CONFIG_BUTTON_ADC=y >> +CONFIG_USB_FUNCTION_FASTBOOT=y >> +CONFIG_FASTBOOT_BUF_ADDR=0x6000000 >> +CONFIG_FASTBOOT_FLASH=y >> +CONFIG_FASTBOOT_FLASH_MMC_DEV=2 >> +CONFIG_FASTBOOT_CMD_OEM_FORMAT=y >> CONFIG_DM_I2C=y >> CONFIG_SYS_I2C_MESON=y >> CONFIG_MMC_MESON_GX=y >> @@ -79,7 +91,6 @@ CONFIG_USB_GADGET_VENDOR_NUM=0x1b8e >> CONFIG_USB_GADGET_PRODUCT_NUM=0xfada >> CONFIG_USB_GADGET_DWC2_OTG=y >> CONFIG_USB_GADGET_DWC2_OTG_PHY_BUS_WIDTH_8=y >> -CONFIG_USB_GADGET_DOWNLOAD=y >> CONFIG_DM_VIDEO=y >> # CONFIG_VIDEO_BPP8 is not set >> # CONFIG_VIDEO_BPP16 is not set >> @@ -92,4 +103,5 @@ CONFIG_VIDEO_BMP_RLE8=y >> CONFIG_BMP_16BPP=y >> CONFIG_BMP_24BPP=y >> CONFIG_BMP_32BPP=y >> +CONFIG_LIBAVB=y >> CONFIG_OF_LIBFDT_OVERLAY=y >> diff --git a/include/configs/khadas-vim3_android.h b/include/configs/khadas-vim3_android.h >> new file mode 100644 >> index 0000000000..a6f6dd0a7a >> --- /dev/null >> +++ b/include/configs/khadas-vim3_android.h >> @@ -0,0 +1,34 @@ >> +/* SPDX-License-Identifier: GPL-2.0+ */ >> +/* >> + * Configuration for the khadas VIM3 Android >> + * >> + * Copyright (C) 2021 Baylibre, SAS >> + * Author: Guillaume LA ROQUE >> + */ >> + >> +#ifndef __CONFIG_H >> +#define __CONFIG_H >> + >> +#define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" >> +#define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" >> + >> +#define PARTS_DEFAULT \ >> + "uuid_disk=${uuid_gpt_disk};" \ >> + "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ >> + "name=misc,size=512K,uuid=${uuid_gpt_misc};" \ >> + "name=dtbo,size=8M,uuid=${uuid_gpt_dtbo};" \ >> + "name=vbmeta,size=512K,uuid=${uuid_gpt_vbmeta};" \ >> + "name=boot,size=32M,bootable,uuid=${uuid_gpt_boot};" \ >> + "name=recovery,size=32M,uuid=${uuid_gpt_recovery};" \ >> + "name=cache,size=256M,uuid=${uuid_gpt_cache};" \ >> + "name=super,size=1792M,uuid=${uuid_gpt_super};" \ >> + "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ >> + "name=rootfs,size=-,uuid=" ROOT_UUID >> + >> +#define EXTRA_ANDROID_ENV_SETTINGS \ >> + "board=vim3\0" \ >> + "board_name=vim3\0" \ >> + >> +#include >> + >> +#endif /* __CONFIG_H */ >> diff --git a/include/configs/khadas-vim3l_android.h b/include/configs/khadas-vim3l_android.h >> new file mode 100644 >> index 0000000000..7affc3e448 >> --- /dev/null >> +++ b/include/configs/khadas-vim3l_android.h >> @@ -0,0 +1,34 @@ >> +/* SPDX-License-Identifier: GPL-2.0+ */ >> +/* >> + * Configuration for the khadas VIM3L Android >> + * >> + * Copyright (C) 2021 Baylibre, SAS >> + * Author: Guillaume LA ROQUE >> + */ >> + >> +#ifndef __CONFIG_H >> +#define __CONFIG_H >> + >> +#define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" >> +#define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" >> + >> +#define PARTS_DEFAULT \ >> + "uuid_disk=${uuid_gpt_disk};" \ >> + "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ >> + "name=misc,size=512K,uuid=${uuid_gpt_misc};" \ >> + "name=dtbo,size=8M,uuid=${uuid_gpt_dtbo};" \ >> + "name=vbmeta,size=512K,uuid=${uuid_gpt_vbmeta};" \ >> + "name=boot,size=32M,bootable,uuid=${uuid_gpt_boot};" \ >> + "name=recovery,size=32M,uuid=${uuid_gpt_recovery};" \ >> + "name=cache,size=256M,uuid=${uuid_gpt_cache};" \ >> + "name=super,size=1792M,uuid=${uuid_gpt_super};" \ >> + "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ >> + "name=rootfs,size=-,uuid=" ROOT_UUID >> + >> +#define EXTRA_ANDROID_ENV_SETTINGS \ >> + "board=vim3l\0" \ >> + "board_name=vim3l\0" \ >> + >> +#include >> + >> +#endif /* __CONFIG_H */ >> diff --git a/include/configs/meson64_android.h b/include/configs/meson64_android.h >> index 56dc9fcb07..0303d187de 100644 >> --- a/include/configs/meson64_android.h >> +++ b/include/configs/meson64_android.h >> @@ -104,6 +104,12 @@ >> "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;" \ >> "else " \ >> "echo Error: Android boot is not supported for $board_name; " \ >> "exit; " \ >> @@ -117,6 +123,12 @@ >> "elif test $board_name = sei610; then " \ >> "echo \" Reading DTBO for sei610...\"; " \ >> "setenv dtbo_index 1;" \ >> + "elif test $board_name = vim3l; then " \ >> + "echo \" Reading DTBO for vim3l...\"; " \ >> + "setenv dtbo_index 2;" \ >> + "elif test $board_name = vim3; then " \ >> + "echo \" Reading DTBO for vim3...\"; " \ >> + "setenv dtbo_index 3;" \ >> "else " \ >> "echo Error: Android boot is not supported for $board_name; " \ >> "exit; " \ >> > > I think you should squash this one with the previous one. Will do in v2. > > Neil