On 21.01.2019 15:22, Lukasz Majewski wrote:
On Mon, 21 Jan 2019 09:33:23 +0100 Lukasz Majewski lukma@denx.de wrote:
Hi Stefan,
This patch series converts PCM052 and BK4 to use Driver Model and Device Tree.
Some notable changes:
- The way how MAC address is read from fuses can now be adjusted
- DTS improvement/sync with kernel (also extract u-boot specific
properties)
- Using generic code instead of one from board
The output of u-boot boot: U-Boot 2019.01-rc3-00076-gc149229be0 (Jan 14 2019 - 08:38:48 +0100)
CPU: Freescale Vybrid VF610 at 396 MHz Reset cause: POWER ON RESET Model: Liebherr (LVF) BK4 Vybrid Board Board: BK4r1 (L333) DRAM: 512 MiB NAND: 1024 MiB MMC: FSL_SDHC: 0 Loading Environment from NAND... OK In: serial@40028000 Out: serial@40028000 Err: serial@40028000 Net: eth0: fec@400d0000, eth1: fec@400d1000 Enter passphrase to stop autoboot, booting in 3 seconds
Buildman CI: ./tools/buildman/buildman.py --branch=HEAD~4 vf610 mx6 vybrid --detail --verbose --show_errors --force-build --count=21 --output-dir=../BUILD/
Travis-CI: https://travis-ci.org/lmajewski/u-boot-dfu/jobs/481963146
I've bisected this error, and it turned out that the following patch is the culprit (on top of which I've rebased my work):
"fs: fat: dynamically allocate memory for temporary buffer"
Without this patch applied: https://travis-ci.org/lmajewski/u-boot-dfu/builds/482301236
With: https://travis-ci.org/lmajewski/u-boot-dfu/builds/482297604
Test: test/py qemu_arm
Error is in: "test_efi_selftest"
Executing 'block device' /home/travis/build/lmajewski/u-boot-dfu/lib/efi_selftest/efi_selftest_block_device.c(386): TODO: Wrong volume label 'xxa1', expected 'U-BOOT TEST' /home/travis/build/lmajewski/u-boot-dfu/lib/efi_selftest/efi_selftest_block_device.c(413): ERROR: Unexpected file content /home/travis/build/lmajewski/u-boot-dfu/lib/efi_selftest/efi_selftest.c(110): ERROR: Executing 'block device' failed
And the full travis-ci build without the above patch (all "green"):
I skimmed through the rest of the patchset but did not do a full review. But I am ok with the rest of the patches.
I also tested the patchset on our Colibri VF50, and U-Boot seems to work fine!
Thanks for your work!
-- Stefan