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

[Jan Kiszka <jan.kiszka@siemens.com>]

On 2018-06-07 19:59, Maxim Yu. Osipov wrote:
> Hi Jan,
> 
> I've tested the resulting image on Banana Pi.
> Banana Pi boots OK - I get login prompt.
> The only complain during booting is from fsck:
> 
>>>>>
> Begin: Loading essential drivers ... done.
> Begin: Running /scripts/init-premount ... done.
> Begin: Mounting root file system ... Begin: Running /scripts/local-top
> ... done.
> Begin: Running /scripts/local-premount ... done.
> [    7.536178] random: fast init done
> Begin: Will now check root file system ... fsck from util-linux 2.29.2
> fsck: error 2 (No such file or directory) while executing fsck.ext4 for
> /dev/mmcblk0p1
> fsck exited with status code 8
> done.
> Warning: File system check failed but did not detect errors
> [   12.782364] EXT4-fs (mmcblk0p1): mounted filesystem with ordered data
> mode. Opts: (null)
> done.
> Begin: Running /scripts/local-bottom ... done.
> Begin: Running /scripts/init-bottom ... done.
> [   13.431139] systemd[1]: System time before build time, advancing clock.
> [   13.582647] systemd[1]: systemd 232 running in system mode. (+PAM
> +AUDIT +SELINUX +IMA +APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP
> +GCRYPT +GNUTLS +ACL +XZ +LZ4 +SECCOMP +BLKID +ELFUTILS +KMOD +IDN)
> [   13.614171] systemd[1]: Detected architecture arm.
> 
> Welcome to Debian GNU/Linux 9 (stretch)!
> <<<<<

I think I saw this flying by as well but didn't look into details. I
assume we are missing the corresponding fschk on the base image.

Jan

> 
> Kind regards,
> Maxim.
> 
> 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!
>>
>> 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
>>
> 
> 


-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux