Re: [PATCH v2 0/6] Enable U-Boot-based images, add Banana Pi demo

["Maxim Yu. Osipov" <mosipov@ilbers.de>]

On 06/07/2018 04:30 PM, Jan Kiszka wrote:
> Changes in v2:
>   - use booti on aarch64 in generated boot.scr
>   - simplify boot script generator internally (fstab parsing, ramdisk
>     size evaluation)
>   - use options for setup_root_file_system()
> 
> Original cover:
> 
> This provides missing bits to build SD-card (or also eMMC) images for
> boards that use U-Boot as bootloader. The images can be described via
> well-known wks files.
> 
> For U-Boot, we rely on scan_dev_for_scripts from the default
> environment. The reason to generate our own boot script is documented in
> the related patch. While that generator may start its life inside Isar,
> I could imagine selling it also to Debian as a seperate package.
> 
> As an example, I've added the necessary configs to build a basic image
> for a Banana Pi board.
> 
> This is surely not the smoothest solution. E.g., I dislike that we still
> need those ugly ROOTFS_TYPE and ROOTFS_DEV variables - the information
> is encoded in the wks file already. But those files are evaluated by wic
> only AFTER we build the rootfs. So, extracting that information from wks
> would require logic like in u-boot-script_1.0.bb, i.e. a certain level
> wic duplication so that we can continue to generate /etc/fstab ahead of
> the package installation. Better suggestions welcome.
> 
> We should probably also convert BUILDCHROOT_PREINSTALL_WIC to
> WIC_FILE_DEPENDS* as indicated in patch 2. But I didn't study the
> implications in details yet.
> 
> Looking forward to comments and tests!

Applied to the 'next',

Thanks,
Maxim.

> Jan
> 
> Jan Kiszka (6):
>    Add DISTRO and DISTRO_ARCH as OVERRIDES suffixes
>    buildchroot: Break up wic build deps into common and arch-specific
>      ones
>    isar-bootstrap: Use options for setup_root_file_system()
>    Set up target image fstab prior to installing any packages
>    Add u-boot script generator
>    Add Banana Pi SD-card image
> 
>   .../conf/distro/debian-stretch-backports.list      |  1 +
>   meta-isar/conf/local.conf.sample                   |  1 +
>   meta-isar/conf/multiconfig/bananapi-stretch.conf   | 24 +++++++++
>   .../conf/multiconfig/preferences.bananapi.conf     |  7 +++
>   .../images/files/debian-configscript.sh            | 14 ------
>   .../images/files/raspbian-configscript.sh          | 14 ------
>   meta-isar/recipes-core/images/isar-image-base.bb   | 20 ++++++--
>   meta-isar/scripts/lib/wic/canned-wks/bananapi.wks  |  5 ++
>   meta/classes/isar-bootstrap-helper.bbclass         | 18 +++++--
>   meta/conf/isar-bitbake.conf                        |  2 +
>   meta/recipes-bsp/u-boot-script/files/u-boot-script |  9 ++++
>   .../u-boot-script/files/update-u-boot-script       | 57 ++++++++++++++++++++++
>   .../u-boot-script/files/zz-u-boot-script           |  3 ++
>   .../recipes-bsp/u-boot-script/u-boot-script_1.0.bb | 53 ++++++++++++++++++++
>   meta/recipes-devtools/buildchroot/buildchroot.bb   | 23 ++++++---
>   15 files changed, 208 insertions(+), 43 deletions(-)
>   create mode 100644 meta-isar/conf/distro/debian-stretch-backports.list
>   create mode 100644 meta-isar/conf/multiconfig/bananapi-stretch.conf
>   create mode 100644 meta-isar/conf/multiconfig/preferences.bananapi.conf
>   create mode 100644 meta-isar/scripts/lib/wic/canned-wks/bananapi.wks
>   create mode 100644 meta/recipes-bsp/u-boot-script/files/u-boot-script
>   create mode 100755 meta/recipes-bsp/u-boot-script/files/update-u-boot-script
>   create mode 100755 meta/recipes-bsp/u-boot-script/files/zz-u-boot-script
>   create mode 100644 meta/recipes-bsp/u-boot-script/u-boot-script_1.0.bb
> 


-- 
Maxim Osipov
ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn
Germany
+49 (151) 6517 6917
mosipov@ilbers.de
http://ilbers.de/
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov