[U-Boot] [PATCH 0/6] Add support for the BK4R1 variant of PCM052

[Stefano Babic]

Hi Albert,

On 26/09/2016 09:08, Albert ARIBAUD (3ADEV) wrote:
> BK4R1 is basically PCM052 with the following differences
> or quirks:
> 
>   1) it has 512MB of DDR using MT41K256M16HA_125IT,
>      while the PCM052 has 256MB using MT41J128M16HA_15EIT;
> 
>   2) it has 1GB of NAND. The size increase is supported
>      by the env directly;
> 
>   3) its Ethernet ports are physicaly tied together until
>      GPIO 122 is raised. As this is a safety feature U-Boot
>      does not untie the ports except if it needs networking,
>      for instance when doing NAND updates via TFTP;
> 
>   4) it has a USB hub which may remain in reset if GPIO 130
>      is not raised. This is done unconditionally at boot;
> 
>   5) It has two NOR SPI flash chips on QSPI.
> 
> This series has been run through checkpatch and has no errors
> or warning except the following one:
> 
>         warning: arch/arm/Kconfig,681: please write a
>         paragraph that describes the config symbol fully
> 
> Which I believe does not apply, as target configs in this file
> never have descriptions.
> 
> 
> Albert ARIBAUD (3ADEV) (6):
>   pcm052: fix MTD partitioning
>   pcm052: remove target-specific dtb name from env
>   pcm052: add 'm4go' command
>   tools: mkimage: add support for Vybrid image format
>   pcm052: allow specifying onboard DDR size in configs
>   pcm052: add new BK4r1 target based on PCM052 SoM
> 
>  Makefile                          |   6 ++
>  arch/arm/Kconfig                  |   4 +
>  arch/arm/config.mk                |   3 +
>  arch/arm/cpu/armv7/vf610/Makefile |   5 +
>  arch/arm/dts/Makefile             |   3 +-
>  arch/arm/dts/bk4r1.dts            |  48 +++++++++
>  arch/arm/dts/vf.dtsi              |   4 +-
>  board/phytec/pcm052/Kconfig       |  24 +++++
>  board/phytec/pcm052/pcm052.c      | 206 ++++++++++++++++++++++++++++----------
>  common/image.c                    |   1 +
>  configs/bk4r1_defconfig           |  32 ++++++
>  include/configs/bk4r1.h           |  33 ++++++
>  include/configs/pcm052.h          |  78 ++++++++++-----
>  include/image.h                   |   1 +
>  tools/Makefile                    |   1 +
>  tools/vybridimage.c               | 164 ++++++++++++++++++++++++++++++
>  16 files changed, 535 insertions(+), 78 deletions(-)
>  create mode 100644 arch/arm/dts/bk4r1.dts
>  create mode 100644 configs/bk4r1_defconfig
>  create mode 100644 include/configs/bk4r1.h
>  create mode 100644 tools/vybridimage.c
> 

It looks like that CONFIG_CMD_UBI for bk4r1 is not set and I get build
errors. Is it ok for you if I add directly this by applying ?

diff --git a/configs/bk4r1_defconfig b/configs/bk4r1_defconfig
index 3994459..26d9e81 100644
--- a/configs/bk4r1_defconfig
+++ b/configs/bk4r1_defconfig
@@ -30,3 +30,4 @@ CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_MTD=y
 CONFIG_CMD_DM=y
+CONFIG_CMD_UBI=y

Regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================