Re: [U-Boot] [PATCH] armv8: layerscape: refine port register configuration

[York Sun]

On 12/04/2017 05:39 PM, Andy Tang wrote:
> Hi York,
> 
> This patch is to adjust the OOB (out of bound) timing of sata port. It is 
> totally hardware timing.
> I was asked to update those timing by hardware/validation team. They 
> calculated those values from clock frequency. You can apply it safely.

Andy, please do not top post. Instead, reply inline with unnecessary
part removed.

Please add details to comments and/or commit message. It will be hard to
maintain the code otherwise.

> 
> Please apply this patch with SATA added.

Please update the subject when you respin this patch.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: ls1046aqds: update IFC NOR timings

[York Sun]

On 11/29/2017 07:02 PM, Wenbin Song wrote:
> Update IFC NOR timings to fix that the NOR flash can not
> be erased with V4 FPGA image on ls1046aqds.
> 
> Signed-off-by: Wenbin Song 
> ---
>  include/configs/ls1046aqds.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/include/configs/ls1046aqds.h b/include/configs/ls1046aqds.h
> index df70bd398f..b619dc7939 100644
> --- a/include/configs/ls1046aqds.h
> +++ b/include/configs/ls1046aqds.h
> @@ -194,10 +194,10 @@ unsigned long get_board_ddr_clk(void);
>  #define CONFIG_SYS_NOR_FTIM1 (FTIM1_NOR_TACO(0x35) | \
>   FTIM1_NOR_TRAD_NOR(0x1a) | \
>   FTIM1_NOR_TSEQRAD_NOR(0x13))
> -#define CONFIG_SYS_NOR_FTIM2 (FTIM2_NOR_TCS(0x4) | \
> - FTIM2_NOR_TCH(0x4) | \
> +#define CONFIG_SYS_NOR_FTIM2 (FTIM2_NOR_TCS(0x8) | \
> + FTIM2_NOR_TCH(0x8) | \
>   FTIM2_NOR_TWPH(0xe) | \
> - FTIM2_NOR_TWP(0x1c))
> + FTIM2_NOR_TWP(0x38))

Wenbin,

You slow it down a lot. Did you actually calculate the timing? Can you
post the numbers? This will impact on NOR flash performance.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] ARM: pxa: Remove unused ifdefs

[Tuomas Tynkkynen]

These ifdefs are protecting #include statements for files that have
never existed. AFAICT this hardware.h has been copied from the kernel
and the ifdefs have never served a role in U-Boot, so delete them.

Signed-off-by: Tuomas Tynkkynen 
---
 arch/arm/include/asm/arch-pxa/hardware.h | 29 -
 scripts/config_whitelist.txt |  6 --
 2 files changed, 35 deletions(-)

diff --git arch/arm/include/asm/arch-pxa/hardware.h 
arch/arm/include/asm/arch-pxa/hardware.h
index e671c143ac..6d0023d7b8 100644
--- arch/arm/include/asm/arch-pxa/hardware.h
+++ arch/arm/include/asm/arch-pxa/hardware.h
@@ -79,33 +79,4 @@
 
 #endif
 
-
-/*
- * Implementation specifics
- */
-
-#ifdef CONFIG_ARCH_LUBBOCK
-#include "lubbock.h"
-#endif
-
-#ifdef CONFIG_ARCH_PXA_IDP
-#include "idp.h"
-#endif
-
-#ifdef CONFIG_ARCH_PXA_CERF
-#include "cerf.h"
-#endif
-
-#ifdef CONFIG_ARCH_CSB226
-#include "csb226.h"
-#endif
-
-#ifdef CONFIG_ARCH_INNOKOM
-#include "innokom.h"
-#endif
-
-#ifdef CONFIG_ARCH_PLEB
-#include "pleb.h"
-#endif
-
 #endif /* _ASM_ARCH_HARDWARE_H */
diff --git scripts/config_whitelist.txt scripts/config_whitelist.txt
index d8d43756b4..b6a1233e82 100644
--- scripts/config_whitelist.txt
+++ scripts/config_whitelist.txt
@@ -45,18 +45,12 @@ CONFIG_APUS_FAST_EXCEPT
 CONFIG_AP_SH4A_4A
 CONFIG_ARCH_ADPAG101P
 CONFIG_ARCH_CPU_INIT
-CONFIG_ARCH_CSB226
 CONFIG_ARCH_HAS_ILOG2_U32
 CONFIG_ARCH_HAS_ILOG2_U64
-CONFIG_ARCH_INNOKOM
 CONFIG_ARCH_KIRKWOOD
-CONFIG_ARCH_LUBBOCK
 CONFIG_ARCH_MAP_SYSMEM
 CONFIG_ARCH_OMAP4
 CONFIG_ARCH_ORION5X
-CONFIG_ARCH_PLEB
-CONFIG_ARCH_PXA_CERF
-CONFIG_ARCH_PXA_IDP
 CONFIG_ARCH_RMOBILE_BOARD_STRING
 CONFIG_ARCH_RMOBILE_EXTRAM_BOOT
 CONFIG_ARCH_TEGRA
-- 
2.15.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-mpc85xx master

[Tom Rini]

On Fri, Dec 08, 2017 at 04:59:39PM +, York Sun wrote:

> Tom,
> 
> The following changes since commit 9da7fb4a39149c3061cb148bfbaa76b4b52b9008:
> 
>   Prepare v2018.01-rc1 (2017-12-04 18:27:17 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-mpc85xx.git
> 
> for you to fetch changes up to 316f0d0f8f3cbeeefda043e69f3e4e6aa57f7f1d:
> 
>   powerpc: mpc85xx: Fix static TLB table for SDRAM (2017-12-06 14:54:12
> -0800)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PULL] Please pull u-boot-rockchip

[Tom Rini]

On Fri, Dec 08, 2017 at 02:45:20PM +0100, Dr. Philipp Tomsich wrote:

> Tom,
> 
> this week has been quiet in the Rockchip tree: I only have a change to
> a snprintf format-string for restoring consistency with Linux.
> 
> Travis-CI results are available at
>   https://travis-ci.org/ptomsich/u-boot-rockchip/builds/313435652
> 
> Thanks,
> Philipp.
> 
> 
> The following changes since commit 6c7010b779ef29a40ed66acc414f65d2f4b75ced:
> 
>   Merge git://git.denx.de/u-boot-fsl-qoriq (2017-12-07 17:56:51 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-rockchip.git master
> 
> for you to fetch changes up to b32b1bd10b86d01a26056f50de89f768e5183ed1:
> 
>   rockchip: rk3399-puma: preserve leading zeros in serial# (2017-12-08 
> 11:49:39 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v2 0/2] arm: ppc: Share DPAA1 fixups between ARCHs

[Ahmed Mansour]

- Support DPAA1 QBMan device tree fixups in a shared location for both
  arm and ppc architectures
- cleanup a define from header files and add as Kconfig with auto
  selection based on SOC

Changes in v2:
- Add get_qman_freq() to replace get_sys_info() for readability
- Correct the copyright year in new files
- Replace !ARM with PPC to wall off PowerPC SOCs specific qman setup
- Rename portals.c -> fsl_portals.c for clarity

Ahmed Mansour (2):
  drivers/misc: Share qbman init between archs
  Move SYS_DPAA_QBMAN to Kconfig

 arch/arm/cpu/armv8/fsl-layerscape/cpu.c|   4 +
 arch/arm/cpu/armv8/fsl-layerscape/fdt.c|   9 +
 .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c |  11 +
 .../include/asm/arch-fsl-layerscape/immap_lsch2.h  |  29 ++
 arch/arm/include/asm/arch-fsl-layerscape/speed.h   |   1 +
 arch/powerpc/cpu/mpc85xx/cpu_init.c|   3 +-
 arch/powerpc/cpu/mpc85xx/fdt.c |   1 +
 arch/powerpc/cpu/mpc85xx/portals.c | 281 ---
 arch/powerpc/include/asm/fsl_liodn.h   |   7 +-
 arch/powerpc/include/asm/fsl_portals.h |   4 -
 arch/powerpc/include/asm/immap_85xx.h  |  60 
 drivers/misc/Makefile  |   1 +
 drivers/misc/fsl_portals.c | 304 +
 drivers/net/Kconfig|  22 ++
 include/configs/B4860QDS.h |   1 -
 include/configs/P1023RDB.h |   1 -
 include/configs/P2041RDB.h |   1 -
 include/configs/T102xQDS.h |   1 -
 include/configs/T102xRDB.h |   1 -
 include/configs/T1040QDS.h |   1 -
 include/configs/T104xRDB.h |   1 -
 include/configs/T208xQDS.h |   1 -
 include/configs/T208xRDB.h |   1 -
 include/configs/T4240QDS.h |   1 -
 include/configs/T4240RDB.h |   1 -
 include/configs/corenet_ds.h   |   1 -
 include/configs/cyrus.h|   1 -
 include/configs/ls1046a_common.h   |   4 -
 include/fsl_qbman.h|  75 +
 29 files changed, 464 insertions(+), 365 deletions(-)
 create mode 100644 drivers/misc/fsl_portals.c
 create mode 100644 include/fsl_qbman.h

-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v2 2/2] Move SYS_DPAA_QBMAN to Kconfig

[Ahmed Mansour]

The CONFIG_SYS_DPAA_QBMAN define is used by DPAA1 freescale SOCs to
add device tree fixups that allow deep sleep in Linux. The define was
placed in header files included by a number of boards, but was not
explicitly documented in any of the Kconfigs. A description was added
to the drivers/networking menuconfig and default selection for
current SOCs that have this part

Signed-off-by: Ahmed Mansour 

---

Changes in v2: None

 drivers/net/Kconfig  | 22 ++
 include/configs/B4860QDS.h   |  1 -
 include/configs/P1023RDB.h   |  1 -
 include/configs/P2041RDB.h   |  1 -
 include/configs/T102xQDS.h   |  1 -
 include/configs/T102xRDB.h   |  1 -
 include/configs/T1040QDS.h   |  1 -
 include/configs/T104xRDB.h   |  1 -
 include/configs/T208xQDS.h   |  1 -
 include/configs/T208xRDB.h   |  1 -
 include/configs/T4240QDS.h   |  1 -
 include/configs/T4240RDB.h   |  1 -
 include/configs/corenet_ds.h |  1 -
 include/configs/cyrus.h  |  1 -
 include/configs/ls1043a_common.h |  2 --
 include/configs/ls1046a_common.h |  4 
 16 files changed, 22 insertions(+), 19 deletions(-)

diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index 52555da..db4c176 100644
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -365,4 +365,26 @@ config FEC2_PHY_NORXERR
  The PHY does not have a RXERR line (RMII only).
  (so program the FEC to ignore it).
 
+config SYS_DPAA_QBMAN
+   bool "Device tree fixup for QBMan on freescale SOCs"
+   depends on (ARM || PPC) && !SPL_BUILD
+   default y if ARCH_B4860 || \
+ARCH_P1023 || \
+ARCH_P2041 || \
+ARCH_T1023 || \
+ARCH_T1024 || \
+ARCH_T1040 || \
+ARCH_T1042 || \
+ARCH_T2080 || \
+ARCH_T2081 || \
+ARCH_T4240 || \
+ARCH_P4080 || \
+ARCH_P3041 || \
+ARCH_P5040 || \
+ARCH_P5020 || \
+ARCH_LS1043A || \
+ARCH_LS1046A
+   help
+ QBman fixups to allow deep sleep in DPAA 1 SOCs
+
 endif # NETDEVICES
diff --git a/include/configs/B4860QDS.h b/include/configs/B4860QDS.h
index 661bc8b..7f9482d 100644
--- a/include/configs/B4860QDS.h
+++ b/include/configs/B4860QDS.h
@@ -572,7 +572,6 @@ unsigned long get_board_ddr_clk(void);
 
 /* Qman/Bman */
 #ifndef CONFIG_NOBQFMAN
-#define CONFIG_SYS_DPAA_QBMAN  /* Support Q/Bman */
 #define CONFIG_SYS_BMAN_NUM_PORTALS25
 #define CONFIG_SYS_BMAN_MEM_BASE   0xf400
 #ifdef CONFIG_PHYS_64BIT
diff --git a/include/configs/P1023RDB.h b/include/configs/P1023RDB.h
index 1b78a4f..56e3f48 100644
--- a/include/configs/P1023RDB.h
+++ b/include/configs/P1023RDB.h
@@ -272,7 +272,6 @@ extern unsigned long get_clock_freq(void);
 #define CONFIG_LOADADDR100
 
 /* Qman/Bman */
-#define CONFIG_SYS_DPAA_QBMAN  /* support Q/Bman */
 #define CONFIG_SYS_QMAN_MEM_BASE   0xff00
 #define CONFIG_SYS_QMAN_MEM_PHYS   CONFIG_SYS_QMAN_MEM_BASE
 #define CONFIG_SYS_QMAN_MEM_SIZE   0x0020
diff --git a/include/configs/P2041RDB.h b/include/configs/P2041RDB.h
index 6008237..5f7ebe7 100644
--- a/include/configs/P2041RDB.h
+++ b/include/configs/P2041RDB.h
@@ -438,7 +438,6 @@ unsigned long get_board_sys_clk(unsigned long dummy);
 #define CONFIG_SYS_PCIE3_IO_SIZE   0x0001  /* 64k */
 
 /* Qman/Bman */
-#define CONFIG_SYS_DPAA_QBMAN  /* Support Q/Bman */
 #define CONFIG_SYS_BMAN_NUM_PORTALS10
 #define CONFIG_SYS_BMAN_MEM_BASE   0xf400
 #ifdef CONFIG_PHYS_64BIT
diff --git a/include/configs/T102xQDS.h b/include/configs/T102xQDS.h
index 259e8a0..13f4c92 100644
--- a/include/configs/T102xQDS.h
+++ b/include/configs/T102xQDS.h
@@ -644,7 +644,6 @@ unsigned long get_board_ddr_clk(void);
 
 /* Qman/Bman */
 #ifndef CONFIG_NOBQFMAN
-#define CONFIG_SYS_DPAA_QBMAN  /* Support Q/Bman */
 #define CONFIG_SYS_BMAN_NUM_PORTALS10
 #define CONFIG_SYS_BMAN_MEM_BASE   0xf400
 #ifdef CONFIG_PHYS_64BIT
diff --git a/include/configs/T102xRDB.h b/include/configs/T102xRDB.h
index 7dee2f0..e1f2196 100644
--- a/include/configs/T102xRDB.h
+++ b/include/configs/T102xRDB.h
@@ -649,7 +649,6 @@ unsigned long get_board_ddr_clk(void);
 
 /* Qman/Bman */
 #ifndef CONFIG_NOBQFMAN
-#define CONFIG_SYS_DPAA_QBMAN  /* Support Q/Bman */
 #define CONFIG_SYS_BMAN_NUM_PORTALS10
 #define CONFIG_SYS_BMAN_MEM_BASE   0xf400
 #ifdef CONFIG_PHYS_64BIT
diff --git a/include/configs/T1040QDS.h b/include/configs/T1040QDS.h
index c694e50..c186d15 100644
--- a/include/configs/T1040QDS.h
+++ b/include/configs/T1040QDS.h
@@ -535,7 +535,6 @@ unsigned long get_board_ddr_clk(void);
 
 /* Qman/Bman */
 #ifndef CONFIG_NOBQFMAN
-#define CONFIG_SYS_DPAA_QBMAN  /* Support Q/Bman */
 

[U-Boot] [PATCH v2 1/2] drivers/misc: Share qbman init between archs

[Ahmed Mansour]

This patch adds changes necessary to move functionality present in
PowerPC folders with ARM architectures that have DPAA1 QBMan hardware

- Create new board/freescale/common/fsl_portals.c to house shared
  device tree fixups for DPAA1 devices with ARM and PowerPC cores
- Add new header file to top includes directory to allow files in
  both architectures to grab the function prototypes
- Port inhibit_portals() from PowerPC to ARM. This function is used in
  setup to disable interrupts on all QMan and BMan portals. It is
  needed because the interrupts are enabled by default for all portals
  including unused/uninitialised portals. When the kernel attempts to
  go to deep sleep the unused portals prevent it from doing so

Signed-off-by: Ahmed Mansour 

---

Changes in v2:
- Add get_qman_freq() to replace get_sys_info() for readability
- Correct the copyright year in new files
- Replace !ARM with PPC to wall off PowerPC SOCs specific qman setup
- Rename portals.c -> fsl_portals.c for clarity

 arch/arm/cpu/armv8/fsl-layerscape/cpu.c|   4 +
 arch/arm/cpu/armv8/fsl-layerscape/fdt.c|   9 +
 .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c |  11 +
 .../include/asm/arch-fsl-layerscape/immap_lsch2.h  |  29 ++
 arch/arm/include/asm/arch-fsl-layerscape/speed.h   |   1 +
 arch/powerpc/cpu/mpc85xx/cpu_init.c|   3 +-
 arch/powerpc/cpu/mpc85xx/fdt.c |   1 +
 arch/powerpc/cpu/mpc85xx/portals.c | 281 ---
 arch/powerpc/include/asm/fsl_liodn.h   |   7 +-
 arch/powerpc/include/asm/fsl_portals.h |   4 -
 arch/powerpc/include/asm/immap_85xx.h  |  60 
 drivers/misc/Makefile  |   1 +
 drivers/misc/fsl_portals.c | 304 +
 include/configs/ls1043a_common.h   |   2 +
 include/fsl_qbman.h|  75 +
 15 files changed, 444 insertions(+), 348 deletions(-)
 create mode 100644 drivers/misc/fsl_portals.c
 create mode 100644 include/fsl_qbman.h

diff --git a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c 
b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
index d082629..3fd352f 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
@@ -30,6 +30,7 @@
 #endif
 #include 
 #include 
+#include 
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -565,6 +566,9 @@ int arch_early_init_r(void)
 #ifdef CONFIG_FMAN_ENET
fman_enet_init();
 #endif
+#ifdef CONFIG_SYS_DPAA_QBMAN
+   setup_qbman_portals();
+#endif
return 0;
 }
 
diff --git a/arch/arm/cpu/armv8/fsl-layerscape/fdt.c 
b/arch/arm/cpu/armv8/fsl-layerscape/fdt.c
index cae59da..382bf3d 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/fdt.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/fdt.c
@@ -26,6 +26,8 @@
 #ifdef CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT
 #include 
 #endif
+#include 
+#include 
 
 int fdt_fixup_phy_connection(void *blob, int offset, phy_interface_t phyc)
 {
@@ -415,6 +417,13 @@ void ft_cpu_setup(void *blob, bd_t *bd)
fdt_fixup_esdhc(blob, bd);
 #endif
 
+#ifdef CONFIG_SYS_DPAA_QBMAN
+   fdt_fixup_bportals(blob);
+   fdt_fixup_qportals(blob);
+   do_fixup_by_compat_u32(blob, "fsl,qman",
+  "clock-frequency", get_qman_freq(), 1);
+#endif
+
 #ifdef CONFIG_SYS_DPAA_FMAN
fdt_fixup_fman_firmware(blob);
 #endif
diff --git a/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c 
b/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c
index 2d7775e..f707205 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c
@@ -155,6 +155,17 @@ void get_sys_info(struct sys_info *sys_info)
sys_info->freq_localbus = sys_info->freq_systembus /
CONFIG_SYS_FSL_IFC_CLK_DIV;
 #endif
+#ifdef CONFIG_SYS_DPAA_QBMAN
+   sys_info->freq_qman = sys_info->freq_systembus;
+#endif
+}
+
+unsigned long get_qman_freq(void)
+{
+   struct sys_info sys_info;
+
+   get_sys_info(_info);
+   return sys_info.freq_systembus;
 }
 
 int get_clocks(void)
diff --git a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h 
b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
index 2561ead..1ff5cac 100644
--- a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
+++ b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
@@ -23,6 +23,8 @@
 #define CONFIG_SYS_FSL_GUTS_ADDR   (CONFIG_SYS_IMMR + 0x00ee)
 #define CONFIG_SYS_FSL_RST_ADDR(CONFIG_SYS_IMMR + 
0x00ee00b0)
 #define CONFIG_SYS_FSL_SCFG_ADDR   (CONFIG_SYS_IMMR + 0x0057)
+#define CONFIG_SYS_FSL_BMAN_ADDR   (CONFIG_SYS_IMMR + 0x0089)
+#define CONFIG_SYS_FSL_QMAN_ADDR   (CONFIG_SYS_IMMR + 0x0088)
 #define CONFIG_SYS_FSL_FMAN_ADDR   (CONFIG_SYS_IMMR + 0x00a0)
 #define CONFIG_SYS_FSL_SERDES_ADDR   

Re: [U-Boot] Swig now required to build U-Boot?

[Stephen Warren]

On 12/08/2017 05:04 PM, Simon Glass wrote:

Hi Stephen,

On 8 December 2017 at 16:57, Stephen Warren  wrote:

On 12/08/2017 04:53 PM, Simon Glass wrote:


Hi Stephen,

On 8 December 2017 at 16:28, Stephen Warren  wrote:


Simon,

Is it expected that the latest u-boot-dm.git master branch now requires
swig? All the non-sandbox builds on my test systems are failing:

CHK include/config/uboot.release
CHK include/generated/timestamp_autogenerated.h
GEN ./Makefile
UPD include/generated/timestamp_autogenerated.h
CHK include/config.h
CFG u-boot.cfg
SHIPPED scripts/dtc/pylibfdt/libfdt.i
PYMOD   scripts/dtc/pylibfdt/_libfdt.so
unable to execute 'swig': No such file or directory
error: command 'swig' failed with exit status 1
make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
make[3]: *** [scripts/dtc/pylibfdt] Error 2
make[2]: *** [scripts/dtc] Error 2
make[1]: *** [scripts] Error 2
make[1]: *** Waiting for unfinished jobs



Yes, unless we have a very recent libfdt on the system (and even then
I'm not sure that it is picked up). We need swig to build pylibfdt
which is needed for binman.



Which version of swig is required? (Perhaps the makefile could check for the
required minimum version and bail out early with a mesage, like it does for
dtc and gcc?)

I have 2.0.11 available in the distro package repository, but I see that
3.0.x exists on swig.org. I hope I can get away with installing the standard
OS package.


Yes v2 should be fine.


OK, installing the swig and python-dev packages made the builds work. I 
do see test failures on Beaver and Jetson TK1 (but not other boards), 
but I probably won't be able to look at those until Monday, and the 
tests haven't finished yet so I can't grab the logs.

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] MBR Signature mismatch on specific SD card

[Jeroen Bobbeldijk]

Hi all,

Recently we have been trying to setup Yocto with U-Boot. We kept getting
errors on U-Boot: MBR Signature mismatch: {some code, I guess sector?}

After many hours of messing around with different builds and write
settings, I tried another SD card and it suddenly started working. Are
there any known issues with some SD cards? Flashing Raspbian on them works
fine.
If required, I'm able to get some more details on the specific SD cards. Or
would you suggest just buying some other SD card?

Kind regards,

Jeroen Bobbeldijk

-- 
Vriendelijke groeten,

JEROEN BOBBELDIJK.
*KLIPPA APP.*

Ook starten met digitaal declareren?

www.klippa.com • +31 50 2111 631 <++31+50+2111+631> • jer...@klippa.com
Lübeckweg 2 • 9723 HE Groningen • Netherlands
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Swig now required to build U-Boot?

[Sergey Kubushyn]

On Fri, 8 Dec 2017, Simon Glass wrote:


Hi Stephen,

On 8 December 2017 at 16:28, Stephen Warren  wrote:

Simon,

Is it expected that the latest u-boot-dm.git master branch now requires
swig? All the non-sandbox builds on my test systems are failing:

  CHK include/config/uboot.release
  CHK include/generated/timestamp_autogenerated.h
  GEN ./Makefile
  UPD include/generated/timestamp_autogenerated.h
  CHK include/config.h
  CFG u-boot.cfg
  SHIPPED scripts/dtc/pylibfdt/libfdt.i
  PYMOD   scripts/dtc/pylibfdt/_libfdt.so
unable to execute 'swig': No such file or directory
error: command 'swig' failed with exit status 1
make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
make[3]: *** [scripts/dtc/pylibfdt] Error 2
make[2]: *** [scripts/dtc] Error 2
make[1]: *** [scripts] Error 2
make[1]: *** Waiting for unfinished jobs


Yes, unless we have a very recent libfdt on the system (and even then
I'm not sure that it is picked up). We need swig to build pylibfdt
which is needed for binman.


Err, would it be long before we need Java/Eclipse/Maven/younameit to just
build U-Boot?

---
**
*  KSI@homeKOI8 Net  < >  The impossible we do immediately.  *
*  Las Vegas   NV, USA   < >  Miracles require 24-hour notice.   *
**
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Swig now required to build U-Boot?

[Simon Glass]

Hi Stephen,

On 8 December 2017 at 16:57, Stephen Warren  wrote:
> On 12/08/2017 04:53 PM, Simon Glass wrote:
>>
>> Hi Stephen,
>>
>> On 8 December 2017 at 16:28, Stephen Warren  wrote:
>>>
>>> Simon,
>>>
>>> Is it expected that the latest u-boot-dm.git master branch now requires
>>> swig? All the non-sandbox builds on my test systems are failing:
>>>
>>>CHK include/config/uboot.release
>>>CHK include/generated/timestamp_autogenerated.h
>>>GEN ./Makefile
>>>UPD include/generated/timestamp_autogenerated.h
>>>CHK include/config.h
>>>CFG u-boot.cfg
>>>SHIPPED scripts/dtc/pylibfdt/libfdt.i
>>>PYMOD   scripts/dtc/pylibfdt/_libfdt.so
>>> unable to execute 'swig': No such file or directory
>>> error: command 'swig' failed with exit status 1
>>> make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
>>> make[3]: *** [scripts/dtc/pylibfdt] Error 2
>>> make[2]: *** [scripts/dtc] Error 2
>>> make[1]: *** [scripts] Error 2
>>> make[1]: *** Waiting for unfinished jobs
>>
>>
>> Yes, unless we have a very recent libfdt on the system (and even then
>> I'm not sure that it is picked up). We need swig to build pylibfdt
>> which is needed for binman.
>
>
> Which version of swig is required? (Perhaps the makefile could check for the
> required minimum version and bail out early with a mesage, like it does for
> dtc and gcc?)
>
> I have 2.0.11 available in the distro package repository, but I see that
> 3.0.x exists on swig.org. I hope I can get away with installing the standard
> OS package.

Yes v2 should be fine. I have:

$ swig -version

SWIG Version 2.0.11

Compiled with g++ [x86_64-unknown-linux-gnu]

Configured options: +pcre

Please see http://www.swig.org for reporting bugs and further information

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Swig now required to build U-Boot?

[Stephen Warren]

On 12/08/2017 04:53 PM, Simon Glass wrote:

Hi Stephen,

On 8 December 2017 at 16:28, Stephen Warren  wrote:

Simon,

Is it expected that the latest u-boot-dm.git master branch now requires
swig? All the non-sandbox builds on my test systems are failing:

   CHK include/config/uboot.release
   CHK include/generated/timestamp_autogenerated.h
   GEN ./Makefile
   UPD include/generated/timestamp_autogenerated.h
   CHK include/config.h
   CFG u-boot.cfg
   SHIPPED scripts/dtc/pylibfdt/libfdt.i
   PYMOD   scripts/dtc/pylibfdt/_libfdt.so
unable to execute 'swig': No such file or directory
error: command 'swig' failed with exit status 1
make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
make[3]: *** [scripts/dtc/pylibfdt] Error 2
make[2]: *** [scripts/dtc] Error 2
make[1]: *** [scripts] Error 2
make[1]: *** Waiting for unfinished jobs


Yes, unless we have a very recent libfdt on the system (and even then
I'm not sure that it is picked up). We need swig to build pylibfdt
which is needed for binman.


Which version of swig is required? (Perhaps the makefile could check for 
the required minimum version and bail out early with a mesage, like it 
does for dtc and gcc?)


I have 2.0.11 available in the distro package repository, but I see that 
3.0.x exists on swig.org. I hope I can get away with installing the 
standard OS package.

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Swig now required to build U-Boot?

[Simon Glass]

Hi Stephen,

On 8 December 2017 at 16:28, Stephen Warren  wrote:
> Simon,
>
> Is it expected that the latest u-boot-dm.git master branch now requires
> swig? All the non-sandbox builds on my test systems are failing:
>
>   CHK include/config/uboot.release
>   CHK include/generated/timestamp_autogenerated.h
>   GEN ./Makefile
>   UPD include/generated/timestamp_autogenerated.h
>   CHK include/config.h
>   CFG u-boot.cfg
>   SHIPPED scripts/dtc/pylibfdt/libfdt.i
>   PYMOD   scripts/dtc/pylibfdt/_libfdt.so
> unable to execute 'swig': No such file or directory
> error: command 'swig' failed with exit status 1
> make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
> make[3]: *** [scripts/dtc/pylibfdt] Error 2
> make[2]: *** [scripts/dtc] Error 2
> make[1]: *** [scripts] Error 2
> make[1]: *** Waiting for unfinished jobs

Yes, unless we have a very recent libfdt on the system (and even then
I'm not sure that it is picked up). We need swig to build pylibfdt
which is needed for binman.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] Swig now required to build U-Boot?

[Stephen Warren]

Simon,

Is it expected that the latest u-boot-dm.git master branch now requires 
swig? All the non-sandbox builds on my test systems are failing:


  CHK include/config/uboot.release
  CHK include/generated/timestamp_autogenerated.h
  GEN ./Makefile
  UPD include/generated/timestamp_autogenerated.h
  CHK include/config.h
  CFG u-boot.cfg
  SHIPPED scripts/dtc/pylibfdt/libfdt.i
  PYMOD   scripts/dtc/pylibfdt/_libfdt.so
unable to execute 'swig': No such file or directory
error: command 'swig' failed with exit status 1
make[4]: *** [scripts/dtc/pylibfdt/_libfdt.so] Error 1
make[3]: *** [scripts/dtc/pylibfdt] Error 2
make[2]: *** [scripts/dtc] Error 2
make[1]: *** [scripts] Error 2
make[1]: *** Waiting for unfinished jobs
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] Please pull u-boot-dm

[Simon Glass]

Hi Tom,

This is the binman symbol feature plus a few other things that I found
in my queue. I'll sort out the test patch problems next week.

Buildman is clean, and travis-ci:

https://travis-ci.org/sglass68/u-boot/builds/313582145


The following changes since commit 6c7010b779ef29a40ed66acc414f65d2f4b75ced:

  Merge git://git.denx.de/u-boot-fsl-qoriq (2017-12-07 17:56:51 -0500)

are available in the Git repository at:

  git://git.denx.de/u-boot-dm.git

for you to fetch changes up to 2b64d049a06131301f86e9deb57919724effb913:

  power: pmic/regulator: Add basic support for TPS65910 (2017-12-08
06:25:34 -0700)


Felix Brack (2):
  power: extend prefix match to regulator-name property
  power: pmic/regulator: Add basic support for TPS65910

Mario Six (1):
  drivers: core: Add translation in live tree case

Philipp Tomsich (1):
  dm: reset: have the reset-command perform a COLD reset

Simon Glass (13):
  binman: Add a function to read ELF symbols
  binman: Add support for including spl/u-boot-spl.dtb
  binman: Add support for including spl/u-boot-spl-nodtb.bin
  binman: Drop a stale comment about the 'board' feature
  binman: Add tests binaries with binman symbols
  binman: Adjust size of test SPL binary
  binman: Support enabling debug in tests
  binman: Support accessing binman tables at run time
  binman: arm: Include the binman symbol table
  binman: Add binman symbol support to SPL
  binman: Add binman support to spl_ram.c
  binman: Add documentation for the symbol feature
  binman: tegra: Convert to use binman

Stefan Roese (1):
  dm: blk: Use uclass_find_first/next_device() in blk_first/next_device()

 Makefile |   6 +
 arch/arm/config.mk   |   6 +-
 arch/arm/cpu/u-boot-spl.lds  |   7 +
 arch/arm/dts/tegra-u-boot.dtsi   |  40 +
 arch/arm/dts/tegra114-u-boot.dtsi|   3 +
 arch/arm/dts/tegra124-nyan-big-u-boot.dtsi   |   2 +
 arch/arm/dts/tegra124-u-boot.dtsi|   3 +
 arch/arm/dts/tegra20-u-boot.dtsi |  11 +-
 arch/arm/dts/tegra210-u-boot.dtsi|   3 +
 arch/arm/dts/tegra30-u-boot.dtsi |   3 +
 arch/arm/mach-tegra/Kconfig  |   1 +
 arch/sandbox/dts/sandbox_pmic.dtsi   |   6 +
 common/spl/spl.c |  16 +-
 common/spl/spl_ram.c |  19 ++-
 doc/device-tree-bindings/regulator/regulator.txt |  16 +-
 drivers/block/blk-uclass.c   |   7 +-
 drivers/core/ofnode.c|   8 +-
 drivers/power/pmic/Kconfig   |   8 +
 drivers/power/pmic/Makefile  |   1 +
 drivers/power/pmic/pmic-uclass.c |  11 +-
 drivers/power/pmic/pmic_tps65910_dm.c|  98 +++
 drivers/power/regulator/Kconfig  |   8 +
 drivers/power/regulator/Makefile |   1 +
 drivers/power/regulator/tps65910_regulator.c | 459
++
 drivers/sysreset/sysreset-uclass.c   |   2 +-
 include/binman_sym.h |  93 ++
 include/power/sandbox_pmic.h |   5 +-
 include/power/tps65910_pmic.h| 130 ++
 include/spl.h|  11 ++
 test/dm/regulator.c  |   2 +
 tools/binman/README  |  32 +++-
 tools/binman/binman.py   |  11 +-
 tools/binman/control.py  |   3 +
 tools/binman/elf.py  | 129 ++
 tools/binman/elf_test.py | 122 ++
 tools/binman/etype/entry.py  |   8 +
 tools/binman/etype/u_boot_spl.py |   6 +
 tools/binman/etype/u_boot_spl_bss_pad.py |   7 +-
 tools/binman/etype/u_boot_spl_dtb.py |  17 ++
 tools/binman/etype/u_boot_spl_nodtb.py   |  17 ++
 tools/binman/etype/u_boot_with_ucode_ptr.py  |   9 +-
 tools/binman/ftest.py|  68 ++--
 tools/binman/image.py|  79 -
 tools/binman/image_test.py   |  46 +
 tools/binman/test/21_image_pad.dts   |   2 +-
 tools/binman/test/24_sorted.dts  |   4 +-
 tools/binman/test/28_pack_4gb_outside.dts|   4 +-
 tools/binman/test/29_x86-rom.dts |   6 +-
 tools/binman/test/51_u_boot_spl_dtb.dts  |  13 ++
 tools/binman/test/52_u_boot_spl_nodtb.dts|  11 ++
 tools/binman/test/53_symbols.dts |  20 +++
 tools/binman/test/Makefile   |  18 +-
 tools/binman/test/bss_data.c |  

[U-Boot] [PATCH 1/1] efi_loader: support device path for ide disks

[Heinrich Schuchardt]

Correctly create the evice path for ide disks.

Signed-off-by: Heinrich Schuchardt 
---
 include/efi_api.h|  9 ++
 lib/efi_loader/efi_device_path.c | 53 
 lib/efi_loader/efi_device_path_to_text.c |  7 +
 3 files changed, 69 insertions(+)

diff --git a/include/efi_api.h b/include/efi_api.h
index 9a66efde45..4f0bfc417d 100644
--- a/include/efi_api.h
+++ b/include/efi_api.h
@@ -329,12 +329,21 @@ struct efi_device_path_acpi_path {
 } __packed;
 
 #define DEVICE_PATH_TYPE_MESSAGING_DEVICE  0x03
+#  define DEVICE_PATH_SUB_TYPE_MSG_ATAPI   0x01
+#  define DEVICE_PATH_SUB_TYPE_MSG_SCSI0x02
 #  define DEVICE_PATH_SUB_TYPE_MSG_USB 0x05
 #  define DEVICE_PATH_SUB_TYPE_MSG_MAC_ADDR0x0b
 #  define DEVICE_PATH_SUB_TYPE_MSG_USB_CLASS   0x0f
 #  define DEVICE_PATH_SUB_TYPE_MSG_SD  0x1a
 #  define DEVICE_PATH_SUB_TYPE_MSG_MMC 0x1d
 
+struct efi_device_path_atapi {
+   struct efi_device_path dp;
+   u8 primary_secondary;
+   u8 slave_master;
+   u8 logical_unit_number;
+} __packed;
+
 struct efi_device_path_usb {
struct efi_device_path dp;
u8 parent_port_number;
diff --git a/lib/efi_loader/efi_device_path.c b/lib/efi_loader/efi_device_path.c
index 31cdd38773..a3eb077e45 100644
--- a/lib/efi_loader/efi_device_path.c
+++ b/lib/efi_loader/efi_device_path.c
@@ -295,6 +295,42 @@ static unsigned dp_size(struct udevice *dev)
}
 }
 
+#ifdef CONFIG_BLK
+/*
+ * Determine the logical unit number as the index of the device
+ * in the device class.
+ *
+ * @uclass_id  device class
+ * @devnum device number from the device descriptor
+ * @return logical unit number or a negative error code
+ */
+static int dev_get_lun(enum uclass_id uclass_id, int devnum)
+{
+   struct uclass *uc;
+   struct udevice *dev;
+   int ret;
+
+   ret = uclass_get(uclass_id, );
+   if (ret)
+   return ret;
+   uclass_foreach_dev(dev, uc) {
+   struct blk_desc *desc = dev_get_uclass_platdata(dev);
+
+   if (desc->devnum == devnum)
+   return ret;
+   ++ret;
+   }
+   return -ENOENT;
+}
+#endif
+
+/*
+ * Recursively build a device path.
+ *
+ * @bufoutput buffer
+ * @devdevice
+ * @return pointer to the end of the device path
+ */
 static void *dp_fill(void *buf, struct udevice *dev)
 {
if (!dev || !dev->driver)
@@ -308,6 +344,23 @@ static void *dp_fill(void *buf, struct udevice *dev)
*vdp = ROOT;
return [1];
}
+#if defined(CONFIG_IDE) && defined(CONFIG_BLK)
+   case UCLASS_IDE: {
+   struct efi_device_path_atapi *idedp =
+   dp_fill(buf, dev->parent);
+   struct blk_desc *ide = dev_get_uclass_platdata(dev);
+   int lun = dev_get_lun(UCLASS_IDE, ide->devnum);
+
+   idedp->dp.type = DEVICE_PATH_TYPE_MESSAGING_DEVICE;
+   idedp->dp.sub_type = DEVICE_PATH_SUB_TYPE_MSG_ATAPI;
+   idedp->dp.length = sizeof(*idedp);
+   idedp->logical_unit_number = lun;
+   idedp->primary_secondary = IDE_BUS(lun);
+   idedp->slave_master = lun %
+   (CONFIG_SYS_IDE_MAXDEVICE / CONFIG_SYS_IDE_MAXBUS);
+   return [1];
+   }
+#endif
 #if defined(CONFIG_DM_MMC) && defined(CONFIG_MMC)
case UCLASS_MMC: {
struct efi_device_path_sd_mmc_path *sddp =
diff --git a/lib/efi_loader/efi_device_path_to_text.c 
b/lib/efi_loader/efi_device_path_to_text.c
index 21c5678d20..7070ac909d 100644
--- a/lib/efi_loader/efi_device_path_to_text.c
+++ b/lib/efi_loader/efi_device_path_to_text.c
@@ -87,6 +87,13 @@ static char *dp_acpi(char *s, struct efi_device_path *dp)
 static char *dp_msging(char *s, struct efi_device_path *dp)
 {
switch (dp->sub_type) {
+   case DEVICE_PATH_SUB_TYPE_MSG_ATAPI: {
+   struct efi_device_path_atapi *ide =
+   (struct efi_device_path_atapi *)dp;
+   s += sprintf(s, "Ata(%d,%d,%d)", ide->primary_secondary,
+ide->slave_master, ide->logical_unit_number);
+   break;
+   }
case DEVICE_PATH_SUB_TYPE_MSG_USB: {
struct efi_device_path_usb *udp =
(struct efi_device_path_usb *)dp;
-- 
2.14.2

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2] test: py: Add an option to skip sleep test

[Michal Simek]

2017-12-08 17:54 GMT+01:00 Stephen Warren :

> On 12/08/2017 07:47 AM, Michal Simek wrote:
>
>> Some QEMUs have a problem with time setup that's why
>> sleep test is failing. Introduce env__sleep_accurate
>> boardenv variable to have an option to skip sleep test.
>>
>
> Nit: Commit descriptions are typically wrapped around 70-74 characters;
> the above is very narrow.
>
> diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
>>
>
> +"""
>> +Note: This test doesn't rely on boardenv_* configuration values but they
>> can
>> +change test behavior.
>> +
>> +# Setup env__sleep_accurate to False if time is not accurate on your
>> platform
>> +env__sleep_accurate = False
>> +
>> +"""
>>
>
> Nit: No need for that trailing empty line in the docstring.
>
> Reviewed-by: Stephen Warren 
>
> (I was going to contemplate a more generic system implemented in
> conftest.py that could work with any test, but I guess this is simple
> enough and solves the only issue we have right now, so it's fine.)
>


I think also that generic solution would be better. I was checking xilinx
repo where we have some simple i2c
device tests which also needs to be run based on platforms. This test is
broken on qemu and some boards are also broken.
There is not an issue with travis because sleep tests are disabled now.
For my qemu testing it is not big deal and I can use -k not sleep or this
patch as temporary workaround till we have generic solution in place.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 02/10] board: common:vid: Add LS1088A VID Supported voltage values

[York Sun]

On 11/29/2017 10:30 PM, Rajesh Bhagat wrote:
> Adds below voltage values supported by LS1088A Soc:
> 
> 1.025 V(default), 0.9875V, 0.9750 V, 0.9V, 1.0 V, 1.0125 V, 1.0250 V
> 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Rajesh Bhagat 
> ---
> Changes in v7:
>   
>  None 
>   
> 
> Changes in v6:
>   
>  None 
>   
>   
>   
> Changes in v5:
>   
>  None 
>   
>   
>   
> Changes in v4:
>   
>  None 
> 
>  board/freescale/common/vid.c | 21 +
>  1 file changed, 21 insertions(+)
> 
> diff --git a/board/freescale/common/vid.c b/board/freescale/common/vid.c
> index d6d1bfc..f4fc95b 100644
> --- a/board/freescale/common/vid.c
> +++ b/board/freescale/common/vid.c
> @@ -296,15 +296,28 @@ int adjust_vdd(ulong vdd_override)
>   unsigned long vdd_string_override;
>   char *vdd_string;
>   static const uint16_t vdd[32] = {
> +#ifdef CONFIG_ARCH_LS1088A
> + 10250,
> + 9875,
> +#else
>   10500,
>   0,  /* reserved */
> +#endif
>   9750,
>   0,  /* reserved */
> +#ifdef CONFIG_ARCH_LS1088A
> + 0,  /* reserved */
> +#else
>   9500,
> +#endif
>   0,  /* reserved */
>   0,  /* reserved */
>   0,  /* reserved */
> +#ifdef CONFIG_ARCH_LS1088A
> + 9000,
> +#else
>   0,  /* reserved */
> +#endif
>   0,  /* reserved */
>   0,  /* reserved */
>   0,  /* reserved */
> @@ -313,10 +326,18 @@ int adjust_vdd(ulong vdd_override)
>   0,  /* reserved */
>   0,  /* reserved */
>   1,  /* 1.V */
> +#ifdef CONFIG_ARCH_LS1088A
> + 10125,
> +#else
>   0,  /* reserved */
> +#endif
>   10250,
>   0,  /* reserved */
> +#ifdef CONFIG_ARCH_LS1088A
> + 0,  /* reserved */
> +#else
>   10500,
> +#endif
>   0,  /* reserved */
>   0,  /* reserved */
>   0,  /* reserved */
> 

Since we have so many differences, would it be cleaner if we simply have
two separated vdd arrays?

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 09/10] common: board_f: vid: Add VID specific API to adjust core voltage

[York Sun]

On 11/29/2017 10:31 PM, Rajesh Bhagat wrote:
> Adds a VID specific API in init_sequence_f and spl code flow
> namely init_func_vid which is required to adjust core voltage.
> 
> VID specific code is required in spl, hence moving flag CONFIG_VID
> out of spl flags.
> 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Rajesh Bhagat 
> ---
>  arch/arm/cpu/armv8/fsl-layerscape/spl.c |  3 +++
>  board/freescale/common/Makefile |  2 +-
>  common/board_f.c| 10 ++
>  include/common.h|  3 +++
>  4 files changed, 17 insertions(+), 1 deletion(-)
> 

Did you gather the information about size increase for SPL image? I am
concerned how much it creases.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 06/10] board: common: vid: Add support for LTC3882 voltage regulator chip

[York Sun]

On 11/29/2017 10:31 PM, Rajesh Bhagat wrote:
> Restructures common driver to support LTC3882 voltage regulator
> chip.
> 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Rajesh Bhagat 
> ---
> Changes in v7:
>   
>  None
> 



> +static int set_voltage_to_LTC(int i2caddress, int vdd)
> +{
> + int ret, vdd_last, vdd_target = vdd;
> +
> + /* Scale up to the LTC resolution is 1/4096V */
> + vdd = (vdd * 4096) / 1000;
> +
> + /* 5-byte buffer which needs to be sent following the
> +  * PMBus command PAGE_PLUS_WRITE.
> +  */
> + u8 buff[5] = {0x04, PWM_CHANNEL0, PMBUS_CMD_VOUT_COMMAND,
> + vdd & 0xFF, (vdd & 0xFF00) >> 8};
> +
> + /* Write the desired voltage code to the regulator */
> + ret = i2c_write(I2C_VOL_MONITOR_ADDR,
> + PMBUS_CMD_PAGE_PLUS_WRITE, 1, (void *), 5);
> + if (ret) {
> + printf("VID: I2C failed to write to the volatge regulator\n");
> + return -1;
> + }
> +
> + /* Wait for the volatge to get to the desired value */
> + do {
> + vdd_last = read_voltage_from_LTC(i2caddress);
> + if (vdd_last < 0) {
> + printf("VID: Couldn't read sensor abort VID adjust\n");
> + return -1;
> + }
> + } while (vdd_last != vdd_target);

nitpick

Insert a blank line before return.

> + return vdd_last;
> +}

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 08/10] ls1088a: ddr: configure DDR for 0.9v for VID support

[York Sun]

On 11/29/2017 10:31 PM, Rajesh Bhagat wrote:
> When VID feature is supported, check the contents of fuse register
> and configure DDR operate at 0.9v.
> 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Rajesh Bhagat 
> ---
>  board/freescale/ls1088a/ddr.c | 21 +
>  1 file changed, 21 insertions(+)
> 
> diff --git a/board/freescale/ls1088a/ddr.c b/board/freescale/ls1088a/ddr.c
> index e24bfd5..f3e70b5 100644
> --- a/board/freescale/ls1088a/ddr.c
> +++ b/board/freescale/ls1088a/ddr.c
> @@ -13,6 +13,23 @@
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> +#if defined(CONFIG_VID) && (!defined(CONFIG_SPL) || 
> defined(CONFIG_SPL_BUILD))
> +void fsl_ddr_setup_0v9_volt(memctl_options_t *popts)

add "static" if this function is not used externally.

> +{
> + int vdd;
> +
> + vdd = get_core_volt_from_fuse();
> + /* Nothing to do for silicons doesn't support VID */
> + if (vdd < 0)
> + return;
> +

Any guarantee the fuse value is actually the voltage on the board? If
you cannot read back the voltage here, you may consider to print a big
warning at the point you know voltage adjustment fails.

> + if (vdd == 900) {
> + popts->ddr_cdr1 |= DDR_CDR1_V0PT9_EN;
> + printf("VID: configure DDR to support 900 mV\n");

s/printf/debug

> + }
> +}
> +#endif
> +
>  void fsl_ddr_board_options(memctl_options_t *popts,
>  dimm_params_t *pdimm,
>  unsigned int ctrl_num)
> @@ -87,6 +104,10 @@ found:
>   popts->addr_hash = 1;
>  
>   popts->ddr_cdr1 = DDR_CDR1_DHC_EN | DDR_CDR1_ODT(DDR_CDR_ODT_60ohm);
> +#if defined(CONFIG_VID) && (!defined(CONFIG_SPL) || 
> defined(CONFIG_SPL_BUILD))
> + fsl_ddr_setup_0v9_volt(popts);
> +#endif
> +
>   popts->ddr_cdr2 = DDR_CDR2_ODT(DDR_CDR_ODT_60ohm) |
> DDR_CDR2_VREF_TRAIN_EN | DDR_CDR2_VREF_RANGE_2;
>  }
> 

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 10/10] ls1088a: Add VID support for QDS and RDB platforms

[York Sun]

On 11/29/2017 10:31 PM, Rajesh Bhagat wrote:
> This patch adds the support for VID on LS1088AQDS and LS1088ARDB systems.
> It reads the fusesr register and changes the VDD accordingly by adjusting
> the voltage via LTC3882 regulator.
> 
> This patch also takes care of the special case of 0.9V VDD is present in
> fusesr register. In that case,it also changes the SERDES voltage by
> disabling the SERDES, changing the SVDD and then re-enabling SERDES.
> 
> Signed-off-by: Raghav Dogra 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Amrita Kumari 
> Signed-off-by: Rajesh Bhagat 
> ---
> Changes in v7:
>   
>  - Defined CONFIG_VID in SPL scenario too
>  - Defined new API init_func_vid for core voltage adjustment for LS1088A
> 
>


> +int get_serdes_volt(void)
> +{
> + int  ret, vcode = 0;
> + u8 chan = PWM_CHANNEL0;
> +
> + /* Select the PAGE 0 using PMBus commands PAGE for VDD */
> + ret = i2c_write(I2C_SVDD_MONITOR_ADDR,
> + PMBUS_CMD_PAGE, 1, , 1);
> + if (ret) {
> + printf("VID: failed to select VDD Page 0\n");
> + return ret;
> + }
> +
> + /* Read the output voltage using PMBus command READ_VOUT */
> + ret = i2c_read(I2C_SVDD_MONITOR_ADDR,
> +PMBUS_CMD_READ_VOUT, 1, (void *), 2);
> + if (ret) {
> + printf("VID: failed to read the volatge\n");
> + return ret;
> + }

nipick
Insert a blank line before return.

> + return vcode;
> +}

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 01/13] binman: Add a function to read ELF symbols

[sjg]

In some cases we need to read symbols from U-Boot. At present we have a
a few cases which does this via 'nm' and 'grep'.

It is better to use objdump since that tells us the size of the symbols
and also whether it is weak or not.

Add a new module which reads ELF information from files. Update existing
uses of 'nm' to use this module.

Signed-off-by: Simon Glass 
---

 tools/binman/binman.py  |  3 +-
 tools/binman/elf.py | 77 +
 tools/binman/elf_test.py| 32 
 tools/binman/etype/u_boot_spl_bss_pad.py|  7 +--
 tools/binman/etype/u_boot_with_ucode_ptr.py |  9 ++--
 tools/binman/ftest.py   |  7 +++
 tools/binman/test/bss_data.c|  2 +-
 tools/binman/test/u_boot_ucode_ptr.c|  2 +-
 8 files changed, 128 insertions(+), 11 deletions(-)
 create mode 100644 tools/binman/elf.py
 create mode 100644 tools/binman/elf_test.py

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 02/13] binman: Add support for including spl/u-boot-spl.dtb

[sjg]

This file contains the SPL device tree. Add support for including this by
itself in images.

Signed-off-by: Simon Glass 
---

 tools/binman/etype/u_boot_spl_dtb.py| 17 +
 tools/binman/ftest.py   |  7 +++
 tools/binman/test/51_u_boot_spl_dtb.dts | 13 +
 3 files changed, 37 insertions(+)
 create mode 100644 tools/binman/etype/u_boot_spl_dtb.py
 create mode 100644 tools/binman/test/51_u_boot_spl_dtb.dts

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] dm: reset: have the reset-command perform a COLD reset

[sjg]

On 24 November 2017 at 10:37, Philipp Tomsich
 wrote:
> The DM version of do_reset has been issuing a warm-reset, which (on
> some platforms keeps GPIOs and other parts of the platform active).
> This may cause unintended behaviour, as calling do_reset usually
> indicates a desire to reset the board/platform and not just the CPU.
>
> This changes do_reset to always request a COLD reset.
> Note that programmatic uses can still invoke a WARM reset through
> reset_cpu() or using sysreset_walk().
>
> Signed-off-by: Philipp Tomsich 
> ---
>
>  drivers/sysreset/sysreset-uclass.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass 

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 12/13] binman: Add documentation for the symbol feature

[sjg]

Hi Simon,

> Add this feature to the README.
>
> Signed-off-by: Simon Glass 
> ---
>
>  tools/binman/README | 32 +++-
>  1 file changed, 31 insertions(+), 1 deletion(-)
>
Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RESEND PATCH v7 01/10] armv8: lsch3: Add serdes and DDR voltage setup

[York Sun]

On 11/29/2017 10:30 PM, Rajesh Bhagat wrote:
> Adds SERDES voltage and reset SERDES lanes API and makes
> enable/disable DDR controller support 0.9V API common.
> 
> Signed-off-by: Ashish Kumar 
> Signed-off-by: Rajesh Bhagat 
> ---
> Changes in v7:
>   
>   - Used APIs clr/set/clrsetbits_le32() for code clarity  
>   
> 

This version is quite good. Some nitpicks below.



> +#endif
> +#ifdef CONFIG_SYS_FSL_SRDS_2
> + {.id = 2,
> +  .mask = FSL_CHASSIS3_SRDS2_PRTCL_MASK,
> +  .shift = FSL_CHASSIS3_SRDS2_PRTCL_SHIFT
> + },
> +#endif
> + {.id = 0, .mask = 0, .shift = 0} /* NULL ENTRY */

An empty {} will do the same.



> +
> + /* For each PLL being reset, and achieved PLL lock set RST_DONE */
> +#ifdef CONFIG_SYS_FSL_SRDS_1
> + cfg_tmp = cfg_rcwsrds1 & 0x3;
> + do_pll_reset_done(cfg_tmp, serdes1_base);
> +#endif
> +#ifdef CONFIG_SYS_FSL_SRDS_2
> + cfg_tmp = cfg_rcwsrds1 & 0xC;
> + cfg_tmp >>= 2;
> + do_pll_reset_done(cfg_tmp, serdes2_base);
> +#endif

Insert a blank line before return.

> + return ret;
> +}
> +

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 10/13] binman: Add binman symbol support to SPL

[sjg]

Allow SPL to access binman symbols and use this to get the address of
U-Boot. This falls back to CONFIG_SYS_TEXT_BASE if the binman symbol
is not available.

Signed-off-by: Simon Glass 
---

 common/spl/spl.c | 16 ++--
 include/spl.h| 11 +++
 2 files changed, 25 insertions(+), 2 deletions(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 04/13] binman: Drop a stale comment about the 'board' feature

[sjg]

This feature is now supported. Drop the incorrect comment.

Signed-off-by: Simon Glass 
---

 tools/binman/ftest.py | 1 -
 1 file changed, 1 deletion(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 05/13] binman: Add tests binaries with binman symbols

[sjg]

For testing we need to build some ELF files containing binman symbols. Add
these to the Makefile and check in the binaries:

   u_boot_binman_syms - normal, valid ELF file
   u_boot_binman_syms_bad - missing the __image_copy_start symbol
   u_boot_binman_syms_size - has a binman symbol with an invalid size

Signed-off-by: Simon Glass 
---

 tools/binman/test/Makefile   |  18 +++-
 tools/binman/test/u_boot_binman_syms | Bin 0 -> 4921 bytes
 tools/binman/test/u_boot_binman_syms.c   |  14 +
 tools/binman/test/u_boot_binman_syms.lds |  30 +++
 tools/binman/test/u_boot_binman_syms_bad | Bin 0 -> 4890 bytes
 tools/binman/test/u_boot_binman_syms_bad.c   |   1 +
 tools/binman/test/u_boot_binman_syms_bad.lds |  29 ++
 tools/binman/test/u_boot_binman_syms_size| Bin 0 -> 4825 bytes
 tools/binman/test/u_boot_binman_syms_size.c  |  12 +++
 9 files changed, 103 insertions(+), 1 deletion(-)
 create mode 100755 tools/binman/test/u_boot_binman_syms
 create mode 100644 tools/binman/test/u_boot_binman_syms.c
 create mode 100644 tools/binman/test/u_boot_binman_syms.lds
 create mode 100755 tools/binman/test/u_boot_binman_syms_bad
 create mode 12 tools/binman/test/u_boot_binman_syms_bad.c
 create mode 100644 tools/binman/test/u_boot_binman_syms_bad.lds
 create mode 100755 tools/binman/test/u_boot_binman_syms_size
 create mode 100644 tools/binman/test/u_boot_binman_syms_size.c

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 07/13] binman: Support enabling debug in tests

[sjg]

The elf module can provide some debugging information to assist with
figuring out what is going wrong. This is also useful in tests. Update the
-D option so that it is passed through to tests as well.

Signed-off-by: Simon Glass 
---

 tools/binman/binman.py  |  6 --
 tools/binman/control.py |  3 +++
 tools/binman/elf.py |  3 +++
 tools/binman/ftest.py   | 17 +
 4 files changed, 23 insertions(+), 6 deletions(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 09/13] binman: arm: Include the binman symbol table

[sjg]

This area of the image contains symbols whose values are filled in by
binman. If this feature is not used, the table is empty.

Add this to the ARM SPL link script.

Signed-off-by: Simon Glass 
---

 arch/arm/config.mk  | 6 --
 arch/arm/cpu/u-boot-spl.lds | 7 +++
 2 files changed, 11 insertions(+), 2 deletions(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 03/13] binman: Add support for including spl/u-boot-spl-nodtb.bin

[sjg]

This file contains SPL image without a device tree. Add support for
including this in images.

Signed-off-by: Simon Glass 
---

 tools/binman/etype/u_boot_spl_nodtb.py| 17 +
 tools/binman/ftest.py |  5 +
 tools/binman/test/52_u_boot_spl_nodtb.dts | 11 +++
 3 files changed, 33 insertions(+)
 create mode 100644 tools/binman/etype/u_boot_spl_nodtb.py
 create mode 100644 tools/binman/test/52_u_boot_spl_nodtb.dts

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 08/13] binman: Support accessing binman tables at run time

[sjg]

Binman construct images consisting of multiple binary files. These files
sometimes need to know (at run timme) where their peers are located. For
example, SPL may want to know where U-Boot is located in the image, so
that it can jump to U-Boot correctly on boot.

In general the positions where the binaries end up after binman has
finished packing them cannot be known at compile time. One reason for
this is that binman does not know the size of the binaries until
everything is compiled, linked and converted to binaries with objcopy.

To make this work, we add a feature to binman which checks each binary
for symbol names starting with '_binman'. These are then decoded to figure
out which entry and property they refer to. Then binman writes the value
of this symbol into the appropriate binary. With this, the symbol will
have the correct value at run time.

Macros are used to make this easier to use. As an example, this declares
a symbol that will access the 'u-boot-spl' entry to find the 'pos' value
(i.e. the position of SPL in the image):

   binman_sym_declare(unsigned long, u_boot_spl, pos);

This converts to a symbol called '_binman_u_boot_spl_prop_pos' in any
binary that includes it. Binman then updates the value in that binary,
ensuring that it can be accessed at runtime with:

   ulong u_boot_pos = binman_sym(ulong, u_boot_spl, pos);

This assigns the variable u_boot_pos to the position of SPL in the image.

Signed-off-by: Simon Glass 
---

 include/binman_sym.h | 80 ++
 tools/binman/binman.py   |  4 +-
 tools/binman/control.py  |  4 +-
 tools/binman/elf.py  | 55 ++--
 tools/binman/elf_test.py | 92 +++-
 tools/binman/etype/entry.py  |  8 
 tools/binman/etype/u_boot_spl.py |  6 +++
 tools/binman/ftest.py| 19 +
 tools/binman/image.py| 79 --
 tools/binman/image_test.py   | 46 
 tools/binman/test/53_symbols.dts | 20 +
 11 files changed, 403 insertions(+), 10 deletions(-)
 create mode 100644 include/binman_sym.h
 create mode 100644 tools/binman/image_test.py
 create mode 100644 tools/binman/test/53_symbols.dts

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2] drivers: core: Add translation in live tree case

[sjg]

On 23 November 2017 at 23:51, Mario Six  wrote:
> The function dev_read_addr calls ofnode_get_addr_index in the live tree
> case, which does not apply bus translations to the address read from the
> device tree. This results in illegal addresses on boards that rely on
> bus translations being applied.
>
> Fix this situation by applying bus translations in the live tree case as
> well.
>
> Signed-off-by: Mario Six 
> ---
>
> Changes v1 -> v2:
> * Added IS_ENABLED(CONFIG_OF_TRANSLATE) case distinction
>
> ---
>  drivers/core/ofnode.c | 8 +++-
>  1 file changed, 7 insertions(+), 1 deletion(-)

Reviewed-by: Simon Glass 

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 13/13] binman: tegra: Convert to use binman

[sjg]

Hi Stephen,

On 14 November 2017 at 13:23, Stephen Warren  wrote:
> On 11/13/2017 06:55 PM, Simon Glass wrote:
>>
>> Update tegra to use binman for image creation. This still includes the
>> current Makefile logic, but a later patch will remove this. Three output
>> files are created, all of which combine
>> SPL and U-Boot:
>>
>> u-boot-tegra.bin  - standard image
>> u-boot-dtb-tegra.bin  - same as u-boot-tegra.bin
>> u-boot-nodtb  - includes U-Boot without the appended device
>> tree
>
>
> I assume that last file is u-boot-nodtb-tegra.bin?

Yes that is correct, will fix.

>
> If this series doesn't change the set of binaries produced by the build
> system (add/drop/rename), then I'm fine with it.

It should not change this.

>
> For the record, I know of tools that use the following files, at least:
>
> u-boot (ELF)
> u-boot.bin
> u-boot.dtb
> u-boot-dtb.bin
> u-boot-nodtb-tegra.bin
> u-boot-dtb-tegra.bin

OK, then I suppose we need them all. I was hoping to drop one of the
special tegra ones, but it seems like they are all used.

Regards,
Simon

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 11/13] binman: Add binman support to spl_ram.c

[sjg]

SPL supports reading U-Boot from a RAM location. At present this is
hard-coded to the U-Boot text base address. Use binman to allow this to
come from the image file, if binman is used.

Signed-off-by: Simon Glass 
---

 common/spl/spl_ram.c | 19 +++
 1 file changed, 15 insertions(+), 4 deletions(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 06/13] binman: Adjust size of test SPL binary

[sjg]

This is only 3 bytes long which is not enough to hold two symbol values,
needed to test the binman symbols feature. Increase it to 15 bytes.

Using very small regions is useful since we can easily compare them in
tests and errors are fairly easy to diagnose.

Signed-off-by: Simon Glass 
---

 tools/binman/ftest.py | 12 ++--
 tools/binman/test/21_image_pad.dts|  2 +-
 tools/binman/test/24_sorted.dts   |  4 ++--
 tools/binman/test/28_pack_4gb_outside.dts |  4 ++--
 tools/binman/test/29_x86-rom.dts  |  6 +++---
 5 files changed, 14 insertions(+), 14 deletions(-)

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v3] power: extend prefix match to regulator-name property

[sjg]

On 27 November 2017 at 01:14, Felix Brack  wrote:
> This patch extends pmic_bind_children prefix matching. In addition to
> the node name the property regulator-name is used while trying to match
> prefixes. This allows assigning different drivers to regulator nodes
> named regulator@1 and regulator@10 for example.
> I have discarded the idea of using other properties then regulator-name
> as I do not see any benefit in using property compatible or even
> regulator-compatible. Of course I am open to change this if there are
> good reasons to do so.
>
> Signed-off-by: Felix Brack 
> ---
>
> Changes in v3:
> - refactoring improving code readability
>
> Changes in v2:
> - add documentation
> - add a regulator to the sandbox for testing
> - extend the test for the new sandbox regulator
>
>  arch/sandbox/dts/sandbox_pmic.dtsi   |  6 ++
>  doc/device-tree-bindings/regulator/regulator.txt | 16 ++--
>  drivers/power/pmic/pmic-uclass.c | 11 +--
>  include/power/sandbox_pmic.h |  5 -
>  test/dm/regulator.c  |  2 ++
>  5 files changed, 35 insertions(+), 5 deletions(-)

Reviewed-by: Simon Glass 

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] dm: blk: Use uclass_find_first/next_device() in blk_first/next_device()

[sjg]

On 29 November 2017 at 08:46, Stefan Roese  wrote:
> This patch changes the calls to uclass_first/next_device() in blk_first/
> next_device() to use uclass_find_first/next_device() instead. These functions
> don't prepare the devices, which is correct in this case.
>
> With this patch applied, the "usb storage" command now works again as
> expected:
>
> => usb storage
>   Device 0: Vendor: SanDisk Rev: 1.00 Prod: Ultra
>   Type: Removable Hard Disk
>   Capacity: 58656.0 MB = 57.2 GB (120127488 x 512)
>
> Without this patch, it used to generate this buggy output:
>
> => usb storage
> Card did not respond to voltage select!
> mmc_init: -95, time 26
> No storage devices, perhaps not 'usb start'ed..?
>
> Signed-off-by: Stefan Roese 
> Suggested-by: Simon Glass 
> Cc: Simon Glass 
> Cc: Bin Meng 
> ---
>  drivers/block/blk-uclass.c | 7 ---
>  1 file changed, 4 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass 

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4] power: pmic/regulator: Add basic support for TPS65910

[sjg]

On 30 November 2017 at 05:52, Felix Brack  wrote:
> Texas Instrument's TPS65910 PMIC contains 3 buck DC-DC converts, one
> boost DC-DC converter and 8 LDOs. This patch implements driver model
> support for the TPS65910 PMIC and its regulators making the get/set
> API for regulator value/enable available.
> This patch depends on the patch "am33xx: Add a function to query MPU
> voltage in uV" to build correctly. For boards relying on the DT
> include file tps65910.dtsi the v3 patch "power: extend prefix match
> to regulator-name property" and an appropriate regulator naming is
> also required.
>
> Signed-off-by: Felix Brack 
> ---
>
> Changes in v4:
> - remove unnecessary test of unit number since it can't be negative
>
> Changes in v3:
> - use dev_get_driver_data() instead of dev->driver_data
> - use dev_get_platdata() instead of dev->platdata
> - store unit address in platdata during ofdata_to_platdata() instead
>   of queering it multiple times by dev_get_driver_data()
> - use pmic_reg_read() instead of pmic_read() wherever possible
> - eliminate the use small data types like u8 if not required
>
> Changes in v2:
> - move information about regulator supply power from the PMIC to the
>   regulator driver which greatly simplifies and streamlines the code
>   of both drivers (thanks Simon!)
> - use dev_get_uclass_platdata() to get the regulator constraints
>   instead of reinventing the wheel and reading them from the DT
> - make pmic_tps65910_write() and pmic_tps65910_read() return the
>   results of the i2c operation instead of obscuring it
> - generally review and eventually fix return values
> - modify regulator description in Kconfig
> - get_ctrl_reg_from_unit_addr() uses a lookup array instead of a
>   switch statement
> - rename some local functions to prevent confusion with generic
>   functions
> - reformatting code for better readability
>
>  drivers/power/pmic/Kconfig   |   8 +
>  drivers/power/pmic/Makefile  |   1 +
>  drivers/power/pmic/pmic_tps65910_dm.c|  98 ++
>  drivers/power/regulator/Kconfig  |   8 +
>  drivers/power/regulator/Makefile |   1 +
>  drivers/power/regulator/tps65910_regulator.c | 459 
> +++
>  include/power/tps65910_pmic.h| 130 
>  7 files changed, 705 insertions(+)
>  create mode 100644 drivers/power/pmic/pmic_tps65910_dm.c
>  create mode 100644 drivers/power/regulator/tps65910_regulator.c
>  create mode 100644 include/power/tps65910_pmic.h

Reviewed-by: Simon Glass 

Applied to u-boot-dm thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] Please pull u-boot-mpc85xx master

[York Sun]

Tom,

The following changes since commit 9da7fb4a39149c3061cb148bfbaa76b4b52b9008:

  Prepare v2018.01-rc1 (2017-12-04 18:27:17 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-mpc85xx.git

for you to fetch changes up to 316f0d0f8f3cbeeefda043e69f3e4e6aa57f7f1d:

  powerpc: mpc85xx: Fix static TLB table for SDRAM (2017-12-06 14:54:12
-0800)


Ran Wang (2):
  powerpc/T104xRDB: Fix endian access issue on EHCI intinalization
  powerpc/p1_p2_rdb_pc: Fix endian access issue on EHCI intinalization

York Sun (1):
  powerpc: mpc85xx: Fix static TLB table for SDRAM

 board/Arcturus/ucp1020/tlb.c   | 2 +-
 board/freescale/b4860qds/tlb.c | 2 +-
 board/freescale/bsc9131rdb/tlb.c   | 2 +-
 board/freescale/bsc9132qds/tlb.c   | 2 +-
 board/freescale/c29xpcie/tlb.c | 4 ++--
 board/freescale/mpc8541cds/tlb.c   | 2 +-
 board/freescale/mpc8548cds/tlb.c   | 2 +-
 board/freescale/mpc8568mds/tlb.c   | 2 +-
 board/freescale/p1010rdb/tlb.c | 2 +-
 board/freescale/p1022ds/tlb.c  | 4 ++--
 board/freescale/p1023rdb/tlb.c | 4 ++--
 board/freescale/p1_p2_rdb_pc/tlb.c | 2 +-
 board/freescale/p1_twr/tlb.c   | 2 +-
 board/freescale/t102xqds/tlb.c | 4 ++--
 board/freescale/t102xrdb/tlb.c | 4 ++--
 board/freescale/t104xrdb/tlb.c | 4 ++--
 board/freescale/t208xqds/tlb.c | 2 +-
 board/freescale/t208xrdb/tlb.c | 2 +-
 board/freescale/t4qds/tlb.c| 2 +-
 board/freescale/t4rdb/tlb.c| 2 +-
 board/gdsys/p1022/tlb.c| 2 +-
 board/sbc8548/tlb.c| 4 ++--
 include/configs/T104xRDB.h | 1 +
 include/configs/p1_p2_rdb_pc.h | 1 +
 24 files changed, 31 insertions(+), 29 deletions(-)

Thanks.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc: mpc85xx: Fix static TLB table for SDRAM

[York Sun]

On 12/06/2017 12:42 PM, York Sun wrote:
> Most predefined TLB tables don't have memory coherence bit set for
> SDRAM. This wasn't an issue before invalidate_dcache_range() function
> was enabled. Without the coherence bit, dcache invalidation doesn't
> automatically flush the cache. The coherence bit is already set when
> dynamic TLB table is used. For some boards with different SPL boot
> method, or with legacy fixed setting, this bit needs to be set in
> TLB files.
> 
> Signed-off-by: York Sun 
> ---


Applied to u-boot-mpc85xx master.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/p1_p2_rdb_pc: Fix endian access issue on EHCI intinalization

[York Sun]

On 11/26/2017 07:10 PM, Ran Wang wrote:
> This issue is exposed after commit 9000eddbae0d
> ("drivers/usb/ehci: Use platform-specific accessors"),
> the wrong endian way of EHCI controller programing will cause
> USB function down.
> 
> Signed-off-by: Ran Wang 
> ---

Applied to u-boot-mpc85xx master. Thanks.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/2] powerpc/T104xRDB: Fix endian access issue on EHCI intinalization

[York Sun]

On 11/26/2017 07:10 PM, Ran Wang wrote:
> This issue is exposed after commit 9000eddbae0d
> ("drivers/usb/ehci: Use platform-specific accessors"),
> the wrong endian way of EHCI controller programing will cause
> USB function down.
> 
> Signed-off-by: Ran Wang 
> ---


Applied to u-boot-mpc85xx master. Thanks.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2] test: py: Add an option to skip sleep test

[Stephen Warren]

On 12/08/2017 07:47 AM, Michal Simek wrote:

Some QEMUs have a problem with time setup that's why
sleep test is failing. Introduce env__sleep_accurate
boardenv variable to have an option to skip sleep test.


Nit: Commit descriptions are typically wrapped around 70-74 characters; 
the above is very narrow.



diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py



+"""
+Note: This test doesn't rely on boardenv_* configuration values but they can
+change test behavior.
+
+# Setup env__sleep_accurate to False if time is not accurate on your platform
+env__sleep_accurate = False
+
+"""


Nit: No need for that trailing empty line in the docstring.

Reviewed-by: Stephen Warren 

(I was going to contemplate a more generic system implemented in 
conftest.py that could work with any test, but I guess this is simple 
enough and solves the only issue we have right now, so it's fine.)

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: ls1088a: Add nand support for ls1088ardb

[York Sun]

On 11/27/2017 09:22 PM, Ashish Kumar wrote:
> Signed-off-by: Ashish Kumar 
> ---

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] driver: fsl-mc: MC object cleanup when DPL not loaded

[York Sun]

On 11/27/2017 08:41 PM, Yogesh Gaur wrote:
> For case when MC is loaded but DPL is not deployed perform MC
> object [DPBP, DPIO, DPNI and DPRC] cleanup.
> 
> Signed-off-by: Yogesh Gaur 
> ---
>  drivers/net/fsl-mc/mc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)


Applied to fsl-qoriq master. Thanks.

York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [Patch v3] LS2080A_SECURE_BOOT: Enable CONFIG_FSL_LS_PPA.

[York Sun]

On 11/21/2017 07:32 PM, Udit Agarwal wrote:
> Adds config CONFIG_FSL_LS_PPA and CONFIG_FSL_CAAM in
> LS2080AQDS and LS2080ARDB secure boot defconfig.
> 
> Removes CONFIG_FIT, as with CONFIG_FSL_LS_PPA enabled,
> CONFIG_FIT is selected.
> 
> Removes CONFIG_SPL_RSA as in NOR boot SPL boot is not done.
> 
> Signed-off-by: Udit Agarwal 
> ---
> Changes in v3:
> Corrects commit message for the patch.
> 

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2] SECURE BOOT: Add fall back option

[York Sun]

On 11/21/2017 09:09 PM, Vinitha Pillai-B57223 wrote:
> Add fall back option, to boot from NOR/QSPI/SD for LS1043, LS1046,
> LS1021 in case of distro boot failure.
> 
> LS1046: Add kernel validation in case of secure boot in sd_bootcmd
> and qspi_bootcmd
> LS1043 and LS1021: Add kernel validation in case of secure boot in
> sd_bootcmd, qspi_bootcmdand  nor_bootcmd
> 
> Signed-off-by: Vinitha Pillai 
> ---


Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4 2/2] LS1088A_QSPI: Add secure boot defconfigs for QSPI boot.

[York Sun]

On 11/21/2017 07:32 PM, Udit Agarwal wrote:
> Add the secure boot defconfigs for QSPI boot on LS1088ARDB
> and LS1088AQDS platforms.
> 
> Signed-off-by: Udit Agarwal 
> Signed-off-by: Vinitha Pillai-B57223 
> ---
> Changes in v4:
> Adds Distro boot support defconfigs.
> Adds config to unset CONFIG_BOOTCOOMAND.
> Keeping defconfig options in sorted manner.
> 

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v4 1/2] LS1088A_QSPI: SECURE_BOOT: Images validation

[York Sun]

On 11/21/2017 07:31 PM, Udit Agarwal wrote:
> Validates PPA, MC, DPC, Bootscript, DPL and Kernel images in ESBC phase
> using esbc_validate command.
> 
> Enable validation of boot.scr script prior to its execution dependent
> on "secureboot" flag in environment
> 
> Add header address for PPA to be validated during ESBC phase for LS1088A
> platform based on LAyerscape Chasis 3.
> 
> Moves sec_init prior to ppa_init as for validation of PPA sec must
> be initialised before the PPA is initialised.
> 
> Signed-off-by: Udit Agarwal 
> Signed-off-by: Vinitha Pillai-B57223 
> Signed-off-by: Sumit Garg 
> ---
> Changes in v4:
> Adds Distro boot support.
> Removes CONFIG_FSL_CAAM. Will be enabled via defconfig

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH V3] armv8: ls1012a: Modify Kernel and Environment offset

[York Sun]

On 11/13/2017 08:25 PM, Bhaskar Upadhaya wrote:
> Kernel is now located at 0x100 instead of 0xa0
> and envirorment variables are located at 3MB offset instead of
> 2MB in Flash
> 
> Signed-off-by: Bhaskar Upadhaya 
> ---
> Changes for V3: Updated Commit message
> 

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: Workaround for USB erratua on LS1012A

[York Sun]

On 11/13/2017 12:33 AM, Ran Wang wrote:
> This is suplement for patch which handle below errata:
> A-009007, A-009008, A-008997, A-009798
> 
> Signed-off-by: Ran Wang 
> ---
>  arch/arm/cpu/armv8/fsl-layerscape/Kconfig |  4 
>  arch/arm/cpu/armv8/fsl-layerscape/soc.c   | 27 ---
>  2 files changed, 24 insertions(+), 7 deletions(-)
> 

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH][v2] armv8: fsl-layerscape: Add support of disabling core prefetch

[York Sun]

On 11/09/2017 10:03 PM, Prabhakar Kushwaha wrote:
> Instruction prefetch feature is by default enabled during core
> release.
> 
> This patch add support of disabling instruction prefetch by setting
> core mask in PPA. Here each core mask bit represents a core and
> prefetch is disabled at the time of core release.
> 
> Signed-off-by: Prabhakar Kushwaha 
> ---
> Changes for v2:  Updated README
> 

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: ls1088: Add fsl_fdt_fixup_flash

[York Sun]

On 11/08/2017 09:44 PM, Ashish Kumar wrote:
> IFC-NOR and QSPI-NOR signals are muxed on SoC to save pins
> 
> Add fsl_fdt_fixup_flash() to disable IFC-NOR node in dts
> if QSPI is enabled and vice-versa
> 
> Signed-off-by: Ashish Kumar 
> ---
>  board/freescale/ls1088a/ls1088a.c | 29 +
>  1 file changed, 29 insertions(+)


Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH][v3] driver: net: fsl-mc: flib changes for MC 10.3.0

[York Sun]

On 11/14/2017 10:29 PM, Yogesh Gaur wrote:
> Existing MC driver framework is based on MC-9.x.x flib.
> This patch migrates MC obj (DPBP, DPNI, DPRC, DPMAC etc) to use latest
> MC flib which is MC-10.3.0.
> 
> Changes introduced due to migration:
> 1. To get OBJ token, pair of create and open API replaces create APIs
> 2. Pair of close and destroy APIs replaces destroy APIs
> 3. For version read, get_version APIs replaces get_attributes APIs
> 4. dpni_get/reset_statistics APIs replaces dpni_get/set_counter APIs
> 5. Simplifies struct dpni_cfg and removes dpni_extended_cfg struct
> 6. Single API dpni_get_buffer_layout/set_buffer_layout replaces
> dpni_get_rx/set_rx, tx related, tx_conf_buffer_layout
> related APIs. New API takes a queue type as an argument.
> 7. Similarly dpni_get_queue/set_queue replaces
> dpni_get_rx_flow/set_rx_flow , tx_flow related,
> tx_conf related APIs
> 
> Signed-off-by: Yogesh Gaur 
> Signed-off-by: Priyanka Jain 
> ---
> Changes in v2:
>  Updated flib version check to make compatible with
>  future MC flib version.
> 
> Changes in v3:
>  Updated copyright years.

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: ls1088a: Unset USE_BOOTCOMMAND in defconfig

[York Sun]

On 11/23/2017 01:03 AM, Ashish Kumar wrote:
> Unset USE_BOOTCOMMAND for all ls1088 defconfig files to fix
> redefinition error
> 
> USE_BOOTCOMMAND was introduced in commit-id b6251db8c3f
> ("Kconfig: Introduce USE_BOOTCOMMAND and migrate BOOTCOMMAND").
> 
> Signed-off-by: Ashish Kumar 
> ---
>  configs/ls1088aqds_qspi_defconfig| 1 +
>  configs/ls1088aqds_sdcard_qspi_defconfig | 1 +
>  configs/ls1088ardb_qspi_defconfig| 1 +
>  configs/ls1088ardb_sdcard_qspi_defconfig | 1 +
>  4 files changed, 4 insertions(+)

Applied to fsl-qoriq master. Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] zynqmp: Access timestamp_ref_ctrl register only if running in el3

[Michal Simek]

From: Siva Durga Prasad Paladugu 

Access the timestamp ref ctrl register only if runinng
at el3 level otherwise just return. This change fixes
the issue when CRL APB is marked as secure and accessing
when not in el3 causes exception.

Signed-off-by: Siva Durga Prasad Paladugu 
Signed-off-by: Michal Simek 
---

 board/xilinx/zynqmp/zynqmp.c | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index 44af49782cce..c198a4d92063 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -277,10 +277,13 @@ int board_early_init_r(void)
 {
u32 val;
 
+   if (current_el() != 3)
+   return 0;
+
val = readl(_base->timestamp_ref_ctrl);
val &= ZYNQMP_CRL_APB_TIMESTAMP_REF_CTRL_CLKACT;
 
-   if (current_el() == 3 && !val) {
+   if (!val) {
val = readl(_base->timestamp_ref_ctrl);
val |= ZYNQMP_CRL_APB_TIMESTAMP_REF_CTRL_CLKACT;
writel(val, _base->timestamp_ref_ctrl);
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] tools: zynqmpimage: Check return values from file functions

[Michal Simek]

Check all return values from file functions.
In case of negative return exit immediately.
Also change fsize return value which can't be negative.

Reported-by: Coverity (CID: 23276, 23304, 169357)
Signed-off-by: Michal Simek 
---

 tools/zynqmpimage.c | 32 +++-
 1 file changed, 27 insertions(+), 5 deletions(-)

diff --git a/tools/zynqmpimage.c b/tools/zynqmpimage.c
index 9667b11b2f8a..f48ac6dbe505 100644
--- a/tools/zynqmpimage.c
+++ b/tools/zynqmpimage.c
@@ -245,16 +245,38 @@ static int zynqmpimage_check_image_types(uint8_t type)
return EXIT_FAILURE;
 }
 
-static int fsize(FILE *fp)
+static uint32_t fsize(FILE *fp)
 {
-   int size;
-   int origin = ftell(fp);
+   int size, ret, origin;
+
+   origin = ftell(fp);
+   if (origin < 0) {
+   fprintf(stderr, "Incorrect file size\n");
+   fclose(fp);
+   exit(2);
+   }
+
+   ret = fseek(fp, 0L, SEEK_END);
+   if (ret) {
+   fprintf(stderr, "Incorrect file SEEK_END\n");
+   fclose(fp);
+   exit(3);
+   }
 
-   fseek(fp, 0L, SEEK_END);
size = ftell(fp);
+   if (size < 0) {
+   fprintf(stderr, "Incorrect file size\n");
+   fclose(fp);
+   exit(4);
+   }
 
/* going back */
-   fseek(fp, origin, SEEK_SET);
+   ret = fseek(fp, origin, SEEK_SET);
+   if (ret) {
+   fprintf(stderr, "Incorrect file SEEK_SET to %d\n", origin);
+   fclose(fp);
+   exit(3);
+   }
 
return size;
 }
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] common/memsize.c: restore content of the base address

[Patrick DELAUNAY]

> From: Wolfgang Denk [mailto:w...@denx.de]
> Subject: Re: [U-Boot] [PATCH] common/memsize.c: restore content of the base
> address
> 
> Dear Patrick,
> 
> In message <6daf1478e4284b8590c2862c6a504...@sfhdag6node3.st.com>
> you wrote:
> >
> > After investigation, I found an potential issue when the current code
> > of get_ram_size() is loaded near of power of 2 offset (just before an 
> > address
> modified by the code)...
> > In fact the content of the RAM is modified during the code execution
> > just after the PC address, and the code is not yet in instruction cache, so 
> > this
> the code crash.
> > I try to found a good solution (limit the min offset to be tested, to
> > avoid to modify the address used by U-Boot code)
> 
> You mean you are running this code from the very memory you are sizing?  This
> is fundamentally broken.  You must not do this!
>

Yes I do it, sorry if it is a error.

But it is recommended in ./doc/README.arm-relocation:
 At board level:

dram_init(): bd pointer is now at this point not accessible, so only
detect the real dramsize, and store it in gd->ram_size. Bst detected
with get_ram_size().

And before to setup by board I check on many device with ARM architecture.
Most the time U-Boot loaded at beginning of the RAM, relocated at the end of 
the RAM 
and dram_init use get_ram_size
(I read also many debate about the end of relocation even when U-Boot in loaded 
in RAM on U-Boot mainling list)

Example1 :
  ./board/toradex/colibri_imx6/colibri_imx6.c
  int dram_init(void)
  {
/* use the DDR controllers configured size */
gd->ram_size = get_ram_size((void *)CONFIG_SYS_SDRAM_BASE,
(ulong)imx_ddr_size());

return 0;
  }

./arch/arm/include/asm/arch-mx6/imx-regs.h:118:#define MMDC0_ARB_BASE_ADDR  
   0x1000
./include/configs/colibri_imx6.h:249:#define PHYS_SDRAM 
MMDC0_ARB_BASE_ADDR
./include/configs/colibri_imx6.h:251:#define CONFIG_SYS_SDRAM_BASE  
PHYS_SDRAM
./include/configs/colibri_imx6.h:126:#define CONFIG_SYS_TEXT_BASE   
0x1780

Example 2
./arch/arm/mach-omap2/am33xx/board.c: int dram_init(void)
/* dram_init must store complete ramsize in gd->ram_size */
gd->ram_size = get_ram_size(
(void *)CONFIG_SYS_SDRAM_BASE,
CONFIG_MAX_RAM_BANK_SIZE);

./include/configs/bur_am335x_common.h:67:#define CONFIG_SYS_SDRAM_BASE  
0x8000
./include/configs/siemens-am33x-common.h:155:#define CONFIG_SYS_TEXT_BASE   
0x8010

So I just do the same for my arm/armv7 platform.

On ARM device device, I think we don't have the banks...

> get_ram_size() is supposed to run from some _different_ memory (at least a
> different memopry bank, usually better from a different memory type).

 I agree that the function is not safe in this case.
So I will change my board to remove this call

> > You can found analysis in :
> >   [U-Boot] questions about get_ram_size usage in U-Boot = DDR modification
> during code execution
> >   From Patrick DELAUNAY Mon Apr 24 16:11:41 UTC 2017
> >   https://lists.denx.de/pipermail/u-boot/2017-April/288709.html
> 
> You write there:
> 
> | Today, I found a issue with  get_ram_size() usage in U-Boot, when
> | U-Boot is executed in the tested memory.
> 
> You must not do this.  The memory under test is supposed to be otherwise idle.
> 
> > and finally the function return max size
> 
> For the test to work correctly, max test size should be chosen at least twice 
> as
> big as the maximum possible memory configuration.

This requirement seems strange to me

I have 32 bits ARM platform, with addressable RAM area 1GiB
DDR start => 0xC000
DDR end => 0x (size = 0x4000)

If I need to test with 2Gib, the code will test access to 0xC000 + max_size 
/ 2 = 0x1 
=> greater than 32bits access to 0x0 address and crash (not allowed to access 
to 0x0 = bootrom)

For me the function is working correctly with  max test size chosen at the 
maximum memory size
=> no overlap detected, so retrun max_size

> 
> > Perhaps other board failed because size of function code increase (not
> > fully inside instruction cache page)
> 
> No.  We are talking of boards that were implemented properly, i. e.
> the tested RAM was NOT the memory where get_ram_size() was running from.
> 
> > or it  is the same issue indicated previously: the address of function 
> > change and
> so code change during execution.
> 
> Code location or size does not play a role here.  The code was running from
> parallel NOR flash, and testing the actual RAM size, so thsi is totally
> independent.

OK, sorry
> 
> > If the same patch come again I think that this function have really a
> > problem and we could solve the issue (with my patch for exmale) and then
> understood the regression on other the boards.
> 
> I agree there appears to be a problem,  but it should be 

Re: [U-Boot] [PATCH v1] test: py: Add an option to skip sleep test

[Michal Simek]

On 8.12.2017 15:47, Tom Rini wrote:
> On Fri, Dec 08, 2017 at 03:44:58PM +0100, Michal Simek wrote:
>> On 8.12.2017 15:34, Tom Rini wrote:
>>> On Fri, Dec 08, 2017 at 02:01:29PM +0100, Michal Simek wrote:
 Some QEMUs have a problem with time setup that's why
 sleep test is failing. Introduce env__sleep_accurate
 boardenv variable to have an option to skip sleep test.

 Signed-off-by: Michal Simek 
 ---

 Changes in v1:
 - Change subject line and patch description
 - Move from RFC to regular patch
 - Remove boardidentity marker and use variable instead

  configs/zynq_zc706_defconfig |  1 +
>>>
>>> This looks unrelated.
>>
>> yes it is sorry - I have tested it on local zc706
>> Will fix in v2.
>>
>>>
  test/py/tests/test_sleep.py  | 13 +
  2 files changed, 14 insertions(+)

 diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
 index 52999debc115..762b892f3fcf 100644
 --- a/configs/zynq_zc706_defconfig
 +++ b/configs/zynq_zc706_defconfig
 @@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y
  CONFIG_CMD_EXT4_WRITE=y
  CONFIG_CMD_FAT=y
  CONFIG_CMD_FS_GENERIC=y
 +CONFIG_OF_EMBED=y
  CONFIG_ENV_IS_IN_SPI_FLASH=y
  CONFIG_NET_RANDOM_ETHADDR=y
  CONFIG_SPL_DM_SEQ_ALIAS=y
 diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
 index 64e057132622..ccef24d7f7b8 100644
 --- a/test/py/tests/test_sleep.py
 +++ b/test/py/tests/test_sleep.py
 @@ -5,10 +5,23 @@
  import pytest
  import time
  
 +"""
 +Note: This test doesn't rely on boardenv_* configuration values but they 
 can
 +change test behavior.
 +
 +# Setup env__sleep_accurate to False if time is not accurate on your 
 platform
 +env__sleep_accurate = False
>>>
>>> This should default to true.  Then a PR to uboot-test-hooks where you
>>> say the file for zynq_zc702 has this false, thanks!
>>
>> This is just comment in that file. By default you can set it up to True
>> or just not setup at all.
> 
> Ah, OK, I was reading too quickly.  Thanks!

No worries. Will wait for Stephen and then will send PR to
uboot-test-hooks and also I can remove sleep from .travis.yml.

Thanks,
Michal

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2] test: py: Add an option to skip sleep test

[Tom Rini]

On Fri, Dec 08, 2017 at 03:47:18PM +0100, Michal Simek wrote:

> Some QEMUs have a problem with time setup that's why
> sleep test is failing. Introduce env__sleep_accurate
> boardenv variable to have an option to skip sleep test.
> 
> Signed-off-by: Michal Simek 

Reviewed-by: Tom Rini 

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v1] test: py: Add an option to skip sleep test

[Tom Rini]

On Fri, Dec 08, 2017 at 03:44:58PM +0100, Michal Simek wrote:
> On 8.12.2017 15:34, Tom Rini wrote:
> > On Fri, Dec 08, 2017 at 02:01:29PM +0100, Michal Simek wrote:
> >> Some QEMUs have a problem with time setup that's why
> >> sleep test is failing. Introduce env__sleep_accurate
> >> boardenv variable to have an option to skip sleep test.
> >>
> >> Signed-off-by: Michal Simek 
> >> ---
> >>
> >> Changes in v1:
> >> - Change subject line and patch description
> >> - Move from RFC to regular patch
> >> - Remove boardidentity marker and use variable instead
> >>
> >>  configs/zynq_zc706_defconfig |  1 +
> > 
> > This looks unrelated.
> 
> yes it is sorry - I have tested it on local zc706
> Will fix in v2.
> 
> > 
> >>  test/py/tests/test_sleep.py  | 13 +
> >>  2 files changed, 14 insertions(+)
> >>
> >> diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
> >> index 52999debc115..762b892f3fcf 100644
> >> --- a/configs/zynq_zc706_defconfig
> >> +++ b/configs/zynq_zc706_defconfig
> >> @@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y
> >>  CONFIG_CMD_EXT4_WRITE=y
> >>  CONFIG_CMD_FAT=y
> >>  CONFIG_CMD_FS_GENERIC=y
> >> +CONFIG_OF_EMBED=y
> >>  CONFIG_ENV_IS_IN_SPI_FLASH=y
> >>  CONFIG_NET_RANDOM_ETHADDR=y
> >>  CONFIG_SPL_DM_SEQ_ALIAS=y
> >> diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
> >> index 64e057132622..ccef24d7f7b8 100644
> >> --- a/test/py/tests/test_sleep.py
> >> +++ b/test/py/tests/test_sleep.py
> >> @@ -5,10 +5,23 @@
> >>  import pytest
> >>  import time
> >>  
> >> +"""
> >> +Note: This test doesn't rely on boardenv_* configuration values but they 
> >> can
> >> +change test behavior.
> >> +
> >> +# Setup env__sleep_accurate to False if time is not accurate on your 
> >> platform
> >> +env__sleep_accurate = False
> > 
> > This should default to true.  Then a PR to uboot-test-hooks where you
> > say the file for zynq_zc702 has this false, thanks!
> 
> This is just comment in that file. By default you can set it up to True
> or just not setup at all.

Ah, OK, I was reading too quickly.  Thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v2] test: py: Add an option to skip sleep test

[Michal Simek]

Some QEMUs have a problem with time setup that's why
sleep test is failing. Introduce env__sleep_accurate
boardenv variable to have an option to skip sleep test.

Signed-off-by: Michal Simek 
---

Changes in v2:
- Remove unrelated zc706 change

Changes in v1:
- Change subject line and patch description
- Move from RFC to regular patch
- Remove boardidentity marker and use variable instead

 test/py/tests/test_sleep.py | 13 +
 1 file changed, 13 insertions(+)

diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
index 64e057132622..ccef24d7f7b8 100644
--- a/test/py/tests/test_sleep.py
+++ b/test/py/tests/test_sleep.py
@@ -5,10 +5,23 @@
 import pytest
 import time
 
+"""
+Note: This test doesn't rely on boardenv_* configuration values but they can
+change test behavior.
+
+# Setup env__sleep_accurate to False if time is not accurate on your platform
+env__sleep_accurate = False
+
+"""
+
 def test_sleep(u_boot_console):
 """Test the sleep command, and validate that it sleeps for approximately
 the correct amount of time."""
 
+sleep_skip = u_boot_console.config.env.get('env__sleep_accurate', True)
+if not sleep_skip:
+pytest.skip('sleep is not accurate')
+
 if u_boot_console.config.buildconfig.get('config_cmd_misc', 'n') != 'y':
 pytest.skip('sleep command not supported')
 # 3s isn't too long, but is enough to cross a few second boundaries.
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v1] test: py: Add an option to skip sleep test

[Michal Simek]

On 8.12.2017 15:34, Tom Rini wrote:
> On Fri, Dec 08, 2017 at 02:01:29PM +0100, Michal Simek wrote:
>> Some QEMUs have a problem with time setup that's why
>> sleep test is failing. Introduce env__sleep_accurate
>> boardenv variable to have an option to skip sleep test.
>>
>> Signed-off-by: Michal Simek 
>> ---
>>
>> Changes in v1:
>> - Change subject line and patch description
>> - Move from RFC to regular patch
>> - Remove boardidentity marker and use variable instead
>>
>>  configs/zynq_zc706_defconfig |  1 +
> 
> This looks unrelated.

yes it is sorry - I have tested it on local zc706
Will fix in v2.

> 
>>  test/py/tests/test_sleep.py  | 13 +
>>  2 files changed, 14 insertions(+)
>>
>> diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
>> index 52999debc115..762b892f3fcf 100644
>> --- a/configs/zynq_zc706_defconfig
>> +++ b/configs/zynq_zc706_defconfig
>> @@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y
>>  CONFIG_CMD_EXT4_WRITE=y
>>  CONFIG_CMD_FAT=y
>>  CONFIG_CMD_FS_GENERIC=y
>> +CONFIG_OF_EMBED=y
>>  CONFIG_ENV_IS_IN_SPI_FLASH=y
>>  CONFIG_NET_RANDOM_ETHADDR=y
>>  CONFIG_SPL_DM_SEQ_ALIAS=y
>> diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
>> index 64e057132622..ccef24d7f7b8 100644
>> --- a/test/py/tests/test_sleep.py
>> +++ b/test/py/tests/test_sleep.py
>> @@ -5,10 +5,23 @@
>>  import pytest
>>  import time
>>  
>> +"""
>> +Note: This test doesn't rely on boardenv_* configuration values but they can
>> +change test behavior.
>> +
>> +# Setup env__sleep_accurate to False if time is not accurate on your 
>> platform
>> +env__sleep_accurate = False
> 
> This should default to true.  Then a PR to uboot-test-hooks where you
> say the file for zynq_zc702 has this false, thanks!

This is just comment in that file. By default you can set it up to True
or just not setup at all.

Thanks,
Michal
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 5/6] arm64: zynqmp: Enable spi flashes

[Michal Simek]

Enabling all spi flashes because some of these boards can have different
flashes compared to public version.

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_ep_defconfig   | 3 +++
 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 6 ++
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 1 +
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
 6 files changed, 13 insertions(+)

diff --git a/configs/xilinx_zynqmp_ep_defconfig 
b/configs/xilinx_zynqmp_ep_defconfig
index 38f7983b0298..1da44bf1e460 100644
--- a/configs/xilinx_zynqmp_ep_defconfig
+++ b/configs/xilinx_zynqmp_ep_defconfig
@@ -64,6 +64,9 @@ CONFIG_NAND=y
 CONFIG_NAND_ARASAN=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_SPANSION=y
+CONFIG_SPI_FLASH_STMICRO=y
+CONFIG_SPI_FLASH_WINBOND=y
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index b3227d0d087a..349ac8a4c91d 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -58,6 +58,7 @@ CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index 502d446ff6d6..dcd660785ae0 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -44,6 +44,12 @@ CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
+CONFIG_SPI_FLASH=y
+CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_MACRONIX=y
+CONFIG_SPI_FLASH_SPANSION=y
+CONFIG_SPI_FLASH_STMICRO=y
+CONFIG_SPI_FLASH_WINBOND=y
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index f0842e625282..8379e1c20929 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -59,6 +59,7 @@ CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index c239b478d44a..3cc1384a9a6a 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -59,6 +59,7 @@ CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index 096a27c3b5a0..9945499f56eb 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -59,6 +59,7 @@ CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
+CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 4/6] arm64: zynqmp: Enable AES command

[Michal Simek]

Enable AES command for zynqmp boards.

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 1 +
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 1 +
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
 7 files changed, 7 insertions(+)

diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index a1b1e801e9b4..b3227d0d087a 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -37,6 +37,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
index 67a6d9ae9b60..eadee00d6be3 100644
--- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
@@ -37,6 +37,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index 36aff62dac73..502d446ff6d6 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -26,6 +26,7 @@ CONFIG_CMD_MMC=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
index ac565ecf8f9c..0ebe16c7dc4b 100644
--- a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
@@ -27,6 +27,7 @@ CONFIG_CMD_MMC=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index 46e992eda502..f0842e625282 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -38,6 +38,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index 7de7f2d237b8..c239b478d44a 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -38,6 +38,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index 0fd76126b14f..096a27c3b5a0 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -38,6 +38,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_TFTPPUT=y
 CONFIG_CMD_TIME=y
 CONFIG_CMD_TIMER=y
+CONFIG_CMD_AES=y
 CONFIG_CMD_EXT4_WRITE=y
 # CONFIG_SPL_ISO_PARTITION is not set
 CONFIG_SPL_OF_CONTROL=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 3/6] arm64: zynqmp: Enable fpga bitstream loading

[Michal Simek]

Enable fpga loadb and loadbp for zynqmp platforms.

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 2 ++
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 2 ++
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 2 ++
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 2 ++
 7 files changed, 14 insertions(+)

diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index 24c623253d67..a1b1e801e9b4 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -27,6 +27,8 @@ CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_GPT=y
 CONFIG_CMD_I2C=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
index dc74360da888..67a6d9ae9b60 100644
--- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
@@ -27,6 +27,8 @@ CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_GPT=y
 CONFIG_CMD_I2C=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index ef072323f3cf..36aff62dac73 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -19,6 +19,8 @@ CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_TFTPPUT=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
index cb0e7874d410..ac565ecf8f9c 100644
--- a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
@@ -20,6 +20,8 @@ CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_I2C=y
 CONFIG_CMD_MMC=y
 CONFIG_CMD_TFTPPUT=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index f144a4f7d1e5..46e992eda502 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -28,6 +28,8 @@ CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_GPT=y
 CONFIG_CMD_I2C=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index 3a3841e8f3eb..7de7f2d237b8 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -28,6 +28,8 @@ CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_GPT=y
 CONFIG_CMD_I2C=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index fb7e4c5c169b..0fd76126b14f 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -28,6 +28,8 @@ CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
+CONFIG_CMD_FPGA_LOADBP=y
+CONFIG_CMD_FPGA_LOADP=y
 CONFIG_CMD_GPIO=y
 CONFIG_CMD_GPT=y
 CONFIG_CMD_I2C=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 6/6] arm64: zynqmp: Dont use 4K sector erase by default for spi-flashes

[Michal Simek]

From: Siva Durga Prasad Paladugu 

Dont use 4K sector erase by default, Disabling this
would use 64K sector erase and decreases erase time.
Also disabled by the fact that UBIFS and JFFS2 won't work
with 4K sector erase.

Signed-off-by: Siva Durga Prasad Paladugu 
Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_ep_defconfig   | 1 +
 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 1 +
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 1 +
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
 6 files changed, 6 insertions(+)

diff --git a/configs/xilinx_zynqmp_ep_defconfig 
b/configs/xilinx_zynqmp_ep_defconfig
index 1da44bf1e460..cd45801ef452 100644
--- a/configs/xilinx_zynqmp_ep_defconfig
+++ b/configs/xilinx_zynqmp_ep_defconfig
@@ -67,6 +67,7 @@ CONFIG_SPI_FLASH_BAR=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index 349ac8a4c91d..e57b92cc5829 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -62,6 +62,7 @@ CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index dcd660785ae0..8e993e557068 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -50,6 +50,7 @@ CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index 8379e1c20929..2f340d368534 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -63,6 +63,7 @@ CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index 3cc1384a9a6a..92da876ac790 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -63,6 +63,7 @@ CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index 9945499f56eb..5251bfa49cdc 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -63,6 +63,7 @@ CONFIG_SPI_FLASH_MACRONIX=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_SPI_FLASH_WINBOND=y
+# CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_DM_ETH=y
 CONFIG_PHY_GIGE=y
 CONFIG_ZYNQ_GEM=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 1/6] arm64: zynqmp: Enable misc devices

[Michal Simek]

Enable misc devices for zynqmp targets.

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_ep_defconfig   | 1 +
 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 1 +
 configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 1 +
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 1 +
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 1 +
 8 files changed, 8 insertions(+)

diff --git a/configs/xilinx_zynqmp_ep_defconfig 
b/configs/xilinx_zynqmp_ep_defconfig
index 4e29cc782032..38f7983b0298 100644
--- a/configs/xilinx_zynqmp_ep_defconfig
+++ b/configs/xilinx_zynqmp_ep_defconfig
@@ -56,6 +56,7 @@ CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_DM_I2C=y
 CONFIG_SYS_I2C_CADENCE=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index c2805ac08125..70de9c944138 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -47,6 +47,7 @@ CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_DM_I2C=y
 CONFIG_SYS_I2C_CADENCE=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
index 3a5fe1331cf1..962e73374179 100644
--- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
@@ -46,6 +46,7 @@ CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_DM_I2C=y
 CONFIG_SYS_I2C_CADENCE=y
+CONFIG_MISC=y
 # CONFIG_MMC is not set
 CONFIG_DM_MMC=y
 CONFIG_NAND=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index 18f2eed7f0aa..c1c6fa94c44c 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -35,6 +35,7 @@ CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_DM_I2C=y
 CONFIG_SYS_I2C_CADENCE=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
index 75ebd223cdf4..889286b4454b 100644
--- a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
@@ -35,6 +35,7 @@ CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_DM_I2C=y
 CONFIG_SYS_I2C_CADENCE=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index a72781000c1e..0419ad2c9c2c 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -48,6 +48,7 @@ CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_CMD_PCA953X=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index 94116ec61b59..685c34ea6961 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -48,6 +48,7 @@ CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_CMD_PCA953X=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index d6c49cdfad5b..50779fd16334 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -48,6 +48,7 @@ CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_ZYNQMPPL=y
 CONFIG_DM_GPIO=y
 CONFIG_CMD_PCA953X=y
+CONFIG_MISC=y
 CONFIG_DM_MMC=y
 CONFIG_MMC_SDHCI=y
 CONFIG_MMC_SDHCI_ZYNQ=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 2/6] arm64: zynqmp: Enable SPL ram support

[Michal Simek]

Signed-off-by: Michal Simek 
---

 configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig | 2 ++
 configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig | 2 ++
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig| 2 ++
 configs/xilinx_zynqmp_zcu102_revA_defconfig  | 2 ++
 configs/xilinx_zynqmp_zcu102_revB_defconfig  | 2 ++
 7 files changed, 14 insertions(+)

diff --git a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
index 70de9c944138..24c623253d67 100644
--- a/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm015_dc1_defconfig
@@ -16,6 +16,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_FASTBOOT=y
 CONFIG_FASTBOOT_FLASH=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
index 962e73374179..dc74360da888 100644
--- a/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm016_dc2_defconfig
@@ -17,6 +17,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_FASTBOOT=y
 CONFIG_FASTBOOT_FLASH=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
index c1c6fa94c44c..ef072323f3cf 100644
--- a/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm018_dc4_defconfig
@@ -13,6 +13,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
diff --git a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig 
b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
index 889286b4454b..cb0e7874d410 100644
--- a/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
+++ b/configs/xilinx_zynqmp_zc1751_xm019_dc5_defconfig
@@ -14,6 +14,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_CMD_MEMTEST=y
 CONFIG_CMD_CLK=y
diff --git a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig 
b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
index 0419ad2c9c2c..f144a4f7d1e5 100644
--- a/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_rev1_0_defconfig
@@ -16,6 +16,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_FASTBOOT=y
 CONFIG_FASTBOOT_FLASH=y
diff --git a/configs/xilinx_zynqmp_zcu102_revA_defconfig 
b/configs/xilinx_zynqmp_zcu102_revA_defconfig
index 685c34ea6961..3a3841e8f3eb 100644
--- a/configs/xilinx_zynqmp_zcu102_revA_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revA_defconfig
@@ -16,6 +16,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_FASTBOOT=y
 CONFIG_FASTBOOT_FLASH=y
diff --git a/configs/xilinx_zynqmp_zcu102_revB_defconfig 
b/configs/xilinx_zynqmp_zcu102_revB_defconfig
index 50779fd16334..fb7e4c5c169b 100644
--- a/configs/xilinx_zynqmp_zcu102_revB_defconfig
+++ b/configs/xilinx_zynqmp_zcu102_revB_defconfig
@@ -16,6 +16,8 @@ CONFIG_SPL_LOAD_FIT=y
 # CONFIG_DISPLAY_BOARDINFO is not set
 CONFIG_SPL=y
 CONFIG_SPL_OS_BOOT=y
+CONFIG_SPL_RAM_SUPPORT=y
+CONFIG_SPL_RAM_DEVICE=y
 CONFIG_SYS_PROMPT="ZynqMP> "
 CONFIG_FASTBOOT=y
 CONFIG_FASTBOOT_FLASH=y
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 2/2] arm64: zynqmp: Add support for zynqmp automotive silicons

[Michal Simek]

Remove silicon prefix. Automotive grade devices are using xazu instead
of xczu prefix.

The patch "fpga: xilinx: Check for substring in device ID validation"
(sha1: f72132673a01216e760864e442f168977cce2bd2)
enables this functionality for zynq devices that only substrings are
checked.
Unfortunately there is no way how to detect device grade that's why
this change is reasonable.

Signed-off-by: Michal Simek 
---

 board/xilinx/zynqmp/zynqmp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
index 7aa9628414fe..44af49782cce 100644
--- a/board/xilinx/zynqmp/zynqmp.c
+++ b/board/xilinx/zynqmp/zynqmp.c
@@ -260,10 +260,10 @@ int board_init(void)
if (current_el() != 3) {
static char version[ZYNQMP_VERSION_SIZE];
 
-   strncat(version, "xczu", 4);
+   strncat(version, "zu", 2);
zynqmppl.name = strncat(version,
zynqmp_get_silicon_idcode_name(),
-   ZYNQMP_VERSION_SIZE - 5);
+   ZYNQMP_VERSION_SIZE - 3);
printf("Chip ID:\t%s\n", zynqmppl.name);
fpga_init();
fpga_add(fpga_xilinx, );
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 1/2] arm64: zynqmp: Add missing zynq_board_read_rom_ethaddr() prototype

[Michal Simek]

Add missing zynq_board_read_rom_ethaddr() prototype reported by sparse.

Signed-off-by: Michal Simek 
---

 arch/arm/include/asm/arch-zynqmp/sys_proto.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/include/asm/arch-zynqmp/sys_proto.h 
b/arch/arm/include/asm/arch-zynqmp/sys_proto.h
index f256c7d4a996..ad2856863363 100644
--- a/arch/arm/include/asm/arch-zynqmp/sys_proto.h
+++ b/arch/arm/include/asm/arch-zynqmp/sys_proto.h
@@ -30,6 +30,7 @@ enum {
TCM_SPLIT,
 };
 
+int zynq_board_read_rom_ethaddr(unsigned char *ethaddr);
 unsigned int zynqmp_get_silicon_version(void);
 
 void psu_init(void);
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] sunxi: OLinuXino: Drop FASTBOOT_FLASH

[Maxime Ripard]

On Fri, Dec 08, 2017 at 05:30:45PM +0530, Jagan Teki wrote:
> On Fri, Dec 8, 2017 at 5:26 PM, Maxime Ripard
>  wrote:
> > On Fri, Dec 08, 2017 at 05:09:51PM +0530, Jagan Teki wrote:
> >> FASTBOOT_FLASH is auto select for sunxi platform now,
> >> so drop explicit addition.
> >>
> >> Signed-off-by: Jagan Teki 
> >
> > There's far more boards than these three, maybe we should do all of
> > them?
> 
> True, but this is because I've tested these.

Still, running moveconfig -s is pretty safe here. You just want to
make sure that the config is the same, and it will be.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] arm64: zynqmp: Use only earlycon bootargs instead of full one

[Michal Simek]

This is the same patch as was done earlier.
Please look at Linux patch:
"arm64: zynqmp: Use only earlycon bootargs instead of full one"
(sha1: f3609c8d4af28b9cc22ca49bf8e529b582ec188c)

Signed-off-by: Michal Simek 
---

 arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts 
b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
index fbf281ed53d6..f3020a57600a 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
@@ -28,7 +28,7 @@
};
 
chosen {
-   bootargs = "earlycon=cdns,mmio,0xff00,115200n8";
+   bootargs = "earlycon";
stdout-path = "serial0:115200n8";
};
 
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] arm64: zynqmp: Remove undocumented dma properties

[Michal Simek]

Remove overfetch, ratectrl, include-sg and src-issue dma properties.
Driver is not using them and they are also not documented in the binding
doc.

Signed-off-by: Michal Simek 
Acked-by: Kedareswara rao Appana 
---

 arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts | 9 -
 arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts | 9 -
 arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts | 9 -
 arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts | 9 -
 arch/arm/dts/zynqmp-zcu102-revA.dts  | 9 -
 5 files changed, 45 deletions(-)

diff --git a/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts 
b/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
index 1f3c30277a41..04d82c4d2ec9 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm015-dc1.dts
@@ -43,16 +43,10 @@
 /* fpd_dma clk 667MHz, lpd_dma 500MHz */
 _dma_chan1 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
-   xlnx,overfetch; /* for testing purpose */
-   xlnx,ratectrl = <0>; /* for testing purpose */
-   xlnx,src-issue = <31>;
 };
 
 _dma_chan2 {
status = "okay";
-   xlnx,ratectrl = <100>; /* for testing purpose */
-   xlnx,src-issue = <4>; /* for testing purpose */
 };
 
 _dma_chan3 {
@@ -61,7 +55,6 @@
 
 _dma_chan4 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan5 {
@@ -70,7 +63,6 @@
 
 _dma_chan6 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan7 {
@@ -79,7 +71,6 @@
 
 _dma_chan8 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
  {
diff --git a/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts 
b/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
index 87df36012ed7..7dfe960135a5 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm016-dc2.dts
@@ -53,16 +53,10 @@
 /* fpd_dma clk 667MHz, lpd_dma 500MHz */
 _dma_chan1 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
-   xlnx,overfetch; /* for testing purpose */
-   xlnx,ratectrl = <0>; /* for testing purpose */
-   xlnx,src-issue = <31>;
 };
 
 _dma_chan2 {
status = "okay";
-   xlnx,ratectrl = <100>; /* for testing purpose */
-   xlnx,src-issue = <4>; /* for testing purpose */
 };
 
 _dma_chan3 {
@@ -71,7 +65,6 @@
 
 _dma_chan4 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan5 {
@@ -80,7 +73,6 @@
 
 _dma_chan6 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan7 {
@@ -89,7 +81,6 @@
 
 _dma_chan8 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
  {
diff --git a/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts 
b/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
index 799b87a04c13..648e3ba79939 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm018-dc4.dts
@@ -55,16 +55,10 @@
 /* fpd_dma clk 667MHz, lpd_dma 500MHz */
 _dma_chan1 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
-   xlnx,overfetch; /* for testing purpose */
-   xlnx,ratectrl = <0>; /* for testing purpose */
-   xlnx,src-issue = <31>;
 };
 
 _dma_chan2 {
status = "okay";
-   xlnx,ratectrl = <100>; /* for testing purpose */
-   xlnx,src-issue = <4>; /* for testing purpose */
 };
 
 _dma_chan3 {
@@ -73,7 +67,6 @@
 
 _dma_chan4 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan5 {
@@ -82,7 +75,6 @@
 
 _dma_chan6 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan7 {
@@ -91,7 +83,6 @@
 
 _dma_chan8 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan1 {
diff --git a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts 
b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
index 6de8296b7353..fbf281ed53d6 100644
--- a/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
+++ b/arch/arm/dts/zynqmp-zc1751-xm019-dc5.dts
@@ -41,16 +41,10 @@
 /* fpd_dma clk 667MHz, lpd_dma 500MHz */
 _dma_chan1 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
-   xlnx,overfetch; /* for testing purpose */
-   xlnx,ratectrl = <0>; /* for testing purpose */
-   xlnx,src-issue = <31>;
 };
 
 _dma_chan2 {
status = "okay";
-   xlnx,ratectrl = <100>; /* for testing purpose */
-   xlnx,src-issue = <4>; /* for testing purpose */
 };
 
 _dma_chan3 {
@@ -59,7 +53,6 @@
 
 _dma_chan4 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan5 {
@@ -68,7 +61,6 @@
 
 _dma_chan6 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
 _dma_chan7 {
@@ -77,7 +69,6 @@
 
 _dma_chan8 {
status = "okay";
-   xlnx,include-sg; /* for testing purpose */
 };
 
  {
diff --git a/arch/arm/dts/zynqmp-zcu102-revA.dts 
b/arch/arm/dts/zynqmp-zcu102-revA.dts
index 

[U-Boot] [PATCH] arm64: zynqmp: Add generic compatible string for I2C EEPROM

[Michal Simek]

From: Javier Martinez Canillas 

The at24 driver allows to register I2C EEPROM chips using different vendor
and devices, but the I2C subsystem does not take the vendor into account
when matching using the I2C table since it only has device entries.

But when matching using an OF table, both the vendor and device has to be
taken into account so the driver defines only a set of compatible strings
using the "atmel" vendor as a generic fallback for compatible I2C devices.

So add this generic fallback to the device node compatible string to make
the device to match the driver using the OF device ID table.

Signed-off-by: Javier Martinez Canillas 
Acked-by: Michal Simek 
Signed-off-by: Michal Simek 
---

 arch/arm/dts/zynqmp-ep108.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/zynqmp-ep108.dts b/arch/arm/dts/zynqmp-ep108.dts
index b0096f14b444..a16ffdc3f0ce 100644
--- a/arch/arm/dts/zynqmp-ep108.dts
+++ b/arch/arm/dts/zynqmp-ep108.dts
@@ -65,7 +65,7 @@
status = "okay";
clock-frequency = <40>;
eeprom@54 {
-   compatible = "at,24c64";
+   compatible = "atmel,24c64";
reg = <0x54>;
};
 };
@@ -74,7 +74,7 @@
status = "okay";
clock-frequency = <40>;
eeprom@55 {
-   compatible = "at,24c64";
+   compatible = "atmel,24c64";
reg = <0x55>;
};
 };
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v1] test: py: Add an option to skip sleep test

[Tom Rini]

On Fri, Dec 08, 2017 at 02:01:29PM +0100, Michal Simek wrote:
> Some QEMUs have a problem with time setup that's why
> sleep test is failing. Introduce env__sleep_accurate
> boardenv variable to have an option to skip sleep test.
> 
> Signed-off-by: Michal Simek 
> ---
> 
> Changes in v1:
> - Change subject line and patch description
> - Move from RFC to regular patch
> - Remove boardidentity marker and use variable instead
> 
>  configs/zynq_zc706_defconfig |  1 +

This looks unrelated.

>  test/py/tests/test_sleep.py  | 13 +
>  2 files changed, 14 insertions(+)
> 
> diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
> index 52999debc115..762b892f3fcf 100644
> --- a/configs/zynq_zc706_defconfig
> +++ b/configs/zynq_zc706_defconfig
> @@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y
>  CONFIG_CMD_EXT4_WRITE=y
>  CONFIG_CMD_FAT=y
>  CONFIG_CMD_FS_GENERIC=y
> +CONFIG_OF_EMBED=y
>  CONFIG_ENV_IS_IN_SPI_FLASH=y
>  CONFIG_NET_RANDOM_ETHADDR=y
>  CONFIG_SPL_DM_SEQ_ALIAS=y
> diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
> index 64e057132622..ccef24d7f7b8 100644
> --- a/test/py/tests/test_sleep.py
> +++ b/test/py/tests/test_sleep.py
> @@ -5,10 +5,23 @@
>  import pytest
>  import time
>  
> +"""
> +Note: This test doesn't rely on boardenv_* configuration values but they can
> +change test behavior.
> +
> +# Setup env__sleep_accurate to False if time is not accurate on your platform
> +env__sleep_accurate = False

This should default to true.  Then a PR to uboot-test-hooks where you
say the file for zynq_zc702 has this false, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-fsl-qoriq master

[Tom Rini]

On Thu, Dec 07, 2017 at 10:43:15PM +, York Sun wrote:

> Tom,
> 
> The following changes since commit 9da7fb4a39149c3061cb148bfbaa76b4b52b9008:
> 
>   Prepare v2018.01-rc1 (2017-12-04 18:27:17 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fsl-qoriq.git
> d798a6ee6436c7d2bcbf3fa6bda01daa4411a493
> 
> for you to fetch changes up to d798a6ee6436c7d2bcbf3fa6bda01daa4411a493:
> 
>   armv8: ls1088a: Add nand support for ls1088ardb (2017-12-06 14:56:31
> -0800)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PULL] Please pull u-boot-rockchip

[Dr . Philipp Tomsich]

Tom,

this week has been quiet in the Rockchip tree: I only have a change to
a snprintf format-string for restoring consistency with Linux.

Travis-CI results are available at
https://travis-ci.org/ptomsich/u-boot-rockchip/builds/313435652

Thanks,
Philipp.


The following changes since commit 6c7010b779ef29a40ed66acc414f65d2f4b75ced:

  Merge git://git.denx.de/u-boot-fsl-qoriq (2017-12-07 17:56:51 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-rockchip.git master

for you to fetch changes up to b32b1bd10b86d01a26056f50de89f768e5183ed1:

  rockchip: rk3399-puma: preserve leading zeros in serial# (2017-12-08 11:49:39 
+0100)


Jakob Unterwurzacher (1):
  rockchip: rk3399-puma: preserve leading zeros in serial#

 board/theobroma-systems/puma_rk3399/puma-rk3399.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 09/10] ata: Migrate CONFIG_LIBATA to Kconfig

[Tuomas Tynkkynen]

This symbol enables some library code used by various SATA drivers,
so make this a non-user-visible symbol select'ed by the respective
drivers, and let moveconfig handle the rest.

Signed-off-by: Tuomas Tynkkynen 
---
 arch/arm/include/asm/arch-ls102xa/config.h |  1 -
 drivers/ata/Kconfig| 11 +++
 include/configs/MPC8315ERDB.h  |  2 --
 include/configs/MPC8349ITX.h   |  1 -
 include/configs/MPC837XEMDS.h  |  2 --
 include/configs/MPC837XERDB.h  |  2 --
 include/configs/MPC8536DS.h|  2 --
 include/configs/MPC8544DS.h|  1 -
 include/configs/MPC8572DS.h|  1 -
 include/configs/MPC8610HPCD.h  |  1 -
 include/configs/MPC8641HPCN.h  |  1 -
 include/configs/P1010RDB.h |  1 -
 include/configs/P1022DS.h  |  1 -
 include/configs/P2041RDB.h |  2 --
 include/configs/P4080DS.h  |  1 -
 include/configs/T102xQDS.h |  1 -
 include/configs/T1040QDS.h |  2 --
 include/configs/T104xRDB.h |  2 --
 include/configs/T208xQDS.h |  1 -
 include/configs/T208xRDB.h |  1 -
 include/configs/T4240QDS.h |  2 --
 include/configs/T4240RDB.h |  4 
 include/configs/UCP1020.h  |  1 -
 include/configs/advantech_dms-ba16.h   |  1 -
 include/configs/am57xx_evm.h   |  1 -
 include/configs/apalis_imx6.h  |  1 -
 include/configs/cgtqmx6eval.h  |  1 -
 include/configs/cl-som-am57x.h |  1 -
 include/configs/cm_fx6.h   |  1 -
 include/configs/cm_t54.h   |  1 -
 include/configs/controlcenterd.h   |  1 -
 include/configs/controlcenterdc.h  |  1 -
 include/configs/corenet_ds.h   |  2 --
 include/configs/cyrus.h|  2 --
 include/configs/db-88f6820-gp.h|  1 -
 include/configs/db-mv784mp-gp.h|  1 -
 include/configs/dh_imx6.h  |  1 -
 include/configs/dra7xx_evm.h   |  1 -
 include/configs/ge_bx50v3.h|  1 -
 include/configs/gw_ventana.h   |  1 -
 include/configs/highbank.h |  1 -
 include/configs/ls1012aqds.h   |  1 -
 include/configs/ls1012ardb.h   |  1 -
 include/configs/ls1021aiot.h   |  1 -
 include/configs/ls1043aqds.h   |  1 -
 include/configs/ls1043ardb.h   |  1 -
 include/configs/ls1046aqds.h   |  1 -
 include/configs/ls1046ardb.h   |  1 -
 include/configs/ls1088a_common.h   |  1 -
 include/configs/ls2080aqds.h   |  1 -
 include/configs/ls2080ardb.h   |  1 -
 include/configs/m53evk.h   |  1 -
 include/configs/mvebu_armada-37xx.h|  1 -
 include/configs/mvebu_armada-8k.h  |  1 -
 include/configs/mx53loco.h |  1 -
 include/configs/mx6cuboxi.h|  1 -
 include/configs/nitrogen6x.h   |  1 -
 include/configs/novena.h   |  1 -
 include/configs/omap5_uevm.h   |  1 -
 include/configs/ot1200.h   |  1 -
 include/configs/p1_p2_rdb_pc.h |  1 -
 include/configs/p1_twr.h   |  1 -
 include/configs/qemu-arm.h |  1 -
 include/configs/sunxi-common.h |  1 -
 include/configs/t4qds.h|  2 --
 include/configs/tbs2910.h  |  1 -
 include/configs/theadorable.h  |  1 -
 include/configs/turris_omnia.h |  1 -
 include/configs/udoo.h |  1 -
 include/configs/wandboard.h|  1 -
 include/configs/x86-common.h   |  1 -
 include/configs/xilinx_zynqmp.h|  1 -
 scripts/config_whitelist.txt   |  1 -
 73 files changed, 11 insertions(+), 86 deletions(-)

diff --git arch/arm/include/asm/arch-ls102xa/config.h 
arch/arm/include/asm/arch-ls102xa/config.h
index 2fbd4ef557..940461137e 100644
--- arch/arm/include/asm/arch-ls102xa/config.h
+++ arch/arm/include/asm/arch-ls102xa/config.h
@@ -80,7 +80,6 @@
 
 /* SATA */
 #define AHCI_BASE_ADDR (CONFIG_SYS_IMMR + 0x0220)
-#define CONFIG_LIBATA
 #define CONFIG_SCSI_AHCI_PLAT
 #define CONFIG_SYS_SCSI_MAX_SCSI_ID1
 #define CONFIG_SYS_SCSI_MAX_LUN1
diff --git drivers/ata/Kconfig drivers/ata/Kconfig
index 38318b4615..43e6626969 100644
--- drivers/ata/Kconfig
+++ drivers/ata/Kconfig
@@ -20,8 +20,14 @@ config SATA
 
  See also CMD_SATA which provides command-line support.
 
+config LIBATA
+   bool
+   help
+ Select this to build and link the libata helper functions.
+
 config SCSI_AHCI
bool "Enable SCSI interface to SATA devices"
+   select LIBATA
help
 

[U-Boot] [PATCH 07/10] ata: Migrate CONFIG_DWC_AHSATA to Kconfig

[Tuomas Tynkkynen]

Signed-off-by: Tuomas Tynkkynen 
---
 configs/apalis_imx6_defconfig   | 1 +
 configs/apalis_imx6_nospl_com_defconfig | 1 +
 configs/apalis_imx6_nospl_it_defconfig  | 1 +
 configs/cgtqmx6eval_defconfig   | 1 +
 configs/cm_fx6_defconfig| 1 +
 configs/dh_imx6_defconfig   | 1 +
 configs/dms-ba16-1g_defconfig   | 1 +
 configs/dms-ba16_defconfig  | 1 +
 configs/gwventana_emmc_defconfig| 1 +
 configs/gwventana_gw5904_defconfig  | 1 +
 configs/gwventana_nand_defconfig| 1 +
 configs/m53evk_defconfig| 1 +
 configs/mx53loco_defconfig  | 1 +
 configs/mx6cuboxi_defconfig | 1 +
 configs/mx6qsabrelite_defconfig | 1 +
 configs/nitrogen6q2g_defconfig  | 1 +
 configs/nitrogen6q_defconfig| 1 +
 configs/novena_defconfig| 1 +
 configs/ot1200_defconfig| 1 +
 configs/ot1200_spl_defconfig| 1 +
 configs/tbs2910_defconfig   | 1 +
 configs/udoo_defconfig  | 1 +
 configs/wandboard_defconfig | 1 +
 drivers/ata/Kconfig | 6 ++
 include/configs/advantech_dms-ba16.h| 1 -
 include/configs/apalis_imx6.h   | 1 -
 include/configs/cgtqmx6eval.h   | 1 -
 include/configs/cm_fx6.h| 1 -
 include/configs/dh_imx6.h   | 1 -
 include/configs/ge_bx50v3.h | 1 -
 include/configs/gw_ventana.h| 1 -
 include/configs/m53evk.h| 1 -
 include/configs/mx53loco.h  | 1 -
 include/configs/mx6cuboxi.h | 1 -
 include/configs/nitrogen6x.h| 1 -
 include/configs/novena.h| 1 -
 include/configs/ot1200.h| 1 -
 include/configs/tbs2910.h   | 1 -
 include/configs/udoo.h  | 1 -
 include/configs/wandboard.h | 1 -
 scripts/config_whitelist.txt| 1 -
 41 files changed, 29 insertions(+), 17 deletions(-)

diff --git configs/apalis_imx6_defconfig configs/apalis_imx6_defconfig
index 2b1b34d80a..18107e88ff 100644
--- configs/apalis_imx6_defconfig
+++ configs/apalis_imx6_defconfig
@@ -45,6 +45,7 @@ CONFIG_CMD_EXT4=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_MMC=y
+CONFIG_DWC_AHSATA=y
 CONFIG_PHYLIB=y
 CONFIG_PHY_MICREL=y
 CONFIG_PHY_MICREL_KSZ90X1=y
diff --git configs/apalis_imx6_nospl_com_defconfig 
configs/apalis_imx6_nospl_com_defconfig
index ddf8c8d1e3..c40e14fdf2 100644
--- configs/apalis_imx6_nospl_com_defconfig
+++ configs/apalis_imx6_nospl_com_defconfig
@@ -34,6 +34,7 @@ CONFIG_CMD_EXT4=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_MMC=y
+CONFIG_DWC_AHSATA=y
 CONFIG_PHYLIB=y
 CONFIG_PHY_MICREL=y
 CONFIG_PHY_MICREL_KSZ90X1=y
diff --git configs/apalis_imx6_nospl_it_defconfig 
configs/apalis_imx6_nospl_it_defconfig
index 6eba47b8ce..5d1243ad92 100644
--- configs/apalis_imx6_nospl_it_defconfig
+++ configs/apalis_imx6_nospl_it_defconfig
@@ -34,6 +34,7 @@ CONFIG_CMD_EXT4=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_MMC=y
+CONFIG_DWC_AHSATA=y
 CONFIG_PHYLIB=y
 CONFIG_PHY_MICREL=y
 CONFIG_PHY_MICREL_KSZ90X1=y
diff --git configs/cgtqmx6eval_defconfig configs/cgtqmx6eval_defconfig
index 47c0b4c622..418e4e2d8c 100644
--- configs/cgtqmx6eval_defconfig
+++ configs/cgtqmx6eval_defconfig
@@ -45,6 +45,7 @@ CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_DWC_AHSATA=y
 CONFIG_DFU_MMC=y
 CONFIG_DFU_SF=y
 CONFIG_PHYLIB=y
diff --git configs/cm_fx6_defconfig configs/cm_fx6_defconfig
index 0773aca0d4..8e0746bcf8 100644
--- configs/cm_fx6_defconfig
+++ configs/cm_fx6_defconfig
@@ -52,6 +52,7 @@ CONFIG_MTDIDS_DEFAULT="nor0=spi0.0"
 
CONFIG_MTDPARTS_DEFAULT="mtdparts=spi0.0:768k(uboot),256k(uboot-environment),-(reserved)"
 CONFIG_OF_CONTROL=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_DWC_AHSATA=y
 CONFIG_DM_MMC=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_ATMEL=y
diff --git configs/dh_imx6_defconfig configs/dh_imx6_defconfig
index 814df1cd27..1fd7c50e31 100644
--- configs/dh_imx6_defconfig
+++ configs/dh_imx6_defconfig
@@ -40,6 +40,7 @@ CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_DWC_AHSATA=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_SPANSION=y
 CONFIG_PHYLIB=y
diff --git configs/dms-ba16-1g_defconfig configs/dms-ba16-1g_defconfig
index cbc9e82eed..144a86bcbf 100644
--- configs/dms-ba16-1g_defconfig
+++ configs/dms-ba16-1g_defconfig
@@ -28,6 +28,7 @@ CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_DWC_AHSATA=y
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_STMICRO=y
 CONFIG_PHYLIB=y
diff --git configs/dms-ba16_defconfig configs/dms-ba16_defconfig
index 9be2d6cc52..075e5070f0 100644
--- configs/dms-ba16_defconfig
+++ configs/dms-ba16_defconfig
@@ -27,6 +27,7 @@ CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
 

[U-Boot] [PATCH 10/10] ata: Migrate CONFIG_MVSATA_IDE to Kconfig

[Tuomas Tynkkynen]

Signed-off-by: Tuomas Tynkkynen 
---
 arch/arm/mach-kirkwood/include/mach/config.h | 1 -
 configs/d2net_v2_defconfig   | 1 +
 configs/dns325_defconfig | 1 +
 configs/dreamplug_defconfig  | 1 +
 configs/ds109_defconfig  | 1 +
 configs/edminiv2_defconfig   | 1 +
 configs/goflexhome_defconfig | 1 +
 configs/guruplug_defconfig   | 1 +
 configs/ib62x0_defconfig | 1 +
 configs/inetspace_v2_defconfig   | 1 +
 configs/lschlv2_defconfig| 1 +
 configs/lsxhl_defconfig  | 1 +
 configs/nas220_defconfig | 1 +
 configs/net2big_v2_defconfig | 1 +
 configs/netspace_lite_v2_defconfig   | 1 +
 configs/netspace_max_v2_defconfig| 1 +
 configs/netspace_mini_v2_defconfig   | 1 +
 configs/netspace_v2_defconfig| 1 +
 configs/nsa310s_defconfig| 1 +
 configs/openrd_base_defconfig| 1 +
 configs/openrd_client_defconfig  | 1 +
 configs/openrd_ultimate_defconfig| 1 +
 configs/sheevaplug_defconfig | 1 +
 drivers/ata/Kconfig  | 6 ++
 include/configs/edminiv2.h   | 1 -
 scripts/config_whitelist.txt | 1 -
 26 files changed, 28 insertions(+), 3 deletions(-)

diff --git arch/arm/mach-kirkwood/include/mach/config.h 
arch/arm/mach-kirkwood/include/mach/config.h
index ba6007186e..efa4e7b6e2 100644
--- arch/arm/mach-kirkwood/include/mach/config.h
+++ arch/arm/mach-kirkwood/include/mach/config.h
@@ -97,7 +97,6 @@
  */
 #ifdef CONFIG_IDE
 #define __io
-#define CONFIG_MVSATA_IDE
 #define CONFIG_IDE_PREINIT
 #define CONFIG_MVSATA_IDE_USE_PORT1
 /* Needs byte-swapping for ATA data register */
diff --git configs/d2net_v2_defconfig configs/d2net_v2_defconfig
index 496c5c02d4..61fd892005 100644
--- configs/d2net_v2_defconfig
+++ configs/d2net_v2_defconfig
@@ -26,6 +26,7 @@ CONFIG_ISO_PARTITION=y
 CONFIG_EFI_PARTITION=y
 # CONFIG_PARTITION_UUIDS is not set
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_MACRONIX=y
diff --git configs/dns325_defconfig configs/dns325_defconfig
index 62ad86ced4..f5556e30aa 100644
--- configs/dns325_defconfig
+++ configs/dns325_defconfig
@@ -23,6 +23,7 @@ 
CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:896k(u-boot),128k(u-boot-env),5m(ke
 CONFIG_CMD_UBI=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_NAND=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
diff --git configs/dreamplug_defconfig configs/dreamplug_defconfig
index 1130faa392..9d24f9c16b 100644
--- configs/dreamplug_defconfig
+++ configs/dreamplug_defconfig
@@ -19,6 +19,7 @@ CONFIG_CMD_EXT4=y
 CONFIG_CMD_FAT=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_MACRONIX=y
diff --git configs/ds109_defconfig configs/ds109_defconfig
index 987a924b7e..6d513cf584 100644
--- configs/ds109_defconfig
+++ configs/ds109_defconfig
@@ -14,6 +14,7 @@ CONFIG_CMD_DATE=y
 CONFIG_CMD_FAT=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SPI_FLASH=y
 CONFIG_SPI_FLASH_BAR=y
diff --git configs/edminiv2_defconfig configs/edminiv2_defconfig
index 14540ca371..4eafb6d5fa 100644
--- configs/edminiv2_defconfig
+++ configs/edminiv2_defconfig
@@ -21,6 +21,7 @@ CONFIG_CMD_USB=y
 CONFIG_CMD_EXT2=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_FLASH=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_MTD_NOR_FLASH=y
 CONFIG_SYS_NS16550=y
diff --git configs/goflexhome_defconfig configs/goflexhome_defconfig
index 80652fec79..bdfd62a576 100644
--- configs/goflexhome_defconfig
+++ configs/goflexhome_defconfig
@@ -24,6 +24,7 @@ 
CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:1m(uboot),6M(uImage),-(root)"
 CONFIG_CMD_UBI=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_NAND=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
diff --git configs/guruplug_defconfig configs/guruplug_defconfig
index d219fee508..66a668a7bb 100644
--- configs/guruplug_defconfig
+++ configs/guruplug_defconfig
@@ -24,6 +24,7 @@ 
CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:896K(uboot),128K(uboot_env),-@1M(ro
 CONFIG_CMD_UBI=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_NAND=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
diff --git configs/ib62x0_defconfig configs/ib62x0_defconfig
index fd7c148860..f6bfa06dcb 100644
--- configs/ib62x0_defconfig
+++ configs/ib62x0_defconfig
@@ -22,6 +22,7 @@ 
CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:0xe@0x0(uboot),0x2@0xe(
 CONFIG_CMD_UBI=y
 CONFIG_ISO_PARTITION=y
 CONFIG_ENV_IS_IN_NAND=y
+CONFIG_MVSATA_IDE=y
 # CONFIG_MMC is not set
 CONFIG_SYS_NS16550=y
 

[U-Boot] [PATCH 08/10] ata: Migrate CONFIG_SCSI_AHCI to Kconfig

[Tuomas Tynkkynen]

And use 'imply' liberally.

Signed-off-by: Tuomas Tynkkynen 
---
 arch/arm/Kconfig  | 3 +++
 arch/arm/cpu/armv7/ls102xa/Kconfig| 1 +
 arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 2 ++
 arch/arm/include/asm/arch-ls102xa/config.h| 1 -
 arch/x86/cpu/baytrail/Kconfig | 1 +
 arch/x86/cpu/braswell/Kconfig | 1 +
 arch/x86/cpu/broadwell/Kconfig| 1 +
 arch/x86/cpu/coreboot/Kconfig | 1 +
 arch/x86/cpu/ivybridge/Kconfig| 1 +
 arch/x86/cpu/qemu/Kconfig | 1 +
 arch/x86/cpu/queensbay/Kconfig| 1 +
 configs/A10-OLinuXino-Lime_defconfig  | 1 +
 configs/A20-OLinuXino-Lime2-eMMC_defconfig| 1 +
 configs/A20-OLinuXino-Lime2_defconfig | 1 +
 configs/A20-OLinuXino-Lime_defconfig  | 1 +
 configs/A20-OLinuXino_MICRO-eMMC_defconfig| 1 +
 configs/A20-OLinuXino_MICRO_defconfig | 1 +
 configs/A20-Olimex-SOM-EVB_defconfig  | 1 +
 configs/Bananapi_M2_Ultra_defconfig   | 1 +
 configs/Bananapi_defconfig| 1 +
 configs/Bananapro_defconfig   | 1 +
 configs/Cubieboard2_defconfig | 1 +
 configs/Cubieboard_defconfig  | 1 +
 configs/Cubietruck_defconfig  | 1 +
 configs/Itead_Ibox_A20_defconfig  | 1 +
 configs/Lamobo_R1_defconfig   | 1 +
 configs/Linksprite_pcDuino3_Nano_defconfig| 1 +
 configs/Linksprite_pcDuino3_defconfig | 1 +
 configs/MPC8544DS_defconfig   | 1 +
 configs/MPC8572DS_36BIT_defconfig | 1 +
 configs/MPC8572DS_defconfig   | 1 +
 configs/MPC8610HPCD_defconfig | 1 +
 configs/MPC8641HPCN_36BIT_defconfig   | 1 +
 configs/MPC8641HPCN_defconfig | 1 +
 configs/Marsboard_A10_defconfig   | 1 +
 configs/Mele_A1000_defconfig  | 1 +
 configs/Mele_M5_defconfig | 1 +
 configs/Orangepi_defconfig| 1 +
 configs/Orangepi_mini_defconfig   | 1 +
 configs/Wits_Pro_A20_DKT_defconfig| 1 +
 configs/am57xx_evm_defconfig  | 1 +
 configs/am57xx_hs_evm_defconfig   | 1 +
 configs/bananapi_m1_plus_defconfig| 1 +
 configs/cl-som-am57x_defconfig| 1 +
 configs/cm_t54_defconfig  | 1 +
 configs/controlcenterdc_defconfig | 1 +
 configs/db-88f6820-gp_defconfig   | 1 +
 configs/highbank_defconfig| 1 +
 configs/ls1012aqds_qspi_defconfig | 1 +
 configs/ls1088aqds_qspi_defconfig | 1 +
 configs/ls1088aqds_sdcard_qspi_defconfig  | 1 +
 configs/ls1088ardb_qspi_defconfig | 1 +
 configs/ls1088ardb_sdcard_qspi_defconfig  | 1 +
 configs/ls2081ardb_defconfig  | 1 +
 configs/mvebu_db-88f3720_defconfig| 1 +
 configs/mvebu_db_armada8k_defconfig   | 1 +
 configs/mvebu_espressobin-88f3720_defconfig   | 1 +
 configs/mvebu_mcbin-88f8040_defconfig | 1 +
 configs/omap5_uevm_defconfig  | 1 +
 configs/qemu_arm_defconfig| 1 +
 configs/turris_omnia_defconfig| 1 +
 configs/xilinx_zynqmp_ep_defconfig| 1 +
 configs/xilinx_zynqmp_zcu102_rev1_0_defconfig | 1 +
 configs/xilinx_zynqmp_zcu102_revA_defconfig   | 1 +
 configs/xilinx_zynqmp_zcu102_revB_defconfig   | 1 +
 drivers/ata/Kconfig   | 5 +
 include/configs/MPC8544DS.h   | 1 -
 include/configs/MPC8572DS.h   | 1 -
 include/configs/MPC8610HPCD.h | 2 --
 include/configs/MPC8641HPCN.h | 2 --
 include/configs/am57xx_evm.h  | 1 -
 include/configs/cl-som-am57x.h| 1 -
 include/configs/cm_t54.h  | 1 -
 include/configs/controlcenterdc.h | 1 -
 include/configs/db-88f6820-gp.h   | 1 -
 include/configs/dra7xx_evm.h  | 1 -
 include/configs/efi-x86.h | 2 --
 include/configs/galileo.h | 3 ---
 include/configs/highbank.h| 1 -
 include/configs/ls1012aqds.h  | 1 -
 include/configs/ls1012ardb.h  | 1 -
 include/configs/ls1021aiot.h  | 1 -
 include/configs/ls1043aqds.h  | 1 -
 include/configs/ls1043ardb.h  | 1 -
 include/configs/ls1046aqds.h  | 1 -
 include/configs/ls1046ardb.h  | 1 -
 include/configs/ls1088a_common.h  | 1 -
 include/configs/ls2080aqds.h  | 1 -
 include/configs/ls2080ardb.h  | 1 -
 include/configs/mvebu_armada-37xx.h   | 1 -
 include/configs/mvebu_armada-8k.h | 1 -
 include/configs/omap5_uevm.h  | 1 -
 

[U-Boot] [PATCH 03/10] ata: Migrate CONFIG_SATA_SIL to Kconfig

[Tuomas Tynkkynen]

Use 'imply' here liberally to avoid the combinatorial explosion of
defconfig changes in the PowerPC boards.

Signed-off-by: Tuomas Tynkkynen 
---
 arch/powerpc/cpu/mpc85xx/Kconfig | 6 ++
 drivers/ata/Kconfig  | 6 ++
 include/configs/P4080DS.h| 1 -
 include/configs/UCP1020.h| 1 -
 include/configs/p1_p2_rdb_pc.h   | 1 -
 scripts/config_whitelist.txt | 1 -
 6 files changed, 12 insertions(+), 4 deletions(-)

diff --git arch/powerpc/cpu/mpc85xx/Kconfig arch/powerpc/cpu/mpc85xx/Kconfig
index 92187d371b..1b0e3d3a2f 100644
--- arch/powerpc/cpu/mpc85xx/Kconfig
+++ arch/powerpc/cpu/mpc85xx/Kconfig
@@ -209,6 +209,7 @@ config TARGET_P1025RDB
select ARCH_P1025
imply CMD_EEPROM
imply CMD_SATA
+   imply SATA_SIL
 
 config TARGET_P2020RDB
bool "Support P2020RDB-PC"
@@ -217,6 +218,7 @@ config TARGET_P2020RDB
select ARCH_P2020
imply CMD_EEPROM
imply CMD_SATA
+   imply SATA_SIL
 
 config TARGET_P1_TWR
bool "Support p1_twr"
@@ -672,6 +674,7 @@ config ARCH_P1020
imply CMD_SATA
imply CMD_PCI
imply CMD_REGINFO
+   imply SATA_SIL
 
 config ARCH_P1021
bool
@@ -690,6 +693,7 @@ config ARCH_P1021
imply CMD_NAND
imply CMD_SATA
imply CMD_REGINFO
+   imply SATA_SIL
 
 config ARCH_P1022
bool
@@ -737,6 +741,7 @@ config ARCH_P1024
imply CMD_SATA
imply CMD_PCI
imply CMD_REGINFO
+   imply SATA_SIL
 
 config ARCH_P1025
bool
@@ -858,6 +863,7 @@ config ARCH_P4080
select FSL_ELBC
imply CMD_SATA
imply CMD_REGINFO
+   imply SATA_SIL
 
 config ARCH_P5020
bool
diff --git drivers/ata/Kconfig drivers/ata/Kconfig
index 803064aaf1..617c80f011 100644
--- drivers/ata/Kconfig
+++ drivers/ata/Kconfig
@@ -47,4 +47,10 @@ config DWC_AHCI
  Enable this driver to support Sata devices through
  Synopsys DWC AHCI module.
 
+config SATA_SIL
+   bool "Enable Silicon Image SIL3131 / SIL3132 / SIL3124 SATA driver 
support"
+   help
+ Enable this driver to support the SIL3131, SIL3132 and SIL3124
+ SATA controllers.
+
 endmenu
diff --git include/configs/P4080DS.h include/configs/P4080DS.h
index f192181c08..8b884411ac 100644
--- include/configs/P4080DS.h
+++ include/configs/P4080DS.h
@@ -12,7 +12,6 @@
 
 #define CONFIG_PCIE3
 
-#define CONFIG_SATA_SIL
 #define CONFIG_SYS_SATA_MAX_DEVICE  2
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
diff --git include/configs/UCP1020.h include/configs/UCP1020.h
index 902abc48ce..455298751c 100644
--- include/configs/UCP1020.h
+++ include/configs/UCP1020.h
@@ -118,7 +118,6 @@
 
 #define CONFIG_ENV_OVERWRITE
 
-#define CONFIG_SATA_SIL
 #define CONFIG_SYS_SATA_MAX_DEVICE 2
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
diff --git include/configs/p1_p2_rdb_pc.h include/configs/p1_p2_rdb_pc.h
index 1b665b2c15..3f073d06cb 100644
--- include/configs/p1_p2_rdb_pc.h
+++ include/configs/p1_p2_rdb_pc.h
@@ -244,7 +244,6 @@
 #define CONFIG_TSEC_ENET   /* tsec ethernet support */
 #define CONFIG_ENV_OVERWRITE
 
-#define CONFIG_SATA_SIL
 #define CONFIG_SYS_SATA_MAX_DEVICE 2
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
diff --git scripts/config_whitelist.txt scripts/config_whitelist.txt
index c2fe81e552..416b6e2aa5 100644
--- scripts/config_whitelist.txt
+++ scripts/config_whitelist.txt
@@ -1905,7 +1905,6 @@ CONFIG_SAR_REG
 CONFIG_SATA1
 CONFIG_SATA2
 CONFIG_SATA_MV
-CONFIG_SATA_SIL
 CONFIG_SATA_SIL3114
 CONFIG_SATA_ULI5288
 CONFIG_SBC8349
-- 
2.15.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 04/10] ata: Migrate CONFIG_SATA_SIL3114 to Kconfig

[Tuomas Tynkkynen]

Signed-off-by: Tuomas Tynkkynen 
---
 configs/MPC8349ITX_LOWBOOT_defconfig | 1 +
 configs/MPC8349ITX_defconfig | 1 +
 configs/TWR-P1025_defconfig  | 1 +
 drivers/ata/Kconfig  | 5 +
 include/configs/MPC8349ITX.h | 1 -
 include/configs/p1_twr.h | 1 -
 scripts/config_whitelist.txt | 1 -
 7 files changed, 8 insertions(+), 3 deletions(-)

diff --git configs/MPC8349ITX_LOWBOOT_defconfig 
configs/MPC8349ITX_LOWBOOT_defconfig
index 0893c4ac02..c95cec698a 100644
--- configs/MPC8349ITX_LOWBOOT_defconfig
+++ configs/MPC8349ITX_LOWBOOT_defconfig
@@ -23,6 +23,7 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_DATE=y
 CONFIG_CMD_EXT2=y
 CONFIG_CMD_FAT=y
+CONFIG_SATA_SIL3114=y
 # CONFIG_MMC is not set
 CONFIG_MTD_NOR_FLASH=y
 CONFIG_PHYLIB=y
diff --git configs/MPC8349ITX_defconfig configs/MPC8349ITX_defconfig
index 8a8f05544f..4403bc0d62 100644
--- configs/MPC8349ITX_defconfig
+++ configs/MPC8349ITX_defconfig
@@ -23,6 +23,7 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_DATE=y
 CONFIG_CMD_EXT2=y
 CONFIG_CMD_FAT=y
+CONFIG_SATA_SIL3114=y
 # CONFIG_MMC is not set
 CONFIG_MTD_NOR_FLASH=y
 CONFIG_PHYLIB=y
diff --git configs/TWR-P1025_defconfig configs/TWR-P1025_defconfig
index 5dda2bbdb5..1eedd3c21f 100644
--- configs/TWR-P1025_defconfig
+++ configs/TWR-P1025_defconfig
@@ -25,6 +25,7 @@ CONFIG_CMD_MTDPARTS=y
 CONFIG_MTDIDS_DEFAULT="nor0=ec00.nor"
 
CONFIG_MTDPARTS_DEFAULT="mtdparts=ec00.nor:256k(vsc7385-firmware),256k(dtb),5632k(kernel),57856k(fs),256k(qe-ucode-firmware),1280k(u-boot)"
 CONFIG_ENV_IS_IN_FLASH=y
+CONFIG_SATA_SIL3114=y
 CONFIG_MTD_NOR_FLASH=y
 CONFIG_PHYLIB=y
 CONFIG_NETDEVICES=y
diff --git drivers/ata/Kconfig drivers/ata/Kconfig
index 617c80f011..77e0fbb217 100644
--- drivers/ata/Kconfig
+++ drivers/ata/Kconfig
@@ -53,4 +53,9 @@ config SATA_SIL
  Enable this driver to support the SIL3131, SIL3132 and SIL3124
  SATA controllers.
 
+config SATA_SIL3114
+   bool "Enable Silicon Image SIL3114 SATA driver support"
+   help
+ Enable this driver to support the SIL3114 SATA controllers.
+
 endmenu
diff --git include/configs/MPC8349ITX.h include/configs/MPC8349ITX.h
index d06d4a2817..23fe158f74 100644
--- include/configs/MPC8349ITX.h
+++ include/configs/MPC8349ITX.h
@@ -67,7 +67,6 @@
 /* The CF card interface on the back of the board */
 #define CONFIG_COMPACT_FLASH
 #define CONFIG_VSC7385_ENET/* VSC7385 ethernet support */
-#define CONFIG_SATA_SIL3114/* SIL3114 SATA controller */
 #define CONFIG_SYS_USB_HOST/* use the EHCI USB controller */
 #endif
 
diff --git include/configs/p1_twr.h include/configs/p1_twr.h
index e9692048d1..5edf07197f 100644
--- include/configs/p1_twr.h
+++ include/configs/p1_twr.h
@@ -50,7 +50,6 @@
 #define CONFIG_TSEC_ENET   /* tsec ethernet support */
 #define CONFIG_ENV_OVERWRITE
 
-#define CONFIG_SATA_SIL3114
 #define CONFIG_SYS_SATA_MAX_DEVICE 2
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
diff --git scripts/config_whitelist.txt scripts/config_whitelist.txt
index 416b6e2aa5..625f610a16 100644
--- scripts/config_whitelist.txt
+++ scripts/config_whitelist.txt
@@ -1905,7 +1905,6 @@ CONFIG_SAR_REG
 CONFIG_SATA1
 CONFIG_SATA2
 CONFIG_SATA_MV
-CONFIG_SATA_SIL3114
 CONFIG_SATA_ULI5288
 CONFIG_SBC8349
 CONFIG_SBC8548
-- 
2.15.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 05/10] ata: Migrate CONFIG_SATA_MV to Kconfig

[Tuomas Tynkkynen]

Signed-off-by: Tuomas Tynkkynen 
---
 configs/db-mv784mp-gp_defconfig | 1 +
 configs/theadorable_debug_defconfig | 1 +
 drivers/ata/Kconfig | 6 ++
 include/configs/db-mv784mp-gp.h | 1 -
 include/configs/theadorable.h   | 1 -
 scripts/config_whitelist.txt| 1 -
 6 files changed, 8 insertions(+), 3 deletions(-)

diff --git configs/db-mv784mp-gp_defconfig configs/db-mv784mp-gp_defconfig
index 23eaa123a2..6724af04bc 100644
--- configs/db-mv784mp-gp_defconfig
+++ configs/db-mv784mp-gp_defconfig
@@ -37,6 +37,7 @@ CONFIG_EFI_PARTITION=y
 # CONFIG_PARTITION_UUIDS is not set
 # CONFIG_SPL_PARTITION_UUIDS is not set
 CONFIG_SPL_OF_TRANSLATE=y
+CONFIG_SATA_MV=y
 # CONFIG_MMC is not set
 CONFIG_NAND=y
 CONFIG_NAND_PXA3XX=y
diff --git configs/theadorable_debug_defconfig 
configs/theadorable_debug_defconfig
index 863d6ce3d0..f602c839c3 100644
--- configs/theadorable_debug_defconfig
+++ configs/theadorable_debug_defconfig
@@ -42,6 +42,7 @@ CONFIG_EFI_PARTITION=y
 # CONFIG_SPL_PARTITION_UUIDS is not set
 CONFIG_NET_RANDOM_ETHADDR=y
 CONFIG_SPL_OF_TRANSLATE=y
+CONFIG_SATA_MV=y
 CONFIG_FPGA_ALTERA=y
 CONFIG_DM_GPIO=y
 # CONFIG_MMC is not set
diff --git drivers/ata/Kconfig drivers/ata/Kconfig
index 77e0fbb217..8dfa4670cc 100644
--- drivers/ata/Kconfig
+++ drivers/ata/Kconfig
@@ -47,6 +47,12 @@ config DWC_AHCI
  Enable this driver to support Sata devices through
  Synopsys DWC AHCI module.
 
+config SATA_MV
+   bool "Enable Marvell SATA controller driver support"
+   help
+ Enable this driver to support the SATA controller found in
+ some Marvell SoCs.
+
 config SATA_SIL
bool "Enable Silicon Image SIL3131 / SIL3132 / SIL3124 SATA driver 
support"
help
diff --git include/configs/db-mv784mp-gp.h include/configs/db-mv784mp-gp.h
index 4a5be6188f..105578ffc6 100644
--- include/configs/db-mv784mp-gp.h
+++ include/configs/db-mv784mp-gp.h
@@ -49,7 +49,6 @@
 
 /* SATA support */
 #define CONFIG_SYS_SATA_MAX_DEVICE 2
-#define CONFIG_SATA_MV
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
 
diff --git include/configs/theadorable.h include/configs/theadorable.h
index a7001e76aa..774324f28b 100644
--- include/configs/theadorable.h
+++ include/configs/theadorable.h
@@ -65,7 +65,6 @@
 
 /* SATA support */
 #define CONFIG_SYS_SATA_MAX_DEVICE 1
-#define CONFIG_SATA_MV
 #define CONFIG_LIBATA
 #define CONFIG_LBA48
 
diff --git scripts/config_whitelist.txt scripts/config_whitelist.txt
index 625f610a16..955369abde 100644
--- scripts/config_whitelist.txt
+++ scripts/config_whitelist.txt
@@ -1904,7 +1904,6 @@ CONFIG_SAR2_REG
 CONFIG_SAR_REG
 CONFIG_SATA1
 CONFIG_SATA2
-CONFIG_SATA_MV
 CONFIG_SATA_ULI5288
 CONFIG_SBC8349
 CONFIG_SBC8548
-- 
2.15.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 02/10] ata: Drop CONFIG_MX51_PATA

[Tuomas Tynkkynen]

The last user of this driver went away in August 2015 in commit:
b6073fd2115 ("arm: Remove mx51_efikamx, mx51_efikasb boards")

Signed-off-by: Tuomas Tynkkynen 
---
 drivers/ata/Makefile  |   1 -
 drivers/ata/mxc_ata.c | 129 --
 2 files changed, 130 deletions(-)
 delete mode 100644 drivers/ata/mxc_ata.c

diff --git drivers/ata/Makefile drivers/ata/Makefile
index 7a4c8c6bf2..a94c804e7a 100644
--- drivers/ata/Makefile
+++ drivers/ata/Makefile
@@ -13,7 +13,6 @@ obj-$(CONFIG_DWC_AHSATA) += dwc_ahsata.o
 obj-$(CONFIG_FSL_SATA) += fsl_sata.o
 obj-$(CONFIG_LIBATA) += libata.o
 obj-$(CONFIG_MVSATA_IDE) += mvsata_ide.o
-obj-$(CONFIG_MX51_PATA) += mxc_ata.o
 obj-$(CONFIG_SATA) += sata.o
 obj-$(CONFIG_SATA_CEVA) += sata_ceva.o
 obj-$(CONFIG_SATA_MV) += sata_mv.o
diff --git drivers/ata/mxc_ata.c drivers/ata/mxc_ata.c
deleted file mode 100644
index 44bb406f4d..00
--- drivers/ata/mxc_ata.c
+++ /dev/null
@@ -1,129 +0,0 @@
-/*
- * Freescale iMX51 ATA driver
- *
- * Copyright (C) 2010 Marek Vasut 
- *
- * Based on code by:
- * Mahesh Mahadevan 
- *
- * Based on code from original FSL ATA driver, which is
- * part of eCos, the Embedded Configurable Operating System.
- * Copyright (C) 1998, 1999, 2000, 2001, 2002 Red Hat, Inc.
- *
- * SPDX-License-Identifier:GPL-2.0+
- */
-
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-
-#include 
-#include 
-
-/* MXC ATA register offsets */
-struct mxc_ata_config_regs {
-   u8  time_off;   /* 0x00 */
-   u8  time_on;
-   u8  time_1;
-   u8  time_2w;
-   u8  time_2r;
-   u8  time_ax;
-   u8  time_pio_rdx;
-   u8  time_4;
-   u8  time_9;
-   u8  time_m;
-   u8  time_jn;
-   u8  time_d;
-   u8  time_k;
-   u8  time_ack;
-   u8  time_env;
-   u8  time_udma_rdx;
-   u8  time_zah;   /* 0x10 */
-   u8  time_mlix;
-   u8  time_dvh;
-   u8  time_dzfs;
-   u8  time_dvs;
-   u8  time_cvh;
-   u8  time_ss;
-   u8  time_cyc;
-   u32 fifo_data_32;   /* 0x18 */
-   u32 fifo_data_16;
-   u32 fifo_fill;
-   u32 ata_control;
-   u32 interrupt_pending;
-   u32 interrupt_enable;
-   u32 interrupt_clear;
-   u32 fifo_alarm;
-};
-
-struct mxc_data_hdd_regs {
-   u32 drive_data; /* 0xa0 */
-   u32 drive_features;
-   u32 drive_sector_count;
-   u32 drive_sector_num;
-   u32 drive_cyl_low;
-   u32 drive_cyl_high;
-   u32 drive_dev_head;
-   u32 command;
-   u32 status;
-   u32 alt_status;
-};
-
-/* PIO timing table */
-#defineNR_PIO_SPECS5
-static uint16_t pio_t1[NR_PIO_SPECS]   = { 70,  50,  30,  30,  25 };
-static uint16_t pio_t2_8[NR_PIO_SPECS] = { 290, 290, 290, 80,  70 };
-static uint16_t pio_t4[NR_PIO_SPECS]   = { 30,  20,  15,  10,  10 };
-static uint16_t pio_t9[NR_PIO_SPECS]   = { 20,  15,  10,  10,  10 };
-static uint16_t pio_tA[NR_PIO_SPECS]   = { 50,  50,  50,  50,  50 };
-
-#defineREG2OFF(reg)uint32_t)reg) & 0x3) * 8)
-static void set_ata_bus_timing(unsigned char mode)
-{
-   uint32_t T = 10 / mxc_get_clock(MXC_IPG_CLK);
-
-   struct mxc_ata_config_regs *ata_regs;
-   ata_regs = (struct mxc_ata_config_regs *)CONFIG_SYS_ATA_BASE_ADDR;
-
-   if (mode >= NR_PIO_SPECS)
-   return;
-
-   /* Write TIME_OFF/ON/1/2W */
-   writeb(3, _regs->time_off);
-   writeb(3, _regs->time_on);
-   writeb((pio_t1[mode] + T) / T, _regs->time_1);
-   writeb((pio_t2_8[mode] + T) / T, _regs->time_2w);
-
-   /* Write TIME_2R/AX/RDX/4 */
-   writeb((pio_t2_8[mode] + T) / T, _regs->time_2r);
-   writeb((pio_tA[mode] + T) / T + 2, _regs->time_ax);
-   writeb(1, _regs->time_pio_rdx);
-   writeb((pio_t4[mode] + T) / T, _regs->time_4);
-
-   /* Write TIME_9 ; the rest of timing registers is irrelevant for PIO */
-   writeb((pio_t9[mode] + T) / T, _regs->time_9);
-}
-
-int ide_preinit(void)
-{
-   struct mxc_ata_config_regs *ata_regs;
-   ata_regs = (struct mxc_ata_config_regs *)CONFIG_SYS_ATA_BASE_ADDR;
-
-   /* 46.3.3.4 @ FSL iMX51 manual */
-   /* FIFO normal op., drive reset */
-   writel(0x80, _regs->ata_control);
-   /* FIFO normal op., drive not reset */
-   writel(0xc0, _regs->ata_control);
-
-   /* Configure the PIO timing */
-   set_ata_bus_timing(CONFIG_MXC_ATA_PIO_MODE);
-
-   /* 46.3.3.4 @ FSL iMX51 manual */
-   /* Drive not reset, IORDY handshake */
-   writel(0x41, _regs->ata_control);
-
-   return 0;
-}
-- 
2.15.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH 06/10] ata: Migrate CONFIG_FSL_SATA to Kconfig

[Tuomas Tynkkynen]

Use 'imply' here liberally to avoid the combinatorial explosion of
defconfig changes in the PowerPC boards.

Signed-off-by: Tuomas Tynkkynen 
---
 arch/powerpc/cpu/mpc83xx/Kconfig  |  1 +
 arch/powerpc/cpu/mpc85xx/Kconfig  | 13 +
 configs/MPC8315ERDB_defconfig |  1 +
 configs/MPC837XERDB_defconfig |  1 +
 configs/controlcenterd_36BIT_SDCARD_DEVELOP_defconfig |  1 +
 configs/controlcenterd_36BIT_SDCARD_defconfig |  1 +
 drivers/ata/Kconfig   |  6 ++
 include/configs/MPC8315ERDB.h |  1 -
 include/configs/MPC837XEMDS.h |  1 -
 include/configs/MPC837XERDB.h |  1 -
 include/configs/MPC8536DS.h   |  1 -
 include/configs/P1010RDB.h|  1 -
 include/configs/P1022DS.h |  1 -
 include/configs/P2041RDB.h|  1 -
 include/configs/T102xQDS.h|  1 -
 include/configs/T1040QDS.h|  1 -
 include/configs/T104xRDB.h|  1 -
 include/configs/T208xQDS.h|  1 -
 include/configs/T208xRDB.h|  1 -
 include/configs/T4240QDS.h|  1 -
 include/configs/T4240RDB.h|  2 --
 include/configs/controlcenterd.h  |  1 -
 include/configs/corenet_ds.h  |  1 -
 include/configs/cyrus.h   |  1 -
 include/configs/t4qds.h   |  1 -
 scripts/config_whitelist.txt  |  1 -
 26 files changed, 24 insertions(+), 20 deletions(-)

diff --git arch/powerpc/cpu/mpc83xx/Kconfig arch/powerpc/cpu/mpc83xx/Kconfig
index a3779734c1..05d29d2fd9 100644
--- arch/powerpc/cpu/mpc83xx/Kconfig
+++ arch/powerpc/cpu/mpc83xx/Kconfig
@@ -55,6 +55,7 @@ config TARGET_MPC837XEMDS
bool "Support MPC837XEMDS"
select BOARD_EARLY_INIT_F
imply CMD_SATA
+   imply FSL_SATA
 
 config TARGET_MPC837XERDB
bool "Support MPC837XERDB"
diff --git arch/powerpc/cpu/mpc85xx/Kconfig arch/powerpc/cpu/mpc85xx/Kconfig
index 1b0e3d3a2f..5df8175f1e 100644
--- arch/powerpc/cpu/mpc85xx/Kconfig
+++ arch/powerpc/cpu/mpc85xx/Kconfig
@@ -92,6 +92,7 @@ config TARGET_MPC8536DS
 # Use DDR3 controller with DDR2 DIMMs on this board
select SYS_FSL_DDRC_GEN3
imply CMD_SATA
+   imply FSL_SATA
 
 config TARGET_MPC8541CDS
bool "Support MPC8541CDS"
@@ -148,6 +149,7 @@ config TARGET_P1022DS
select SUPPORT_SPL
select SUPPORT_TPL
imply CMD_SATA
+   imply FSL_SATA
 
 config TARGET_P1023RDB
bool "Support P1023RDB"
@@ -230,6 +232,7 @@ config TARGET_P2041RDB
select BOARD_LATE_INIT if CHAIN_OF_TRUST
select PHYS_64BIT
imply CMD_SATA
+   imply FSL_SATA
 
 config TARGET_QEMU_PPCE500
bool "Support qemu-ppce500"
@@ -244,6 +247,7 @@ config TARGET_T1024QDS
select PHYS_64BIT
imply CMD_EEPROM
imply CMD_SATA
+   imply FSL_SATA
 
 config TARGET_T1023RDB
bool "Support T1023RDB"
@@ -642,6 +646,7 @@ config ARCH_P1010
imply CMD_SATA
imply CMD_PCI
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_P1011
bool
@@ -826,6 +831,7 @@ config ARCH_P3041
imply CMD_NAND
imply CMD_SATA
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_P4080
bool
@@ -887,6 +893,7 @@ config ARCH_P5020
select FSL_ELBC
imply CMD_SATA
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_P5040
bool
@@ -910,6 +917,7 @@ config ARCH_P5040
select FSL_ELBC
imply CMD_SATA
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_QEMU_E500
bool
@@ -976,6 +984,7 @@ config ARCH_T1040
imply CMD_NAND
imply CMD_SATA
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_T1042
bool
@@ -998,6 +1007,7 @@ config ARCH_T1042
imply CMD_NAND
imply CMD_SATA
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_T2080
bool
@@ -1023,6 +1033,7 @@ config ARCH_T2080
imply CMD_SATA
imply CMD_NAND
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_T2081
bool
@@ -1069,6 +1080,7 @@ config ARCH_T4160
imply CMD_SATA
imply CMD_NAND
imply CMD_REGINFO
+   imply FSL_SATA
 
 config ARCH_T4240
bool
@@ -1096,6 +1108,7 @@ config ARCH_T4240
imply CMD_SATA
imply CMD_NAND
imply CMD_REGINFO
+   imply FSL_SATA
 
 config BOOKE
bool
diff --git configs/MPC8315ERDB_defconfig configs/MPC8315ERDB_defconfig
index fc846b26bd..473b9a6d5a 100644
--- configs/MPC8315ERDB_defconfig
+++ 

[U-Boot] [PATCH 01/10] ata: Drop CONFIG_SATA_DWC

[Tuomas Tynkkynen]

The last user of this driver went away in June 2017, in commit:
98f705c9ce ("powerpc: remove 4xx support")

Signed-off-by: Tuomas Tynkkynen 
---
 drivers/ata/Makefile   |1 -
 drivers/ata/sata_dwc.c | 2077 
 drivers/ata/sata_dwc.h |  458 ---
 3 files changed, 2536 deletions(-)
 delete mode 100644 drivers/ata/sata_dwc.c
 delete mode 100644 drivers/ata/sata_dwc.h

diff --git drivers/ata/Makefile drivers/ata/Makefile
index 4e2de93025..7a4c8c6bf2 100644
--- drivers/ata/Makefile
+++ drivers/ata/Makefile
@@ -16,7 +16,6 @@ obj-$(CONFIG_MVSATA_IDE) += mvsata_ide.o
 obj-$(CONFIG_MX51_PATA) += mxc_ata.o
 obj-$(CONFIG_SATA) += sata.o
 obj-$(CONFIG_SATA_CEVA) += sata_ceva.o
-obj-$(CONFIG_SATA_DWC) += sata_dwc.o
 obj-$(CONFIG_SATA_MV) += sata_mv.o
 obj-$(CONFIG_SATA_SIL3114) += sata_sil3114.o
 obj-$(CONFIG_SATA_SIL) += sata_sil.o
diff --git drivers/ata/sata_dwc.c drivers/ata/sata_dwc.c
deleted file mode 100644
index 2f3b2ddb41..00
--- drivers/ata/sata_dwc.c
+++ /dev/null
@@ -1,2077 +0,0 @@
-/*
- * sata_dwc.c
- *
- * Synopsys DesignWare Cores (DWC) SATA host driver
- *
- * Author: Mark Miesfeld 
- *
- * Ported from 2.6.19.2 to 2.6.25/26 by Stefan Roese 
- * Copyright 2008 DENX Software Engineering
- *
- * Based on versions provided by AMCC and Synopsys which are:
- *  Copyright 2006 Applied Micro Circuits Corporation
- *  COPYRIGHT (C) 2005  SYNOPSYS, INC.  ALL RIGHTS RESERVED
- *
- * SPDX-License-Identifier:GPL-2.0+
- */
-/*
- * SATA support based on the chip canyonlands.
- *
- * 04-17-2009
- * The local version of this driver for the canyonlands board
- * does not use interrupts but polls the chip instead.
- */
-
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-
-#include "sata_dwc.h"
-
-#define DMA_NUM_CHANS  1
-#define DMA_NUM_CHAN_REGS  8
-
-#define AHB_DMA_BRST_DFLT  16
-
-struct dmareg {
-   u32 low;
-   u32 high;
-};
-
-struct dma_chan_regs {
-   struct dmareg sar;
-   struct dmareg dar;
-   struct dmareg llp;
-   struct dmareg ctl;
-   struct dmareg sstat;
-   struct dmareg dstat;
-   struct dmareg sstatar;
-   struct dmareg dstatar;
-   struct dmareg cfg;
-   struct dmareg sgr;
-   struct dmareg dsr;
-};
-
-struct dma_interrupt_regs {
-   struct dmareg tfr;
-   struct dmareg block;
-   struct dmareg srctran;
-   struct dmareg dsttran;
-   struct dmareg error;
-};
-
-struct ahb_dma_regs {
-   struct dma_chan_regschan_regs[DMA_NUM_CHAN_REGS];
-   struct dma_interrupt_regs   interrupt_raw;
-   struct dma_interrupt_regs   interrupt_status;
-   struct dma_interrupt_regs   interrupt_mask;
-   struct dma_interrupt_regs   interrupt_clear;
-   struct dmareg   statusInt;
-   struct dmareg   rq_srcreg;
-   struct dmareg   rq_dstreg;
-   struct dmareg   rq_sgl_srcreg;
-   struct dmareg   rq_sgl_dstreg;
-   struct dmareg   rq_lst_srcreg;
-   struct dmareg   rq_lst_dstreg;
-   struct dmareg   dma_cfg;
-   struct dmareg   dma_chan_en;
-   struct dmareg   dma_id;
-   struct dmareg   dma_test;
-   struct dmareg   res1;
-   struct dmareg   res2;
-   /* DMA Comp Params
-* Param 6 = dma_param[0], Param 5 = dma_param[1],
-* Param 4 = dma_param[2] ...
-*/
-   struct dmareg   dma_params[6];
-};
-
-#define DMA_EN 0x0001
-#define DMA_DI 0x
-#define DMA_CHANNEL(ch)(0x0001 << (ch))
-#define DMA_ENABLE_CHAN(ch)((0x0001 << (ch)) | \
-   ((0x1 << (ch)) << 8))
-#define DMA_DISABLE_CHAN(ch)   (0x |   \
-   ((0x1 << (ch)) << 8))
-
-#define SATA_DWC_MAX_PORTS 1
-#define SATA_DWC_SCR_OFFSET0x24
-#define SATA_DWC_REG_OFFSET0x64
-
-struct sata_dwc_regs {
-   u32 fptagr;
-   u32 fpbor;
-   u32 fptcr;
-   u32 dmacr;
-   u32 dbtsr;
-   u32 intpr;
-   u32 intmr;
-   u32 errmr;
-   u32 llcr;
-   u32 phycr;
-   u32 physr;
-   u32 rxbistpd;
-   u32 rxbistpd1;
-   u32 rxbistpd2;
-   u32 txbistpd;
-   u32 txbistpd1;
-   u32 txbistpd2;
-   u32 bistcr;
-   u32 bistfctr;
-   u32 bistsr;
-   u32 bistdecr;
-   u32 res[15];
-   u32 testr;
-   u32 versionr;
-   u32 idr;
-   u32 unimpl[192];
-   u32 dmadr[256];
-};
-
-#define SATA_DWC_TXFIFO_DEPTH  0x01FF
-#define SATA_DWC_RXFIFO_DEPTH  

[U-Boot] [PATCH 00/10] Migrate ATA drivers to Kconfig

[Tuomas Tynkkynen]

This series migrates ATA drivers (that is, CONFIG_x symbols
listed in drivers/ata/Makefile) to Kconfig. In the process
we get to remove two unused drivers.

Potential improvements on the help texts and/or Kconfig
dependencies on the added symbols are welcome.

Tuomas Tynkkynen (10):
  ata: Drop CONFIG_SATA_DWC
  ata: Drop CONFIG_MX51_PATA
  ata: Migrate CONFIG_SATA_SIL to Kconfig
  ata: Migrate CONFIG_SATA_SIL3114 to Kconfig
  ata: Migrate CONFIG_SATA_MV to Kconfig
  ata: Migrate CONFIG_FSL_SATA to Kconfig
  ata: Migrate CONFIG_DWC_AHSATA to Kconfig
  ata: Migrate CONFIG_SCSI_AHCI to Kconfig
  ata: Migrate CONFIG_LIBATA to Kconfig
  ata: Migrate CONFIG_MVSATA_IDE to Kconfig

 arch/arm/Kconfig   |3 +
 arch/arm/cpu/armv7/ls102xa/Kconfig |1 +
 arch/arm/cpu/armv8/fsl-layerscape/Kconfig  |2 +
 arch/arm/include/asm/arch-ls102xa/config.h |2 -
 arch/arm/mach-kirkwood/include/mach/config.h   |1 -
 arch/powerpc/cpu/mpc83xx/Kconfig   |1 +
 arch/powerpc/cpu/mpc85xx/Kconfig   |   19 +
 arch/x86/cpu/baytrail/Kconfig  |1 +
 arch/x86/cpu/braswell/Kconfig  |1 +
 arch/x86/cpu/broadwell/Kconfig |1 +
 arch/x86/cpu/coreboot/Kconfig  |1 +
 arch/x86/cpu/ivybridge/Kconfig |1 +
 arch/x86/cpu/qemu/Kconfig  |1 +
 arch/x86/cpu/queensbay/Kconfig |1 +
 configs/A10-OLinuXino-Lime_defconfig   |1 +
 configs/A20-OLinuXino-Lime2-eMMC_defconfig |1 +
 configs/A20-OLinuXino-Lime2_defconfig  |1 +
 configs/A20-OLinuXino-Lime_defconfig   |1 +
 configs/A20-OLinuXino_MICRO-eMMC_defconfig |1 +
 configs/A20-OLinuXino_MICRO_defconfig  |1 +
 configs/A20-Olimex-SOM-EVB_defconfig   |1 +
 configs/Bananapi_M2_Ultra_defconfig|1 +
 configs/Bananapi_defconfig |1 +
 configs/Bananapro_defconfig|1 +
 configs/Cubieboard2_defconfig  |1 +
 configs/Cubieboard_defconfig   |1 +
 configs/Cubietruck_defconfig   |1 +
 configs/Itead_Ibox_A20_defconfig   |1 +
 configs/Lamobo_R1_defconfig|1 +
 configs/Linksprite_pcDuino3_Nano_defconfig |1 +
 configs/Linksprite_pcDuino3_defconfig  |1 +
 configs/MPC8315ERDB_defconfig  |1 +
 configs/MPC8349ITX_LOWBOOT_defconfig   |1 +
 configs/MPC8349ITX_defconfig   |1 +
 configs/MPC837XERDB_defconfig  |1 +
 configs/MPC8544DS_defconfig|1 +
 configs/MPC8572DS_36BIT_defconfig  |1 +
 configs/MPC8572DS_defconfig|1 +
 configs/MPC8610HPCD_defconfig  |1 +
 configs/MPC8641HPCN_36BIT_defconfig|1 +
 configs/MPC8641HPCN_defconfig  |1 +
 configs/Marsboard_A10_defconfig|1 +
 configs/Mele_A1000_defconfig   |1 +
 configs/Mele_M5_defconfig  |1 +
 configs/Orangepi_defconfig |1 +
 configs/Orangepi_mini_defconfig|1 +
 configs/TWR-P1025_defconfig|1 +
 configs/Wits_Pro_A20_DKT_defconfig |1 +
 configs/am57xx_evm_defconfig   |1 +
 configs/am57xx_hs_evm_defconfig|1 +
 configs/apalis_imx6_defconfig  |1 +
 configs/apalis_imx6_nospl_com_defconfig|1 +
 configs/apalis_imx6_nospl_it_defconfig |1 +
 configs/bananapi_m1_plus_defconfig |1 +
 configs/cgtqmx6eval_defconfig  |1 +
 configs/cl-som-am57x_defconfig |1 +
 configs/cm_fx6_defconfig   |1 +
 configs/cm_t54_defconfig   |1 +
 .../controlcenterd_36BIT_SDCARD_DEVELOP_defconfig  |1 +
 configs/controlcenterd_36BIT_SDCARD_defconfig  |1 +
 configs/controlcenterdc_defconfig  |1 +
 configs/d2net_v2_defconfig |1 +
 configs/db-88f6820-gp_defconfig|1 +
 configs/db-mv784mp-gp_defconfig|1 +
 configs/dh_imx6_defconfig  |1 +
 configs/dms-ba16-1g_defconfig  |1 +
 configs/dms-ba16_defconfig |1 +
 configs/dns325_defconfig   |1 +
 configs/dreamplug_defconfig|1 +
 configs/ds109_defconfig|1 +
 configs/edminiv2_defconfig  

Re: [U-Boot] [U-Boot, v2, 1/2] drivers/reset: support rockchip reset drivers

[Dr. Philipp Tomsich]

> On 8 Dec 2017, at 14:27, Dr. Philipp Tomsich 
>  wrote:
> 
>> 
>> On 4 Dec 2017, at 02:04, Kever Yang > > wrote:
>> 
>> Philipp,
>> 
>> 
>> On 11/29/2017 03:49 AM, Dr. Philipp Tomsich wrote:
>>> Kever,
>>> 
>>> This series causes build-breakage (even outside of the rockchip targets)...
>>> See https://travis-ci.org/ptomsich/u-boot-rockchip/builds/308522589 
>>> 
>>> 
>> I'm not understand this error, sometimes I also met this, which depends on 
>> the build system.
>> The asm/arch is a link to asm/arch-rockchip which is dynamic generated by 
>> build system,
>> is there a option to make sure the build system generate the link before 
>> build the c source?
>> +drivers/reset/reset-rockchip.c:11:31: fatal error: asm/arch/hardware.h: No 
>> such file or directory
>> 
>>> Please fix and resubmit.
>>> Also: did you test this using buildman or travis?
>> 
>> I use buildman, but no expperience with travis.

You might want to check your buildman configuration, as the following should 
have
been caught by buildman on your end...

The following snippet from the patch

@@ -1221,6 +1227,18 @@ static int rk3399_pmuclk_ofdata_to_platdata(struct 
udevice *dev)
return 0;
 }
 
+static int rk3399_pmuclk_bind(struct udevice *dev)
+{
+   int ret = 0;
+
+   offset = offsetof(struct rk3399_pmucru, pmucru_softrst_con[0]);
+   ret = rockchip_reset_bind(dev, offset, 2);
+   if (ret)
+   debug("Warning: software reset driver bind faile\n");
+
+   return 0;
+}
+

results in the buildman failure (due to offset not being declared):

   aarch64:  +   firefly-rk3399
+drivers/clk/rockchip/clk_rk3399.c: In function 'rk3399_pmuclk_bind':
+drivers/clk/rockchip/clk_rk3399.c:1234:2: error: 'offset' undeclared (first 
use in this function); did you mean 'off_t'?
+  offset = offsetof(struct rk3399_pmucru, pmucru_softrst_con[0]);
+  ^~
+  off_t
+drivers/clk/rockchip/clk_rk3399.c:1234:2: note: each undeclared identifier is 
reported only once for each function it appears in
+make[4]: *** [drivers/clk/rockchip/clk_rk3399.o] Error 1
+make[3]: *** [drivers/clk/rockchip] Error 2
+make[2]: *** [drivers/clk] Error 2
+make[1]: *** [drivers] Error 2
+make: *** [sub-make] Error 2

Thanks,
Philipp.

> 
>> Thanks,
>> - Kever
>>> 
>>> Thanks,
>>> Philipp.
>>> 
>>> 
 On 28 Nov 2017, at 11:06, Philipp Tomsich 
 > wrote:
 
> From: Elaine Zhang  >
> 
> Create driver to support all Rockchip SoCs soft reset.
> Example of usage:
> i2c driver:
>   ret = reset_get_by_name(dev, "i2c", _ctl);
>   if (ret) {
>   error("reset_get_by_name() failed: %d\n", ret);
>   }
> 
>   reset_assert(_ctl);
>   udelay(50);
>   reset_deassert(_ctl);
> 
> i2c dts node:
> resets = < SRST_P_I2C1>, < SRST_I2C1>;
> reset-names = "p_i2c", "i2c";
> 
> Signed-off-by: Elaine Zhang  >
> Signed-off-by: Kever Yang  >
> Acked-by: Philipp Tomsich  >
> ---
> 
> Changes in v2:
> - fix Kconfig more than 80 length
> - use MACRO for reset bits in one reg
> - use rkclr/set_reg for reg access
> - add rockchip_reset_bind()
> - use dev_read_addr_size() instead of fdtdec_
> 
> drivers/reset/Kconfig  |   9 +++
> drivers/reset/Makefile |   1 +
> drivers/reset/reset-rockchip.c | 133 
> +
> 3 files changed, 143 insertions(+)
> create mode 100644 drivers/reset/reset-rockchip.c
> 
 
 Reviewed-by: Philipp Tomsich >
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de 
 https://lists.denx.de/listinfo/u-boot 
 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [U-Boot, v2, 1/2] drivers/reset: support rockchip reset drivers

[Dr. Philipp Tomsich]

> On 4 Dec 2017, at 02:04, Kever Yang  wrote:
> 
> Philipp,
> 
> 
> On 11/29/2017 03:49 AM, Dr. Philipp Tomsich wrote:
>> Kever,
>> 
>> This series causes build-breakage (even outside of the rockchip targets)...
>> See https://travis-ci.org/ptomsich/u-boot-rockchip/builds/308522589 
>> 
>> 
> I'm not understand this error, sometimes I also met this, which depends on 
> the build system.
> The asm/arch is a link to asm/arch-rockchip which is dynamic generated by 
> build system,
> is there a option to make sure the build system generate the link before 
> build the c source?
> +drivers/reset/reset-rockchip.c:11:31: fatal error: asm/arch/hardware.h: No 
> such file or directory
> 
>> Please fix and resubmit.
>> Also: did you test this using buildman or travis?
> 
> I use buildman, but no expperience with travis.

This reproduces with buildman for me.
I wonder what difference in our setups causes this not to show up on your end…
Also: why is reset-rockchip.o even being built for the evb-ast2500 target?

ptomsich@android:~/u-boot-rockchip$ tools/buildman/buildman arm11
Building current source for 11 boards (4 threads, 1 job per thread)
   arm:  +   evb-ast2500   
+drivers/reset/reset-rockchip.c:11:31: fatal error: asm/arch/hardware.h: No 
such file or directory
+ #include 
+   ^
+compilation terminated.
+make[3]: *** [drivers/reset/reset-rockchip.o] Error 1
+make[2]: *** [drivers/reset] Error 2
+make[1]: *** [drivers] Error 2
+make: *** [sub-make] Error 2
   arm:  +   rpi   
+arch/arm/dts/bcm2835-rpi-a-plus.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2835-rpi-a.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2835-rpi-b-plus.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2835-rpi-b-rev2.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2835-rpi-b.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2836-rpi-2-b.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
+arch/arm/dts/bcm2837-rpi-3-b.dtb: Warning (phys_property): Missing property 
'#phy-cells' in node /phy or bad phandle (referred from 
/soc/usb@7e98:phys[0])
911 /11 0:00:07  : mx31pdk


> Thanks,
> - Kever
>> 
>> Thanks,
>> Philipp.
>> 
>> 
>>> On 28 Nov 2017, at 11:06, Philipp Tomsich 
>>> >> > wrote:
>>> 
 From: Elaine Zhang >
 
 Create driver to support all Rockchip SoCs soft reset.
 Example of usage:
 i2c driver:
ret = reset_get_by_name(dev, "i2c", _ctl);
if (ret) {
error("reset_get_by_name() failed: %d\n", ret);
}
 
reset_assert(_ctl);
udelay(50);
reset_deassert(_ctl);
 
 i2c dts node:
 resets = < SRST_P_I2C1>, < SRST_I2C1>;
 reset-names = "p_i2c", "i2c";
 
 Signed-off-by: Elaine Zhang >
 Signed-off-by: Kever Yang >
 Acked-by: Philipp Tomsich >
 ---
 
 Changes in v2:
 - fix Kconfig more than 80 length
 - use MACRO for reset bits in one reg
 - use rkclr/set_reg for reg access
 - add rockchip_reset_bind()
 - use dev_read_addr_size() instead of fdtdec_
 
 drivers/reset/Kconfig  |   9 +++
 drivers/reset/Makefile |   1 +
 drivers/reset/reset-rockchip.c | 133 
 +
 3 files changed, 143 insertions(+)
 create mode 100644 drivers/reset/reset-rockchip.c
 
>>> 
>>> Reviewed-by: Philipp Tomsich >> >
>>> ___
>>> U-Boot mailing list
>>> U-Boot@lists.denx.de 
>>> https://lists.denx.de/listinfo/u-boot 
>>> 
> 

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [RFC PATCH] test: py: Disable sleep test for qemu targets

[Michal Simek]

On 6.12.2017 13:17, Tom Rini wrote:
> On Wed, Dec 06, 2017 at 10:53:08AM +0100, Michal Simek wrote:
>> On 5.12.2017 16:13, Tom Rini wrote:
>>> On Tue, Dec 05, 2017 at 01:10:47PM +0100, Michal Simek wrote:
 On 4.12.2017 18:14, Stephen Warren wrote:
> On 12/04/2017 08:30 AM, Tom Rini wrote:
>> On Mon, Dec 04, 2017 at 03:21:04PM +0100, Michal Simek wrote:
>>> On 4.12.2017 15:03, Tom Rini wrote:
 On Mon, Dec 04, 2017 at 02:55:45PM +0100, Michal Simek wrote:
> On 1.12.2017 23:44, Tom Rini wrote:
>> On Fri, Dec 01, 2017 at 10:07:54AM -0700, Stephen Warren wrote:
>>> On 12/01/2017 08:19 AM, Michal Simek wrote:
 Hi,

 On 1.12.2017 16:06, Heinrich Schuchardt wrote:
>
>
> On 12/01/2017 03:46 PM, Michal Simek wrote:
>> Qemu for arm32/arm64 has a problem with time setup.
>
> Wouldn't it be preferable to fix the root cause?

 Definitely that would be the best and IIRC I have tried to
 convince our
 qemu guy to do that but they have never done that.
>>>
>>> What is the exact failure condition? Is it simply that the test
>>> is still
>>> slightly too strict about which delays it accepts, or is sleep
>>> outright
>>> broken?
>>>
>>> You can use command-line option -k to avoid some tests. For
>>> example "-k not
>>> sleep". That way, we don't have to hard-code the dependency into
>>> the test
>>> source. Depending on the root cause (issue in U-Boot, or issue in
>>> just your
>>> local version of qemu, or something that will never work) this
>>> might be
>>> better?
>>
>> Even with the most recent relaxing of the sleep test requirements,
>> I can
>> still (depending on overall system load) have 'sleep' take too
>> long, on
>> QEMU.  I think it might have been half a second slow, but I don't
>> have
>> the log handy anymore.  Both locally and in travis we -k not sleep
>> all
>> of the qemu instances.
>
> ok. By locally do you mean just using -k not sleep?

 Yes, I have that in my CI scripts and similar.
>>>
>>> Wouldn't be easier to keep this in uboot-test-hooks repo with other
>>> target setting?
>>
>> Or do as you did did and mark the tests as not allowed for qemu, yes.
>>
>>> What we are trying to do is that our testing group will run these tests
>>> for me that's why it is just easier for me to change local
>>> uboot-test-hooks repo instead of communicate with them what -k not XXX
>>> parameters to add to certain scripts.
>>>
>>> It means in loop they will just run all tests on qemu, local targets and
>>> in boardfarm. It is probably not big deal to tell them to add -k not
>>> sleep for all qemu runs but I know that for some i2c testing qemu
>>> doesn't emulate these devices that's why these tests fails. And the
>>> amount of tests which we shouldn't run on qemu will probably grow.
>>
>> Well, I'm still open to possibly tweaking the allowed variance in the
>> sleep test.  OTOH, if we just say "no QEMU" here, we can then go back to
>> "sleep should be pretty darn accurate on HW" for the test too, and
>> perhaps that's best.
>
> The fundamental problem of "over-sleeping" due to host system load/..
> exists with all boards. There's nothing specific to qemu here except
> that running U-Boot on qemu on the host rather than on separate HW might
> more easily trigger the "high load on the host" condition; I see the
> issue now and then and manually retry that test, although that is a bit
> annoying.
>
> The original test was mostly intended to make sure that e U-Boot clock
> didn't run at a significantly different rate to the host, since I had
> seen that issue during development of some board support or as a
> regression sometime. Perhaps the definition of "significantly different"
> should be more like "1/2 rate or twice rate or more" rather than "off by
> a small fraction of a second". That might avoid so many false positives.

 We had this issue with silicon v1 and having accurate sleep measuring is
 definitely good thing to have (Probably make sense to enable margin
 setup via variable anyway).

 But still I would extend this to more wider discussion how to disable
 just one particular test case which is verified that it is broken on
 certain target/target configuration.
 Using -k not XXX option is possible but as I said before it is not ideal
 to keep track of these problematic tests in two locations and share this
 between two teams.

 Better would be to add to u_boot_boardenv...py file line like 

[U-Boot] [PATCH v1] test: py: Add an option to skip sleep test

[Michal Simek]

Some QEMUs have a problem with time setup that's why
sleep test is failing. Introduce env__sleep_accurate
boardenv variable to have an option to skip sleep test.

Signed-off-by: Michal Simek 
---

Changes in v1:
- Change subject line and patch description
- Move from RFC to regular patch
- Remove boardidentity marker and use variable instead

 configs/zynq_zc706_defconfig |  1 +
 test/py/tests/test_sleep.py  | 13 +
 2 files changed, 14 insertions(+)

diff --git a/configs/zynq_zc706_defconfig b/configs/zynq_zc706_defconfig
index 52999debc115..762b892f3fcf 100644
--- a/configs/zynq_zc706_defconfig
+++ b/configs/zynq_zc706_defconfig
@@ -37,6 +37,7 @@ CONFIG_CMD_EXT4=y
 CONFIG_CMD_EXT4_WRITE=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
+CONFIG_OF_EMBED=y
 CONFIG_ENV_IS_IN_SPI_FLASH=y
 CONFIG_NET_RANDOM_ETHADDR=y
 CONFIG_SPL_DM_SEQ_ALIAS=y
diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
index 64e057132622..ccef24d7f7b8 100644
--- a/test/py/tests/test_sleep.py
+++ b/test/py/tests/test_sleep.py
@@ -5,10 +5,23 @@
 import pytest
 import time
 
+"""
+Note: This test doesn't rely on boardenv_* configuration values but they can
+change test behavior.
+
+# Setup env__sleep_accurate to False if time is not accurate on your platform
+env__sleep_accurate = False
+
+"""
+
 def test_sleep(u_boot_console):
 """Test the sleep command, and validate that it sleeps for approximately
 the correct amount of time."""
 
+sleep_skip = u_boot_console.config.env.get('env__sleep_accurate', True)
+if not sleep_skip:
+pytest.skip('sleep is not accurate')
+
 if u_boot_console.config.buildconfig.get('config_cmd_misc', 'n') != 'y':
 pytest.skip('sleep command not supported')
 # 3s isn't too long, but is enough to cross a few second boundaries.
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] sunxi: OLinuXino: Drop FASTBOOT_FLASH

[Jagan Teki]

On Fri, Dec 8, 2017 at 5:26 PM, Maxime Ripard
 wrote:
> On Fri, Dec 08, 2017 at 05:09:51PM +0530, Jagan Teki wrote:
>> FASTBOOT_FLASH is auto select for sunxi platform now,
>> so drop explicit addition.
>>
>> Signed-off-by: Jagan Teki 
>
> There's far more boards than these three, maybe we should do all of
> them?

True, but this is because I've tested these.

-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH] sunxi: OLinuXino: Drop FASTBOOT_FLASH

[Maxime Ripard]

On Fri, Dec 08, 2017 at 05:09:51PM +0530, Jagan Teki wrote:
> FASTBOOT_FLASH is auto select for sunxi platform now,
> so drop explicit addition.
> 
> Signed-off-by: Jagan Teki 

There's far more boards than these three, maybe we should do all of
them?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] sunxi: OLinuXino: Drop FASTBOOT_FLASH

[Jagan Teki]

FASTBOOT_FLASH is auto select for sunxi platform now,
so drop explicit addition.

Signed-off-by: Jagan Teki 
---
 configs/A13-OLinuXino_defconfig| 1 -
 configs/A20-OLinuXino-Lime2-eMMC_defconfig | 1 -
 configs/A20-OLinuXino-Lime2_defconfig  | 1 -
 3 files changed, 3 deletions(-)

diff --git a/configs/A13-OLinuXino_defconfig b/configs/A13-OLinuXino_defconfig
index 3c2a6f8..4e65698 100644
--- a/configs/A13-OLinuXino_defconfig
+++ b/configs/A13-OLinuXino_defconfig
@@ -16,7 +16,6 @@ CONFIG_DEFAULT_DEVICE_TREE="sun5i-a13-olinuxino"
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SPL=y
 CONFIG_SPL_I2C_SUPPORT=y
-CONFIG_FASTBOOT_FLASH=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
 # CONFIG_CMD_FPGA is not set
diff --git a/configs/A20-OLinuXino-Lime2-eMMC_defconfig 
b/configs/A20-OLinuXino-Lime2-eMMC_defconfig
index cd1fa64..2f2b5d1 100644
--- a/configs/A20-OLinuXino-Lime2-eMMC_defconfig
+++ b/configs/A20-OLinuXino-Lime2-eMMC_defconfig
@@ -13,7 +13,6 @@ CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SPL=y
 CONFIG_SPL_I2C_SUPPORT=y
-CONFIG_FASTBOOT_FLASH=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
 # CONFIG_CMD_FPGA is not set
diff --git a/configs/A20-OLinuXino-Lime2_defconfig 
b/configs/A20-OLinuXino-Lime2_defconfig
index 4a90ab6..9492cae 100644
--- a/configs/A20-OLinuXino-Lime2_defconfig
+++ b/configs/A20-OLinuXino-Lime2_defconfig
@@ -12,7 +12,6 @@ CONFIG_AHCI=y
 # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
 CONFIG_SPL=y
 CONFIG_SPL_I2C_SUPPORT=y
-CONFIG_FASTBOOT_FLASH=y
 CONFIG_CMD_DFU=y
 # CONFIG_CMD_FLASH is not set
 # CONFIG_CMD_FPGA is not set
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot