[U-Boot] [PATCH v2 00/21] imx: vybrid: Update BK4 and PCM052 boards to only use DM/DTS

Mon Jan 21 08:33:23 UTC 2019

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 at 40028000
> Out:   serial at 40028000
> Err:   serial at 40028000
> Net:   eth0: fec at 400d0000, eth1: fec at 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

> 
> U-boot master branch: SHA1: c4d323793ba2e0616d93ca104e1e2b9a9fbccf9b
> 
> 
> Changes in v2:
>     - Rename pcm052.dts to vf610-pcm052.dts
>     - Rename bk4r1.dts to vf610-bk4r1.dts
>     - Extract 'u-boot,dm-pre-reloc;' property to separate file (to
> facilitate sync with Linux kernel dts files)
>         - New patch
>         - Disable EFI related support and commands (as we do not plan
> to use EFI on this setup)
> 
> Lukasz Majewski (21):
>   net: FEC: Add compatible for vybrid (vf610) to reuse fec_mxc.c
> driver net: Kconfig: FEC: Add dependency on VF610
>   vybrid: ddr: Extend vf610-pinfunc.h with DDR pads definitions
>   vybrid: clock: Provide enable_i2c_clk() function for Vybrid
>   vybrid: Define the imx_get_mac_from_fuse() as a __weak function
>   pcm052: board: Remove "m4go" command as it is superseded by
> "bootaux" pcm052: board: vybrid: Update the board name for BK4 device
>   ARM: DTS: vybrid: Update vf.dtsi file to descibe more vf610 hardware
>   pcm052: board: cosmetic: Add copyright notice to pcm052.c
>   ARM: DTS: Update pcm052 based dts files (bk4r1/pcm052)
>   ARM: DTS: Provide vf610-bk4r1-u-boot.dtsi for U-Boot specific
>     properties
>   defconfig: bk4/pcm052: Update bk4r1|pcm052_defconfig to support
> DM/DT config: pcm052: Use SZ_X{MK} from linux/sizes.h for
>     include/configs/pcm052.h
>   config: bk4: Update include/configs/bk4r1.h file
>   pcm052: board: Remove in-board setup code (it is now replaced by DM
>     setup)
>   config: bk4: Update u-boot script to support recovery via SD card
>   config: bk4: Update u-boot envs to support NOR memories initial
> setup pcm052: bk4: sdcard: Add support for SD card booting/recovery
>   pcm052: board: Add code to setup LED default states
>   pcm052: mac: Provide board specific imx_get_mac_from_fuse() function
>   pcm052: bk4: Add board_phy_config() for BK4 to setup ksz8081 phy
> 
>  arch/arm/cpu/armv7/vf610/Kconfig              |   1 +
>  arch/arm/cpu/armv7/vf610/generic.c            |  21 +-
>  arch/arm/dts/Makefile                         |   4 +-
>  arch/arm/dts/bk4r1.dts                        |  47 ----
>  arch/arm/dts/vf.dtsi                          |  62 +++++
>  arch/arm/dts/vf610-bk4r1-u-boot.dtsi          |  27 ++
>  arch/arm/dts/vf610-bk4r1.dts                  |  97 +++++++
>  arch/arm/dts/{pcm052.dts => vf610-pcm052.dts} |   6 +-
>  arch/arm/dts/vf610-pcm052.dtsi                | 259 +++++++++++++++++
>  arch/arm/dts/vf610-pinfunc.h                  |  50 ++++
>  arch/arm/include/asm/arch-vf610/clock.h       |   3 +
>  arch/arm/include/asm/arch-vf610/imx-regs.h    |   2 +
>  board/phytec/pcm052/pcm052.c                  | 386
> ++++++--------------------
> configs/bk4r1_defconfig                       |  49 +++-
> configs/pcm052_defconfig                      |  36 ++-
> drivers/net/Kconfig                           |   2 +-
> drivers/net/fec_mxc.c                         |   1 +
> include/configs/bk4r1.h                       | 263
> +++++++++++++++++- include/configs/pcm052.h                      |
> 51 +--- 19 files changed, 956 insertions(+), 411 deletions(-) delete
> mode 100644 arch/arm/dts/bk4r1.dts create mode 100644
> arch/arm/dts/vf610-bk4r1-u-boot.dtsi create mode 100644
> arch/arm/dts/vf610-bk4r1.dts rename arch/arm/dts/{pcm052.dts =>
> vf610-pcm052.dts} (81%) create mode 100644
> arch/arm/dts/vf610-pcm052.dtsi
> 

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190121/4764fda3/attachment.sig>