Message ID | 20211119100829.843139-6-mkorpershoek@baylibre.com |
---|---|
State | Superseded |
Delegated to: | Neil Armstrong |
Headers | show |
Series | meson64_android: add support for vim3/vim3l boards | expand |
Hi, On 19/11/2021 11:08, Mattijs Korpershoek wrote: > meson64_android.h also relies on CMD_AB_SELECT so enable that as well. > > Signed-off-by: Guillaume La Roque <glaroque@baylibre.com> > Signed-off-by: Mattijs Korpershoek <mkorpershoek@baylibre.com> > --- > configs/khadas-vim3_android_ab_defconfig | 2 ++ > configs/khadas-vim3l_android_ab_defconfig | 2 ++ > include/configs/khadas-vim3_android.h | 16 ++++++++++++++++ > include/configs/khadas-vim3l_android.h | 16 ++++++++++++++++ > 4 files changed, 36 insertions(+) > > diff --git a/configs/khadas-vim3_android_ab_defconfig b/configs/khadas-vim3_android_ab_defconfig > index 9305a54b1b..88325c37db 100644 > --- a/configs/khadas-vim3_android_ab_defconfig > +++ b/configs/khadas-vim3_android_ab_defconfig > @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y > # CONFIG_DISPLAY_CPUINFO is not set > CONFIG_MISC_INIT_R=y > CONFIG_AVB_VERIFY=y > +CONFIG_ANDROID_AB=y > # CONFIG_CMD_BDI is not set > CONFIG_CMD_ADTIMG=y > CONFIG_CMD_ABOOTIMG=y > @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y > CONFIG_CMD_USB=y > CONFIG_CMD_USB_MASS_STORAGE=y > # CONFIG_CMD_SETEXPR is not set > +CONFIG_CMD_AB_SELECT=y > CONFIG_CMD_REGULATOR=y > CONFIG_CMD_AVB=y > CONFIG_OF_CONTROL=y > diff --git a/configs/khadas-vim3l_android_ab_defconfig b/configs/khadas-vim3l_android_ab_defconfig > index 5eed79b263..608b54dca2 100644 > --- a/configs/khadas-vim3l_android_ab_defconfig > +++ b/configs/khadas-vim3l_android_ab_defconfig > @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y > # CONFIG_DISPLAY_CPUINFO is not set > CONFIG_MISC_INIT_R=y > CONFIG_AVB_VERIFY=y > +CONFIG_ANDROID_AB=y > # CONFIG_CMD_BDI is not set > CONFIG_CMD_ADTIMG=y > CONFIG_CMD_ABOOTIMG=y > @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y > CONFIG_CMD_USB=y > CONFIG_CMD_USB_MASS_STORAGE=y > # CONFIG_CMD_SETEXPR is not set > +CONFIG_CMD_AB_SELECT=y > CONFIG_CMD_REGULATOR=y > CONFIG_CMD_AVB=y > CONFIG_OF_CONTROL=y > diff --git a/include/configs/khadas-vim3_android.h b/include/configs/khadas-vim3_android.h > index a6f6dd0a7a..da6adf6c41 100644 > --- a/include/configs/khadas-vim3_android.h > +++ b/include/configs/khadas-vim3_android.h > @@ -12,6 +12,21 @@ > #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" > #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" > > +#if defined(CONFIG_CMD_AB_SELECT) > +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ > + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ > + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ > + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ > + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ > + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ > + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ > + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ > + "name=rootfs,size=-,uuid=" ROOT_UUID > +#else > #define PARTS_DEFAULT \ > "uuid_disk=${uuid_gpt_disk};" \ > "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ > @@ -24,6 +39,7 @@ > "name=super,size=1792M,uuid=${uuid_gpt_super};" \ > "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ > "name=rootfs,size=-,uuid=" ROOT_UUID > +#endif > > #define EXTRA_ANDROID_ENV_SETTINGS \ > "board=vim3\0" \ > diff --git a/include/configs/khadas-vim3l_android.h b/include/configs/khadas-vim3l_android.h > index 7affc3e448..b1768e2d82 100644 > --- a/include/configs/khadas-vim3l_android.h > +++ b/include/configs/khadas-vim3l_android.h > @@ -12,6 +12,21 @@ > #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" > #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" > > +#if defined(CONFIG_CMD_AB_SELECT) > +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ > + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ > + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ > + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ > + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ > + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ > + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ > + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ > + "name=rootfs,size=-,uuid=" ROOT_UUID > +#else > #define PARTS_DEFAULT \ > "uuid_disk=${uuid_gpt_disk};" \ > "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ > @@ -24,6 +39,7 @@ > "name=super,size=1792M,uuid=${uuid_gpt_super};" \ > "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ > "name=rootfs,size=-,uuid=" ROOT_UUID > +#endif > > #define EXTRA_ANDROID_ENV_SETTINGS \ > "board=vim3l\0" \ > You should move this khadas-vim3l_android.h change adding AB support before adding defconfig for AB, move this patch in position 4 and add CONFIG_*AB*=y in the XXXXX_android_ab_defconfig configs you add. Neil
Hi Neil, Thank you for your review. Neil Armstrong <narmstrong@baylibre.com> writes: > Hi, > > On 19/11/2021 11:08, Mattijs Korpershoek wrote: >> meson64_android.h also relies on CMD_AB_SELECT so enable that as well. >> >> Signed-off-by: Guillaume La Roque <glaroque@baylibre.com> >> Signed-off-by: Mattijs Korpershoek <mkorpershoek@baylibre.com> >> --- >> configs/khadas-vim3_android_ab_defconfig | 2 ++ >> configs/khadas-vim3l_android_ab_defconfig | 2 ++ >> include/configs/khadas-vim3_android.h | 16 ++++++++++++++++ >> include/configs/khadas-vim3l_android.h | 16 ++++++++++++++++ >> 4 files changed, 36 insertions(+) >> >> diff --git a/configs/khadas-vim3_android_ab_defconfig b/configs/khadas-vim3_android_ab_defconfig >> index 9305a54b1b..88325c37db 100644 >> --- a/configs/khadas-vim3_android_ab_defconfig >> +++ b/configs/khadas-vim3_android_ab_defconfig >> @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y >> # CONFIG_DISPLAY_CPUINFO is not set >> CONFIG_MISC_INIT_R=y >> CONFIG_AVB_VERIFY=y >> +CONFIG_ANDROID_AB=y >> # CONFIG_CMD_BDI is not set >> CONFIG_CMD_ADTIMG=y >> CONFIG_CMD_ABOOTIMG=y >> @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y >> CONFIG_CMD_USB=y >> CONFIG_CMD_USB_MASS_STORAGE=y >> # CONFIG_CMD_SETEXPR is not set >> +CONFIG_CMD_AB_SELECT=y >> CONFIG_CMD_REGULATOR=y >> CONFIG_CMD_AVB=y >> CONFIG_OF_CONTROL=y >> diff --git a/configs/khadas-vim3l_android_ab_defconfig b/configs/khadas-vim3l_android_ab_defconfig >> index 5eed79b263..608b54dca2 100644 >> --- a/configs/khadas-vim3l_android_ab_defconfig >> +++ b/configs/khadas-vim3l_android_ab_defconfig >> @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y >> # CONFIG_DISPLAY_CPUINFO is not set >> CONFIG_MISC_INIT_R=y >> CONFIG_AVB_VERIFY=y >> +CONFIG_ANDROID_AB=y >> # CONFIG_CMD_BDI is not set >> CONFIG_CMD_ADTIMG=y >> CONFIG_CMD_ABOOTIMG=y >> @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y >> CONFIG_CMD_USB=y >> CONFIG_CMD_USB_MASS_STORAGE=y >> # CONFIG_CMD_SETEXPR is not set >> +CONFIG_CMD_AB_SELECT=y >> CONFIG_CMD_REGULATOR=y >> CONFIG_CMD_AVB=y >> CONFIG_OF_CONTROL=y >> diff --git a/include/configs/khadas-vim3_android.h b/include/configs/khadas-vim3_android.h >> index a6f6dd0a7a..da6adf6c41 100644 >> --- a/include/configs/khadas-vim3_android.h >> +++ b/include/configs/khadas-vim3_android.h >> @@ -12,6 +12,21 @@ >> #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" >> #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" >> >> +#if defined(CONFIG_CMD_AB_SELECT) >> +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ >> + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ >> + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ >> + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ >> + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ >> + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ >> + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ >> + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ >> + "name=rootfs,size=-,uuid=" ROOT_UUID >> +#else >> #define PARTS_DEFAULT \ >> "uuid_disk=${uuid_gpt_disk};" \ >> "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ >> @@ -24,6 +39,7 @@ >> "name=super,size=1792M,uuid=${uuid_gpt_super};" \ >> "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ >> "name=rootfs,size=-,uuid=" ROOT_UUID >> +#endif >> >> #define EXTRA_ANDROID_ENV_SETTINGS \ >> "board=vim3\0" \ >> diff --git a/include/configs/khadas-vim3l_android.h b/include/configs/khadas-vim3l_android.h >> index 7affc3e448..b1768e2d82 100644 >> --- a/include/configs/khadas-vim3l_android.h >> +++ b/include/configs/khadas-vim3l_android.h >> @@ -12,6 +12,21 @@ >> #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" >> #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" >> >> +#if defined(CONFIG_CMD_AB_SELECT) >> +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ >> + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ >> + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ >> + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ >> + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ >> + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ >> + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ >> + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ >> + "name=rootfs,size=-,uuid=" ROOT_UUID >> +#else >> #define PARTS_DEFAULT \ >> "uuid_disk=${uuid_gpt_disk};" \ >> "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ >> @@ -24,6 +39,7 @@ >> "name=super,size=1792M,uuid=${uuid_gpt_super};" \ >> "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ >> "name=rootfs,size=-,uuid=" ROOT_UUID >> +#endif >> >> #define EXTRA_ANDROID_ENV_SETTINGS \ >> "board=vim3l\0" \ >> > > > You should move this khadas-vim3l_android.h change adding AB support before adding defconfig for AB, > move this patch in position 4 and add CONFIG_*AB*=y in the XXXXX_android_ab_defconfig configs > you add. Will do in v2. > > Neil
diff --git a/configs/khadas-vim3_android_ab_defconfig b/configs/khadas-vim3_android_ab_defconfig index 9305a54b1b..88325c37db 100644 --- a/configs/khadas-vim3_android_ab_defconfig +++ b/configs/khadas-vim3_android_ab_defconfig @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y # CONFIG_DISPLAY_CPUINFO is not set CONFIG_MISC_INIT_R=y CONFIG_AVB_VERIFY=y +CONFIG_ANDROID_AB=y # CONFIG_CMD_BDI is not set CONFIG_CMD_ADTIMG=y CONFIG_CMD_ABOOTIMG=y @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y CONFIG_CMD_USB=y CONFIG_CMD_USB_MASS_STORAGE=y # CONFIG_CMD_SETEXPR is not set +CONFIG_CMD_AB_SELECT=y CONFIG_CMD_REGULATOR=y CONFIG_CMD_AVB=y CONFIG_OF_CONTROL=y diff --git a/configs/khadas-vim3l_android_ab_defconfig b/configs/khadas-vim3l_android_ab_defconfig index 5eed79b263..608b54dca2 100644 --- a/configs/khadas-vim3l_android_ab_defconfig +++ b/configs/khadas-vim3l_android_ab_defconfig @@ -17,6 +17,7 @@ CONFIG_OF_BOARD_SETUP=y # CONFIG_DISPLAY_CPUINFO is not set CONFIG_MISC_INIT_R=y CONFIG_AVB_VERIFY=y +CONFIG_ANDROID_AB=y # CONFIG_CMD_BDI is not set CONFIG_CMD_ADTIMG=y CONFIG_CMD_ABOOTIMG=y @@ -33,6 +34,7 @@ CONFIG_CMD_SPI=y CONFIG_CMD_USB=y CONFIG_CMD_USB_MASS_STORAGE=y # CONFIG_CMD_SETEXPR is not set +CONFIG_CMD_AB_SELECT=y CONFIG_CMD_REGULATOR=y CONFIG_CMD_AVB=y CONFIG_OF_CONTROL=y diff --git a/include/configs/khadas-vim3_android.h b/include/configs/khadas-vim3_android.h index a6f6dd0a7a..da6adf6c41 100644 --- a/include/configs/khadas-vim3_android.h +++ b/include/configs/khadas-vim3_android.h @@ -12,6 +12,21 @@ #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" +#if defined(CONFIG_CMD_AB_SELECT) +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ + "name=rootfs,size=-,uuid=" ROOT_UUID +#else #define PARTS_DEFAULT \ "uuid_disk=${uuid_gpt_disk};" \ "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ @@ -24,6 +39,7 @@ "name=super,size=1792M,uuid=${uuid_gpt_super};" \ "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ "name=rootfs,size=-,uuid=" ROOT_UUID +#endif #define EXTRA_ANDROID_ENV_SETTINGS \ "board=vim3\0" \ diff --git a/include/configs/khadas-vim3l_android.h b/include/configs/khadas-vim3l_android.h index 7affc3e448..b1768e2d82 100644 --- a/include/configs/khadas-vim3l_android.h +++ b/include/configs/khadas-vim3l_android.h @@ -12,6 +12,21 @@ #define LOGO_UUID "43a3305d-150f-4cc9-bd3b-38fca8693846;" #define ROOT_UUID "ddb8c3f6-d94d-4394-b633-3134139cc2e0;" +#if defined(CONFIG_CMD_AB_SELECT) +#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_a,size=8M,uuid=${uuid_gpt_dtbo_a};" \ + "name=dtbo_b,size=8M,uuid=${uuid_gpt_dtbo_b};" \ + "name=vbmeta_a,size=512K,uuid=${uuid_gpt_vbmeta_a};" \ + "name=vbmeta_b,size=512K,uuid=${uuid_gpt_vbmeta_b};" \ + "name=boot_a,size=32M,bootable,uuid=${uuid_gpt_boot_a};" \ + "name=boot_b,size=32M,bootable,uuid=${uuid_gpt_boot_b};" \ + "name=super,size=3072M,uuid=${uuid_gpt_super};" \ + "name=userdata,size=11282M,uuid=${uuid_gpt_userdata};" \ + "name=rootfs,size=-,uuid=" ROOT_UUID +#else #define PARTS_DEFAULT \ "uuid_disk=${uuid_gpt_disk};" \ "name=logo,start=512K,size=2M,uuid=" LOGO_UUID \ @@ -24,6 +39,7 @@ "name=super,size=1792M,uuid=${uuid_gpt_super};" \ "name=userdata,size=12786M,uuid=${uuid_gpt_userdata};" \ "name=rootfs,size=-,uuid=" ROOT_UUID +#endif #define EXTRA_ANDROID_ENV_SETTINGS \ "board=vim3l\0" \