[U-Boot] [PATCH v2 7/8] env: ti: boot: Boot Android with dynamic partitions

[Sam Protsenko]

Hi Tom,

On Sun, Nov 3, 2019 at 4:35 PM Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Oct 23, 2019 at 05:34:26PM +0300, Sam Protsenko wrote:
>
> > Changes:
> >   - use boot.img instead of boot_fit.img
> >   - use .dtb from boot.img v2
> >   - implement recovery boot
> >   - always boot ramdisk from boot.img, we can't mount system as root
> >     now, as system is a logical partition inside of super partition
> >   - don't add "skip_initramfs" to cmdline anymore
> >   - to boot into recovery, use boot image from recovery partition
> >   - prepare partition table:
> >     - A/B scheme
> >     - use 'super' partition instead of 'system' and 'vendor'
> >     - add dtbo partitions
> >     - introduce metadata partition
> >
> > Not implemented: reading and applying dtbo files from dtbo partition.
> >
> > Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
>
> This breaks at least "dra7xx_evm" building:
>        arm:  +   dra7xx_evm
> +(dra7xx_evm) In file included from include/configs/ti_omap5_common.h:57:0,
> +(dra7xx_evm)                  from include/configs/dra7xx_evm.h:62,
> +(dra7xx_evm)                  from include/config.h:5,
> +(dra7xx_evm)                  from include/common.h:23,
> +(dra7xx_evm)                  from env/common.c:10:
> +(dra7xx_evm) include/environment/ti/boot.h:104:3: error: expected '}' before 'AB_SELECT_SLOT'
> +(dra7xx_evm)    AB_SELECT_SLOT \
> +(dra7xx_evm)    ^
> +(dra7xx_evm) include/configs/ti_omap5_common.h:65:2: note: in expansion of macro 'DEFAULT_COMMON_BOOT_TI_ARGS'
> +(dra7xx_evm)   DEFAULT_COMMON_BOOT_TI_ARGS \
> +(dra7xx_evm)   ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> +(dra7xx_evm) include/env_default.h:111:2: note: in expansion of macro 'CONFIG_EXTRA_ENV_SETTINGS'
> +(dra7xx_evm)   CONFIG_EXTRA_ENV_SETTINGS
> +(dra7xx_evm)   ^~~~~~~~~~~~~~~~~~~~~~~~~
> +(dra7xx_evm) make[2]: *** [env/common.o] Error 1
> +(dra7xx_evm) make[1]: *** [env] Error 2
> +(dra7xx_evm) make[1]: *** wait: No child processes.  Stop.
> +(dra7xx_evm) make: *** [sub-make] Error 2
>

Thanks, will be fixed in new version. I should run buildman more often :)

> --
> Tom