Re: [U-Boot] [PATCH 0/8] samsung: Use common config files with Samsung boards (help wanted)

2014-09-09 Thread Simon Glass
Hi Minkyu,

On 25 July 2014 08:43, Simon Glass s...@chromium.org wrote:

 Hi Lucasz,

 On 24 July 2014 09:34, Lukasz Majewski l.majew...@samsung.com wrote:
  Hi Simon,
 
  This series tries to unify the Samsung board configs into a few header
  files for exynos5 and exynos5.
 
  The purpose is to make it easier to move to driver model. In that case
  I would like things like the GPIO drivers and serial drivers to work
  in a standard way, and not need to support device tree and platform
  data at the same time. That would be quite painful.
 
  Another reason is that the Chrome OS EC drivers are currently
  included in boards that don't have a Chrome OS EC. This concern was
  raised by the Samsung maintainer (Minkyu) a while back.
 
  There are still a few boards that don't use CONFIG_OF_CONTROL so I
  have updated these with the most rudimentary of device tree files.
 
  Unfortunately I don't have boards for most of these (for testing) and
  I am hoping that the maintainers can come to the rescue and fix up any
  patches that have problems. I am also worried that I have used a
  common exynos file for things like smdkv310, when in fact they have
  some other chip in common.
 
  So maintainers, please can you test this and re-issue the patch, or
  make comments on my attempts?
 
  We will do our best to test our Exynos4 based boards.

 Thanks - please feel free to re-issue the patches and I will drop mine
 (if you cc me).


Can you please review this series? We are already at rc2.

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


[U-Boot] [PATCH] mpc8xx: move common linker scripts into the CPU directory

2014-09-09 Thread Masahiro Yamada
Each CPU of PowerPC has its default linker script under the CPU
directory, except mpc8xx.

Every mpc8xx board has its own linker script under the board
directory, resulting in lots of duplication of linker scripts.

I notice eight mpc8xx boards have the same linker script.
We can decrease the number of linker scripts by putting a single
default linker script, arch/powerpc/cpu/mpc8xx/u-boot.lds.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Wolfgang Denk w...@denx.de
---

 .../ip860 = arch/powerpc/cpu/mpc8xx}/u-boot.lds   |  0
 board/eltec/mhpc/u-boot.lds| 82 -
 board/emk/top860/u-boot.lds| 83 -
 board/ivm/u-boot.lds   | 82 -
 board/kup/kup4k/u-boot.lds | 82 -
 board/lwmon/u-boot.lds | 82 -
 board/manroland/uc100/u-boot.lds   | 85 --
 board/netvia/u-boot.lds| 82 -
 8 files changed, 578 deletions(-)
 rename {board/ip860 = arch/powerpc/cpu/mpc8xx}/u-boot.lds (100%)
 delete mode 100644 board/eltec/mhpc/u-boot.lds
 delete mode 100644 board/emk/top860/u-boot.lds
 delete mode 100644 board/ivm/u-boot.lds
 delete mode 100644 board/kup/kup4k/u-boot.lds
 delete mode 100644 board/lwmon/u-boot.lds
 delete mode 100644 board/manroland/uc100/u-boot.lds
 delete mode 100644 board/netvia/u-boot.lds

diff --git a/board/ip860/u-boot.lds b/arch/powerpc/cpu/mpc8xx/u-boot.lds
similarity index 100%
rename from board/ip860/u-boot.lds
rename to arch/powerpc/cpu/mpc8xx/u-boot.lds
diff --git a/board/eltec/mhpc/u-boot.lds b/board/eltec/mhpc/u-boot.lds
deleted file mode 100644
index 7ae91ff..000
--- a/board/eltec/mhpc/u-boot.lds
+++ /dev/null
@@ -1,82 +0,0 @@
-/*
- * (C) Copyright 2001-2010
- * Wolfgang Denk, DENX Software Engineering, w...@denx.de.
- *
- * SPDX-License-Identifier:GPL-2.0+
- */
-
-OUTPUT_ARCH(powerpc)
-
-SECTIONS
-{
-  /* Read-only sections, merged into text segment: */
-  . = + SIZEOF_HEADERS;
-  .text  :
-  {
-arch/powerpc/cpu/mpc8xx/start.o(.text*)
-arch/powerpc/cpu/mpc8xx/traps.o(.text*)
-
-*(.text*)
-  }
-  _etext = .;
-  PROVIDE (etext = .);
-  .rodata:
-  {
-*(SORT_BY_ALIGNMENT(SORT_BY_NAME(.rodata*)))
-  }
-
-  /* Read-write section, merged into data segment: */
-  . = (. + 0x00FF)  0xFF00;
-  _erotext = .;
-  PROVIDE (erotext = .);
-  .reloc   :
-  {
-_GOT2_TABLE_ = .;
-KEEP(*(.got2))
-KEEP(*(.got))
-PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4);
-_FIXUP_TABLE_ = .;
-KEEP(*(.fixup))
-  }
-  __got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  2) - 1;
-  __fixup_entries = (. - _FIXUP_TABLE_)2;
-
-  .data:
-  {
-*(.data*)
-*(.sdata*)
-  }
-  _edata  =  .;
-  PROVIDE (edata = .);
-
-  . = .;
-
-  . = ALIGN(4);
-  .u_boot_list : {
-   KEEP(*(SORT(.u_boot_list*)));
-  }
-
-
-  . = .;
-  __start___ex_table = .;
-  __ex_table : { *(__ex_table) }
-  __stop___ex_table = .;
-
-  . = ALIGN(256);
-  __init_begin = .;
-  .text.init : { *(.text.init) }
-  .data.init : { *(.data.init) }
-  . = ALIGN(256);
-  __init_end = .;
-
-  __bss_start = .;
-  .bss (NOLOAD)   :
-  {
-   *(.bss*)
-   *(.sbss*)
-   *(COMMON)
-   . = ALIGN(4);
-  }
-  __bss_end = . ;
-  PROVIDE (end = .);
-}
diff --git a/board/emk/top860/u-boot.lds b/board/emk/top860/u-boot.lds
deleted file mode 100644
index 79fcbf4..000
--- a/board/emk/top860/u-boot.lds
+++ /dev/null
@@ -1,83 +0,0 @@
-/*
- * (C) Copyright 2000-2010
- * Wolfgang Denk, DENX Software Engineering, w...@denx.de.
- *
- * SPDX-License-Identifier:GPL-2.0+
- */
-
-OUTPUT_ARCH(powerpc)
-
-SECTIONS
-{
-  /* Read-only sections, merged into text segment: */
-  . = + SIZEOF_HEADERS;
-  .text  :
-  {
-arch/powerpc/cpu/mpc8xx/start.o(.text*)
-arch/powerpc/cpu/mpc8xx/traps.o(.text*)
-
-*(.text*)
-  }
-  _etext = .;
-  PROVIDE (etext = .);
-  .rodata:
-  {
-*(SORT_BY_ALIGNMENT(SORT_BY_NAME(.rodata*)))
-  }
-
-  /* Read-write section, merged into data segment: */
-  . = (. + 0x00FF)  0xFF00;
-  _erotext = .;
-  PROVIDE (erotext = .);
-  .reloc   :
-  {
-_GOT2_TABLE_ = .;
-KEEP(*(.got2))
-KEEP(*(.got))
-PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4);
-_FIXUP_TABLE_ = .;
-KEEP(*(.fixup))
-  }
-  __got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  2) - 1;
-  __fixup_entries = (. - _FIXUP_TABLE_)2;
-
-  .data:
-  {
-*(.data*)
-*(.sdata*)
-  }
-  _edata  =  .;
-  PROVIDE (edata = .);
-
-
-  . = .;
-
-  . = ALIGN(4);
-  .u_boot_list : {
-   KEEP(*(SORT(.u_boot_list*)));
-  }
-
-
-  . = .;
-  __start___ex_table = .;
-  __ex_table : { *(__ex_table) }
-  __stop___ex_table = .;
-
-  . = ALIGN(256);
-  __init_begin = .;
-  .text.init : { *(.text.init) }
-  .data.init : { *(.data.init) }
-  . = ALIGN(256);
-  __init_end = .;
-
-  

Re: [U-Boot] [PATCH] mpc8xx: move common linker scripts into the CPU directory

2014-09-09 Thread Stefan Roese

On 09.09.2014 08:12, Masahiro Yamada wrote:

Each CPU of PowerPC has its default linker script under the CPU
directory, except mpc8xx.

Every mpc8xx board has its own linker script under the board
directory, resulting in lots of duplication of linker scripts.

I notice eight mpc8xx boards have the same linker script.
We can decrease the number of linker scripts by putting a single
default linker script, arch/powerpc/cpu/mpc8xx/u-boot.lds.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Wolfgang Denk w...@denx.de


Thanks again for spotting such issues:

Acked-by: Stefan Roese s...@denx.de

Thanks,
Stefan

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


Re: [U-Boot] relocation problem on powerpc

2014-09-09 Thread Wolfgang Denk
Dear Chris,

In message CAFOYHZCtPSJRvL8asB9O6dUPtNHZSE_gUd=n8+6hyqsnyhd...@mail.gmail.com 
you wrote:
 
 Do you mean the address or the value? FYI the 77fb addresses are
 what I believe to be the correct relocated address. It's the f0d0
 address for NetArpWaitTimerStart that I believe is causing the problem
 I am seeing.

Of course you are right.  Silly me. 

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Hindsight is an exact science.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/5] ARM: sheevaplug: refresh for 201410

2014-09-09 Thread Nobuhiro Iwamatsu
Hi, Albert.

These patches are not taken for a long time.
If possible and Prafulla is OK, can you apply to u-boot-arm directly?

Best regards,
  Nobuhiro

2014-09-08 1:01 GMT+09:00 Gerald Kerma drea...@doukki.net:
 This serie of patches add some needed features to sheevaplug

 Gerald Kerma (5):
   ARM: sheevaplug: change env location
   ARM: sheevaplug: add MVSATA driver
   ARM: sheevaplug: add MTD defaults
   ARM: sheevaplug: redefine MTDPARTS
   ARM: sheevaplug: add HUSH parser

  include/configs/sheevaplug.h | 46 
 +++-
  1 file changed, 41 insertions(+), 5 deletions(-)

 --
 1.9.1




-- 
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 4/4] imx: mx6solosabreauto: Add the i.MX6Solo SABREAUTO board support

2014-09-09 Thread Ye . Li
This patch is to add the i.MX6solo sabreauto support,

The i.MX6solo sabreauto board configuration has the following
difference with i.MX6dl sabreauto:

- DDR bus width: 32bit
- DDR capacity:  1024M

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Rework the short log subject

 board/freescale/mx6qsabreauto/mx6solo.cfg |  106 +
 configs/mx6solosabreauto_defconfig|3 +
 2 files changed, 109 insertions(+), 0 deletions(-)
 create mode 100644 board/freescale/mx6qsabreauto/mx6solo.cfg
 create mode 100644 configs/mx6solosabreauto_defconfig

diff --git a/board/freescale/mx6qsabreauto/mx6solo.cfg 
b/board/freescale/mx6qsabreauto/mx6solo.cfg
new file mode 100644
index 000..4311f68
--- /dev/null
+++ b/board/freescale/mx6qsabreauto/mx6solo.cfg
@@ -0,0 +1,106 @@
+/*
+ * Copyright (C) 2014 Freescale Semiconductor, Inc.
+ * Jason Liu r64...@freescale.com
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ *
+ * Refer doc/README.imximage for more details about how-to configure
+ * and create imximage boot image
+ *
+ * The syntax is taken as close as possible with the kwbimage
+ */
+/* image version */
+
+IMAGE_VERSION 2
+
+/*
+ * Boot Device : one of
+ * spi, sd (the board has no nand neither onenand)
+ */
+BOOT_FROM  sd
+
+/*
+ * Device Configuration Data (DCD)
+ *
+ * Each entry must have the format:
+ * Addr-type   AddressValue
+ *
+ * where:
+ * Addr-type register length (1,2 or 4 bytes)
+ * Address   absolute address of the register
+ * value value to be stored in the register
+ */
+DATA 4 0x020e0774 0x000C
+DATA 4 0x020e0754 0x
+DATA 4 0x020e04ac 0x0030
+DATA 4 0x020e04b0 0x0030
+DATA 4 0x020e0464 0x0030
+DATA 4 0x020e0490 0x0030
+DATA 4 0x020e074c 0x0030
+DATA 4 0x020e0494 0x0030
+DATA 4 0x020e04a0 0x
+DATA 4 0x020e04b4 0x0030
+DATA 4 0x020e04b8 0x0030
+DATA 4 0x020e076c 0x0030
+DATA 4 0x020e0750 0x0002
+DATA 4 0x020e04bc 0x0028
+DATA 4 0x020e04c0 0x0028
+DATA 4 0x020e04c4 0x0028
+DATA 4 0x020e04c8 0x0028
+DATA 4 0x020e0760 0x0002
+DATA 4 0x020e0764 0x0028
+DATA 4 0x020e0770 0x0028
+DATA 4 0x020e0778 0x0028
+DATA 4 0x020e077c 0x0028
+DATA 4 0x020e0470 0x0028
+DATA 4 0x020e0474 0x0028
+DATA 4 0x020e0478 0x0028
+DATA 4 0x020e047c 0x0028
+DATA 4 0x021b0800 0xa1390003
+DATA 4 0x021b080c 0x001F001F
+DATA 4 0x021b0810 0x001F001F
+DATA 4 0x021b083c 0x421C0216
+DATA 4 0x021b0840 0x017B017A
+DATA 4 0x021b0848 0x4B4A4E4C
+DATA 4 0x021b0850 0x3F3F3334
+DATA 4 0x021b081c 0x
+DATA 4 0x021b0820 0x
+DATA 4 0x021b0824 0x
+DATA 4 0x021b0828 0x
+DATA 4 0x021b08b8 0x0800
+DATA 4 0x021b0004 0x00020025
+DATA 4 0x021b0008 0x00333030
+DATA 4 0x021b000c 0x676B5313
+DATA 4 0x021b0010 0xB66E8B63
+DATA 4 0x021b0014 0x01FF00DB
+DATA 4 0x021b0018 0x1740
+DATA 4 0x021b001c 0x8000
+DATA 4 0x021b002c 0x26d2
+DATA 4 0x021b0030 0x006B1023
+DATA 4 0x021b0040 0x0027
+DATA 4 0x021b 0x8419
+DATA 4 0x021b001c 0x04008032
+DATA 4 0x021b001c 0x8033
+DATA 4 0x021b001c 0x00048031
+DATA 4 0x021b001c 0x05208030
+DATA 4 0x021b001c 0x04008040
+DATA 4 0x021b0020 0x5800
+DATA 4 0x021b0818 0x0007
+DATA 4 0x021b0004 0x00025565
+DATA 4 0x021b0404 0x00011006
+DATA 4 0x021b001c 0x
+
+/* set the default clock gate to save power */
+DATA 4 0x020c4068 0x00C03F3F
+DATA 4 0x020c406c 0x0030FC03
+DATA 4 0x020c4070 0x0FFFC000
+DATA 4 0x020c4074 0x3FF0
+DATA 4 0x020c4078 0xF300
+DATA 4 0x020c407c 0x0FC3
+DATA 4 0x020c4080 0x0FFF
+
+/* enable AXI cache for VDOA/VPU/IPU */
+DATA 4 0x020e0010 0xF0CF
+/* set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7 */
+DATA 4 0x020e0018 0x007F007F
+DATA 4 0x020e001c 0x007F007F
diff --git a/configs/mx6solosabreauto_defconfig 
b/configs/mx6solosabreauto_defconfig
new file mode 100644
index 000..17610c6
--- /dev/null
+++ b/configs/mx6solosabreauto_defconfig
@@ -0,0 +1,3 @@
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/mx6solo.cfg,MX6S,DEFAULT_FDT_FILE=\imx6dl-sabreauto.dtb\,DDR_MB=1024
+CONFIG_ARM=y
+CONFIG_TARGET_MX6QSABREAUTO=y
-- 
1.7.4.1

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


[U-Boot] [PATCH v2 1/4] imx: mx6sabresd/sabreauto: Move MX6Q/DL DDR and FDT configs to defconfig

2014-09-09 Thread Ye . Li
To support more iMX6 variants,
1. Make the DDR size configurable based on the defconfig file
2. Make the FDT file configurable based on the defconfig file

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Rework the short log subject

 board/freescale/mx6qsabreauto/mx6qsabreauto.c |2 +-
 board/freescale/mx6sabresd/mx6sabresd.c   |2 +-
 configs/mx6dlsabreauto_defconfig  |2 +-
 configs/mx6dlsabresd_defconfig|2 +-
 configs/mx6qsabreauto_defconfig   |2 +-
 configs/mx6qsabresd_defconfig |2 +-
 include/configs/mx6qsabreauto.h   |9 ++---
 include/configs/mx6sabresd.h  |8 +---
 8 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/board/freescale/mx6qsabreauto/mx6qsabreauto.c 
b/board/freescale/mx6qsabreauto/mx6qsabreauto.c
index 928dadf..bfb9b6a 100644
--- a/board/freescale/mx6qsabreauto/mx6qsabreauto.c
+++ b/board/freescale/mx6qsabreauto/mx6qsabreauto.c
@@ -45,7 +45,7 @@ DECLARE_GLOBAL_DATA_PTR;
 
 int dram_init(void)
 {
-   gd-ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE);
+   gd-ram_size = (phys_size_t)CONFIG_DDR_MB * 1024 * 1024;
 
return 0;
 }
diff --git a/board/freescale/mx6sabresd/mx6sabresd.c 
b/board/freescale/mx6sabresd/mx6sabresd.c
index 80c8ebd..5f65f1b 100644
--- a/board/freescale/mx6sabresd/mx6sabresd.c
+++ b/board/freescale/mx6sabresd/mx6sabresd.c
@@ -53,7 +53,7 @@ DECLARE_GLOBAL_DATA_PTR;
 
 int dram_init(void)
 {
-   gd-ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE);
+   gd-ram_size = (phys_size_t)CONFIG_DDR_MB * 1024 * 1024;
 
return 0;
 }
diff --git a/configs/mx6dlsabreauto_defconfig b/configs/mx6dlsabreauto_defconfig
index b649935..ce755d1 100644
--- a/configs/mx6dlsabreauto_defconfig
+++ b/configs/mx6dlsabreauto_defconfig
@@ -1,3 +1,3 @@
-CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/mx6dl.cfg,MX6DL
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/mx6dl.cfg,MX6DL,DEFAULT_FDT_FILE=\imx6dl-sabreauto.dtb\,DDR_MB=2048
 CONFIG_ARM=y
 CONFIG_TARGET_MX6QSABREAUTO=y
diff --git a/configs/mx6dlsabresd_defconfig b/configs/mx6dlsabresd_defconfig
index 9ce960e..b8e6d29 100644
--- a/configs/mx6dlsabresd_defconfig
+++ b/configs/mx6dlsabresd_defconfig
@@ -1,3 +1,3 @@
-CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DEFAULT_FDT_FILE=\imx6dl-sabresd.dtb\,DDR_MB=1024
 CONFIG_ARM=y
 CONFIG_TARGET_MX6SABRESD=y
diff --git a/configs/mx6qsabreauto_defconfig b/configs/mx6qsabreauto_defconfig
index 7d86700..25085a9 100644
--- a/configs/mx6qsabreauto_defconfig
+++ b/configs/mx6qsabreauto_defconfig
@@ -1,3 +1,3 @@
-CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/imximage.cfg,MX6Q
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/imximage.cfg,MX6Q,DEFAULT_FDT_FILE=\imx6q-sabreauto.dtb\,DDR_MB=2048
 CONFIG_ARM=y
 CONFIG_TARGET_MX6QSABREAUTO=y
diff --git a/configs/mx6qsabresd_defconfig b/configs/mx6qsabresd_defconfig
index dc8e254..edfb988 100644
--- a/configs/mx6qsabresd_defconfig
+++ b/configs/mx6qsabresd_defconfig
@@ -1,3 +1,3 @@
-CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg,MX6Q
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/imx/ddr/mx6q_4x_mt41j128.cfg,MX6Q,DEFAULT_FDT_FILE=\imx6q-sabresd.dtb\,DDR_MB=1024
 CONFIG_ARM=y
 CONFIG_TARGET_MX6SABRESD=y
diff --git a/include/configs/mx6qsabreauto.h b/include/configs/mx6qsabreauto.h
index d1639c4..e8580e6 100644
--- a/include/configs/mx6qsabreauto.h
+++ b/include/configs/mx6qsabreauto.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012 Freescale Semiconductor, Inc.
+ * Copyright (C) 2014 Freescale Semiconductor, Inc.
  *
  * Configuration settings for the Freescale i.MX6Q SabreAuto board.
  *
@@ -12,13 +12,8 @@
 #define CONFIG_MACH_TYPE   3529
 #define CONFIG_MXC_UART_BASE   UART4_BASE
 #define CONFIG_CONSOLE_DEV ttymxc3
-#if defined CONFIG_MX6Q
-#define CONFIG_DEFAULT_FDT_FILEimx6q-sabreauto.dtb
-#elif defined CONFIG_MX6DL
-#define CONFIG_DEFAULT_FDT_FILEimx6dl-sabreauto.dtb
-#endif
+
 #define CONFIG_MMCROOT /dev/mmcblk0p2
-#define PHYS_SDRAM_SIZE(2u * 1024 * 1024 * 1024)
 
 /* USB Configs */
 #define CONFIG_CMD_USB
diff --git a/include/configs/mx6sabresd.h b/include/configs/mx6sabresd.h
index e666ebb..c8ac5aa 100644
--- a/include/configs/mx6sabresd.h
+++ b/include/configs/mx6sabresd.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012 Freescale Semiconductor, Inc.
+ * Copyright (C) 2014 Freescale Semiconductor, Inc.
  *
  * Configuration settings for the Freescale i.MX6Q SabreSD board.
  *
@@ -16,12 +16,6 @@
 #define CONFIG_MXC_UART_BASE   UART1_BASE
 #define CONFIG_CONSOLE_DEV ttymxc0
 #define CONFIG_MMCROOT /dev/mmcblk1p2
-#if defined(CONFIG_MX6Q)

[U-Boot] [PATCH v2 3/4] imx: mx6solosabresd: Add the i.MX6Solo SABRESD board support

2014-09-09 Thread Ye . Li
The i.MX6solo SABRE-SD board configuration has the following
difference with i.MX6dl sabre-sd:

- DDR bus width: 32bit
- DDR capacity:  512M

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Remove the nosmp kernel parameter, since kernel can detect cores number by 
SCU.
- Rework the short log subject

 board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg |  106 
 configs/mx6solosabresd_defconfig   |3 +
 include/configs/mx6sabre_common.h  |4 +
 3 files changed, 113 insertions(+), 0 deletions(-)
 create mode 100644 board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg
 create mode 100644 configs/mx6solosabresd_defconfig

diff --git a/board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg 
b/board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg
new file mode 100644
index 000..c886d66
--- /dev/null
+++ b/board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg
@@ -0,0 +1,106 @@
+/*
+ * Copyright (C) 2014 Freescale Semiconductor, Inc.
+ * Jason Liu r64...@freescale.com
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ *
+ * Refer doc/README.imximage for more details about how-to configure
+ * and create imximage boot image
+ *
+ * The syntax is taken as close as possible with the kwbimage
+ */
+
+/* image version */
+IMAGE_VERSION 2
+
+/*
+ * Boot Device : one of
+ * spi, sd (the board has no nand neither onenand)
+ */
+BOOT_FROM  sd
+
+/*
+ * Device Configuration Data (DCD)
+ *
+ * Each entry must have the format:
+ * Addr-type   AddressValue
+ *
+ * where:
+ *  Addr-type register length (1,2 or 4 bytes)
+ *  Address   absolute address of the register
+ *  value value to be stored in the register
+ */
+DATA 4 0x020e0774 0x000C
+DATA 4 0x020e0754 0x
+DATA 4 0x020e04ac 0x0030
+DATA 4 0x020e04b0 0x0030
+DATA 4 0x020e0464 0x0030
+DATA 4 0x020e0490 0x0030
+DATA 4 0x020e074c 0x0030
+DATA 4 0x020e0494 0x0030
+DATA 4 0x020e04a0 0x
+DATA 4 0x020e04b4 0x0030
+DATA 4 0x020e04b8 0x0030
+DATA 4 0x020e076c 0x0030
+DATA 4 0x020e0750 0x0002
+DATA 4 0x020e04bc 0x0030
+DATA 4 0x020e04c0 0x0030
+DATA 4 0x020e04c4 0x0030
+DATA 4 0x020e04c8 0x0030
+DATA 4 0x020e0760 0x0002
+DATA 4 0x020e0764 0x0030
+DATA 4 0x020e0770 0x0030
+DATA 4 0x020e0778 0x0030
+DATA 4 0x020e077c 0x0030
+DATA 4 0x020e0470 0x0030
+DATA 4 0x020e0474 0x0030
+DATA 4 0x020e0478 0x0030
+DATA 4 0x020e047c 0x0030
+DATA 4 0x021b0800 0xa1390003
+DATA 4 0x021b080c 0x001F001F
+DATA 4 0x021b0810 0x001F001F
+DATA 4 0x021b083c 0x42190219
+DATA 4 0x021b0840 0x017B0177
+DATA 4 0x021b0848 0x4B4D4E4D
+DATA 4 0x021b0850 0x3F3E2D36
+DATA 4 0x021b081c 0x
+DATA 4 0x021b0820 0x
+DATA 4 0x021b0824 0x
+DATA 4 0x021b0828 0x
+DATA 4 0x021b08b8 0x0800
+DATA 4 0x021b0004 0x0002002D
+DATA 4 0x021b0008 0x00333030
+DATA 4 0x021b000c 0x3F435313
+DATA 4 0x021b0010 0xB66E8B63
+DATA 4 0x021b0014 0x01FF00DB
+DATA 4 0x021b0018 0x1740
+DATA 4 0x021b001c 0x8000
+DATA 4 0x021b002c 0x26d2
+DATA 4 0x021b0030 0x00431023
+DATA 4 0x021b0040 0x0017
+DATA 4 0x021b 0x8319
+DATA 4 0x021b001c 0x04008032
+DATA 4 0x021b001c 0x8033
+DATA 4 0x021b001c 0x00048031
+DATA 4 0x021b001c 0x05208030
+DATA 4 0x021b001c 0x04008040
+DATA 4 0x021b0020 0x5800
+DATA 4 0x021b0818 0x0007
+DATA 4 0x021b0004 0x0002556D
+DATA 4 0x021b0404 0x00011006
+DATA 4 0x021b001c 0x
+
+/* set the default clock gate to save power */
+DATA 4 0x020c4068 0x00C03F3F
+DATA 4 0x020c406c 0x0030FC03
+DATA 4 0x020c4070 0x0FFFC000
+DATA 4 0x020c4074 0x3FF0
+DATA 4 0x020c4078 0x00FFF300
+DATA 4 0x020c407c 0x0FC3
+DATA 4 0x020c4080 0x03FF
+
+/* enable AXI cache for VDOA/VPU/IPU */
+DATA 4 0x020e0010 0xF0CF
+/* set IPU AXI-id0 Qos=0xf(bypass) AXI-id1 Qos=0x7 */
+DATA 4 0x020e0018 0x007F007F
+DATA 4 0x020e001c 0x007F007F
diff --git a/configs/mx6solosabresd_defconfig b/configs/mx6solosabresd_defconfig
new file mode 100644
index 000..d41754d
--- /dev/null
+++ b/configs/mx6solosabresd_defconfig
@@ -0,0 +1,3 @@
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6sabresd/mx6solo_4x_mt41j128.cfg,MX6S,DEFAULT_FDT_FILE=\imx6dl-sabresd.dtb\,DDR_MB=512
+CONFIG_ARM=y
+CONFIG_TARGET_MX6SABRESD=y
diff --git a/include/configs/mx6sabre_common.h 
b/include/configs/mx6sabre_common.h
index e59a3b4..cf43404 100644
--- a/include/configs/mx6sabre_common.h
+++ b/include/configs/mx6sabre_common.h
@@ -11,6 +11,10 @@
 
 #define CONFIG_MX6
 
+#ifdef CONFIG_MX6S
+#define CONFIG_MX6DL
+#endif
+
 #include mx6_common.h
 #include linux/sizes.h
 
-- 
1.7.4.1

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


[U-Boot] [PATCH v2 2/4] imx: mx6qsabreauto: Rename the imximage.cfg to mx6q.cfg

2014-09-09 Thread Ye . Li
Rename the imximage.cfg to mx6q.cfg.
No function change at all

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Use format-patch -M to detect renames
- Rework the short log subject

 .../mx6qsabreauto/{imximage.cfg = mx6q.cfg}   |2 +-
 configs/mx6qsabreauto_defconfig|2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename board/freescale/mx6qsabreauto/{imximage.cfg = mx6q.cfg} (98%)

diff --git a/board/freescale/mx6qsabreauto/imximage.cfg 
b/board/freescale/mx6qsabreauto/mx6q.cfg
similarity index 98%
rename from board/freescale/mx6qsabreauto/imximage.cfg
rename to board/freescale/mx6qsabreauto/mx6q.cfg
index 16bf473..075d51a 100644
--- a/board/freescale/mx6qsabreauto/imximage.cfg
+++ b/board/freescale/mx6qsabreauto/mx6q.cfg
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012 Freescale Semiconductor, Inc.
+ * Copyright (C) 2014 Freescale Semiconductor, Inc.
  *
  * SPDX-License-Identifier:GPL-2.0+
  *
diff --git a/configs/mx6qsabreauto_defconfig b/configs/mx6qsabreauto_defconfig
index 25085a9..82b0b8e 100644
--- a/configs/mx6qsabreauto_defconfig
+++ b/configs/mx6qsabreauto_defconfig
@@ -1,3 +1,3 @@
-CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/imximage.cfg,MX6Q,DEFAULT_FDT_FILE=\imx6q-sabreauto.dtb\,DDR_MB=2048
+CONFIG_SYS_EXTRA_OPTIONS=IMX_CONFIG=board/freescale/mx6qsabreauto/mx6q.cfg,MX6Q,DEFAULT_FDT_FILE=\imx6q-sabreauto.dtb\,DDR_MB=2048
 CONFIG_ARM=y
 CONFIG_TARGET_MX6QSABREAUTO=y
-- 
1.7.4.1

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


[U-Boot] [PATCH v2 2/2] imx: mx6slevk: Change to use generic board

2014-09-09 Thread Ye . Li
Enable CONFIG_SYS_GENERIC_BOARD for imx6slevk to use generic board.

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Rework the short log subject

 include/configs/mx6slevk.h |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/include/configs/mx6slevk.h b/include/configs/mx6slevk.h
index 3d05a64..194d7bd 100644
--- a/include/configs/mx6slevk.h
+++ b/include/configs/mx6slevk.h
@@ -26,6 +26,8 @@
 #define CONFIG_INITRD_TAG
 #define CONFIG_REVISION_TAG
 
+#define CONFIG_SYS_GENERIC_BOARD
+
 /* Size of malloc() pool */
 #define CONFIG_SYS_MALLOC_LEN  (3 * SZ_1M)
 
-- 
1.7.4.1

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


[U-Boot] [PATCH v2 1/2] imx: mx6q/dlarm2: Change to use generic board

2014-09-09 Thread Ye . Li
Enable the CONFIG_SYS_GENERIC_BOARD for imx6q/dl arm2 board to
use generic board.

Signed-off-by: Ye.Li b37...@freescale.com
---
Changes since v1:
- Rework the short log subject

 include/configs/mx6qarm2.h |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/include/configs/mx6qarm2.h b/include/configs/mx6qarm2.h
index f314b31..3566b15 100644
--- a/include/configs/mx6qarm2.h
+++ b/include/configs/mx6qarm2.h
@@ -22,6 +22,8 @@
 #define CONFIG_SETUP_MEMORY_TAGS
 #define CONFIG_INITRD_TAG
 
+#define CONFIG_SYS_GENERIC_BOARD
+
 /* Size of malloc() pool */
 #define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + 2 * 1024 * 1024)
 
-- 
1.7.4.1

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


[U-Boot] [PATCH 2/2] fsl_sec : Change accessor function to take care of endianness

2014-09-09 Thread Ruchika Gupta
SEC registers can be of type Little Endian or big Endian depending upon
Freescale SoC. Here SoC defines the register type of SEC IP.

So update acessor functions with common SEC acessor functions to take care
both type of endianness.

Signed-off-by: Ruchika Gupta ruchika.gu...@freescale.com
CC: York Sun york...@freescale.com
---
 README|  6 ++
 arch/powerpc/cpu/mpc85xx/cpu_init.c   |  2 +-
 arch/powerpc/cpu/mpc85xx/fdt.c|  2 +-
 arch/powerpc/cpu/mpc85xx/liodn.c  |  4 ++--
 arch/powerpc/cpu/mpc8xxx/fdt.c|  4 ++--
 arch/powerpc/include/asm/config_mpc85xx.h |  1 +
 include/fsl_sec.h | 16 
 7 files changed, 29 insertions(+), 6 deletions(-)

diff --git a/README b/README
index 0a0f528..0befb9b 100644
--- a/README
+++ b/README
@@ -538,6 +538,12 @@ The following options need to be configured:
interleaving mode, handled by Dickens for Freescale layerscape
SoCs with ARM core.
 
+   CONFIG_SYS_FSL_SEC_BE
+   Defines the SEC controller register space as Big Endian
+
+   CONFIG_SYS_FSL_SEC_LE
+   Defines the SEC controller register space as Little Endian
+
 - Intel Monahans options:
CONFIG_SYS_MONAHANS_RUN_MODE_OSC_RATIO
 
diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init.c 
b/arch/powerpc/cpu/mpc85xx/cpu_init.c
index 5bfab70..bf9fbbf 100644
--- a/arch/powerpc/cpu/mpc85xx/cpu_init.c
+++ b/arch/powerpc/cpu/mpc85xx/cpu_init.c
@@ -803,7 +803,7 @@ int cpu_init_r(void)
 #ifdef CONFIG_SYS_FSL_ERRATUM_SEC_A003571
 #define MCFGR_AXIPIPE 0x00f0
if (IS_SVR_REV(svr, 1, 0))
-   clrbits_be32(sec-mcfgr, MCFGR_AXIPIPE);
+   sec_clrbits32(sec-mcfgr, MCFGR_AXIPIPE);
 #endif
 
 #ifdef CONFIG_SYS_FSL_ERRATUM_A005871
diff --git a/arch/powerpc/cpu/mpc85xx/fdt.c b/arch/powerpc/cpu/mpc85xx/fdt.c
index 3222e26..d4c3d9d 100644
--- a/arch/powerpc/cpu/mpc85xx/fdt.c
+++ b/arch/powerpc/cpu/mpc85xx/fdt.c
@@ -714,7 +714,7 @@ void ft_cpu_setup(void *blob, bd_t *bd)
ccsr_sec_t __iomem *sec;
 
sec = (void __iomem *)CONFIG_SYS_FSL_SEC_ADDR;
-   fdt_fixup_crypto_node(blob, in_be32(sec-secvid_ms));
+   fdt_fixup_crypto_node(blob, sec_in32(sec-secvid_ms));
}
 #endif
 
diff --git a/arch/powerpc/cpu/mpc85xx/liodn.c b/arch/powerpc/cpu/mpc85xx/liodn.c
index 19e130e..7a2d4be 100644
--- a/arch/powerpc/cpu/mpc85xx/liodn.c
+++ b/arch/powerpc/cpu/mpc85xx/liodn.c
@@ -66,12 +66,12 @@ static void setup_sec_liodn_base(void)
return;
 
/* QILCR[QSLOM] */
-   out_be32(sec-qilcr_ms, 0x3ff16);
+   sec_out32(sec-qilcr_ms, 0x3ff16);
 
base = (liodn_bases[FSL_HW_PORTAL_SEC].id[0]  16) |
liodn_bases[FSL_HW_PORTAL_SEC].id[1];
 
-   out_be32(sec-qilcr_ls, base);
+   sec_out32(sec-qilcr_ls, base);
 }
 
 #ifdef CONFIG_SYS_DPAA_FMAN
diff --git a/arch/powerpc/cpu/mpc8xxx/fdt.c b/arch/powerpc/cpu/mpc8xxx/fdt.c
index 9273745..946ab70 100644
--- a/arch/powerpc/cpu/mpc8xxx/fdt.c
+++ b/arch/powerpc/cpu/mpc8xxx/fdt.c
@@ -289,8 +289,8 @@ static u8 caam_get_era(void)
};
 
ccsr_sec_t __iomem *sec = (void __iomem *)CONFIG_SYS_FSL_SEC_ADDR;
-   u32 secvid_ms = in_be32(sec-secvid_ms);
-   u32 ccbvid = in_be32(sec-ccbvid);
+   u32 secvid_ms = sec_in32(sec-secvid_ms);
+   u32 ccbvid = sec_in32(sec-ccbvid);
u16 ip_id = (secvid_ms  SEC_SECVID_MS_IPID_MASK) 
SEC_SECVID_MS_IPID_SHIFT;
u8 maj_rev = (secvid_ms  SEC_SECVID_MS_MAJ_REV_MASK) 
diff --git a/arch/powerpc/include/asm/config_mpc85xx.h 
b/arch/powerpc/include/asm/config_mpc85xx.h
index 712f2ef..4c1774f 100644
--- a/arch/powerpc/include/asm/config_mpc85xx.h
+++ b/arch/powerpc/include/asm/config_mpc85xx.h
@@ -24,6 +24,7 @@
 
 /* IP endianness */
 #define CONFIG_SYS_FSL_IFC_BE
+#define CONFIG_SYS_FSL_SEC_BE
 
 /* Number of TLB CAM entries we have on FSL Book-E chips */
 #if defined(CONFIG_E500MC)
diff --git a/include/fsl_sec.h b/include/fsl_sec.h
index b31999f..a11f58d 100644
--- a/include/fsl_sec.h
+++ b/include/fsl_sec.h
@@ -11,6 +11,22 @@
 #include common.h
 #include asm/io.h
 
+#ifdef CONFIG_SYS_FSL_SEC_LE
+#define sec_in32(a)   in_le32(a)
+#define sec_out32(a, v)   out_le32(a, v)
+#define sec_in16(a)   in_le16(a)
+#define sec_clrbits32 clrbits_le32
+#define sec_setbits32 setbits_le32
+#elif defined(CONFIG_SYS_FSL_SEC_BE)
+#define sec_in32(a)   in_be32(a)
+#define sec_out32(a, v)   out_be32(a, v)
+#define sec_in16(a)   in_be16(a)
+#define sec_clrbits32 clrbits_be32
+#define sec_setbits32 setbits_be32
+#else
+#error Neither CONFIG_SYS_FSL_SEC_LE nor CONFIG_SYS_FSL_SEC_BE is defined
+#endif
+
 /* Security Engine Block (MS = Most Sig., LS = Least Sig.) */
 #if CONFIG_SYS_FSL_SEC_COMPAT = 4
 typedef struct ccsr_sec {
-- 
1.8.1.4


[U-Boot] [PATCH 1/2] fsl_sec : Move SEC CCSR definition to common include

2014-09-09 Thread Ruchika Gupta
Freescale SEC controller has been used for mpc8xxx. It will be used
for ARM-based SoC as well. This patch moves the CCSR defintion of
SEC to common include

Signed-off-by: Ruchika Gupta ruchika.gu...@freescale.com
CC: York Sun york...@freescale.com
---
 arch/powerpc/include/asm/immap_85xx.h | 67 +-
 include/fsl_sec.h | 88 +++
 2 files changed, 89 insertions(+), 66 deletions(-)
 create mode 100644 include/fsl_sec.h

diff --git a/arch/powerpc/include/asm/immap_85xx.h 
b/arch/powerpc/include/asm/immap_85xx.h
index dfb370e..e426314 100644
--- a/arch/powerpc/include/asm/immap_85xx.h
+++ b/arch/powerpc/include/asm/immap_85xx.h
@@ -16,6 +16,7 @@
 #include asm/fsl_dma.h
 #include asm/fsl_i2c.h
 #include fsl_ifc.h
+#include fsl_sec.h
 #include asm/fsl_lbc.h
 #include asm/fsl_fman.h
 #include fsl_immap.h
@@ -2675,72 +2676,6 @@ enum {
FSL_SRDS_B3_LANE_D = 23,
 };
 
-/* Security Engine Block (MS = Most Sig., LS = Least Sig.) */
-#if CONFIG_SYS_FSL_SEC_COMPAT = 4
-typedef struct ccsr_sec {
-   u32 res0;
-   u32 mcfgr;  /* Master CFG Register */
-   u8  res1[0x8];
-   struct {
-   u32 ms; /* Job Ring LIODN Register, MS */
-   u32 ls; /* Job Ring LIODN Register, LS */
-   } jrliodnr[4];
-   u8  res2[0x30];
-   struct {
-   u32 ms; /* RTIC LIODN Register, MS */
-   u32 ls; /* RTIC LIODN Register, LS */
-   } rticliodnr[4];
-   u8  res3[0x1c];
-   u32 decorr; /* DECO Request Register */
-   struct {
-   u32 ms; /* DECO LIODN Register, MS */
-   u32 ls; /* DECO LIODN Register, LS */
-   } decoliodnr[8];
-   u8  res4[0x40];
-   u32 dar;/* DECO Avail Register */
-   u32 drr;/* DECO Reset Register */
-   u8  res5[0xe78];
-   u32 crnr_ms;/* CHA Revision Number Register, MS */
-   u32 crnr_ls;/* CHA Revision Number Register, LS */
-   u32 ctpr_ms;/* Compile Time Parameters Register, MS */
-   u32 ctpr_ls;/* Compile Time Parameters Register, LS */
-   u8  res6[0x10];
-   u32 far_ms; /* Fault Address Register, MS */
-   u32 far_ls; /* Fault Address Register, LS */
-   u32 falr;   /* Fault Address LIODN Register */
-   u32 fadr;   /* Fault Address Detail Register */
-   u8  res7[0x4];
-   u32 csta;   /* CAAM Status Register */
-   u8  res8[0x8];
-   u32 rvid;   /* Run Time Integrity Checking Version ID Reg.*/
-   u32 ccbvid; /* CHA Cluster Block Version ID Register */
-   u32 chavid_ms;  /* CHA Version ID Register, MS */
-   u32 chavid_ls;  /* CHA Version ID Register, LS */
-   u32 chanum_ms;  /* CHA Number Register, MS */
-   u32 chanum_ls;  /* CHA Number Register, LS */
-   u32 secvid_ms;  /* SEC Version ID Register, MS */
-   u32 secvid_ls;  /* SEC Version ID Register, LS */
-   u8  res9[0x6020];
-   u32 qilcr_ms;   /* Queue Interface LIODN CFG Register, MS */
-   u32 qilcr_ls;   /* Queue Interface LIODN CFG Register, LS */
-   u8  res10[0x8fd8];
-} ccsr_sec_t;
-
-#define SEC_CTPR_MS_AXI_LIODN  0x0800
-#define SEC_CTPR_MS_QI 0x0200
-#define SEC_RVID_MA0x0f00
-#define SEC_CHANUM_MS_JRNUM_MASK   0xf000
-#define SEC_CHANUM_MS_JRNUM_SHIFT  28
-#define SEC_CHANUM_MS_DECONUM_MASK 0x0f00
-#define SEC_CHANUM_MS_DECONUM_SHIFT24
-#define SEC_SECVID_MS_IPID_MASK0x
-#define SEC_SECVID_MS_IPID_SHIFT   16
-#define SEC_SECVID_MS_MAJ_REV_MASK 0xff00
-#define SEC_SECVID_MS_MAJ_REV_SHIFT8
-#define SEC_CCBVID_ERA_MASK0xff00
-#define SEC_CCBVID_ERA_SHIFT   24
-#endif
-
 typedef struct ccsr_qman {
 #ifdef CONFIG_SYS_FSL_QMAN_V3
u8  res0[0x200];
diff --git a/include/fsl_sec.h b/include/fsl_sec.h
new file mode 100644
index 000..b31999f
--- /dev/null
+++ b/include/fsl_sec.h
@@ -0,0 +1,88 @@
+/*
+ * Common internal memory map for some Freescale SoCs
+ *
+ * Copyright 2014 Freescale Semiconductor, Inc.
+ *
+ */
+
+#ifndef __FSL_SEC_H
+#define __FSL_SEC_H
+
+#include common.h
+#include asm/io.h
+
+/* Security Engine Block (MS = Most Sig., LS = Least Sig.) */
+#if CONFIG_SYS_FSL_SEC_COMPAT = 4
+typedef struct ccsr_sec {
+   u32 res0;
+   u32 mcfgr;  /* Master CFG Register */
+   u8  res1[0x4];
+   u32 scfgr;
+   struct {
+   u32 ms; /* Job Ring LIODN Register, MS */
+   u32 ls; /* Job Ring LIODN Register, LS */
+   } jrliodnr[4];
+   u8  res2[0x2c];
+   u32 

Re: [U-Boot] [PATCH 0/5] ARM: sheevaplug: refresh for 201410

2014-09-09 Thread Prafulla Wadaskar
Hi Everybody,

Sorry for being inactive on the thread...
I will pull these patches and provide a pull request to albert soon...

Regards...
Prafulla . . .

 -Original Message-
 From: Nobuhiro Iwamatsu [mailto:iwama...@nigauri.org]
 Sent: 09 September 2014 12:08
 To: Albert ARIBAUD
 Cc: U-Boot; Prafulla Wadaskar; Pantelis Antoniou;
 Gerald Kerma; Tom Rini
 Subject: Re: [PATCH 0/5] ARM: sheevaplug: refresh for
 201410
 
 Hi, Albert.
 
 These patches are not taken for a long time.
 If possible and Prafulla is OK, can you apply to u-
 boot-arm directly?
 
 Best regards,
   Nobuhiro
 
 2014-09-08 1:01 GMT+09:00 Gerald Kerma
 drea...@doukki.net:
  This serie of patches add some needed features to
 sheevaplug
 
  Gerald Kerma (5):
ARM: sheevaplug: change env location
ARM: sheevaplug: add MVSATA driver
ARM: sheevaplug: add MTD defaults
ARM: sheevaplug: redefine MTDPARTS
ARM: sheevaplug: add HUSH parser
 
   include/configs/sheevaplug.h | 46
 +++-
   1 file changed, 41 insertions(+), 5 deletions(-)
 
  --
  1.9.1
 
 
 
 
 --
 Nobuhiro Iwamatsu
iwamatsu at {nigauri.org / debian.org}
GPG ID: 40AD1FA6
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/7] ARM: sunxi: Add basic support for Allwinner A31 (sun6i)

2014-09-09 Thread Hans de Goede
Hi,

On 09/08/2014 03:28 PM, Chen-Yu Tsai wrote:
 Hi everyone,
 
 This series add basic support for Allwinner's A31 SoC. The patches,
 excluding the first one, were cherry-picked from u-boot-sunxi. Due to
 the difference between u-boot mainline and u-boot-sunxi, some patches
 were rearranged or squashed to better fit the current state of u-boot,
 and not introduce any build breaks. It follows Ian's initial merge
 method of sun7i support: introducing various components first, then
 enabling them in the last commit. I tried to keep the commits separate,
 thus retaining the original author and Signed-off-bys.
 
 Patch 1 adds a wrapper around func(USB, usb, 0) in BOOT_TARGET_DEVICES
 to deal with breakage when USB support is not enabled.
 
 Patch 2 adds memory addresses for some hardware blocks new in sun6i.
 
 Patch 3 adds support for the new PRCM (power reset and clock management)
 block, which also contains PLL bias voltage control.
 
 Patch 4 adds support for the clock module. This patch is a bunch of
 different sun6i related patches on the clock code, from when sun6i
 support was introduced to u-boot-sunxi, up to its current form.
 This is done to avoid various conflicts and needlessly introducing
 then removing macros.
 
 Patch 5 adds mmc support on sun6i.
 
 Patch 6 adds uart0 muxing on sun6i.
 
 Patch 7 enables sun6i support and adds defconfig for the Colombus board.

Chen,

Many thanks for working on this!

Just a quick not for people celebrating too early, this is the *incomplete*
sun7i support from the linux-sunxi/u-boot-sunxi git repo. It is fine to
merge this upstream, but this does not include SPL support.

This allows replacing u-boot.bin on allwinnner sd-card images, which is
very useful. But it does not get us all the way to booting sun7i devices
we still need boot0 and boot1 binaries from allwinner for that (for now).

Regards,

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


[U-Boot] Spansion SPI flash read timeout with AM335x

2014-09-09 Thread Andy Pont
Hello,

I am doing some work with the TI AM3358 Starter Kit and a Spansion S25FL164K
SPI NOR flash.

Trying both U-Boot 2013.10, which ships with the TI kit, and 2014.04 I have
made the changes to board configuration file to enable SPI0 and disable I2C1
in the board pin mux, have included Spansion flash support and have added
the following entry into the Spansion devices listed in sf_params.c

{S25FL164K,  0x014017, 0x0140, 4 * 1024,  2048, RD_FULL,
SECT_4K},

Using the am335x_evm_spiboot configuration I have MLO and u-boot.img files
which I can program to the SPI flash successfully and also boot the board
from.  I have a problem with read timeouts in the omap3_spi_read() function
in drivers/spi/omap3_spi.c trying to read the Linux kernel.

Having run some more tests (results below) it appears that with an sf read
command requesting more than about 0x16E000 bytes will fail.  On occasion,
typically the first failed attempt, the status returns 0x0002 rather
than 0x0007.

U-Boot# sf read ${loadaddr} 0xE 0x10
SF: 1048576 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x14
SF: 1310720 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x15
SF: 1376256 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x16
SF: 1441792 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x17
omap3_spi_read SPI RXS timed out, status=0x0007
SF: 1507328 bytes @ 0xe Read: ERROR
U-Boot# sf read ${loadaddr} 0xE 0x168000
SF: 1474560 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x16C000
SF: 1490944 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x16E000
SF: 1499136 bytes @ 0xe Read: OK
U-Boot# sf read ${loadaddr} 0xE 0x16F000
omap3_spi_read SPI RXS timed out, status=0x0007
SF: 1503232 bytes @ 0xe Read: ERROR

Can anyone give me some guidance on how to resolve this?

Thanks,

Andy.

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


Re: [U-Boot] Please pull u-boot-sunxi.git/master

2014-09-09 Thread Albert ARIBAUD
Hi Ian,

On Fri, 05 Sep 2014 14:23:01 +0100, Ian Campbell i...@hellion.org.uk
wrote:

 (resend since I managed to forget the list in the CC, sorry about that)
 
 The following changes since commit d6c1ffc7d23f4fe4ae8c91101861055b8e1501b6:
 
   Prepare v2014.10-rc2 (2014-09-02 16:58:29 -0400)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-sunxi.git 
 
 for you to fetch changes up to b86d54b2c38a4beb97d580d7d9c8c6a5e57fc510:
 
   sun7i: Add support for Olimex A20-OLinuXino-LIME (2014-09-05 08:30:38 +0100)
 
 
 FUKAUMI Naoki (1):
   sun7i: Add support for Olimex A20-OLinuXino-LIME
 
 Ian Campbell (1):
   sunxi: Correct typo CONFIG_FTDFILE = CONFIG_FDTFILE
 
 Masahiro Yamada (1):
   kconfig: remove redundant SPL from CONFIG_SYS_EXTRA_OPTIONS
 
  board/sunxi/Kconfig   |  4 ++--
  board/sunxi/MAINTAINERS   |  6 ++
  board/sunxi/Makefile  |  1 +
  board/sunxi/dram_a20_olinuxino_l.c| 31 +++
  configs/A10-OLinuXino-Lime_defconfig  |  4 ++--
  configs/A10s-OLinuXino-M_defconfig|  4 ++--
  configs/A13-OLinuXinoM_defconfig  |  4 ++--
  configs/A13-OLinuXino_defconfig   |  4 ++--
  configs/A20-OLinuXino-Lime_defconfig  |  5 +
  configs/A20-OLinuXino_MICRO_defconfig |  4 ++--
  configs/Auxtek-T004_defconfig |  4 ++--
  configs/Bananapi_defconfig|  4 ++--
  configs/Cubieboard2_FEL_defconfig |  2 +-
  configs/Cubieboard2_defconfig |  4 ++--
  configs/Cubieboard_defconfig  |  4 ++--
  configs/Cubietruck_FEL_defconfig  |  2 +-
  configs/Cubietruck_defconfig  |  4 ++--
  configs/Linksprite_pcDuino3_defconfig |  4 ++--
  configs/Mele_A1000G_defconfig |  4 ++--
  configs/Mele_A1000_defconfig  |  4 ++--
  configs/Mini-X-1Gb_defconfig  |  4 ++--
  configs/Mini-X_defconfig  |  4 ++--
  configs/ba10_tv_box_defconfig |  4 ++--
  configs/gwventana_defconfig   |  2 +-
  configs/i12-tvbox_defconfig   |  4 ++--
  configs/qt840a_defconfig  |  4 ++--
  configs/r7-tv-dongle_defconfig|  4 ++--
  include/configs/sunxi-common.h|  2 +-
  28 files changed, 87 insertions(+), 44 deletions(-)
  create mode 100644 board/sunxi/dram_a20_olinuxino_l.c
  create mode 100644 configs/A20-OLinuXino-Lime_defconfig
 
 

Applied to u-boot-arm/master, thanks!

(still having 5 errors and 2 warnings, so there is no regression)

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH RESEND] vf610twr: Tune DDR initialization settings

2014-09-09 Thread Albert ARIBAUD
Hi Stefan,

On Sat,  6 Sep 2014 19:47:06 +0200, Stefan Agner ste...@agner.ch
wrote:

 From: Anthony Felice tony.fel...@timesys.com
 
 Removed settings in unsupported register fields. They didn’t
 do anything, and in most cases, were not documented in the
 reference manual.
 
 Changed register settings to comply with JEDEC required values.
 
 Changed timing parameters because they included full clock
 periods that were doing nothing.
 
 Signed-off-by: Anthony Felice tony.fel...@timesys.com
 [rebased on v2014.10-rc2]
 Signed-off-by: Stefan Agner ste...@agner.ch
 ---
 As discuessed in the initial patch set this fixes a lot of wrong/
 undocummented access and it would be nice to have in the next U-Boot
 release. Verified the patchset after rebase again using memtester
 on Vybrid Tower.
 
  arch/arm/include/asm/arch-vf610/imx-regs.h| 49 +++---
  arch/arm/include/asm/arch-vf610/iomux-vf610.h | 44 +++--
  arch/arm/include/asm/imx-common/iomux-v3.h|  2 +
  board/freescale/vf610twr/vf610twr.c   | 94 
 +--
  4 files changed, 127 insertions(+), 62 deletions(-)
 
 diff --git a/arch/arm/include/asm/arch-vf610/imx-regs.h 
 b/arch/arm/include/asm/arch-vf610/imx-regs.h
 index bb00217..9d797db 100644
 --- a/arch/arm/include/asm/arch-vf610/imx-regs.h
 +++ b/arch/arm/include/asm/arch-vf610/imx-regs.h
 @@ -103,9 +103,11 @@
  /* DDRMC */
  #define DDRMC_PHY_DQ_TIMING  0x2613
  #define DDRMC_PHY_DQS_TIMING 0x2615
 -#define DDRMC_PHY_CTRL   0x01210080
 +#define DDRMC_PHY_CTRL   0x0021
  #define DDRMC_PHY_MASTER_CTRL0x0001012a
 -#define DDRMC_PHY_SLAVE_CTRL 0x00012020
 +#define DDRMC_PHY_SLAVE_CTRL 0x2000
 +#define DDRMC_PHY_OFF0x
 +#define DDRMC_PHY_PROC_PAD_ODT   0x00010101
  
  #define DDRMC_PHY50_DDR3_MODE(1  12)
  #define DDRMC_PHY50_EN_SW_HALF_CYCLE (1  8)
 @@ -138,7 +140,7 @@
  #define DDRMC_CR21_CCMAP_EN  1
  #define DDRMC_CR22_TDAL(v)   (((v)  0x3f)  16)
  #define DDRMC_CR23_BSTLEN(v) (((v)  0x7)  24)
 -#define DDRMC_CR23_TDLL(v)   ((v)  0xff)
 +#define DDRMC_CR23_TDLL(v)   ((v)  0x)
  #define DDRMC_CR24_TRP_AB(v) ((v)  0x1f)
  #define DDRMC_CR25_TREF_EN   (1  16)
  #define DDRMC_CR26_TREF(v)   (((v)  0x)  16)
 @@ -151,7 +153,7 @@
  #define DDRMC_CR33_EN_QK_SREF(1  16)
  #define DDRMC_CR34_CKSRX(v)  (((v)  0xf)  16)
  #define DDRMC_CR34_CKSRE(v)  (((v)  0xf)  8)
 -#define DDRMC_CR38_FREQ_CHG_EN   (1  8)
 +#define DDRMC_CR38_FREQ_CHG_EN(v)(((v)  0x1)  8)
  #define DDRMC_CR39_PHY_INI_COM(v)(((v)  0x)  16)
  #define DDRMC_CR39_PHY_INI_STA(v)(((v)  0xff)  8)
  #define DDRMC_CR39_FRQ_CH_DLLOFF(v)  ((v)  0x3)
 @@ -163,7 +165,7 @@
  #define DDRMC_CR67_ZQCS(v)   ((v)  0xfff)
  #define DDRMC_CR69_ZQ_ON_SREF_EX(v)  (((v)  0xf)  8)
  #define DDRMC_CR70_REF_PER_ZQ(v) (v)
 -#define DDRMC_CR72_ZQCS_ROTATE   (1  24)
 +#define DDRMC_CR72_ZQCS_ROTATE(v)(((v)  0x1)  24)
  #define DDRMC_CR73_APREBIT(v)(((v)  0xf)  
 24)
  #define DDRMC_CR73_COL_DIFF(v)   (((v)  0x7)  
 16)
  #define DDRMC_CR73_ROW_DIFF(v)   (((v)  0x3)  
 8)
 @@ -182,9 +184,10 @@
  #define DDRMC_CR77_CS_MAP(1  24)
  #define DDRMC_CR77_DI_RD_INTLEAVE(1  8)
  #define DDRMC_CR77_SWAP_EN   1
 +#define DDRMC_CR78_Q_FULLNESS(v) (((v)  0x7)  24)
  #define DDRMC_CR78_BUR_ON_FLY_BIT(v) ((v)  0xf)
 -#define DDRMC_CR79_CTLUPD_AREF   (1  24)
 -#define DDRMC_CR82_INT_MASK  0x1fff
 +#define DDRMC_CR79_CTLUPD_AREF(v)(((v)  0x1)  24)
 +#define DDRMC_CR82_INT_MASK  0x1000
  #define DDRMC_CR87_ODT_WR_MAPCS0 (1  24)
  #define DDRMC_CR87_ODT_RD_MAPCS0 (1  16)
  #define DDRMC_CR88_TODTL_CMD(v)  (((v)  0x1f) 
  16)
 @@ -192,9 +195,17 @@
  #define DDRMC_CR91_R2W_SMCSDL(v) (((v)  0x7)  16)
  #define DDRMC_CR96_WLMRD(v)  (((v)  0x3f)  8)
  #define DDRMC_CR96_WLDQSEN(v)((v)  0x3f)
 

Re: [U-Boot] Please pull u-boot-sunxi.git/master

2014-09-09 Thread Ian Campbell
On Tue, 2014-09-09 at 09:58 +0200, Albert ARIBAUD wrote:
 Applied to u-boot-arm/master, thanks!

Thank you.

 (still having 5 errors and 2 warnings, so there is no regression)

MAKEALL -s sunxi is only showing a single generic error for me (on every
target):

In file included from scripts/kconfig/zconf.tab.c:2537:0:
scripts/kconfig/menu.c: In function ‘get_symbol_str’:
scripts/kconfig/menu.c:590:18: warning: ‘jump’ may be used 
uninitialized in this function [-Wmaybe-uninitialized]
 jump-offset = strlen(r-s);
  ^
In file included from scripts/kconfig/zconf.tab.c:2537:0:
scripts/kconfig/menu.c:551:19: note: ‘jump’ was declared here
  struct jump_key *jump;
   ^

What are you seeing?

Ian.

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


Re: [U-Boot] [PATCH] dfu, usb: set usb descriptor to values defined in dfu spec 1.1

2014-09-09 Thread Heiko Schocher

Hello Marek,

Am 05.09.2014 00:42, schrieb Marek Vasut:

On Thursday, September 04, 2014 at 12:21:40 PM, Heiko Schocher wrote:

Hi Heiko,

I'll just rant a bit, but please wait until Lukasz does a proper runthrough.


set bDeviceClass, bDeviceSubClass and bcdUSB to the values
defined in dfu spec 1.1 chapter 4.2.1 found here:

http://www.usb.org/developers/devclass_docs/DFU_1.1.pdf


This URL gives me a website with:

Resource not found.
Please check the URL and try again.



Ah! Here the correct link:

http://www.usb.org/developers/docs/devclass_docs/DFU_1.1.pdf

I change this for v2.

bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] generic board for socfpga

2014-09-09 Thread Albert ARIBAUD
Hi Pavel,

On Tue, 27 May 2014 14:50:30 +0200, Pavel Machek pa...@denx.de wrote:

 
 Socfpga u-boot works fine with CONFIG_SYS_GENERIC_BOARD, so enable
 that option as documentation suggests.
 
 Signed-off-by: Pavel Machek pa...@denx.de
 
 diff --git a/include/configs/socfpga_cyclone5.h 
 b/include/configs/socfpga_cyclone5.h
 index 2fc16ef..72d77f4 100644
 --- a/include/configs/socfpga_cyclone5.h
 +++ b/include/configs/socfpga_cyclone5.h
 @@ -16,6 +16,8 @@
  /* Virtual target or real hardware */
  #define CONFIG_SOCFPGA_VIRTUAL_TARGET
  
 +#define CONFIG_SYS_GENERIC_BOARD
 +
  #define CONFIG_ARMV7
  #define CONFIG_SYS_DCACHE_OFF
  #undef CONFIG_USE_IRQ

Sorry, I let this last too long before I reviewed it and now it does
not apply cleanly at all. Can you have a look?

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] relocation problem on powerpc

2014-09-09 Thread Joakim Tjernlund
Chris Packham judge.pack...@gmail.com wrote on 2014/09/09 01:07:51:

 From: Chris Packham judge.pack...@gmail.com
 To: Joakim Tjernlund joakim.tjernl...@transmode.se, 
 Cc: u-boot u-boot@lists.denx.de
 Date: 2014/09/09 01:08
 Subject: Re: [U-Boot] relocation problem on powerpc
 
 Hi Jocke,
 
 On Tue, Sep 9, 2014 at 12:22 AM, Joakim Tjernlund
 joakim.tjernl...@transmode.se wrote:
 
  
   There are some disadvantages to living in a weird timezone, mostly
   that you end up having conversations with yourself.
  
   On Mon, Sep 8, 2014 at 4:32 PM, Chris Packham
  judge.pack...@gmail.com
  wrote:
Hi All,
   
I have come across what I think is a relocation problem for 
powerpc.
   
I've added the following to ArpTimeoutCheck
   
+   printf(NetArpWaitTimerStart = %ld\n,
  NetArpWaitTimerStart);
+   printf(NetArpWaitTimerStart = %p\n,
  NetArpWaitTimerStart);
+   printf(NetArpWaitTry = %d\n, NetArpWaitTry);
+   printf(NetArpWaitTry = %p\n, NetArpWaitTry);
+   printf(NetArpWaitTxPacketSize = %d\n,
  NetArpWaitTxPacketSize);
+   printf(NetArpWaitTxPacketSize = %p\n,
  NetArpWaitTxPacketSize);
   
Which yields the following output
   
  NetArpWaitTimerStart = 0
  NetArpWaitTimerStart = f0d0
  NetArpWaitTry = 1
  NetArpWaitTry = 7ffb0058
  NetArpWaitTxPacketSize = 42
  NetArpWaitTxPacketSize = 7ffb0078
   
That looks to me like NetArpWaitTimerStart hasn't been relocated 
for
some reason. Looking at my u-boot.map NetArpWaitTimerStart is the
  last
item in the .sbss section
   
Here's the relevant snippets for the variables I'm displaying
   
 0xf058NetArpWaitTry
 0xf078NetArpWaitTxPacketSize
 0xf0d0NetArpWaitTimerStart
   
The actual problem for me is that ARPs timeout and various 
network
things fail. Has anyone got any clues as to why this one 
particular
variable isn't getting relocated.
   
Thanks,
Chris
   
More info:
  I'm building P2041RDB_config
   
  $ git describe
  v2014.10-rc2-15-g2ec8915
   
  = version
  U-Boot 2014.10-rc2-00015-g2ec8915-dirty (Sep 08 2014 - 
16:18:42)
  powerpc-e500-linux-gnu-gcc (Gentoo 4.6.3-r1 p1.9, pie-0.5.2) 
4.6.3
  GNU ld (GNU Binutils) 2.21
  
   I'm not confident enough to say it's a fix but the following seems 
to
   solve the relocation problem for NetArpWaitTimerStart.
  
   diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds
   b/arch/powerpc/cpu/mpc85xx/u-boot.lds
   index 2cf0b25..36711b0 100644
   --- a/arch/powerpc/cpu/mpc85xx/u-boot.lds
   +++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds
   @@ -54,7 +54,7 @@ SECTIONS
_FIXUP_TABLE_ = .;
KEEP(*(.fixup))
  }
   -  __got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  2) - 
1;
   +  __got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  2);
  __fixup_entries = (. - _FIXUP_TABLE_)  2;
 
  This looks like the correct fix, The relevant code that uses
  __got2_entries is in start.S:
 
  in_ram:
 
  /*
   * Relocation Function, r12 point to got2+0x8000
   *
   * Adjust got2 pointers, no need to check for 0, this code
   * already puts a few entries in the table.
   */
  li  r0,__got2_entries@sectoff@l
  la  r3,GOT(_GOT2_TABLE_)
  lwz r11,GOT(_GOT2_TABLE_)
  mtctr   r0
  sub r11,r3,r11
  addir3,r3,-4
  1:  lwzur0,4(r3)
  cmpwi   r0,0
  beq-2f
  add r0,r0,r11
  stw r0,0(r3)
  2:  bdnz1b
 
  bdnz does decrement then test for zero so __got2_entries should hold 
the
 
  number of entries to relocate.
 
  Seems like most(all?) ppc cpu has this error.
 
  Looking at u-boot.lds:
   .reloc   :
{
  _GOT2_TABLE_ = .;
  KEEP(*(.got2))
  KEEP(*(.got))
  PROVIDE(_GLOBAL_OFFSET_TABLE_ = . + 4);
  _FIXUP_TABLE_ = .;
  KEEP(*(.fixup))
}
__got2_entries = ((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  2) - 1;
__fixup_entries = (. - _FIXUP_TABLE_)  2;
 
 
  There is something more to this error. Since _GLOBAL_OFFSET_TABLE_  is
  defined to
  _GLOBAL_OFFSET_TABLE_ = . + 4
  there is one extra entry added( for brlr insn inserted by the linker) 
so
  something is
  amiss here, is there a brlr insns at _GLOBAL_OFFSET_TABLE_-4 in your
  u-boot?
 
   Jocke
 
 
 Here's some more info from a build without any of my modifications.
 
   $ git describe
   v2014.10-rc2-12-g0b703db
 
   $ powerpc-e500-linux-gnu-gcc --version
   powerpc-e500-linux-gnu-gcc (Gentoo 4.6.3-r1 p1.9, pie-0.5.2) 4.6.3
 
   $ grep '_GOT2_TABLE_\|_GLOBAL_OFFSET_TABLE_\|__got2_entries' 
u-boot.map
   0xeffb5100_GOT2_TABLE_ = .
   0xeffb7d38_GLOBAL_OFFSET_TABLE_
   0xeffb7d48PROVIDE (_GLOBAL_OFFSET_TABLE_, (. + 
0x4))
   0x0b0d  

Re: [U-Boot] [PATCH 7/8] samsung: Move s5p_goni to use exynos-common config

2014-09-09 Thread Przemyslaw Marczak

Hello Simon,

On 07/23/2014 02:11 PM, Simon Glass wrote:

Change this board to use the exynos common config and add a device tree.

This also adds a pinmux header file - but it is just a copy of the exynos
one so may be incorrect.

Signed-off-by: Simon Glass s...@chromium.org
---

  arch/arm/dts/Makefile  |  1 +
  arch/arm/dts/s5pc1xx-goni.dts  | 21 ++
  arch/arm/include/asm/arch-s5pc1xx/periph.h | 61 ++
  arch/arm/include/asm/arch-s5pc1xx/pinmux.h | 50 
  drivers/mmc/s5p_sdhci.c|  2 -
  include/configs/s5p_goni.h | 55 +--
  6 files changed, 143 insertions(+), 47 deletions(-)
  create mode 100644 arch/arm/dts/s5pc1xx-goni.dts
  create mode 100644 arch/arm/include/asm/arch-s5pc1xx/periph.h
  create mode 100644 arch/arm/include/asm/arch-s5pc1xx/pinmux.h

diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 3b760ff..6a6e664 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -1,3 +1,4 @@
+dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
  dtb-$(CONFIG_EXYNOS4) += exynos4210-origen.dtb \
exynos4210-smdkv310.dtb \
exynos4210-universal_c210.dtb \
diff --git a/arch/arm/dts/s5pc1xx-goni.dts b/arch/arm/dts/s5pc1xx-goni.dts
new file mode 100644
index 000..c390c8f
--- /dev/null
+++ b/arch/arm/dts/s5pc1xx-goni.dts
@@ -0,0 +1,21 @@
+/*
+ * Samsung's Exynos4210-based SMDKV310 board device tree source
+ *
+ * Copyright (c) 2014 Google, Inc
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+/dts-v1/;


This is a mistake. Exynos4 is different than s5pc1xx(which was later 
called exynos3 and is also called s5pv210(kernel)).

So the s5pc1xx-goni.dts should include:
  model = Samsung Goni based on S5PC110;
  compatible = samsung,goni, samsung,s5pc110;

And the base addresses are different than exynos4. Please look at:
arch/arm/include/asm/arch-s5pc1xx/cpu.h

This will require also a common file for s5pc1xx: s5pc1xx-common.dts.
Moreover, the S5PC100 and S5PC110 have lot of differences.

Simon, I think that this is a job for goni's maintainer, since you can't 
test it on a hardware.


Robert, can you look at this?


+/include/ exynos4.dtsi
+
+/ {
+   model = Samsung SMDKV310 on Exynos4210;
+   compatible = samsung,smdkv310, samsung,exynos4210;
+
+   aliases {
+   serial0 = /serial@1380;
+   console = /serial@1382;
+   };
+
+};



Best regards,
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 8/8] samsung: Move smdkc100 to use exynos-common config

2014-09-09 Thread Przemyslaw Marczak

Hello Simon,

On 07/23/2014 02:11 PM, Simon Glass wrote:

Change this board to use the exynos common config and add a device tree.

Signed-off-by: Simon Glass s...@chromium.org
---

  arch/arm/dts/Makefile |  1 +
  arch/arm/dts/s5pc1xx-smdkc100.dts | 21 +++
  include/configs/smdkc100.h| 57 +--
  3 files changed, 35 insertions(+), 44 deletions(-)
  create mode 100644 arch/arm/dts/s5pc1xx-smdkc100.dts

diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 6a6e664..ed67844 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -1,3 +1,4 @@
+dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb
  dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
  dtb-$(CONFIG_EXYNOS4) += exynos4210-origen.dtb \
exynos4210-smdkv310.dtb \
diff --git a/arch/arm/dts/s5pc1xx-smdkc100.dts 
b/arch/arm/dts/s5pc1xx-smdkc100.dts
new file mode 100644
index 000..409812d
--- /dev/null
+++ b/arch/arm/dts/s5pc1xx-smdkc100.dts
@@ -0,0 +1,21 @@
+/*
+ * Samsung's Exynos4210-based SMDKV310 board device tree source
+ *
+ * Copyright (c) 2014 Google, Inc
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+/dts-v1/;


The same as in s5pc110:

+/include/ exynos4.dtsi
+
+/ {
+   model = Samsung SMDKC100;
+   compatible = samsung,smdkc100;
+
+   aliases {

Bad base addresses:

+   serial0 = /serial@1380;
+   console = /serial@1382;
+   };
+


Best regards,
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/8] samsung: Use common config files with Samsung boards (help wanted)

2014-09-09 Thread Przemyslaw Marczak

Hi,

I will test this patch set on a trats2 board - today evening.

On 09/09/2014 08:05 AM, Simon Glass wrote:

Hi Minkyu,

On 25 July 2014 08:43, Simon Glass s...@chromium.org wrote:


Hi Lucasz,

On 24 July 2014 09:34, Lukasz Majewski l.majew...@samsung.com wrote:

Hi Simon,


This series tries to unify the Samsung board configs into a few header
files for exynos5 and exynos5.

The purpose is to make it easier to move to driver model. In that case
I would like things like the GPIO drivers and serial drivers to work
in a standard way, and not need to support device tree and platform
data at the same time. That would be quite painful.

Another reason is that the Chrome OS EC drivers are currently
included in boards that don't have a Chrome OS EC. This concern was
raised by the Samsung maintainer (Minkyu) a while back.

There are still a few boards that don't use CONFIG_OF_CONTROL so I
have updated these with the most rudimentary of device tree files.

Unfortunately I don't have boards for most of these (for testing) and
I am hoping that the maintainers can come to the rescue and fix up any
patches that have problems. I am also worried that I have used a
common exynos file for things like smdkv310, when in fact they have
some other chip in common.

So maintainers, please can you test this and re-issue the patch, or
make comments on my attempts?


We will do our best to test our Exynos4 based boards.


Thanks - please feel free to re-issue the patches and I will drop mine
(if you cc me).



Can you please review this series? We are already at rc2.

Regards,
Simon



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



Best regards,
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCHv2] generic board for socfpga

2014-09-09 Thread Pavel Machek
Socfpga u-boot works fine with CONFIG_SYS_GENERIC_BOARD, so enable
that option as documentation suggests.

Signed-off-by: Pavel Machek pa...@denx.de

---

 Sorry, I let this last too long before I reviewed it and now it does
 not apply cleanly at all. Can you have a look?

Here you go.


diff --git a/include/configs/socfpga_cyclone5.h 
b/include/configs/socfpga_cyclone5.h
index 5d145cd..d3d1e48 100644
--- a/include/configs/socfpga_cyclone5.h
+++ b/include/configs/socfpga_cyclone5.h
@@ -11,6 +11,8 @@
 #include ../../board/altera/socfpga/iocsr_config.h
 #include ../../board/altera/socfpga/pll_config.h
 
+#define CONFIG_SYS_GENERIC_BOARD
+
 /*
  * High level configuration
  */

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] VxWorks Booting using U-boot as Bootloader

2014-09-09 Thread Ashish Khetan
Thanks Again,
which u-boot version are you referring, because I am using 2013.04 version
and in this there is no directory named board/BuR/KWB. can you please send
the Boot Arguments which you used to boot the vxworks and which method are
you using to boot vxworks?


On Mon, Sep 8, 2014 at 10:16 PM, Hannes Petermaier han...@petermaier.org
wrote:

 Hi Ashish,

 okay - i am using vxworks 6.9.3.3, and there is is a issue around the
 interrupt vectors. In later versions 6.9.4 upwards windriver takes care
 about this and sets up the interruptvectors.
 U-Boot does sets them to some address within u-boot code, but vxWorks
 expects that the interrupt vector are at the place where they are after
 poweron-reset.

 In my case (using AM3352 from TI) i've set the interrupt vectors back to
 original state (have a look to my KWB-Board, board/BuR/kwb/board.c).

 BTW: i am using a stripped image which i directly start with go
 address, but that shouldn't matter.

 best regards,
 Hannes



 On 2014-09-09 03:56, Ashish Khetan wrote:

 Hi Hannes
 Thanks for your reply.  I am using VxWorks 6.7 and the image is not
 stripped.  The loadaddress is also same within kernel configuration. I am
 using bootvx command to boot VxWorks.  Do I need to update boot
 parameters?
 On Sep 8, 2014 4:11 PM, Hannes Petermaier 
 hannes.peterma...@br-automation.com wrote:

  Hi Ashish,

 i had a problem booting vxworks in the past around interrupt vectors.
 Which version of vxworks you are using ? do you use a stripped image ?
 is the loadaddress the same within kernel configuration ?

 best regards,
 hannes

  From: Ashish Khetan curieux.khe...@gmail.com
 To: u-boot@lists.denx.de
 Date: 04.09.2014 14:13
 Subject: [U-Boot] VxWorks Booting using U-boot as Bootloader
 Sent by: u-boot-boun...@lists.denx.de

 Hi,
 I was trying to boot vxworks using u-boot bootloader for learning

 purpose

 on MPC8640DHPCN board. Previously I successfully ported QNX os along

 with

 linux on same board using u-boot. while booting using bootvx command its
 not booting and stucking after showing the following message:

 tftp 0100 vxWorks
 Speed: 1000, full duplex
 Using eTSEC1 device
 TFTP from server 192.168.10.2; our IP address is 192.168.10.4
 Filename 'vxWorks'.
 Load address: 0x100
 Loading:

 * #

   #
   #
   12.3 MiB/s
 done
 Bytes transferred = 2392432 (248170 hex)
 BOSSv2.0= bootvx 0010 100
 ## Ethernet MAC address not copied to NV RAM
 ## Using bootline (@ 0x4200): root=/dev/nfs rw
 nfsroot=192.168.10.2:/opt/eldk-5.3/powerpc/rootfs-basic
 ip=192.168.10.4:192.168.10.2:192.168.10.1:255.255.255.0:Boss:eth0:off
 panic=1 console=ttyS0,115200
 ## Starting vxWorks at 0x0010 ...


 please somebody provide me some link or any pointer to help me get more
 understanding about booting multiple os using uboot.
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot





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


Re: [U-Boot] [PATCH] arch/arm: Add individual TLB size support.

2014-09-09 Thread Albert ARIBAUD
Hi Xiubo,

On Mon, 7 Jul 2014 13:19:11 +0800, Xiubo Li li.xi...@freescale.com
wrote:

 This adds CONFIG_TLB_SIZE for individual board, whose TLB size maybe
 larger than PGTABLE_SIZE.
 
 Signed-off-by: Xiubo Li li.xi...@freescale.com
 ---
  arch/arm/lib/board.c | 4 
  1 file changed, 4 insertions(+)
 
 diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
 index dc34190..b7327ce 100644
 --- a/arch/arm/lib/board.c
 +++ b/arch/arm/lib/board.c
 @@ -353,7 +353,11 @@ void board_init_f(ulong bootflag)
  
  #if !(defined(CONFIG_SYS_ICACHE_OFF)  defined(CONFIG_SYS_DCACHE_OFF))
   /* reserve TLB table */
 +#ifdef CONFIG_TLB_SIZE
 + gd-arch.tlb_size = CONFIG_TLB_SIZE;
 +#else
   gd-arch.tlb_size = PGTABLE_SIZE;
 +#endif
   addr -= gd-arch.tlb_size;
  
   /* round down to next 64 kB limit */

There is no code in current mainline which defines CONFIG_TLB_SIZE;
that makes the patch a dead code addition.

Besides, what's the point of this as opposed to, e.g., just defining the
right PGTABLE_SIZE, or renaming PGTABLE_SIZE as CONFIG_TLB_SIZE?

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Pavel Machek

This adds watchdog disable. It is neccessary for running Linux kernel.

Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/arch/arm/cpu/armv7/socfpga/reset_manager.c 
b/arch/arm/cpu/armv7/socfpga/reset_manager.c
index e320c01..5503059 100644
--- a/arch/arm/cpu/armv7/socfpga/reset_manager.c
+++ b/arch/arm/cpu/armv7/socfpga/reset_manager.c
@@ -14,6 +14,19 @@ DECLARE_GLOBAL_DATA_PTR;
 static const struct socfpga_reset_manager *reset_manager_base =
(void *)SOCFPGA_RSTMGR_ADDRESS;
 
+#define RSTMGR_PERMODRST_L4WD0_LSB 6
+
+/* Disable the watchdog (toggle reset to watchdog) */
+void watchdog_disable(void)
+{
+   /* assert reset for watchdog */
+   setbits_le32(reset_manager_base-per_mod_reset,
+(1RSTMGR_PERMODRST_L4WD0_LSB));
+   /* deassert watchdog from reset (watchdog in not running state) */
+   clrbits_le32(reset_manager_base-per_mod_reset,
+(1RSTMGR_PERMODRST_L4WD0_LSB));
+}
+
 /*
  * Write the reset manager register to cause reset
  */
diff --git a/arch/arm/include/asm/arch-socfpga/reset_manager.h 
b/arch/arm/include/asm/arch-socfpga/reset_manager.h
index 3e95476..eb95c5b 100644
--- a/arch/arm/include/asm/arch-socfpga/reset_manager.h
+++ b/arch/arm/include/asm/arch-socfpga/reset_manager.h
@@ -10,6 +10,8 @@
 void reset_cpu(ulong addr);
 void reset_deassert_peripherals_handoff(void);
 
+void watchdog_disable(void);
+
 struct socfpga_reset_manager {
u32 status;
u32 ctrl;

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] base addresses for more subsystems

2014-09-09 Thread Pavel Machek

Add base addresses for more subsystems.

Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/arch/arm/include/asm/arch-socfpga/socfpga_base_addrs.h 
b/arch/arm/include/asm/arch-socfpga/socfpga_base_addrs.h
index 2d3152d..39a7874 100644
--- a/arch/arm/include/asm/arch-socfpga/socfpga_base_addrs.h
+++ b/arch/arm/include/asm/arch-socfpga/socfpga_base_addrs.h
@@ -7,9 +7,15 @@
 #ifndef _SOCFPGA_BASE_ADDRS_H_
 #define _SOCFPGA_BASE_ADDRS_H_
 
+#define SOCFPGA_EMAC0_ADDRESS 0xff70
+#define SOCFPGA_EMAC1_ADDRESS 0xff702000
+#define SOCFPGA_SDMMC_ADDRESS 0xff704000
+#define SOCFPGA_FPGAMGRREGS_ADDRESS 0xff706000
 #define SOCFPGA_L3REGS_ADDRESS 0xff80
+#define SOCFPGA_FPGAMGRDATA_ADDRESS 0xffb9
 #define SOCFPGA_UART0_ADDRESS 0xffc02000
 #define SOCFPGA_UART1_ADDRESS 0xffc03000
+#define SOCFPGA_SDR_ADDRESS 0xffc2
 #define SOCFPGA_OSC1TIMER0_ADDRESS 0xffd0
 #define SOCFPGA_L4WD0_ADDRESS 0xffd02000
 #define SOCFPGA_CLKMGR_ADDRESS 0xffd04000

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] fix compilation of socfpga_dw_mmc

2014-09-09 Thread Pavel Machek

Fix compilation of socfpga_dw_mmc.
 
Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/drivers/mmc/socfpga_dw_mmc.c b/drivers/mmc/socfpga_dw_mmc.c
index 1f96382..d8da7f5 100644
--- a/drivers/mmc/socfpga_dw_mmc.c
+++ b/drivers/mmc/socfpga_dw_mmc.c
@@ -16,6 +16,8 @@ static const struct socfpga_clock_manager *clock_manager_base 
=
 static const struct socfpga_system_manager *system_manager_base =
(void *)SOCFPGA_SYSMGR_ADDRESS;
 
+#define CLKMGR_PERPLLGRP_EN_SDMMCCLK_MASK (1  8)
+
 static void socfpga_dwmci_clksel(struct dwmci_host *host)
 {
unsigned int drvsel;

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] cleanup drivers/net/phy/micrel.c

2014-09-09 Thread Pavel Machek
Old saying says that more than three exclamation marks in a row are
sign of mental disease. Cleanup micrel.c.

Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/drivers/net/phy/micrel.c b/drivers/net/phy/micrel.c
index 5d7e3be..06a31b0 100644
--- a/drivers/net/phy/micrel.c
+++ b/drivers/net/phy/micrel.c
@@ -25,8 +25,7 @@ static struct phy_driver KSZ804_driver = {
 #ifndef CONFIG_PHY_MICREL_KSZ9021
 /*
  * I can't believe Micrel used the exact same part number
- * for the KSZ9021
- * Shame Micrel, Shame!
+ * for the KSZ9021. Shame Micrel, Shame!
  */
 static struct phy_driver KS8721_driver = {
.name = Micrel KS8721BL,
@@ -40,7 +39,7 @@ static struct phy_driver KS8721_driver = {
 #endif
 
 
-/**
+/*
  * KSZ9021 - KSZ9031 common
  */
 
@@ -69,8 +68,8 @@ static int ksz90xx_startup(struct phy_device *phydev)
phydev-speed = SPEED_10;
return 0;
 }
-#ifdef CONFIG_PHY_MICREL_KSZ9021
 
+#ifdef CONFIG_PHY_MICREL_KSZ9021
 /*
  * KSZ9021
  */


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] fix compilation of socfpga_dw_mmc

2014-09-09 Thread Marek Vasut
On Tuesday, September 09, 2014 at 02:08:13 PM, Pavel Machek wrote:
 Fix compilation of socfpga_dw_mmc.
 
 Signed-off-by: Pavel Machek pa...@denx.de
 
 diff --git a/drivers/mmc/socfpga_dw_mmc.c b/drivers/mmc/socfpga_dw_mmc.c
 index 1f96382..d8da7f5 100644
 --- a/drivers/mmc/socfpga_dw_mmc.c
 +++ b/drivers/mmc/socfpga_dw_mmc.c
 @@ -16,6 +16,8 @@ static const struct socfpga_clock_manager
 *clock_manager_base = static const struct socfpga_system_manager
 *system_manager_base = (void *)SOCFPGA_SYSMGR_ADDRESS;
 
 +#define CLKMGR_PERPLLGRP_EN_SDMMCCLK_MASK (1  8)
 +
  static void socfpga_dwmci_clksel(struct dwmci_host *host)
  {
   unsigned int drvsel;

This macro should go into arch/arm/include/asm/arch-socfpga/clock_manager.h

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] dfu, usb: set usb descriptor to values defined in dfu spec 1.1

2014-09-09 Thread Marek Vasut
On Tuesday, September 09, 2014 at 10:45:57 AM, Heiko Schocher wrote:
 Hello Marek,
 
 Am 05.09.2014 00:42, schrieb Marek Vasut:
  On Thursday, September 04, 2014 at 12:21:40 PM, Heiko Schocher wrote:
  
  Hi Heiko,
  
  I'll just rant a bit, but please wait until Lukasz does a proper
  runthrough.
  
  set bDeviceClass, bDeviceSubClass and bcdUSB to the values
  defined in dfu spec 1.1 chapter 4.2.1 found here:
  
  http://www.usb.org/developers/devclass_docs/DFU_1.1.pdf
  
  This URL gives me a website with:
  
  Resource not found.
  Please check the URL and try again.
  
 
 Ah! Here the correct link:
 
 http://www.usb.org/developers/docs/devclass_docs/DFU_1.1.pdf
 
 I change this for v2.

Cool, thank you! :)

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Marek Vasut
On Tuesday, September 09, 2014 at 02:03:28 PM, Pavel Machek wrote:
 This adds watchdog disable. It is neccessary for running Linux kernel.

Why do we not enable WDT in Linux instead ? Also, I recall there was a call to 
explicitly enable the L4 watchdog, so why do we not get rid of that instead ?

 Signed-off-by: Pavel Machek pa...@denx.de
 
 diff --git a/arch/arm/cpu/armv7/socfpga/reset_manager.c
 b/arch/arm/cpu/armv7/socfpga/reset_manager.c index e320c01..5503059 100644
 --- a/arch/arm/cpu/armv7/socfpga/reset_manager.c
 +++ b/arch/arm/cpu/armv7/socfpga/reset_manager.c
 @@ -14,6 +14,19 @@ DECLARE_GLOBAL_DATA_PTR;
  static const struct socfpga_reset_manager *reset_manager_base =
   (void *)SOCFPGA_RSTMGR_ADDRESS;
 
 +#define RSTMGR_PERMODRST_L4WD0_LSB 6
 +
 +/* Disable the watchdog (toggle reset to watchdog) */
 +void watchdog_disable(void)
 +{
 + /* assert reset for watchdog */
 + setbits_le32(reset_manager_base-per_mod_reset,
 +  (1RSTMGR_PERMODRST_L4WD0_LSB));
 + /* deassert watchdog from reset (watchdog in not running state) */
 + clrbits_le32(reset_manager_base-per_mod_reset,
 +  (1RSTMGR_PERMODRST_L4WD0_LSB));
 +}
 +
  /*
   * Write the reset manager register to cause reset
   */
 diff --git a/arch/arm/include/asm/arch-socfpga/reset_manager.h
 b/arch/arm/include/asm/arch-socfpga/reset_manager.h index 3e95476..eb95c5b
 100644
 --- a/arch/arm/include/asm/arch-socfpga/reset_manager.h
 +++ b/arch/arm/include/asm/arch-socfpga/reset_manager.h
 @@ -10,6 +10,8 @@
  void reset_cpu(ulong addr);
  void reset_deassert_peripherals_handoff(void);
 
 +void watchdog_disable(void);
 +
  struct socfpga_reset_manager {
   u32 status;
   u32 ctrl;

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Pavel Machek
On Tue 2014-09-09 14:20:23, Marek Vasut wrote:
 On Tuesday, September 09, 2014 at 02:03:28 PM, Pavel Machek wrote:
  This adds watchdog disable. It is neccessary for running Linux kernel.
 
 Why do we not enable WDT in Linux instead ? Also, I recall there was a call 
 to 
 explicitly enable the L4 watchdog, so why do we not get rid of that
 instead ?

Booting linux kernel without watchdog support should be valid
operation.
Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] relocation problem on powerpc

2014-09-09 Thread Joakim Tjernlund
  Here's some more info from a build without any of my modifications.
  
$ git describe
v2014.10-rc2-12-g0b703db
  
$ powerpc-e500-linux-gnu-gcc --version
powerpc-e500-linux-gnu-gcc (Gentoo 4.6.3-r1 p1.9, pie-0.5.2) 4.6.3
  
$ grep '_GOT2_TABLE_\|_GLOBAL_OFFSET_TABLE_\|__got2_entries' 
u-boot.map
0xeffb5100_GOT2_TABLE_ = .
0xeffb7d38_GLOBAL_OFFSET_TABLE_
0xeffb7d48PROVIDE (_GLOBAL_OFFSET_TABLE_, (. 
+ 0x4))
0x0b0d__got2_entries =
  (((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  0x2) - 0x1)
  
$ gdb-multiarch u-boot
(gdb) disassemble /r 0xeffb5100,0xeffb7d48
Dump of assembler code from 0xeffb5100 to 0xeffb7d48:
   0xeffb5100 _GOT2_TABLE_+0: ef fb 51 00 .long 0xeffb5100
   0xeffb5104 _GOT2_TABLE_+4: ef fb 7d 44 dtstsf  cr7,f27,f15
   0xeffb5108 _GOT2_TABLE_+8: ef f4 00 00 .long 0xeff4
   0xeffb510c _GOT2_TABLE_+12:ef f4 01 00 .long 
0xeff40100
  snip
   0xeffb7d30 _GOT2_TABLE_+11312: ef f6 6e 74 .long 
0xeff66e74
   0xeffb7d34 _GOT2_TABLE_+11316: f0 00 00 d0 psq_st 
f0,208(r0),0,0
   0xeffb7d38 _GLOBAL_OFFSET_TABLE_+0:00 00 00 00 .long 
0x0
   0xeffb7d3c _GLOBAL_OFFSET_TABLE_+4:00 00 00 00 .long 
0x0
   0xeffb7d40 _GLOBAL_OFFSET_TABLE_+8:00 00 00 00 .long 
0x0
   0xeffb7d44 _FIXUP_TABLE_+0:ef fb 99 d4 .long 
0xeffb99d4
  
  So no there is no brlr at the end of the table.
 
 There should be, see 
http://www.freescale.com/files/32bit/doc/ref_manual/E500ABIUG.pdf, chap 
4.3.2

So I did a quick test too with my copy of u-boot:
(gdb) disass /r 0xeffa0a48,0xeffa0a68
   Dump of assembler code from 0xeffa0a48 to 0xeffa0a68:
   0xeffa0a48 _GOT2_TABLE_+9800:  f0 00 00 dc .long 0xf0dc
   0xeffa0a4c _GOT2_TABLE_+9804:  ef f6 29 48 .long 0xeff62948
   0xeffa0a50 _GOT2_TABLE_+9808:  f0 00 00 e0 .long 0xf0e0
   0xeffa0a54 _GOT2_TABLE_+9812:  4e 80 00 21 blrl
^
   0xeffa0a58 _GLOBAL_OFFSET_TABLE_+0:00 00 00 00 .long 0x0
   0xeffa0a5c _GLOBAL_OFFSET_TABLE_+4:00 00 00 00 .long 0x0
   0xeffa0a60 _GLOBAL_OFFSET_TABLE_+8:00 00 00 00 .long 0x0
   0xeffa0a64 _FIXUP_TABLE_+0:ef fa 23 1c .long 0xeffa231c

$ powerpc-softfloat_4.5.3-linux-gnu-ld --version
GNU ld (GNU Binutils) 2.21.1

I don't know it this is tool(ld) problem or if something in u-boot's build 
env. Try dropping
--gc-sections, see if that changes anything, or add --bss-plt or similar
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Marek Vasut
On Tuesday, September 09, 2014 at 02:30:01 PM, Pavel Machek wrote:
 On Tue 2014-09-09 14:20:23, Marek Vasut wrote:
  On Tuesday, September 09, 2014 at 02:03:28 PM, Pavel Machek wrote:
   This adds watchdog disable. It is neccessary for running Linux kernel.
  
  Why do we not enable WDT in Linux instead ? Also, I recall there was a
  call to explicitly enable the L4 watchdog, so why do we not get rid of
  that instead ?
 
 Booting linux kernel without watchdog support should be valid
 operation.

If you watchdog is enabled, then not so much. Also, why can we not just disable 
the WDT in U-Boot instead using a config option ?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: nitrogen6x: Replace 'fatload' by 'load' command in env settings to be filesystem independent

2014-09-09 Thread Guillaume Gardet

Ping.

Guillaume


Le 01/09/2014 20:27, Eric Nelson a écrit :

Hi all,

On 08/31/2014 07:24 PM, Fabio Estevam wrote:

On Tue, Aug 26, 2014 at 7:05 AM, Guillaume GARDET
guillaume.gar...@free.fr wrote:

nitrogen6x.h file defines CONFIG_CMD_FS_GENERIC, so we are able to use generic
'load' command instead of 'fatload'. It allows to use ext filesystem and keep
compatibilty with fat filesystem.

Signed-off-by: Guillaume GARDET guillaume.gar...@free.fr
Cc: Stefano Babic sba...@denx.de

Adding Eric Nelson on Cc.


Thanks Fabio. I stopped delivery from the list while on vacation and
hadn't re-started it.


---
  include/configs/nitrogen6x.h | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h
index b2b17ce..d4b0ac9 100644
--- a/include/configs/nitrogen6x.h
+++ b/include/configs/nitrogen6x.h
@@ -192,11 +192,11 @@

I've been wondering if anyone was using this section of nitrogen6x.h,
and I guess this answers the question.


 mmcargs=setenv bootargs console=${console},${baudrate}  \
 root=${mmcroot}\0 \
 loadbootscript= \
-   fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0 \
+   load mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0 \
 bootscript=echo Running bootscript from mmc ...;  \
 source\0 \
-   loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0 \
-   loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0 \
+   loaduimage=load mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0 \
+   loadfdt=load mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0 \
 mmcboot=echo Booting from mmc ...;  \
 run mmcargs;  \
 if test ${boot_fdt} = yes || test ${boot_fdt} = try; then  \
--
1.8.4.5


I don't have a copy of the original e-mail, but:

Acked-By: Eric Nelson eric.nel...@boundarydevices.com




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


[U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only

2014-09-09 Thread Heiko Schocher
This patchserie adds the new config option CONFIG_DFU_FULLSPEED.
If this is enabled DFU uses fullspeed only. This is used on the
siemens boards.

Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com

Heiko Schocher (2):
  usb: dfu: add config option to use in dfu mode fullspeed only
  arm: am335x: siemens board use in DFU mode fullspeed only

 README | 3 +++
 drivers/usb/gadget/f_dfu.c | 3 +++
 drivers/usb/gadget/g_dnl.c | 3 +++
 include/configs/siemens-am33x-common.h | 3 ++-
 4 files changed, 11 insertions(+), 1 deletion(-)

-- 
1.8.3.1

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


[U-Boot] [PATCH v1 2/2] arm: am335x: siemens board use in DFU mode fullspeed only

2014-09-09 Thread Heiko Schocher
Siemens boards are now using DFU in fullspeed only.

Signed-off-by: Heiko Schocher h...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com
---
 include/configs/siemens-am33x-common.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/configs/siemens-am33x-common.h 
b/include/configs/siemens-am33x-common.h
index b8fb77e..6fa5ddf 100644
--- a/include/configs/siemens-am33x-common.h
+++ b/include/configs/siemens-am33x-common.h
@@ -233,7 +233,7 @@
 #define CONFIG_MUSB_GADGET
 #define CONFIG_MUSB_PIO_ONLY
 #define CONFIG_MUSB_DISABLE_BULK_COMBINE_SPLIT
-#define CONFIG_USB_GADGET_DUALSPEED
+#undef CONFIG_USB_GADGET_DUALSPEED
 #define CONFIG_USB_GADGET_VBUS_DRAW2
 #define CONFIG_MUSB_HOST
 
@@ -267,6 +267,7 @@
 #define CONFIG_CMD_DFU
 #define CONFIG_SYS_DFU_DATA_BUF_SIZE   (1  20)
 #define DFU_MANIFEST_POLL_TIMEOUT  25000
+#define CONFIG_DFU_FULLSPEED
 
 #endif /* CONFIG_SPL_BUILD */
 
-- 
1.8.3.1

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


[U-Boot] [PATCH v1 1/2] usb: dfu: add config option to use in dfu mode fullspeed only

2014-09-09 Thread Heiko Schocher
add the new config option CONFIG_DFU_FULLSPEED. With this
option enabled, DFU uses fullspeed only.

Signed-off-by: Heiko Schocher h...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com
---
 README | 3 +++
 drivers/usb/gadget/f_dfu.c | 3 +++
 drivers/usb/gadget/g_dnl.c | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/README b/README
index 0a0f528..1413392 100644
--- a/README
+++ b/README
@@ -1607,6 +1607,9 @@ The following options need to be configured:
entering dfuMANIFEST state. Host waits this timeout, before
sending again an USB request to the device.
 
+   CONFIG_DFU_FULLSPEED
+   use for the dfu functionality fullspeed only.
+
 - USB Device Android Fastboot support:
CONFIG_CMD_FASTBOOT
This enables the command fastboot which enables the Android
diff --git a/drivers/usb/gadget/f_dfu.c b/drivers/usb/gadget/f_dfu.c
index 1145aab..dfa9f3b 100644
--- a/drivers/usb/gadget/f_dfu.c
+++ b/drivers/usb/gadget/f_dfu.c
@@ -238,6 +238,7 @@ static inline void to_dfu_mode(struct f_dfu *f_dfu)
 {
f_dfu-usb_function.strings = dfu_strings;
f_dfu-usb_function.hs_descriptors = f_dfu-function;
+   f_dfu-usb_function.descriptors = f_dfu-function;
f_dfu-dfu_state = DFU_STATE_dfuIDLE;
 }
 
@@ -245,6 +246,7 @@ static inline void to_runtime_mode(struct f_dfu *f_dfu)
 {
f_dfu-usb_function.strings = NULL;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+   f_dfu-usb_function.descriptors = dfu_runtime_descs;
 }
 
 static int handle_upload(struct usb_request *req, u16 len)
@@ -809,6 +811,7 @@ static int dfu_bind_config(struct usb_configuration *c)
return -ENOMEM;
f_dfu-usb_function.name = dfu;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+   f_dfu-usb_function.descriptors = dfu_runtime_descs;
f_dfu-usb_function.bind = dfu_bind;
f_dfu-usb_function.unbind = dfu_unbind;
f_dfu-usb_function.set_alt = dfu_set_alt;
diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c
index 58cad11..79e0c9c 100644
--- a/drivers/usb/gadget/g_dnl.c
+++ b/drivers/usb/gadget/g_dnl.c
@@ -143,6 +143,9 @@ static int g_dnl_config_register(struct usb_composite_dev 
*cdev)
config-bConfigurationValue = CONFIGURATION_NUMBER;
config-iConfiguration = STRING_USBDOWN;
config-bind = g_dnl_do_config;
+#if defined(CONFIG_DFU_FULLSPEED)
+   config-fullspeed = 1;
+#endif
 
return usb_add_config(cdev, config);
 }
-- 
1.8.3.1

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


Re: [U-Boot] [PATCH] imx: nitrogen6x: Replace 'fatload' by 'load' command in env settings to be filesystem independent

2014-09-09 Thread Stefano Babic
Hi Guillaume,

On 09/09/2014 14:31, Guillaume Gardet wrote:
 Ping.

Sorry, I was not in office in the last two weeks. I will go through all
i.MX patches (it seems I have something to do..) and I will apply the
ones are ready, as yours.

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Pavel Machek
On Tue 2014-09-09 14:31:46, Marek Vasut wrote:
 On Tuesday, September 09, 2014 at 02:30:01 PM, Pavel Machek wrote:
  On Tue 2014-09-09 14:20:23, Marek Vasut wrote:
   On Tuesday, September 09, 2014 at 02:03:28 PM, Pavel Machek wrote:
This adds watchdog disable. It is neccessary for running Linux kernel.
   
   Why do we not enable WDT in Linux instead ? Also, I recall there was a
   call to explicitly enable the L4 watchdog, so why do we not get rid of
   that instead ?
  
  Booting linux kernel without watchdog support should be valid
  operation.
 
 If you watchdog is enabled, then not so much. Also, why can we not just 
 disable 
 the WDT in U-Boot instead using a config option ?

It seems that watchdog is enabled by default (by hardware or SPL), so
yes, watchdog disable operation (and this patch) in u-boot is a good
idea.

Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v7 1/2] net: fec_mxc: Adjust RX DMA alignment for mx6solox

2014-09-09 Thread Stefano Babic
Hi Fabio,

On 09/09/2014 01:52, Fabio Estevam wrote:
 Hi Tom,
 
 On Sat, Aug 30, 2014 at 4:21 PM, Tom Rini tr...@ti.com wrote:
 On Sat, Aug 30, 2014 at 02:22:22PM -0300, Fabio Estevam wrote:
 Tom, Joe or Stefano,

 On Mon, Aug 25, 2014 at 4:51 PM, Marek Vasut ma...@denx.de wrote:
 On Monday, August 25, 2014 at 06:34:16 PM, Fabio Estevam wrote:
 mx6solox has a requirement for 64 bytes alignment for RX DMA transfer.
 Other SoCs work with the standard 32 bytes alignment.

 Adjust it accordingly by using 64 bytes aligment in the FEC RX DMA 
 buffers,
 which addresses the needs from mx6solox and also works for the other SoCs.

 Signed-off-by: Fabio Estevam fabio.este...@freescale.com

 Acked-by: Marek Vasut ma...@denx.de

 Could this one be applied for 2014.10-rc?

 I'd like this via the imx tree.
 
 Unfortunately, I am not getting any response from Stefano for quite some time.
 

Sorry, I was not in office in last two weeks.

 Can you apply this series as we are already in rc2 now?

Applied to u-boot-imx, thanks !

Fabio, I will try to dig in all patches that queue in last two weeks. If
you can, please take a look next week what is still missing and send me
a remind - thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v7 2/2] net: fec_mxc: Poll FEC_TBD_READY after polling TDAR

2014-09-09 Thread Stefano Babic
On 25/08/2014 18:34, Fabio Estevam wrote:
 When testing the FEC driver on a mx6solox we noticed that the TDAR bit gets
 always cleared prior then the READY bit is cleared in the last BD, which 
 causes
 FEC packets reception to always fail.
 
 As explained by Ye Li:
 
 The TDAR bit is cleared when the descriptors are all out from TX ring, but 
 on 
 mx6solox we noticed that the READY bit is still not cleared right after TDAR. 
 These are two distinct signals, and in IC simulation, we found that TDAR 
 always
 gets cleared prior than the READY bit of last BD becomes cleared. 
 In mx6solox, we use a later version of FEC IP. It looks like that this 
 intrinsic behaviour of TDAR bit has changed in this newer FEC version.
 
 Fix this by polling the READY bit of BD after the TDAR polling, which covers 
 the
 mx6solox case and does not harm the other SoCs.
 
 No performance drop has been noticed with this patch applied when testing TFTP
 transfers on several boards of different i.mx SoCs.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: ventana: base SPL MMDC calibration on width and size not board

2014-09-09 Thread Stefano Babic
On 21/08/2014 08:31, Tim Harvey wrote:
 The IMX6 MMDC calibration registers depend on propagation delay and capacitive
 loading between the SoC's MMDC and the DDR3 chips. On the Ventana boards the
 board layout varies little in trace-lengths such that propagation delays are
 irrelevant thus we can simply things by using calibration values obtained
 from various board layouts based on a common SoC and DDR chip configuration.
 
 This eliminates board-model from being needed allowing more flexibility. These
 values were tested on a large sample size of Gateworks Ventana boards ranging
 in layout, and memory configuration over the entire temperature range 
 supported.
 
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: ventana: add GW5520 support

2014-09-09 Thread Stefano Babic
On 21/08/2014 08:35, Tim Harvey wrote:
 The GW5520 has an IMX6Q SoC with 512MB of DDR3, 256MB of NAND flash as well 
 as:
  * 2x MiniPCIe sockets
  * 2x USB host sockets
  * 2x i210 GigE
  * HDMI out
  * digital I/O expansion
 
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: ventana: updated notes regarding NAND boot errata

2014-09-09 Thread Stefano Babic
On 21/08/2014 08:30, Tim Harvey wrote:
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---


Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] sort drivers/mmc/Makefile

2014-09-09 Thread Pavel Machek

Sort drivers/mmc makefile, as requested by wd.

Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
index 464cee1..461d7d8 100644
--- a/drivers/mmc/Makefile
+++ b/drivers/mmc/Makefile
@@ -5,37 +5,39 @@
 # SPDX-License-Identifier: GPL-2.0+
 #
 
+obj-$(CONFIG_ARM_PL180_MMCI) += arm_pl180_mmci.o
+obj-$(CONFIG_BCM2835_SDHCI) += bcm2835_sdhci.o
 obj-$(CONFIG_BFIN_SDH) += bfin_sdh.o
 obj-$(CONFIG_DAVINCI_MMC) += davinci_mmc.o
+obj-$(CONFIG_DWMMC) += dw_mmc.o
+obj-$(CONFIG_EXYNOS_DWMMC) += exynos_dw_mmc.o
 obj-$(CONFIG_FSL_ESDHC) += fsl_esdhc.o
 obj-$(CONFIG_FTSDC010) += ftsdc010_mci.o
 obj-$(CONFIG_FTSDC021) += ftsdc021_sdhci.o
 obj-$(CONFIG_GENERIC_MMC) += mmc.o
 obj-$(CONFIG_GENERIC_ATMEL_MCI) += gen_atmel_mci.o
+obj-$(CONFIG_KONA_SDHCI) += kona_sdhci.o
 obj-$(CONFIG_MMC_SPI) += mmc_spi.o
-obj-$(CONFIG_ARM_PL180_MMCI) += arm_pl180_mmci.o
+obj-$(CONFIG_MMC_SUNXI) += sunxi_mmc.o
 obj-$(CONFIG_MV_SDHCI) += mv_sdhci.o
+obj-$(CONFIG_MVEBU_MMC) += mvebu_mmc.o
 obj-$(CONFIG_MXC_MMC) += mxcmmc.o
 obj-$(CONFIG_MXS_MMC) += mxsmmc.o
 obj-$(CONFIG_OMAP_HSMMC) += omap_hsmmc.o
 obj-$(CONFIG_PXA_MMC_GENERIC) += pxa_mmc_gen.o
-obj-$(CONFIG_SDHCI) += sdhci.o
-obj-$(CONFIG_BCM2835_SDHCI) += bcm2835_sdhci.o
-obj-$(CONFIG_KONA_SDHCI) += kona_sdhci.o
+obj-$(CONFIG_SUPPORT_EMMC_RPMB) += rpmb.o
 obj-$(CONFIG_S3C_SDI) += s3c_sdi.o
 obj-$(CONFIG_S5P_SDHCI) += s5p_sdhci.o
+obj-$(CONFIG_SDHCI) += sdhci.o
 obj-$(CONFIG_SH_MMCIF) += sh_mmcif.o
+obj-$(CONFIG_SOCFPGA_DWMMC) += socfpga_dw_mmc.o
 obj-$(CONFIG_SPEAR_SDHCI) += spear_sdhci.o
 obj-$(CONFIG_TEGRA_MMC) += tegra_mmc.o
-obj-$(CONFIG_DWMMC) += dw_mmc.o
-obj-$(CONFIG_EXYNOS_DWMMC) += exynos_dw_mmc.o
-obj-$(CONFIG_MMC_SUNXI) += sunxi_mmc.o
 obj-$(CONFIG_ZYNQ_SDHCI) += zynq_sdhci.o
-obj-$(CONFIG_SOCFPGA_DWMMC) += socfpga_dw_mmc.o
-obj-$(CONFIG_SUPPORT_EMMC_RPMB) += rpmb.o
+
 ifdef CONFIG_SPL_BUILD
 obj-$(CONFIG_SPL_MMC_BOOT) += fsl_esdhc_spl.o
 else
 obj-$(CONFIG_GENERIC_MMC) += mmc_write.o
 endif
-obj-$(CONFIG_MVEBU_MMC) += mvebu_mmc.o
+

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 02/11] imx: ventana: added cputype env var

2014-09-09 Thread Stefano Babic
On 08/08/2014 07:35, Tim Harvey wrote:
 There are many similarities between the IMX6QUAD/IMX6DUAL and there are
 many similarities between the IMX6SOLO/IMX6DUALITE. Add a 'soctype' env
 variable that tells you which type you have.
 
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] cleanup disk/part.c whitespace

2014-09-09 Thread Pavel Machek
Cleanup disk/part.c

Signed-off-by: Pavel Machek pa...@denx.de

diff --git a/disk/part.c b/disk/part.c
index ecc5e7e..cfd77b0 100644
--- a/disk/part.c
+++ b/disk/part.c
@@ -133,7 +133,7 @@ typedef lbaint_t lba512_t;
  * Overflowless variant of (block_count * mul_by / div_by)
  * when div_by  mul_by
  */
-static lba512_t lba512_muldiv (lba512_t block_count, lba512_t mul_by, lba512_t 
div_by)
+static lba512_t lba512_muldiv(lba512_t block_count, lba512_t mul_by, lba512_t 
div_by)
 {
lba512_t bc_quot, bc_rem;
 
@@ -215,7 +215,8 @@ void dev_print (block_dev_desc_t *dev_desc)
 
lba512 = (lba * (dev_desc-blksz/512));
/* round to 1 digit */
-   mb = lba512_muldiv(lba512, 10, 2048);   /* 2048 = (1024 * 1024) 
/ 512 MB */
+   /* 2048 = (1024 * 1024) / 512 MB */
+   mb = lba512_muldiv(lba512, 10, 2048);
 
mb_quot = mb / 10;
mb_rem  = mb - (10 * mb_quot);
@@ -248,7 +249,7 @@ void dev_print (block_dev_desc_t *dev_desc)
 
 #ifdef HAVE_BLOCK_DEVICE
 
-void init_part (block_dev_desc_t * dev_desc)
+void init_part(block_dev_desc_t *dev_desc)
 {
 #ifdef CONFIG_ISO_PARTITION
if (test_part_iso(dev_desc) == 0) {
@@ -295,7 +296,7 @@ void init_part (block_dev_desc_t * dev_desc)
defined(CONFIG_AMIGA_PARTITION) || \
defined(CONFIG_EFI_PARTITION)
 
-static void print_part_header (const char *type, block_dev_desc_t * dev_desc)
+static void print_part_header(const char *type, block_dev_desc_t *dev_desc)
 {
puts (\nPartition Map for );
switch (dev_desc-if_type) {

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] relocation problem on powerpc

2014-09-09 Thread Joakim Tjernlund
Joakim Tjernlund/Transmode wrote on 2014/09/09 14:30:13:

 From: Joakim Tjernlund/Transmode
 To: 
 Cc: Chris Packham judge.pack...@gmail.com, u-boot 
u-boot@lists.denx.de
 Date: 2014/09/09 14:30
 Subject: Re: [U-Boot] relocation problem on powerpc
 
 
   Here's some more info from a build without any of my modifications.
   
 $ git describe
 v2014.10-rc2-12-g0b703db
   
 $ powerpc-e500-linux-gnu-gcc --version
 powerpc-e500-linux-gnu-gcc (Gentoo 4.6.3-r1 p1.9, pie-0.5.2) 4.6.3
   
 $ grep '_GOT2_TABLE_\|_GLOBAL_OFFSET_TABLE_\|__got2_entries' 
u-boot.map
 0xeffb5100_GOT2_TABLE_ = .
 0xeffb7d38_GLOBAL_OFFSET_TABLE_
 0xeffb7d48PROVIDE (_GLOBAL_OFFSET_TABLE_, 
(. + 0x4))
 0x0b0d__got2_entries =
   (((_GLOBAL_OFFSET_TABLE_ - _GOT2_TABLE_)  0x2) - 0x1)
   
 $ gdb-multiarch u-boot
 (gdb) disassemble /r 0xeffb5100,0xeffb7d48
 Dump of assembler code from 0xeffb5100 to 0xeffb7d48:
0xeffb5100 _GOT2_TABLE_+0: ef fb 51 00 .long 0xeffb5100
0xeffb5104 _GOT2_TABLE_+4: ef fb 7d 44 dtstsf cr7,f27,f15
0xeffb5108 _GOT2_TABLE_+8: ef f4 00 00 .long 0xeff4
0xeffb510c _GOT2_TABLE_+12:ef f4 01 00 .long 
0xeff40100
   snip
0xeffb7d30 _GOT2_TABLE_+11312: ef f6 6e 74 .long 
0xeff66e74
0xeffb7d34 _GOT2_TABLE_+11316: f0 00 00 d0 psq_st 
f0,208(r0),0,0
0xeffb7d38 _GLOBAL_OFFSET_TABLE_+0:00 00 00 00 .long 
0x0
0xeffb7d3c _GLOBAL_OFFSET_TABLE_+4:00 00 00 00 .long 
0x0
0xeffb7d40 _GLOBAL_OFFSET_TABLE_+8:00 00 00 00 .long 
0x0
0xeffb7d44 _FIXUP_TABLE_+0:ef fb 99 d4 .long 
0xeffb99d4
   
   So no there is no brlr at the end of the table.
  
  There should be, see 
http://www.freescale.com/files/32bit/doc/ref_manual/E500ABIUG.pdf, chap 
4.3.2
 
 So I did a quick test too with my copy of u-boot:
 (gdb) disass /r 0xeffa0a48,0xeffa0a68
Dump of assembler code from 0xeffa0a48 to 0xeffa0a68:
0xeffa0a48 _GOT2_TABLE_+9800: f0 00 00 dc .long 0xf0dc
0xeffa0a4c _GOT2_TABLE_+9804: ef f6 29 48 .long 0xeff62948
0xeffa0a50 _GOT2_TABLE_+9808: f0 00 00 e0 .long 0xf0e0
0xeffa0a54 _GOT2_TABLE_+9812: 4e 80 00 21 blrl
 ^
0xeffa0a58 _GLOBAL_OFFSET_TABLE_+0: 00 00 00 00 .long 0x0
0xeffa0a5c _GLOBAL_OFFSET_TABLE_+4: 00 00 00 00 .long 0x0
0xeffa0a60 _GLOBAL_OFFSET_TABLE_+8: 00 00 00 00 .long 0x0
0xeffa0a64 _FIXUP_TABLE_+0: ef fa 23 1c .long 0xeffa231c

 $ powerpc-softfloat_4.5.3-linux-gnu-ld --version
 GNU ld (GNU Binutils) 2.21.1
 
 I don't know it this is tool(ld) problem or if something in u-boot's 
build env. Try dropping
 --gc-sections, see if that changes anything, or add --bss-plt or similar

--bss-plt seems to be the answer:
http://lists.denx.de/pipermail/u-boot/2014-February/173134.html

  Jocke

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


Re: [U-Boot] [PATCH v7 1/2] net: fec_mxc: Adjust RX DMA alignment for mx6solox

2014-09-09 Thread Fabio Estevam
Hi Stefano,

On Tue, Sep 9, 2014 at 10:09 AM, Stefano Babic sba...@denx.de wrote:

 Fabio, I will try to dig in all patches that queue in last two weeks. If
 you can, please take a look next week what is still missing and send me
 a remind - thanks !

Excellent, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only

2014-09-09 Thread Lukasz Majewski
Hi Heiko,

 This patchserie adds the new config option CONFIG_DFU_FULLSPEED.

Is there any special reason to support Full Speed (12 Mbit/sec - USB
1.1) and not rely solely on the High Speed (USB 2.0) as we do now?

 If this is enabled DFU uses fullspeed only. This is used on the
 siemens boards.

Is there any DFU problem with the mentioned Siemens board, that we must
use USB 1.1?

I'd also appreciate more verbose rationale for this patch series - why
this change is needed? What is the expected improvement?

Thanks in advance.

 
 Cc: Tom Rini tr...@ti.com
 Cc: Lukasz Majewski l.majew...@samsung.com
 Cc: Marek Vasut ma...@denx.de
 Cc: Liu Bin b-...@ti.com
 Cc: Lukas Stockmann lukas.stockm...@siemens.com
 
 Heiko Schocher (2):
   usb: dfu: add config option to use in dfu mode fullspeed only
   arm: am335x: siemens board use in DFU mode fullspeed only
 
  README | 3 +++
  drivers/usb/gadget/f_dfu.c | 3 +++
  drivers/usb/gadget/g_dnl.c | 3 +++
  include/configs/siemens-am33x-common.h | 3 ++-
  4 files changed, 11 insertions(+), 1 deletion(-)
 



-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH V4 00/19] Introduce cm-fx6 board

2014-09-09 Thread Stefano Babic
Hi Nikita,

On 31/08/2014 10:09, Nikita Kiryanov wrote:
 Hi Tom, Stefano,
 
 We really want this series to make it into the next release (V1 was
 submitted only a day after the merge window closed). If there are no
 other comments, can we maybe start applying it?
 

Patch 1/19, 4/19 and 5/19 were already applied by Jagan - I do not merge
them into u-boot-imx.
I applied 2/19 after rebasing on current u-boot-imx.

I have nothing against the helper introduced in 3/19, but this touch
more boards - I will make a short check before applying it. However, it
is orthogonal and can be applied later.

I applied as well 6/19 through 19/19 into u-boot-imx - thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only

2014-09-09 Thread Bin Liu

Lukasz,

On 09/09/2014 08:43 AM, Lukasz Majewski wrote:

Hi Heiko,


This patchserie adds the new config option CONFIG_DFU_FULLSPEED.


Is there any special reason to support Full Speed (12 Mbit/sec - USB
1.1) and not rely solely on the High Speed (USB 2.0) as we do now?



The drivers must support full-speed anyway based on the USB2.0 Specs.

I have seen many cases in which only full-speed is suitable due to 
either board design or specific USB use case.


One example is that to better fit to the production firmware flashing 
automation, a board does not use standard USB receptacle or plug, but a 
four touch pads on the PCB, so that the robot can grab the board and 
touch the pads to download the firmware. In this case signal integrity 
is a concern, but not download speed.


Regards,
-Bin.


If this is enabled DFU uses fullspeed only. This is used on the
siemens boards.


Is there any DFU problem with the mentioned Siemens board, that we must
use USB 1.1?

I'd also appreciate more verbose rationale for this patch series - why
this change is needed? What is the expected improvement?

Thanks in advance.



Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com

Heiko Schocher (2):
   usb: dfu: add config option to use in dfu mode fullspeed only
   arm: am335x: siemens board use in DFU mode fullspeed only

  README | 3 +++
  drivers/usb/gadget/f_dfu.c | 3 +++
  drivers/usb/gadget/g_dnl.c | 3 +++
  include/configs/siemens-am33x-common.h | 3 ++-
  4 files changed, 11 insertions(+), 1 deletion(-)







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


Re: [U-Boot] [PATCH v1 1/2] usb: dfu: add config option to use in dfu mode fullspeed only

2014-09-09 Thread Bin Liu

Heiko,

On 09/09/2014 07:37 AM, Heiko Schocher wrote:

add the new config option CONFIG_DFU_FULLSPEED. With this
option enabled, DFU uses fullspeed only.


Can we not introduce the new config option but check the gadget driver 
speed in runtime as what the ether gadget driver does?


I don't want two config options (CONFIG_USB_GADGET_DUALSPEED and 
CONFIG_DFU_FULLSPEED) to control one feature, as in your patch 2/2.


Regards,
-Bin.



Signed-off-by: Heiko Schocher h...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com
---
  README | 3 +++
  drivers/usb/gadget/f_dfu.c | 3 +++
  drivers/usb/gadget/g_dnl.c | 3 +++
  3 files changed, 9 insertions(+)

diff --git a/README b/README
index 0a0f528..1413392 100644
--- a/README
+++ b/README
@@ -1607,6 +1607,9 @@ The following options need to be configured:
entering dfuMANIFEST state. Host waits this timeout, before
sending again an USB request to the device.

+   CONFIG_DFU_FULLSPEED
+   use for the dfu functionality fullspeed only.
+
  - USB Device Android Fastboot support:
CONFIG_CMD_FASTBOOT
This enables the command fastboot which enables the Android
diff --git a/drivers/usb/gadget/f_dfu.c b/drivers/usb/gadget/f_dfu.c
index 1145aab..dfa9f3b 100644
--- a/drivers/usb/gadget/f_dfu.c
+++ b/drivers/usb/gadget/f_dfu.c
@@ -238,6 +238,7 @@ static inline void to_dfu_mode(struct f_dfu *f_dfu)
  {
f_dfu-usb_function.strings = dfu_strings;
f_dfu-usb_function.hs_descriptors = f_dfu-function;
+   f_dfu-usb_function.descriptors = f_dfu-function;
f_dfu-dfu_state = DFU_STATE_dfuIDLE;
  }

@@ -245,6 +246,7 @@ static inline void to_runtime_mode(struct f_dfu *f_dfu)
  {
f_dfu-usb_function.strings = NULL;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+   f_dfu-usb_function.descriptors = dfu_runtime_descs;
  }

  static int handle_upload(struct usb_request *req, u16 len)
@@ -809,6 +811,7 @@ static int dfu_bind_config(struct usb_configuration *c)
return -ENOMEM;
f_dfu-usb_function.name = dfu;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+   f_dfu-usb_function.descriptors = dfu_runtime_descs;
f_dfu-usb_function.bind = dfu_bind;
f_dfu-usb_function.unbind = dfu_unbind;
f_dfu-usb_function.set_alt = dfu_set_alt;
diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c
index 58cad11..79e0c9c 100644
--- a/drivers/usb/gadget/g_dnl.c
+++ b/drivers/usb/gadget/g_dnl.c
@@ -143,6 +143,9 @@ static int g_dnl_config_register(struct usb_composite_dev 
*cdev)
config-bConfigurationValue = CONFIGURATION_NUMBER;
config-iConfiguration = STRING_USBDOWN;
config-bind = g_dnl_do_config;
+#if defined(CONFIG_DFU_FULLSPEED)
+   config-fullspeed = 1;
+#endif

return usb_add_config(cdev, config);
  }



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


Re: [U-Boot] [PATCH] watchdog disable for socfpga

2014-09-09 Thread Marek Vasut
On Tuesday, September 09, 2014 at 03:09:48 PM, Pavel Machek wrote:
 On Tue 2014-09-09 14:31:46, Marek Vasut wrote:
  On Tuesday, September 09, 2014 at 02:30:01 PM, Pavel Machek wrote:
   On Tue 2014-09-09 14:20:23, Marek Vasut wrote:
On Tuesday, September 09, 2014 at 02:03:28 PM, Pavel Machek wrote:
 This adds watchdog disable. It is neccessary for running Linux
 kernel.

Why do we not enable WDT in Linux instead ? Also, I recall there was
a call to explicitly enable the L4 watchdog, so why do we not get
rid of that instead ?
   
   Booting linux kernel without watchdog support should be valid
   operation.
  
  If you watchdog is enabled, then not so much. Also, why can we not just
  disable the WDT in U-Boot instead using a config option ?
 
 It seems that watchdog is enabled by default (by hardware or SPL), so
 yes, watchdog disable operation (and this patch) in u-boot is a good
 idea.

OK.

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/2] imx: ventana: Add pci fixup for PLX switch

2014-09-09 Thread Stefano Babic
On 21/08/2014 08:37, Tim Harvey wrote:
 On Thu, Aug 7, 2014 at 10:49 PM, Tim Harvey thar...@gateworks.com wrote:
 Many of the Gateworks Ventana boards have a PLX PCIe switch where GPIO
 on the switch is used as the PERST# of the downstream ports. In the Linux
 kernel there is a PCI fixup that asserts these properly when the upstream
 port is enumerated.

 The first patch in this serial adds a board_pci_fixup_dev function define
 to allow hooking similarly to the kernel pci fixup functionality.

 The second patch uses the board_pci_fixup_dev function to perform the
 GPIO configuration on the switch for the downstream ports.

 Tim Harvey (2):
   pci: add support for board_pci_fixup_dev function
   imx: ventana: add pci fixup for PLX PEX860x switch GPIO

  board/gateworks/gw_ventana/gw_ventana.c | 30 ++
  drivers/pci/pci.c   |  4 
  include/configs/gw_ventana.h|  1 +
  include/pci.h   |  7 +++
  4 files changed, 42 insertions(+)

 --
 1.8.3.2

 
 Any comments on these? I'm wondering if there is someone else I should
 be including with regards to reviewing pci changes.
 

No comment, and I do not see any bad influence on other boards - I am
going to apply them.

Regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] pci: add support for board_pci_fixup_dev function

2014-09-09 Thread Stefano Babic
Hi Tim,

On 08/08/2014 07:49, Tim Harvey wrote:
 Some board-level drivers may wish to have per-device fixup functions
 for PCI devices.
 
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] imx: ventana: add pci fixup for PLX PEX860x switch GPIO

2014-09-09 Thread Stefano Babic
Hi Tim,

On 08/08/2014 07:49, Tim Harvey wrote:
 Most Gateworks Ventana boards use a PLX PEX860x PCIe switch for PCIe 
 expansion.
 These boards use GPIO on the PLX device as PERST# for the downstream ports
 thus we assert this when the PLX is enumerated.
 
 Signed-off-by: Tim Harvey thar...@gateworks.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mtd: nand: am335x: Fix 'bit-flip' errors in SPL

2014-09-09 Thread Rostislav Lisovy
OMAP GPMC driver used with some NAND Flash devices
(e.g. Spansion S34ML08G1) causes that U-boot shows
hundreds of 'nand: bit-flip corrected' error messages.
Possible cause was discussed in the mailinglist thread:
http://lists.denx.de/pipermail/u-boot/2014-April/177508.html

The issue was partially fixed with the cc81a5291910d7a.git
however this has to be done to fix the SPL.

The original author of the code is Belisko Marek
marek.beli...@gmail.com

Signed-off-by: Rostislav Lisovy lis...@merica.cz
---
 drivers/mtd/nand/am335x_spl_bch.c | 12 
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/mtd/nand/am335x_spl_bch.c 
b/drivers/mtd/nand/am335x_spl_bch.c
index ce65d8e..bf8b2ee 100644
--- a/drivers/mtd/nand/am335x_spl_bch.c
+++ b/drivers/mtd/nand/am335x_spl_bch.c
@@ -64,14 +64,18 @@ static int nand_command(int block, int page, uint32_t offs,
   NAND_CTRL_ALE | NAND_CTRL_CHANGE); /* A[7:0] */
hwctrl(nand_info[0], (offs  8)  0xff, NAND_CTRL_ALE); /* A[11:9] */
/* Row address */
-   hwctrl(nand_info[0], (page_addr  0xff), NAND_CTRL_ALE); /* A[19:12] */
-   hwctrl(nand_info[0], ((page_addr  8)  0xff),
+   if (cmd != NAND_CMD_RNDOUT) {
+   hwctrl(nand_info[0], (page_addr  0xff),
+  NAND_CTRL_ALE); /* A[19:12] */
+   hwctrl(nand_info[0], ((page_addr  8)  0xff),
   NAND_CTRL_ALE); /* A[27:20] */
 #ifdef CONFIG_SYS_NAND_5_ADDR_CYCLE
-   /* One more address cycle for devices  128MiB */
-   hwctrl(nand_info[0], (page_addr  16)  0x0f,
+   /* One more address cycle for devices  128MiB */
+   hwctrl(nand_info[0], (page_addr  16)  0x0f,
   NAND_CTRL_ALE); /* A[31:28] */
 #endif
+   }
+
hwctrl(nand_info[0], NAND_CMD_NONE, NAND_NCE | NAND_CTRL_CHANGE);
 
if (cmd == NAND_CMD_READ0) {
-- 
1.9.1

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


Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only

2014-09-09 Thread Heiko Schocher

Hello Lukasz,

Am 09.09.2014 15:43, schrieb Lukasz Majewski:

Hi Heiko,


This patchserie adds the new config option CONFIG_DFU_FULLSPEED.


Is there any special reason to support Full Speed (12 Mbit/sec - USB
1.1) and not rely solely on the High Speed (USB 2.0) as we do now?


If this is enabled DFU uses fullspeed only. This is used on the
siemens boards.


Is there any DFU problem with the mentioned Siemens board, that we must
use USB 1.1?

I'd also appreciate more verbose rationale for this patch series - why
this change is needed? What is the expected improvement?


They have a USB HW protection circuit and therefore they only allowed
to use fullspeed ...

@Lukas Stockman: Maybe you could answer this better than me?

bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6

2014-09-09 Thread Stefano Babic
On 01/09/2014 14:56, Fabio Estevam wrote:
 According to mx53 and mx6 reference manuals:
 
 The maximum size of the DCD limited to 1768 bytes.
 
 As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and
 excluding the first entry, which is the header leads to 220 as the maximum
 number for DCD size.
 
 Reported-by: Jonas Karlsson jonas.d.karls...@gmail.com 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 0/2] usb: dfu: am335x: allow dfu in fullspeed mode only

2014-09-09 Thread Lukasz Majewski
Hi Bin,

 Lukasz,
 
 On 09/09/2014 08:43 AM, Lukasz Majewski wrote:
  Hi Heiko,
 
  This patchserie adds the new config option CONFIG_DFU_FULLSPEED.
 
  Is there any special reason to support Full Speed (12 Mbit/sec - USB
  1.1) and not rely solely on the High Speed (USB 2.0) as we do now?
 
 
 The drivers must support full-speed anyway based on the USB2.0 Specs.
 
 I have seen many cases in which only full-speed is suitable due to 
 either board design or specific USB use case.
 
 One example is that to better fit to the production firmware flashing 
 automation, a board does not use standard USB receptacle or plug, but
 a four touch pads on the PCB, so that the robot can grab the board
 and touch the pads to download the firmware. In this case signal
 integrity is a concern, but not download speed.

Thanks for example. 

Best regards,
Lukasz

 
 Regards,
 -Bin.
 
  If this is enabled DFU uses fullspeed only. This is used on the
  siemens boards.
 
  Is there any DFU problem with the mentioned Siemens board, that we
  must use USB 1.1?
 
  I'd also appreciate more verbose rationale for this patch series -
  why this change is needed? What is the expected improvement?
 
  Thanks in advance.
 
 
  Cc: Tom Rini tr...@ti.com
  Cc: Lukasz Majewski l.majew...@samsung.com
  Cc: Marek Vasut ma...@denx.de
  Cc: Liu Bin b-...@ti.com
  Cc: Lukas Stockmann lukas.stockm...@siemens.com
 
  Heiko Schocher (2):
 usb: dfu: add config option to use in dfu mode fullspeed only
 arm: am335x: siemens board use in DFU mode fullspeed only
 
README | 3 +++
drivers/usb/gadget/f_dfu.c | 3 +++
drivers/usb/gadget/g_dnl.c | 3 +++
include/configs/siemens-am33x-common.h | 3 ++-
4 files changed, 11 insertions(+), 1 deletion(-)
 
 
 
 
 



-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 1/2] usb: dfu: add config option to use in dfu mode fullspeed only

2014-09-09 Thread Lukasz Majewski
Hi Bin,

 Heiko,
 
 On 09/09/2014 07:37 AM, Heiko Schocher wrote:
  add the new config option CONFIG_DFU_FULLSPEED. With this
  option enabled, DFU uses fullspeed only.
 
 Can we not introduce the new config option but check the gadget
 driver speed in runtime as what the ether gadget driver does?
 
 I don't want two config options (CONFIG_USB_GADGET_DUALSPEED and 
 CONFIG_DFU_FULLSPEED) to control one feature, as in your patch 2/2.

+1

 
 Regards,
 -Bin.
 
 
  Signed-off-by: Heiko Schocher h...@denx.de
  Cc: Tom Rini tr...@ti.com
  Cc: Lukasz Majewski l.majew...@samsung.com
  Cc: Marek Vasut ma...@denx.de
  Cc: Liu Bin b-...@ti.com
  Cc: Lukas Stockmann lukas.stockm...@siemens.com
  ---
README | 3 +++
drivers/usb/gadget/f_dfu.c | 3 +++
drivers/usb/gadget/g_dnl.c | 3 +++
3 files changed, 9 insertions(+)
 
  diff --git a/README b/README
  index 0a0f528..1413392 100644
  --- a/README
  +++ b/README
  @@ -1607,6 +1607,9 @@ The following options need to be configured:
  entering dfuMANIFEST state. Host waits this
  timeout, before sending again an USB request to the device.
 
  +   CONFIG_DFU_FULLSPEED
  +   use for the dfu functionality fullspeed only.
  +
- USB Device Android Fastboot support:
  CONFIG_CMD_FASTBOOT
  This enables the command fastboot which enables
  the Android diff --git a/drivers/usb/gadget/f_dfu.c
  b/drivers/usb/gadget/f_dfu.c index 1145aab..dfa9f3b 100644
  --- a/drivers/usb/gadget/f_dfu.c
  +++ b/drivers/usb/gadget/f_dfu.c
  @@ -238,6 +238,7 @@ static inline void to_dfu_mode(struct f_dfu
  *f_dfu) {
  f_dfu-usb_function.strings = dfu_strings;
  f_dfu-usb_function.hs_descriptors = f_dfu-function;
  +   f_dfu-usb_function.descriptors = f_dfu-function;
  f_dfu-dfu_state = DFU_STATE_dfuIDLE;
}
 
  @@ -245,6 +246,7 @@ static inline void to_runtime_mode(struct f_dfu
  *f_dfu) {
  f_dfu-usb_function.strings = NULL;
  f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
  +   f_dfu-usb_function.descriptors = dfu_runtime_descs;
}
 
static int handle_upload(struct usb_request *req, u16 len)
  @@ -809,6 +811,7 @@ static int dfu_bind_config(struct
  usb_configuration *c) return -ENOMEM;
  f_dfu-usb_function.name = dfu;
  f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
  +   f_dfu-usb_function.descriptors = dfu_runtime_descs;
  f_dfu-usb_function.bind = dfu_bind;
  f_dfu-usb_function.unbind = dfu_unbind;
  f_dfu-usb_function.set_alt = dfu_set_alt;
  diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c
  index 58cad11..79e0c9c 100644
  --- a/drivers/usb/gadget/g_dnl.c
  +++ b/drivers/usb/gadget/g_dnl.c
  @@ -143,6 +143,9 @@ static int g_dnl_config_register(struct
  usb_composite_dev *cdev) config-bConfigurationValue =
  CONFIGURATION_NUMBER; config-iConfiguration = STRING_USBDOWN;
  config-bind = g_dnl_do_config;
  +#if defined(CONFIG_DFU_FULLSPEED)
  +   config-fullspeed = 1;
  +#endif
 
  return usb_add_config(cdev, config);
}
 
 



-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: ventana: Avoid undefined behaviour

2014-09-09 Thread Stefano Babic
On 22/08/2014 09:46, Thierry Reding wrote:
 From: Thierry Reding tred...@nvidia.com
 
 The leds array within struct ventana has space for 3 elements, but the
 setup_board_gpio() function tries to set up 4 GPIOs for LEDs. Recent
 versions of GCC complain about that:
 
   board/gateworks/gw_ventana/gw_ventana.c: In function 'setup_board_gpio':
   board/gateworks/gw_ventana/gw_ventana.c:987:27: warning: iteration 3u 
 invokes undefined behavior [-Waggressive-loop-optimizations]
  if (gpio_cfg[board].leds[i])
  ^
   board/gateworks/gw_ventana/gw_ventana.c:986:2: note: containing loop
 for (i = 0; i  4; i++) {
 ^
 
 Fix this by making the upper bound of the loop match the array size.
 
 Signed-off-by: Thierry Reding tred...@nvidia.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] iMX6: Disable the L2 before chaning the PL310 latency

2014-09-09 Thread Stefano Babic
On 20/08/2014 11:18, Ye.Li wrote:
 From: Ye.Li ye...@freescale.com
 
 The Latency parameters of PL310 Tag RAM latency control register and
 Data RAM Latency control register are set in L2 cache enable. And
 setting these registers must have PL310 NOT enabled.
 
 But when using Plugin mode boot, the PL310 is enabled by bootrom.
 The patch disables the PL310 before applying this setting.
 
 Signed-off-by: Ye.Li ye...@freescale.com
 ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 7/8] Makefile: default to cc for host compiler

2014-09-09 Thread Albert ARIBAUD
Hi Jeroen, Masahiro,

On Thu, 31 Jul 2014 19:01:22 +0900, Masahiro Yamada
yamad...@jp.panasonic.com wrote:

   HOSTCXX  = g++
   HOSTCFLAGS   = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer

 For consistency,
 
 HOSTCXX = c++
 
 ?

So, Jeroen, what is your pick ? Will you send a v3 7/8, or are you
sticking with g++?

(or if everyone agrees, I could to the change to v2 7/8 when applying
it, and add a comment about it in the commit message.)

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] usb: eth: add ASIX AX88179 DRIVER

2014-09-09 Thread Marek Vasut
On Wednesday, September 03, 2014 at 04:31:20 PM, Rene Griessl wrote:
 changes in v2:
   -added usb_ether.h to change list
   -added 2nd patch to enable driver for arndale board
 
 Signed-off-by: Rene Griessl rgrie...@cit-ec.uni-bielefeld.de

I see that in Linux, there is asix_common.c stuff. Can this driver share any 
parts with drivers/net/ax88* ?

 ---
  drivers/usb/eth/Makefile|   3 +
  drivers/usb/eth/asix88179.c | 641
  drivers/usb/eth/usb_ether.c |
   7 +
  include/usb_ether.h |   6 +
  4 files changed, 657 insertions(+)
  create mode 100644 drivers/usb/eth/asix88179.c
 
 diff --git a/drivers/usb/eth/Makefile b/drivers/usb/eth/Makefile
 index 94551c4..fad4acd 100644
 --- a/drivers/usb/eth/Makefile
 +++ b/drivers/usb/eth/Makefile
 @@ -8,5 +8,8 @@ obj-$(CONFIG_USB_HOST_ETHER) += usb_ether.o
  ifdef CONFIG_USB_ETHER_ASIX
  obj-y += asix.o
  endif
 +ifdef CONFIG_USB_ETHER_ASIX_88179
 +obj-y += asix88179.o
 +endif

This should be obj-$(CONFIG) as seen below. Fix the asix one in a separate 
patch please.

[...]

 +/* ASIX AX88179 based USB 3.0 Ethernet Devices */
 +#define AX88179_PHY_ID   0x03
 +#define AX_EEPROM_LEN0x100
 +#define AX88179_EEPROM_MAGIC 0x17900b95
 +#define AX_MCAST_FLTSIZE 8
 +#define AX_MAX_MCAST 64
 +#define AX_INT_PPLS_LINK ((u32)BIT(16))

The u32 cast is not needed. Also, please drop the BIT() macro, it's just 
obfuscating the code, just use (1  16) instead. Please fix globally.

 +#define AX_RXHDR_L4_TYPE_MASK0x1c
 +#define AX_RXHDR_L4_TYPE_UDP 4
 +#define AX_RXHDR_L4_TYPE_TCP 16
 +#define AX_RXHDR_L3CSUM_ERR  2
 +#define AX_RXHDR_L4CSUM_ERR  1
 +#define AX_RXHDR_CRC_ERR ((u32)BIT(29))
 +#define AX_RXHDR_DROP_ERR((u32)BIT(31))
 +#define AX_ACCESS_MAC0x01
 +#define AX_ACCESS_PHY0x02
 +#define AX_ACCESS_EEPROM 0x04
 +#define AX_ACCESS_EFUS   0x05
 +#define AX_PAUSE_WATERLVL_HIGH   0x54
 +#define AX_PAUSE_WATERLVL_LOW0x55

[...]

 +static inline int asix_get_phy_addr(struct ueth_data *dev)
 +{
 + ALLOC_CACHE_ALIGN_BUFFER(u8, buf, 2);
 + int ret = -1;
 + if (dev-pusb_dev-descriptor.idProduct == 0x1790) {
 + ret = asix_read_cmd(dev, AX_ACCESS_MAC, 0x10, 6, 6, buf);
 + debug(asix_get_phy_addr() returning 
0x%02x%02x%02x%02x%02x%02x\n,
 +   buf[0], buf[1], buf[2], buf[3], buf[4], buf[5]);
 + }
 + else {
 + }

Uh, this check needs some rework, right ? Also, you want to lint your patches 
with ./scripts/checkpatch.pl tool before resubmitting.

 + return ret;
 +}
 +
 +
 +static int asix_read_mac(struct eth_device *eth)
 +{
 + struct ueth_data *dev = (struct ueth_data *)eth-priv;
 + ALLOC_CACHE_ALIGN_BUFFER(unsigned char, buf, ETH_ALEN);
 +
 + if (dev-pusb_dev-descriptor.idProduct == 0x1790) {
 + asix_read_cmd(dev, AX_ACCESS_MAC, 0x10, 6, 6, buf);
 + debug(asix_read_mac() returning 0x%02x%02x%02x%02x%02x%02x\n,
 +   buf[0], buf[1], buf[2], buf[3], buf[4], buf[5]);
 + memcpy(eth-enetaddr, buf, ETH_ALEN);
 + }
 + return 0;
 +}
 +
 +
 +
 +static int asix_basic_reset(struct ueth_data *dev)
 +{
 + ALLOC_CACHE_ALIGN_BUFFER(u8, buf, 6);

Why does the buffer need to be aligned here ? It's just a buffer that is not 
used for DMA, no ?

 + u16 *tmp16;

Is it because you were getting unaligned accesses , since when the buffer 
itself 
was not aligned and you did cast it to u16, the CPU triggered unaligned access ?

 + u8 *tmp;
[...]
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 1/2] usb: dfu: add config option to use in dfu mode fullspeed only

2014-09-09 Thread Heiko Schocher

Hello Bin Liu,

Am 09.09.2014 16:09, schrieb Bin Liu:

Heiko,

On 09/09/2014 07:37 AM, Heiko Schocher wrote:

add the new config option CONFIG_DFU_FULLSPEED. With this
option enabled, DFU uses fullspeed only.


Can we not introduce the new config option but check the gadget driver speed in 
runtime as what the ether gadget driver does?
I don't want two config options (CONFIG_USB_GADGET_DUALSPEED and 
CONFIG_DFU_FULLSPEED) to control one feature, as in your patch 2/2.


Hmm.. I am not sure, if I understand you correct. I could use
CONFIG_USB_GADGET_DUALSPEED instead the new config option
for setting config-fullspeed = 1; in g_dnl_config_register()

If you mean this, yes, thats would be a good change...

bye,
Heiko


Regards,
-Bin.



Signed-off-by: Heiko Schocher h...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com
---
README | 3 +++
drivers/usb/gadget/f_dfu.c | 3 +++
drivers/usb/gadget/g_dnl.c | 3 +++
3 files changed, 9 insertions(+)

diff --git a/README b/README
index 0a0f528..1413392 100644
--- a/README
+++ b/README
@@ -1607,6 +1607,9 @@ The following options need to be configured:
entering dfuMANIFEST state. Host waits this timeout, before
sending again an USB request to the device.

+ CONFIG_DFU_FULLSPEED
+ use for the dfu functionality fullspeed only.
+
- USB Device Android Fastboot support:
CONFIG_CMD_FASTBOOT
This enables the command fastboot which enables the Android
diff --git a/drivers/usb/gadget/f_dfu.c b/drivers/usb/gadget/f_dfu.c
index 1145aab..dfa9f3b 100644
--- a/drivers/usb/gadget/f_dfu.c
+++ b/drivers/usb/gadget/f_dfu.c
@@ -238,6 +238,7 @@ static inline void to_dfu_mode(struct f_dfu *f_dfu)
{
f_dfu-usb_function.strings = dfu_strings;
f_dfu-usb_function.hs_descriptors = f_dfu-function;
+ f_dfu-usb_function.descriptors = f_dfu-function;
f_dfu-dfu_state = DFU_STATE_dfuIDLE;
}

@@ -245,6 +246,7 @@ static inline void to_runtime_mode(struct f_dfu *f_dfu)
{
f_dfu-usb_function.strings = NULL;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+ f_dfu-usb_function.descriptors = dfu_runtime_descs;
}

static int handle_upload(struct usb_request *req, u16 len)
@@ -809,6 +811,7 @@ static int dfu_bind_config(struct usb_configuration *c)
return -ENOMEM;
f_dfu-usb_function.name = dfu;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+ f_dfu-usb_function.descriptors = dfu_runtime_descs;
f_dfu-usb_function.bind = dfu_bind;
f_dfu-usb_function.unbind = dfu_unbind;
f_dfu-usb_function.set_alt = dfu_set_alt;
diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c
index 58cad11..79e0c9c 100644
--- a/drivers/usb/gadget/g_dnl.c
+++ b/drivers/usb/gadget/g_dnl.c
@@ -143,6 +143,9 @@ static int g_dnl_config_register(struct usb_composite_dev 
*cdev)
config-bConfigurationValue = CONFIGURATION_NUMBER;
config-iConfiguration = STRING_USBDOWN;
config-bind = g_dnl_do_config;
+#if defined(CONFIG_DFU_FULLSPEED)
+ config-fullspeed = 1;
+#endif

return usb_add_config(cdev, config);
}







--
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imximage: Fix imximage IVT bug for EIM-NOR boot

2014-09-09 Thread Stefano Babic
Hi Ye,

On 21/08/2014 09:04, Li Ye-B37916 wrote:

 There are two minor impacts if putting the check in the function
 parse_cfg_cmd:
 
 1. The imximage_version must be got before parsing a CMD_BOOT_FROM
 command. This compels the CMD_IMAGE_VERSION preceding the
 CMD_BOOT_FROM in script.  imximage_init_loadsize is only needed by V2
 version.

I know abot this, and this is a minor impact. It is common that a
version number of a document must be set first in the document.

 
 2. Since the imximage_generate function already implements post
 fixing for imximage_init_loadsize, this post fixing needs be
 removed.
 
 Actually, putting the check in the parsing or post the parsing are ok
 for me. Both can resolve the issue. The comment for
 FLASH_LOADSIZE_NOR sources from iMX reference manual, it is
 correct.

Well, I think it is clear for both of us because we worked with i-IM
images. But setting a size to 0x0 with the comment that this is the size
of whole image can be confusing (I think I was the author of this
comment, so I am guilty for that). Anyway, it is a very minor issue.

I hope that people will read carefully the manual together with code.

Anyway, comments are very minor issues - I will apply the patch.

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 1/2] usb: dfu: add config option to use in dfu mode fullspeed only

2014-09-09 Thread Bin Liu

Heiko,

On 09/09/2014 09:37 AM, Heiko Schocher wrote:

Hello Bin Liu,

Am 09.09.2014 16:09, schrieb Bin Liu:

Heiko,

On 09/09/2014 07:37 AM, Heiko Schocher wrote:

add the new config option CONFIG_DFU_FULLSPEED. With this
option enabled, DFU uses fullspeed only.


Can we not introduce the new config option but check the gadget driver
speed in runtime as what the ether gadget driver does?
I don't want two config options (CONFIG_USB_GADGET_DUALSPEED and
CONFIG_DFU_FULLSPEED) to control one feature, as in your patch 2/2.


Hmm.. I am not sure, if I understand you correct. I could use
CONFIG_USB_GADGET_DUALSPEED instead the new config option
for setting config-fullspeed = 1; in g_dnl_config_register()

If you mean this, yes, thats would be a good change...


No, I did not mean this build time macro, but was thinking you can check 
g-speed at the same place in runtime.


I think you can refer to ether.c, which supports full-speed but does not 
use any macro.


Regards,
-Bin.



bye,
Heiko


Regards,
-Bin.



Signed-off-by: Heiko Schocher h...@denx.de
Cc: Tom Rini tr...@ti.com
Cc: Lukasz Majewski l.majew...@samsung.com
Cc: Marek Vasut ma...@denx.de
Cc: Liu Bin b-...@ti.com
Cc: Lukas Stockmann lukas.stockm...@siemens.com
---
README | 3 +++
drivers/usb/gadget/f_dfu.c | 3 +++
drivers/usb/gadget/g_dnl.c | 3 +++
3 files changed, 9 insertions(+)

diff --git a/README b/README
index 0a0f528..1413392 100644
--- a/README
+++ b/README
@@ -1607,6 +1607,9 @@ The following options need to be configured:
entering dfuMANIFEST state. Host waits this timeout, before
sending again an USB request to the device.

+ CONFIG_DFU_FULLSPEED
+ use for the dfu functionality fullspeed only.
+
- USB Device Android Fastboot support:
CONFIG_CMD_FASTBOOT
This enables the command fastboot which enables the Android
diff --git a/drivers/usb/gadget/f_dfu.c b/drivers/usb/gadget/f_dfu.c
index 1145aab..dfa9f3b 100644
--- a/drivers/usb/gadget/f_dfu.c
+++ b/drivers/usb/gadget/f_dfu.c
@@ -238,6 +238,7 @@ static inline void to_dfu_mode(struct f_dfu *f_dfu)
{
f_dfu-usb_function.strings = dfu_strings;
f_dfu-usb_function.hs_descriptors = f_dfu-function;
+ f_dfu-usb_function.descriptors = f_dfu-function;
f_dfu-dfu_state = DFU_STATE_dfuIDLE;
}

@@ -245,6 +246,7 @@ static inline void to_runtime_mode(struct f_dfu
*f_dfu)
{
f_dfu-usb_function.strings = NULL;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+ f_dfu-usb_function.descriptors = dfu_runtime_descs;
}

static int handle_upload(struct usb_request *req, u16 len)
@@ -809,6 +811,7 @@ static int dfu_bind_config(struct
usb_configuration *c)
return -ENOMEM;
f_dfu-usb_function.name = dfu;
f_dfu-usb_function.hs_descriptors = dfu_runtime_descs;
+ f_dfu-usb_function.descriptors = dfu_runtime_descs;
f_dfu-usb_function.bind = dfu_bind;
f_dfu-usb_function.unbind = dfu_unbind;
f_dfu-usb_function.set_alt = dfu_set_alt;
diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c
index 58cad11..79e0c9c 100644
--- a/drivers/usb/gadget/g_dnl.c
+++ b/drivers/usb/gadget/g_dnl.c
@@ -143,6 +143,9 @@ static int g_dnl_config_register(struct
usb_composite_dev *cdev)
config-bConfigurationValue = CONFIGURATION_NUMBER;
config-iConfiguration = STRING_USBDOWN;
config-bind = g_dnl_do_config;
+#if defined(CONFIG_DFU_FULLSPEED)
+ config-fullspeed = 1;
+#endif

return usb_add_config(cdev, config);
}









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


[U-Boot] [PATCH] imx6: add Bachmann OT1200 board

2014-09-09 Thread Christian Gmeiner
This patch adds support for the OT1200 series of devices.

Following components are used in u-boot:
+ ethernet
+ i2c
+ emmc
+ gpio

The main difference between the different models of the OT1200
series is how ethernet is connected (directly to a switch or
to a normal phy).

Signed-off-by: Christian Gmeiner christian.gmei...@gmail.com
---
 arch/arm/Kconfig  |   4 +
 board/bachmann/ot1200/Kconfig |  23 
 board/bachmann/ot1200/MAINTAINERS |   6 +
 board/bachmann/ot1200/Makefile|   9 ++
 board/bachmann/ot1200/ot1200.c| 262 ++
 configs/ot1200_defconfig  |   3 +
 include/configs/ot1200.h  | 195 
 7 files changed, 502 insertions(+)
 create mode 100644 board/bachmann/ot1200/Kconfig
 create mode 100644 board/bachmann/ot1200/MAINTAINERS
 create mode 100644 board/bachmann/ot1200/Makefile
 create mode 100644 board/bachmann/ot1200/ot1200.c
 create mode 100644 configs/ot1200_defconfig
 create mode 100644 include/configs/ot1200.h

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 22f0f09..c93dcb6 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -412,6 +412,9 @@ config TARGET_HUMMINGBOARD
 config TARGET_TQMA6
bool TQ Systems TQMa6 board
 
+config TARGET_OT1200
+   bool Bachmann OT1200
+
 config OMAP34XX
bool OMAP34XX SoC
 
@@ -564,6 +567,7 @@ source board/atmel/at91sam9rlek/Kconfig
 source board/atmel/at91sam9x5ek/Kconfig
 source board/atmel/sama5d3_xplained/Kconfig
 source board/atmel/sama5d3xek/Kconfig
+source board/bachmann/ot1200/Kconfig
 source board/balloon3/Kconfig
 source board/barco/titanium/Kconfig
 source board/bluegiga/apx4devkit/Kconfig
diff --git a/board/bachmann/ot1200/Kconfig b/board/bachmann/ot1200/Kconfig
new file mode 100644
index 000..55a825d
--- /dev/null
+++ b/board/bachmann/ot1200/Kconfig
@@ -0,0 +1,23 @@
+if TARGET_OT1200
+
+config SYS_CPU
+   string
+   default armv7
+
+config SYS_BOARD
+   string
+   default ot1200
+
+config SYS_VENDOR
+   string
+   default bachmann
+
+config SYS_SOC
+   string
+   default mx6
+
+config SYS_CONFIG_NAME
+   string
+   default ot1200
+
+endif
diff --git a/board/bachmann/ot1200/MAINTAINERS 
b/board/bachmann/ot1200/MAINTAINERS
new file mode 100644
index 000..ad75c24
--- /dev/null
+++ b/board/bachmann/ot1200/MAINTAINERS
@@ -0,0 +1,6 @@
+BACHMANN ELECTRONIC OT1200 BOARD
+M: Christian Gmeiner christian.gmei...@gmail.com
+S: Maintained
+F: board/bachmann/ot1200
+F: include/configs/ot1200.h
+F: configs/ot1200*_defconfig
diff --git a/board/bachmann/ot1200/Makefile b/board/bachmann/ot1200/Makefile
new file mode 100644
index 000..1bd42e8
--- /dev/null
+++ b/board/bachmann/ot1200/Makefile
@@ -0,0 +1,9 @@
+#
+# Copyright (C) 2012-2013, Guennadi Liakhovetski l...@denx.de
+# (C) Copyright 2012-2013 Freescale Semiconductor, Inc.
+# Copyright (C) 2013, Boundary Devices i...@boundarydevices.com
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := ot1200.o
diff --git a/board/bachmann/ot1200/ot1200.c b/board/bachmann/ot1200/ot1200.c
new file mode 100644
index 000..1c04f45
--- /dev/null
+++ b/board/bachmann/ot1200/ot1200.c
@@ -0,0 +1,262 @@
+/*
+ * Copyright (C) 2010-2013 Freescale Semiconductor, Inc.
+ * Copyright (C) 2014, Bachmann electronic GmbH
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#include common.h
+#include asm/io.h
+#include asm/arch/clock.h
+#include asm/arch/imx-regs.h
+#include asm/arch/iomux.h
+#include malloc.h
+#include asm/arch/mx6-pins.h
+#include asm/imx-common/iomux-v3.h
+#include asm/imx-common/mxc_i2c.h
+#include asm/imx-common/sata.h
+#include asm/imx-common/boot_mode.h
+#include asm/arch/crm_regs.h
+#include mmc.h
+#include fsl_esdhc.h
+#include netdev.h
+#include i2c.h
+#include pca953x.h
+#include asm/gpio.h
+#include phy.h
+
+DECLARE_GLOBAL_DATA_PTR;
+
+#define UART_PAD_CTRL  (PAD_CTL_PUS_100K_UP |  \
+   PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm | \
+   PAD_CTL_SRE_FAST  | PAD_CTL_HYS)
+
+#define USDHC_PAD_CTRL (PAD_CTL_PUS_47K_UP |   \
+   PAD_CTL_SPEED_LOW | PAD_CTL_DSE_80ohm | \
+   PAD_CTL_SRE_FAST  | PAD_CTL_HYS)
+
+#define ENET_PAD_CTRL  (PAD_CTL_PUS_100K_UP |  \
+   PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm | PAD_CTL_HYS)
+
+#define SPI_PAD_CTRL (PAD_CTL_HYS | PAD_CTL_SPEED_MED |\
+   PAD_CTL_DSE_40ohm | PAD_CTL_SRE_FAST)
+
+#define I2C_PAD_CTRL   (PAD_CTL_PUS_100K_UP |  \
+   PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm | PAD_CTL_HYS |   \
+   PAD_CTL_ODE | PAD_CTL_SRE_FAST)
+
+#define OUTPUT_40OHM (PAD_CTL_SPEED_MED|PAD_CTL_DSE_40ohm)
+
+int dram_init(void)
+{
+   gd-ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE);
+
+   return 0;
+}
+
+static iomux_v3_cfg_t const uart1_pads[] = {
+   MX6_PAD_CSI0_DAT10__UART1_TX_DATA | MUX_PAD_CTRL(UART_PAD_CTRL),
+   

Re: [U-Boot] [PATCH] mx6: Fix ECSPI typo in soc_boot_modes

2014-09-09 Thread Stefano Babic
On 10/08/2014 19:03, picmas...@mail.bg wrote:
 From: Nikolay Dimitrov picmas...@mail.bg
 
 Signed-off-by: Nikolay Dimitrov picmas...@mail.bg
 Cc: Stefano Babic sba...@denx.de
 ---


Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] arm: vf610: lpuart: fix status register handling

2014-09-09 Thread Stefano Babic
On 19/08/2014 17:54, Stefan Agner wrote:
 The status register 1 (S1) is not writeable, hence we should not
 write it. In order to clear the RDRF flag we only need to read
 the data register.
 
 Also, when stressing U-Boot a lot with serial input, an overflow can
 occur which asserts the S1_OR flag (while not asserting the S1_RDRF
 flag). To clear this flag we again just need to read the data
 register, hence add this flag to the abort conditions for the while
 loop.
 
 Insert a compiler barrier to make sure reading the data register
 gets executed after reading the status register.
 
 Signed-off-by: Stefan Agner ste...@agner.ch
 ---

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] arm: vf610: lpuart: disable FIFO on initializaton

2014-09-09 Thread Stefano Babic
On 19/08/2014 17:54, Stefan Agner wrote:
 UART does not use the UART FIFO, but we should also not rely that
 the UART FIFO is diabled by default. For instance, when loading
 U-Boot using the boot ROMs serial downloader protocol over UART,
 FIFO is enabled at U-Boot start time.
 
 This patch disables the RX and TX FIFO, sets back their thresholds
 and flushes them.
 
 Signed-off-by: Stefan Agner ste...@agner.ch
 ---

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mx6dlsabresd: Use its own DCD table

2014-09-09 Thread Stefano Babic
On 21/08/2014 15:02, Fabio Estevam wrote:
 Currently mx6dlsabresd shares the same DCD settings with the nitrogen board.
 
 Provide a DCD configuration file specific to mx6dlsabresd with the settings
 recommended by the Freescale hardware team.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx: nitrogen6x: Replace 'fatload' by 'load' command in env settings to be filesystem independent

2014-09-09 Thread Stefano Babic
On 26/08/2014 12:05, Guillaume GARDET wrote:
 nitrogen6x.h file defines CONFIG_CMD_FS_GENERIC, so we are able to use generic
 'load' command instead of 'fatload'. It allows to use ext filesystem and keep
 compatibilty with fat filesystem.
 
 Signed-off-by: Guillaume GARDET guillaume.gar...@free.fr
 Cc: Stefano Babic sba...@denx.de
  
 ---


Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH V4 00/19] Introduce cm-fx6 board

2014-09-09 Thread Nikita Kiryanov



On 09/09/14 16:45, Stefano Babic wrote:

Hi Nikita,

On 31/08/2014 10:09, Nikita Kiryanov wrote:

Hi Tom, Stefano,

We really want this series to make it into the next release (V1 was
submitted only a day after the merge window closed). If there are no
other comments, can we maybe start applying it?



Patch 1/19, 4/19 and 5/19 were already applied by Jagan - I do not merge
them into u-boot-imx.
I applied 2/19 after rebasing on current u-boot-imx.

I have nothing against the helper introduced in 3/19, but this touch
more boards - I will make a short check before applying it. However, it
is orthogonal and can be applied later.

I applied as well 6/19 through 19/19 into u-boot-imx - thanks !


Thanks!
I'm a little confused about what you meant by patch 3 though...
Jagan took patch number 3 (and 1, and 4), and I can see in your tree
that all 16 remaining patches are there. What patch were you referring
to?



Best regards,
Stefano Babic



--
Regards,
Nikita Kiryanov
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH V4 00/19] Introduce cm-fx6 board

2014-09-09 Thread Stefano Babic
Hi Nikita,

On 09/09/2014 17:00, Nikita Kiryanov wrote:
 
 
 On 09/09/14 16:45, Stefano Babic wrote:
 Hi Nikita,

 On 31/08/2014 10:09, Nikita Kiryanov wrote:
 Hi Tom, Stefano,

 We really want this series to make it into the next release (V1 was
 submitted only a day after the merge window closed). If there are no
 other comments, can we maybe start applying it?


 Patch 1/19, 4/19 and 5/19 were already applied by Jagan - I do not merge
 them into u-boot-imx.
 I applied 2/19 after rebasing on current u-boot-imx.

 I have nothing against the helper introduced in 3/19, but this touch
 more boards - I will make a short check before applying it. However, it
 is orthogonal and can be applied later.

 I applied as well 6/19 through 19/19 into u-boot-imx - thanks !
 
 Thanks!
 I'm a little confused about what you meant by patch 3 though...
 Jagan took patch number 3 (and 1, and 4),

I have not seen that Jagan picked up patch 3, I found only 1, 4 and 5.

 and I can see in your tree
 that all 16 remaining patches are there. What patch were you referring
 to?

Then all patches should be already merged and will flow into Tom's tree
at next pull request.

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH RESEND] vf610twr: Tune DDR initialization settings

2014-09-09 Thread Stefan Agner
Hi Albert,

The RESEND version of the patch is actually an updated version (maybe I
should have increased the version number?)

For me, that patch applies cleanly on U-Boot master

0b703dbcee7103f07804d0a4328d1593355c4324
patman: Fix detection of git version

Also I tested on U-Boot ARM master and next branch, applies without
errors for me.

--
Stefan

Am 2014-09-09 10:26, schrieb Albert ARIBAUD:
 Hi Stefan,
 
 On Sat,  6 Sep 2014 19:47:06 +0200, Stefan Agner ste...@agner.ch
 wrote:
 
 From: Anthony Felice tony.fel...@timesys.com

 Removed settings in unsupported register fields. They didn’t
 do anything, and in most cases, were not documented in the
 reference manual.

 Changed register settings to comply with JEDEC required values.

 Changed timing parameters because they included full clock
 periods that were doing nothing.

 Signed-off-by: Anthony Felice tony.fel...@timesys.com
 [rebased on v2014.10-rc2]
 Signed-off-by: Stefan Agner ste...@agner.ch
 ---
 As discuessed in the initial patch set this fixes a lot of wrong/
 undocummented access and it would be nice to have in the next U-Boot
 release. Verified the patchset after rebase again using memtester
 on Vybrid Tower.

  arch/arm/include/asm/arch-vf610/imx-regs.h| 49 +++---
  arch/arm/include/asm/arch-vf610/iomux-vf610.h | 44 +++--
  arch/arm/include/asm/imx-common/iomux-v3.h|  2 +
  board/freescale/vf610twr/vf610twr.c   | 94 
 +--
  4 files changed, 127 insertions(+), 62 deletions(-)

 diff --git a/arch/arm/include/asm/arch-vf610/imx-regs.h 
 b/arch/arm/include/asm/arch-vf610/imx-regs.h
 index bb00217..9d797db 100644
 --- a/arch/arm/include/asm/arch-vf610/imx-regs.h
 +++ b/arch/arm/include/asm/arch-vf610/imx-regs.h
 @@ -103,9 +103,11 @@
  /* DDRMC */
  #define DDRMC_PHY_DQ_TIMING 0x2613
  #define DDRMC_PHY_DQS_TIMING0x2615
 -#define DDRMC_PHY_CTRL  0x01210080
 +#define DDRMC_PHY_CTRL  0x0021
  #define DDRMC_PHY_MASTER_CTRL   0x0001012a
 -#define DDRMC_PHY_SLAVE_CTRL0x00012020
 +#define DDRMC_PHY_SLAVE_CTRL0x2000
 +#define DDRMC_PHY_OFF   0x
 +#define DDRMC_PHY_PROC_PAD_ODT  0x00010101

  #define DDRMC_PHY50_DDR3_MODE   (1  12)
  #define DDRMC_PHY50_EN_SW_HALF_CYCLE(1  8)
 @@ -138,7 +140,7 @@
  #define DDRMC_CR21_CCMAP_EN 1
  #define DDRMC_CR22_TDAL(v)  (((v)  0x3f)  16)
  #define DDRMC_CR23_BSTLEN(v)(((v)  0x7)  
 24)
 -#define DDRMC_CR23_TDLL(v)  ((v)  0xff)
 +#define DDRMC_CR23_TDLL(v)  ((v)  0x)
  #define DDRMC_CR24_TRP_AB(v)((v)  0x1f)
  #define DDRMC_CR25_TREF_EN  (1  16)
  #define DDRMC_CR26_TREF(v)  (((v)  0x)  16)
 @@ -151,7 +153,7 @@
  #define DDRMC_CR33_EN_QK_SREF   (1  16)
  #define DDRMC_CR34_CKSRX(v) (((v)  0xf)  16)
  #define DDRMC_CR34_CKSRE(v) (((v)  0xf)  8)
 -#define DDRMC_CR38_FREQ_CHG_EN  (1  8)
 +#define DDRMC_CR38_FREQ_CHG_EN(v)   (((v)  0x1)  8)
  #define DDRMC_CR39_PHY_INI_COM(v)   (((v)  0x)  16)
  #define DDRMC_CR39_PHY_INI_STA(v)   (((v)  0xff)  8)
  #define DDRMC_CR39_FRQ_CH_DLLOFF(v) ((v)  0x3)
 @@ -163,7 +165,7 @@
  #define DDRMC_CR67_ZQCS(v)  ((v)  0xfff)
  #define DDRMC_CR69_ZQ_ON_SREF_EX(v) (((v)  0xf)  8)
  #define DDRMC_CR70_REF_PER_ZQ(v)(v)
 -#define DDRMC_CR72_ZQCS_ROTATE  (1  24)
 +#define DDRMC_CR72_ZQCS_ROTATE(v)   (((v)  0x1)  24)
  #define DDRMC_CR73_APREBIT(v)   (((v)  0xf)  
 24)
  #define DDRMC_CR73_COL_DIFF(v)  (((v)  0x7)  
 16)
  #define DDRMC_CR73_ROW_DIFF(v)  (((v)  0x3)  
 8)
 @@ -182,9 +184,10 @@
  #define DDRMC_CR77_CS_MAP   (1  24)
  #define DDRMC_CR77_DI_RD_INTLEAVE   (1  8)
  #define DDRMC_CR77_SWAP_EN  1
 +#define DDRMC_CR78_Q_FULLNESS(v)(((v)  0x7)  24)
  #define DDRMC_CR78_BUR_ON_FLY_BIT(v)((v)  0xf)
 -#define DDRMC_CR79_CTLUPD_AREF  (1  24)
 -#define DDRMC_CR82_INT_MASK 0x1fff
 +#define DDRMC_CR79_CTLUPD_AREF(v)   (((v)  0x1)  24)
 +#define DDRMC_CR82_INT_MASK 0x1000
  #define DDRMC_CR87_ODT_WR_MAPCS0

Re: [U-Boot] [PATCH] mx6qsabreauto: Remove imx6q-sabreauto.dts

2014-09-09 Thread Stefano Babic
Hi Masahiro,

On 09/09/2014 07:11, Masahiro Yamada wrote:
 Hi Stefano,
 
 
 
 On Fri, 5 Sep 2014 16:26:48 -0300
 Otavio Salvador ota...@ossystems.com.br wrote:
 
 On Fri, Sep 5, 2014 at 3:36 PM, Fabio Estevam feste...@gmail.com wrote:
 From: Fabio Estevam fabio.este...@freescale.com

 Commit fa9c021632473 (mx6: add example DTB for mx6qsabreauto) introduced
 'imx6q-sabreauto.dts' but it adds no real value as the dts file only 
 contains
 the 'model' and 'compatible' strings.

 After this commit the final binary is also changed from 'u-boot.imx' to
 'u-boot-dtb.imx', which may confuse users.

 So revert it until a more complete and useful device tree could be provided.

 Signed-off-by: Fabio Estevam fabio.este...@freescale.com

 Acked-by: Otavio Salvador ota...@ossystems.com.br

 
 
 Could you pick up this trivial one and send a pull-req?
 
 I am moving CONFIG_OF_SEPARATE and CONFIG_DEFAULT_DEVICE_TREE
 to Kconfig.
 
 Ideally, this patch should come first.
 
 
 See this thread:
 http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/
 

I am preparing the PR - it comes soon.

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 1/3] mx6: imx-regs: Provide a structure for GPC registers

2014-09-09 Thread Stefano Babic
On 25/08/2014 19:26, Fabio Estevam wrote:
 Introduce a structure for accessing the General Power Controller block (GPC)
 registers.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3] env_mmc: correct fini partition to match init partition

2014-09-09 Thread Igor Grinberg
Hi Peter,

On 09/03/14 20:22, Peter A. Bigot wrote:
 The code to set the MMC partition uses an weak function to obtain the
 correct partition number.  Use that instead of the compile-time default
 when deciding whether it needs to switch back.
 
 Fixes: 6e7b7df4df43574 (env_mmc: support env partition setup in runtime)

It is sometimes also useful to Cc the original author of the patch.
Cc: Dmitry Lifshitz lifsh...@compulab.co.il

 Signed-off-by: Peter A. Bigot p...@pabigot.com
 ---
 V3:
 * Add Fixes line as requested
 
 V2:
 * Preserve desired behavior of avoiding diagnostic when no HW partition 
 supported
 * Supersedes https://patchwork.ozlabs.org/patch/385355/
 
  common/env_mmc.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/common/env_mmc.c b/common/env_mmc.c
 index a7621a8..14648e3 100644
 --- a/common/env_mmc.c
 +++ b/common/env_mmc.c
 @@ -113,7 +113,7 @@ static void fini_mmc_for_env(struct mmc *mmc)
  #ifdef CONFIG_SPL_BUILD
   dev = 0;
  #endif
 - if (CONFIG_SYS_MMC_ENV_PART != mmc-part_num)
 + if (mmc_get_env_part(mmc) != mmc-part_num)
   mmc_switch_part(dev, mmc-part_num);
  #endif
  }
 

-- 
Regards,
Igor.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH RESEND] vf610twr: Tune DDR initialization settings

2014-09-09 Thread Stefano Babic
Hi Stefan, Albert,

On 09/09/2014 17:14, Stefan Agner wrote:
 Hi Albert,
 
 The RESEND version of the patch is actually an updated version (maybe I
 should have increased the version number?)
 
 For me, that patch applies cleanly on U-Boot master
 
 0b703dbcee7103f07804d0a4328d1593355c4324
 patman: Fix detection of git version
 
 Also I tested on U-Boot ARM master and next branch, applies without
 errors for me.
 

I have tried myself and I can confirm that patch can be applied fine.
Albert, should I apply it to u-boot-imx (vf610twr is part of iMX) and
then send it in my next PR ?

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 2/3] pcie_imx: Add mx6solox support

2014-09-09 Thread Stefano Babic
On 25/08/2014 19:26, Fabio Estevam wrote:
 Let PCI on mx6solox also be supported.
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic



-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] README.imximage: Fix the maximum DCD size

2014-09-09 Thread Stefano Babic
Hi Fabio,

On 09/09/2014 17:28, Fabio Estevam wrote:
 In commit 021e79c85371 (tools: imximage: Fix the maximum DCD size for
 mx53/mx6) we have fixed the maximum DCD size for mx53/mx53. 
 
 Do the same on the README document for consistency.
 
 Reported-by: Jonas Karlsson jonas.d.karls...@gmail.com
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---
  doc/README.imximage | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/doc/README.imximage b/doc/README.imximage
 index dcda200..27d3354 100644
 --- a/doc/README.imximage
 +++ b/doc/README.imximage
 @@ -120,7 +120,7 @@ Configuration command line syntax:
   DATA 4 0x73FA88a0 0x200
  
  The processor support up to 60 register programming commands for 
 IMXIMAGE_VERSION 1
 -and 121 register programming commands for IMXIMAGE_VERSION 2.
 +and 220 register programming commands for IMXIMAGE_VERSION 2.
  An error is generated if more commands are found in the configuration file.
  

Correct. Thanks for fixing the documentation, too.

Acked-by: Stefano Babic sba...@denx.de

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 3/3] mx6sxsabresd: Add PCI support

2014-09-09 Thread Stefano Babic
On 25/08/2014 19:26, Fabio Estevam wrote:
 Tested with an Intel Wireless PCI 7260HMW card:
 
 U-Boot 2014.10-rc1-16576-g4a8a8a8-dirty (Aug 23 2014 - 16:05:11)  
   
   
   
 CPU:   Freescale i.MX6SX rev1.0 at 792 MHz
   
 Reset cause: WDOG 
   
 Board: MX6SX SABRE SDB
   
 I2C:   ready  
   
 DRAM:  1 GiB  
   
 MMC:   FSL_SDHC: 0
   
   00:01.0 - 16c3:abcd - Bridge device 
   
01:00.0- 8086:08b1 - Network controller
   
 
 Signed-off-by: Fabio Estevam fabio.este...@freescale.com
 ---

I have no problem with the commit message.

Applied to u-boot-imx, thanks!

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] README.imximage: Fix the maximum DCD size

2014-09-09 Thread Fabio Estevam
In commit 021e79c85371 (tools: imximage: Fix the maximum DCD size for
mx53/mx6) we have fixed the maximum DCD size for mx53/mx53. 

Do the same on the README document for consistency.

Reported-by: Jonas Karlsson jonas.d.karls...@gmail.com
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
 doc/README.imximage | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/README.imximage b/doc/README.imximage
index dcda200..27d3354 100644
--- a/doc/README.imximage
+++ b/doc/README.imximage
@@ -120,7 +120,7 @@ Configuration command line syntax:
DATA 4 0x73FA88a0 0x200
 
 The processor support up to 60 register programming commands for 
IMXIMAGE_VERSION 1
-and 121 register programming commands for IMXIMAGE_VERSION 2.
+and 220 register programming commands for IMXIMAGE_VERSION 2.
 An error is generated if more commands are found in the configuration file.
 
 3. All commands are optional to program.
-- 
1.9.1

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


[U-Boot] [PATCH v2] README.imximage: Fix the maximum DCD size

2014-09-09 Thread Fabio Estevam
In commit 021e79c85371 (tools: imximage: Fix the maximum DCD size for
mx53/mx6) we have fixed the maximum DCD size for mx53/mx6. 

Do the same on the README document for consistency.

Reported-by: Jonas Karlsson jonas.d.karls...@gmail.com
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
Changes since v1:
- Fixed a typo in the commit log (mx53/mx53 -- mx53/mx6)
- Added Stefano's Acked-by tag

 doc/README.imximage | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/README.imximage b/doc/README.imximage
index dcda200..27d3354 100644
--- a/doc/README.imximage
+++ b/doc/README.imximage
@@ -120,7 +120,7 @@ Configuration command line syntax:
DATA 4 0x73FA88a0 0x200
 
 The processor support up to 60 register programming commands for 
IMXIMAGE_VERSION 1
-and 121 register programming commands for IMXIMAGE_VERSION 2.
+and 220 register programming commands for IMXIMAGE_VERSION 2.
 An error is generated if more commands are found in the configuration file.
 
 3. All commands are optional to program.
-- 
1.9.1

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


[U-Boot] commit 83e359adf9f578a58f20daf2e4425a754defac7b breaks mmc

2014-09-09 Thread Felipe Balbi
Hi,

commit 83e359a (am335x_evm: Enable CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT)
breaks MMC on some boards. I believe it only breaks if EMMC has no
partition on it, so that would point out to a bug on spl env support
itself and this has only exposed it.

Here's a bisection log anyway:

git bisect start
# good: [524123a70761110c5cf3ccc5f52f6d4da071b959] Prepare v2014.07
git bisect good 524123a70761110c5cf3ccc5f52f6d4da071b959
# bad: [0b703dbcee7103f07804d0a4328d1593355c4324] patman: Fix detection of git 
version
git bisect bad 0b703dbcee7103f07804d0a4328d1593355c4324
# good: [63b85adcecdd019f049cbbebf10119cea45d3645] imx: ventana: set dynamic 
env var for flash layout
git bisect good 63b85adcecdd019f049cbbebf10119cea45d3645
# good: [63b85adcecdd019f049cbbebf10119cea45d3645] imx: ventana: set dynamic 
env var for flash layout
git bisect good 63b85adcecdd019f049cbbebf10119cea45d3645
# good: [04b43f32731c1171877541050bb3f2bfeb100e3d] tools/genboardscfg.py: 
ignore defconfigs starting with a dot
git bisect good 04b43f32731c1171877541050bb3f2bfeb100e3d
# bad: [6defdc0b5552ab1af4a66a8abac8196cbb6b9e15] Merge branch 'master' of 
git://git.denx.de/u-boot-ti
git bisect bad 6defdc0b5552ab1af4a66a8abac8196cbb6b9e15
# good: [6af857c50df4e62ec08e51ad73c96f63f1480386] Merge branch 'master' of 
http://git.denx.de/u-boot-sunxi
git bisect good 6af857c50df4e62ec08e51ad73c96f63f1480386
# good: [7f14fb20f895016fb38d30ce71aeb4d441b5bcb8] Merge branch 'zynq' of 
git://www.denx.de/git/u-boot-microblaze
git bisect good 7f14fb20f895016fb38d30ce71aeb4d441b5bcb8
# bad: [61f66fd5a81b97478e9d14326c1059baa6626680] keystone2: use EFUSE_BOOTROM 
information to configure PLLs
git bisect bad 61f66fd5a81b97478e9d14326c1059baa6626680
# bad: [fea9543f1bd1d068a372ef378f624941c25989a8] board/ti/am335x: update 
configs for parallel NAND
git bisect bad fea9543f1bd1d068a372ef378f624941c25989a8
# good: [e017fd61c5a89e32db682d94d8d669df1709edbb] tseries: Set 
CONFIG_ENV_IS_NOWHERE for SPL+NAND
git bisect good e017fd61c5a89e32db682d94d8d669df1709edbb
# bad: [83e359adf9f578a58f20daf2e4425a754defac7b] am335x_evm: Enable 
CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT
git bisect bad 83e359adf9f578a58f20daf2e4425a754defac7b
# good: [00e385325fce36fa13d48091d73b1b3428394b6b] common/Makefile: Consolidate 
SPL ENV options, correct inclusion
git bisect good 00e385325fce36fa13d48091d73b1b3428394b6b
# first bad commit: [83e359adf9f578a58f20daf2e4425a754defac7b] am335x_evm: 
Enable CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT

Revert that commit also makes it work, btw.

On a further dig into this, it looks like when enabling
CONFIG_SPL_ENV_SUPPORT, somehow MMC numbering changes and u-boot ends up
trying to read environment from a non-formatted EMMC.

-- 
balbi


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] commit 83e359adf9f578a58f20daf2e4425a754defac7b breaks mmc

2014-09-09 Thread Felipe Balbi
Hi,

On Tue, Sep 09, 2014 at 12:09:02PM -0500, Felipe Balbi wrote:
 Hi,
 
 commit 83e359a (am335x_evm: Enable CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT)
 breaks MMC on some boards. I believe it only breaks if EMMC has no
 partition on it, so that would point out to a bug on spl env support
 itself and this has only exposed it.
 
 Here's a bisection log anyway:
 
 git bisect start
 # good: [524123a70761110c5cf3ccc5f52f6d4da071b959] Prepare v2014.07
 git bisect good 524123a70761110c5cf3ccc5f52f6d4da071b959
 # bad: [0b703dbcee7103f07804d0a4328d1593355c4324] patman: Fix detection of 
 git version
 git bisect bad 0b703dbcee7103f07804d0a4328d1593355c4324
 # good: [63b85adcecdd019f049cbbebf10119cea45d3645] imx: ventana: set dynamic 
 env var for flash layout
 git bisect good 63b85adcecdd019f049cbbebf10119cea45d3645
 # good: [63b85adcecdd019f049cbbebf10119cea45d3645] imx: ventana: set dynamic 
 env var for flash layout
 git bisect good 63b85adcecdd019f049cbbebf10119cea45d3645
 # good: [04b43f32731c1171877541050bb3f2bfeb100e3d] tools/genboardscfg.py: 
 ignore defconfigs starting with a dot
 git bisect good 04b43f32731c1171877541050bb3f2bfeb100e3d
 # bad: [6defdc0b5552ab1af4a66a8abac8196cbb6b9e15] Merge branch 'master' of 
 git://git.denx.de/u-boot-ti
 git bisect bad 6defdc0b5552ab1af4a66a8abac8196cbb6b9e15
 # good: [6af857c50df4e62ec08e51ad73c96f63f1480386] Merge branch 'master' of 
 http://git.denx.de/u-boot-sunxi
 git bisect good 6af857c50df4e62ec08e51ad73c96f63f1480386
 # good: [7f14fb20f895016fb38d30ce71aeb4d441b5bcb8] Merge branch 'zynq' of 
 git://www.denx.de/git/u-boot-microblaze
 git bisect good 7f14fb20f895016fb38d30ce71aeb4d441b5bcb8
 # bad: [61f66fd5a81b97478e9d14326c1059baa6626680] keystone2: use 
 EFUSE_BOOTROM information to configure PLLs
 git bisect bad 61f66fd5a81b97478e9d14326c1059baa6626680
 # bad: [fea9543f1bd1d068a372ef378f624941c25989a8] board/ti/am335x: update 
 configs for parallel NAND
 git bisect bad fea9543f1bd1d068a372ef378f624941c25989a8
 # good: [e017fd61c5a89e32db682d94d8d669df1709edbb] tseries: Set 
 CONFIG_ENV_IS_NOWHERE for SPL+NAND
 git bisect good e017fd61c5a89e32db682d94d8d669df1709edbb
 # bad: [83e359adf9f578a58f20daf2e4425a754defac7b] am335x_evm: Enable 
 CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT
 git bisect bad 83e359adf9f578a58f20daf2e4425a754defac7b
 # good: [00e385325fce36fa13d48091d73b1b3428394b6b] common/Makefile: 
 Consolidate SPL ENV options, correct inclusion
 git bisect good 00e385325fce36fa13d48091d73b1b3428394b6b
 # first bad commit: [83e359adf9f578a58f20daf2e4425a754defac7b] am335x_evm: 
 Enable CONFIG_SPL_ENV_SUPPORT on EMMC_BOOT
 
 Revert that commit also makes it work, btw.
 
 On a further dig into this, it looks like when enabling
 CONFIG_SPL_ENV_SUPPORT, somehow MMC numbering changes and u-boot ends up
 trying to read environment from a non-formatted EMMC.

forgot to mention, this is all on BBB and affects only am335x-based
boards, of course.

-- 
balbi


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 7/8] Makefile: default to cc for host compiler

2014-09-09 Thread Jeroen Hofstee

Hello Albert,

On 09-09-14 16:31, Albert ARIBAUD wrote:

On Thu, 31 Jul 2014 19:01:22 +0900, Masahiro Yamada
yamad...@jp.panasonic.com wrote:


  HOSTCXX  = g++
  HOSTCFLAGS   = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer

For consistency,

HOSTCXX = c++

?

So, Jeroen, what is your pick ? Will you send a v3 7/8, or are you
sticking with g++?

(or if everyone agrees, I could to the change to v2 7/8 when applying
it, and add a comment about it in the commit message.)


I did check Masahiro's statement that cpp is actually used in u-boot
and it is; make xconfig uses it, if I recall correctly. And as Masahiro
suggested I sent it to linux-kbuild mailinglist, but I am in the impression
it never arrived, perhaps I need to be subscribed or something...

Anyway, Masahiro's suggestion is sound and I am fine you changing
it while applying. I tested it on FreeBSD, Xubuntu and Debian.

Regards,
Jeroen

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


Re: [U-Boot] [PATCH 7/8] samsung: Move s5p_goni to use exynos-common config

2014-09-09 Thread Simon Glass
Hi Przemyslaw,

On 9 September 2014 04:26, Przemyslaw Marczak p.marc...@samsung.com wrote:

 Hello Simon,


 On 07/23/2014 02:11 PM, Simon Glass wrote:

 Change this board to use the exynos common config and add a device tree.

 This also adds a pinmux header file - but it is just a copy of the exynos
 one so may be incorrect.

 Signed-off-by: Simon Glass s...@chromium.org
 ---

   arch/arm/dts/Makefile  |  1 +
   arch/arm/dts/s5pc1xx-goni.dts  | 21 ++
   arch/arm/include/asm/arch-s5pc1xx/periph.h | 61
 ++
   arch/arm/include/asm/arch-s5pc1xx/pinmux.h | 50
 
   drivers/mmc/s5p_sdhci.c|  2 -
   include/configs/s5p_goni.h | 55
 +--
   6 files changed, 143 insertions(+), 47 deletions(-)
   create mode 100644 arch/arm/dts/s5pc1xx-goni.dts
   create mode 100644 arch/arm/include/asm/arch-s5pc1xx/periph.h
   create mode 100644 arch/arm/include/asm/arch-s5pc1xx/pinmux.h

 diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
 index 3b760ff..6a6e664 100644
 --- a/arch/arm/dts/Makefile
 +++ b/arch/arm/dts/Makefile
 @@ -1,3 +1,4 @@
 +dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
   dtb-$(CONFIG_EXYNOS4) += exynos4210-origen.dtb \
 exynos4210-smdkv310.dtb \
 exynos4210-universal_c210.dtb \
 diff --git a/arch/arm/dts/s5pc1xx-goni.dts b/arch/arm/dts/s5pc1xx-goni.
 dts
 new file mode 100644
 index 000..c390c8f
 --- /dev/null
 +++ b/arch/arm/dts/s5pc1xx-goni.dts
 @@ -0,0 +1,21 @@
 +/*
 + * Samsung's Exynos4210-based SMDKV310 board device tree source
 + *
 + * Copyright (c) 2014 Google, Inc
 + *
 + * SPDX-License-Identifier:GPL-2.0+
 + */
 +
 +/dts-v1/;


 This is a mistake. Exynos4 is different than s5pc1xx(which was later
 called exynos3 and is also called s5pv210(kernel)).
 So the s5pc1xx-goni.dts should include:
   model = Samsung Goni based on S5PC110;
   compatible = samsung,goni, samsung,s5pc110;


Thanks for the review.


 And the base addresses are different than exynos4. Please look at:
 arch/arm/include/asm/arch-s5pc1xx/cpu.h


OK


 This will require also a common file for s5pc1xx: s5pc1xx-common.dts.
 Moreover, the S5PC100 and S5PC110 have lot of differences.

 Simon, I think that this is a job for goni's maintainer, since you can't
 test it on a hardware.


Yes that would be great. To be clear, my intent is to make all these boards
similar so that it is possible to change over the GPIO and serial drivers
to use driver model.


 Robert, can you look at this?

  +/include/ exynos4.dtsi
 +
 +/ {
 +   model = Samsung SMDKV310 on Exynos4210;
 +   compatible = samsung,smdkv310, samsung,exynos4210;
 +
 +   aliases {
 +   serial0 = /serial@1380;
 +   console = /serial@1382;
 +   };
 +
 +};



 Best regards,
 --
 Przemyslaw Marczak
 Samsung RD Institute Poland
 Samsung Electronics
 p.marc...@samsung.com


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


Re: [U-Boot] [PATCH v7 05/13] tegra: Convert tegra GPIO driver to use driver model

2014-09-09 Thread Simon Glass
Hi,

On 3 September 2014 17:37, Simon Glass s...@chromium.org wrote:

 This is an implementation of GPIOs for Tegra that uses driver model. It has
 been tested on trimslice and also using the new iotrace feature.

 The implementation uses a top-level GPIO device (which has no actual
 GPIOS).
 Under this all the banks are created as separate GPIO devices.

 The GPIOs are named as per the Tegra datasheet/header files: A0..A7,
 B0..B7,
 ..., Z0..Z7, AA0..AA7, etc.

 Since driver model is not yet available before relocation, or in SPL, a
 special function is provided for seaboard's SPL code.

 Signed-off-by: Simon Glass s...@chromium.org


I've tested this series as much as I can, and in particular on Tegra 20, 30
and 124. Are there any final comments before I pull it into dm/master? I'd
like to allow plenty of test time before the release.

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


Re: [U-Boot] [PATCH v5 14/14] dm: tegra: Enable driver model for serial

2014-09-09 Thread Simon Glass
Hi,

On 4 September 2014 16:27, Simon Glass s...@chromium.org wrote:

 Use driver model for serial ports.

 Since Tegra now uses driver model for serial, adjust the definition of
 V_NS16550_CLK so that it is clear that this is only used for SPL.

 Signed-off-by: Simon Glass s...@chromium.org
 ---


I've tested this series as much as I can, and in particular on Tegra 20, 30
and 124. Are there any final comments before I pull it into dm/master? I'd
like to allow plenty of test time before the release.

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


Re: [U-Boot] [PATCH v2 7/8] Makefile: default to cc for host compiler

2014-09-09 Thread Albert ARIBAUD
Hi Jeroen,

On Tue, 09 Sep 2014 19:34:44 +0200, Jeroen Hofstee
jer...@myspectrum.nl wrote:

 Hello Albert,
 
 On 09-09-14 16:31, Albert ARIBAUD wrote:
  On Thu, 31 Jul 2014 19:01:22 +0900, Masahiro Yamada
  yamad...@jp.panasonic.com wrote:
 
HOSTCXX  = g++
HOSTCFLAGS   = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer
  For consistency,
 
  HOSTCXX = c++
 
  ?
  So, Jeroen, what is your pick ? Will you send a v3 7/8, or are you
  sticking with g++?
 
  (or if everyone agrees, I could to the change to v2 7/8 when applying
  it, and add a comment about it in the commit message.)
 
 I did check Masahiro's statement that cpp is actually used in u-boot
 and it is; make xconfig uses it, if I recall correctly. And as Masahiro
 suggested I sent it to linux-kbuild mailinglist, but I am in the impression
 it never arrived, perhaps I need to be subscribed or something...
 
 Anyway, Masahiro's suggestion is sound and I am fine you changing
 it while applying. I tested it on FreeBSD, Xubuntu and Debian.

I've tried building rpi_b as per the README, but I keep getting

/usr/bin/as: unrecognized option '-mfloat-abi=soft'
clang: error: assembler command failed with exit code 1 (use -v
to see invocation)

Shouldn't rpi_b build properly since it is the one given as an example
for Debian-based building?

 Regards,
 Jeroen

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] arm: Make reset position-independent

2014-09-09 Thread Fabio Estevam
Hi Albert,

On Fri, Sep 5, 2014 at 3:41 PM, Magnus Lilja lilja.mag...@gmail.com wrote:
 Hi

 On 3 September 2014 23:32, Benoît Thébaudeau
 benoit.thebaudeau@gmail.com wrote:
 Some boards, like mx31pdk and tx25, require the beginning of the SPL
 code to be position-independent. For these two boards, this is because
 they use the i.MX external NAND boot, which starts by executing the
 first NAND Flash page from the NFC page buffer. The SPL then needs to
 copy itself to its actual link address in order to free the NFC page
 buffer and use it to load the non-SPL image from Flash before running
 it. This means that the SPL runtime address differs from its link
 address between the reset and the initial copy performed by
 board_init_f(), so this part of the SPL binary must be
 position-independent.

 This requirement was broken by commit 41623c9 'arm: move exception
 handling out of start.S files', which used an absolute address to branch
 to the reset routine. This new commit restores the original behavior,
 which just performed a relative branch. This fixes the boot of mx31pdk
 and tx25.

 Signed-off-by: Benoît Thébaudeau benoit.thebaudeau@gmail.com
 Reported-by: Helmut Raiger helmut.rai...@hale.at
 Cc: Albert Aribaud albert.u.b...@aribaud.net
 Cc: Magnus Lilja lilja.mag...@gmail.com
 Cc: John Rigby jcri...@gmail.com
 ---

 Tested-by: Magnus Lilja lilja.mag...@gmail.com

Should this one go through your tree?
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


  1   2   >