Re: [U-Boot] [U-Boot, 2/2] mkimage: Refactor mxsimage to use common crc32 code

2014-05-13 Thread Charles Manning
On Tuesday 13 May 2014 08:26:44 Tom Rini wrote:
 On Tue, May 06, 2014 at 10:46:46AM +1200, Charles Manning wrote:
  mxsimage uses the same crc32 function as pblimage.
 
  Signed-off-by: Charles Manning cdhmann...@gmail.com
  Acked-by: Stefano Babic sba...@denx.de

 This introduces warnings:
 tools/mxsimage.c: In function ‘sb_build_command_load’:
 tools/mxsimage.c:979:2: warning: pointer targets in passing argument 2 of
 ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
 note: expected ‘const char *’ but argument is of type ‘uint8_t *’
 u-boot/tools/mxsimage.c: In function ‘sb_verify_command’:
 tools/mxsimage.c:1815:3: warning: pointer targets in passing argument 2 of
 ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
 note: expected ‘const char *’ but argument is of type ‘uint8_t *’

Thanks for the feedback.

Will it be acceptable to do the following:

*Change the called function from uint8_t * to const uint8_t
*Cast the const char * pointer to const uint8_t *.

Thanks

Charles

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


[U-Boot] [GIT PULL v2 ] fpga changes

2014-05-13 Thread Michal Simek
Hi Tom, [cc: Albert]

please pull these fpga changes to your tree.
I have run buildman to ensure that there are no new compilation warnings.
As you see from log below there are some warnings for m68k and astro board
but they are there before my first commit.

When this is in the tree I will send pull request to Albert to get our
Zynq changes to ARM tree.

Thanks,
Michal

[u-boot]$ ./tools/buildman/buildman -b fpga apf27 zynq spear astro balloon3 
pmc440 gen860t mvsmr mt_ventoux -sSed
Summary of 10 commits for 35 boards (4 threads, 1 job per thread)
01: kbuild: Fix trailing whitespaces
  m68k: +   astro_mcf5373l
+/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c: In function 
'cpu_init_f':
+/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c:211:10: warning: 
unused variable 'wdog' [-Wunused-variable]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c: In function 'do_bootm_linux':
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:53:8: warning: unused 
variable 'rd_len' [-Wunused-variable]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
'initrd_start' may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'initrd_end' 
may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_start' 
may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_end' may 
be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c: In function 
'initdram':
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c:83:5: warning: 
pointer targets in passing argument 1 of 'get_ram_size' differ in signedness 
[-Wpointer-sign]
+/mnt/disk/fpga/.bm-work/01/include/common.h:470:6: note: expected 'long int *' 
but argument is of type 'long unsigned int *'
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: 
initialization from incompatible pointer type [enabled by default]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: (near 
initialization for 'altera_fns.write') [enabled by default]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c: In function 
'astro5373l_altera_load':
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:196:20: warning: 
assignment from incompatible pointer type [enabled by default]
02: fpga: spartan2: Avoid CamelCase
03: fpga: spartan3: Avoid CamelCase
04: fpga: virtex2: Avoid CamelCase
05: fpga: xilinx: Avoid CamelCase for in Xilinx_desc
06: fpga: xilinx: Fix the rest of CamelCases
07: fpga: xilinx: Simplify load/dump/info function handling
   arm: (for 30/30 boards)  all -191.0  bss -1.7  data +9.3  rodata -54.9  
text -143.7
zynq_zed   :  all -489  bss +24  data +36  rodata -153  text 
-396
zynq_zc770_xm013:  all -509  bss +4  data +36  rodata -153  text 
-396
zynq_zc770_xm012:  all -509  bss +4  data +36  rodata -153  text 
-396
zynq_zc70x :  all -517  bss -4  data +36  rodata -153  text -396
zynq_zc770_xm010:  all -521  bss -8  data +36  rodata -153  text 
-396
zynq_microzed  :  all -521  bss -8  data +36  rodata -153  text -396
x600   :  all -646  bss +4  data +16  rodata -182  text -484
apf27  :  all -650  data +16  rodata -182  text -484
mt_ventoux :  all -654  bss -4  data +16  rodata -182  text -484
balloon3   :  all -714  bss -64  data +16  rodata -182  text 
-484
  m68k: (for 1/1 boards)  all -389.0  data -24.0  rodata -149.0  text -216.0
astro_mcf5373l :  all -389  data -24  rodata -149  text -216
   powerpc: (for 4/4 boards)  all -431.0  data +10.0  rodata -133.0  text -308.0
PMC440 :  all -364  data +40  rodata -148  text -256
MVSMR  :  all -448  text -448
GEN860T:  all -456  rodata -192  text -264
GEN860T_SC :  all -456  rodata -192  text -264
08: fpga: zynq: Remove sparse warnings
09: fpga: zynq: Use helper functions for zynq dma
   arm: (for 30/30 boards)  all +22.2  rodata +11.8  text +10.4
zynq_zc770_xm010:  all +119  bss +8  rodata +59  text +52
zynq_microzed  :  all +119  bss +8  rodata +59  text +52
zynq_zc70x :  all +119  bss +8  rodata +59  text +52
zynq_zc770_xm013:  all +111  rodata +59  text +52
zynq_zc770_xm012:  all +111  rodata +59  text +52
zynq_zed   :  all +87  bss -24  rodata +59  text +52
10: fpga: zynq: Use helper function zynq_validate_bitstream
   arm: (for 30/30 boards)  all +6.4  rodata +4.8  text +1.6
zynq_zc770_xm010:  all +40  bss +8  rodata +24  text +8
zynq_microzed  :  all +40  bss +8  rodata +24  text +8
zynq_zed   :  all +40  bss 

[U-Boot] [PATCH] powerpc/mpc85xx: SECURE BOOT- corrected CSPR settings for BSC9132QDS NAND

2014-05-13 Thread Aneesh Bansal
In case of NAND boot, CSPR0 is configured as NAND and CSPR1
is configured as NOR.

Signed-off-by: Aneesh Bansal aneesh.ban...@freescale.com
---
 include/configs/BSC9132QDS.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index e76a04b..7bb5d33 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -360,7 +360,7 @@ combinations. this should be removed later
 #endif
 
 /* Set up IFC registers for boot location NOR/NAND */
-#if defined(CONFIG_NAND)
+#if defined(CONFIG_NAND) || defined(CONFIG_NAND_SECBOOT)
 #define CONFIG_SYS_CSPR0   CONFIG_SYS_NAND_CSPR
 #define CONFIG_SYS_AMASK0  CONFIG_SYS_NAND_AMASK
 #define CONFIG_SYS_CSOR0   CONFIG_SYS_NAND_CSOR
-- 
1.8.1.4

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


Re: [U-Boot] [PATCH v13 0/2] S5P: Exynos: Add GPIO numbering feature

2014-05-13 Thread Minkyu Kang
Dear Akshay Saraswat,

On 13/05/14 14:00, Akshay Saraswat wrote:
 Used a script to recheck/verify pin mapping and existing mapping
 appears to be fine, returning correct bank and pin values.
 Did ./MAKEALL -a arm and found all Exynos/S5P related boards
 compiled successfully.
 Couldn't test booting over all of them.
 Tested U-Boot bootup over SMDK5420, SMDK5250, Snow.
 Requesting maintaners to please test over affected SoCs/boards.
 
 Changes in V2:
   - Enabled CMD_GPIO as suggested by Simon Glass
 and supported same for EXYNOS5.
 Changes in V3:
   - New patch added to rename S5P GPIO definitions
 to S5P_GPIO.
   - GPIO Table added to calculate the base address
 of input gpio bank.
 Changes in V4:
   - To have consistent 0..n-1 GPIO numbering the banks
 are divided into different parts where ever they
 have holes in them.
   - Function and table to support gpio command moved
 to s5p-gpio driver.
   - Rebased on latest u-boot-samsung tree.
 Changes in V5:
   - Rebased on latest u-boot-samsung tree.
   - Removed Exynos5 specific code in gpio driver api to
 get bank.
   - Added #define HAVE_GENERIC_GPIO in config file
 to remove conditinal CPU check in gpio driver.
 Changes in V6:
   - Isolated config changes in a new patch.
   - Updated patches with corresponding changes for Exynos 5420.
 Changes in V7:
   - Added changes for other SoCs like Exynos 4412, 4210 etc.
 Changes in V8:
   - Changed Subject of patch 2/2 to reflect affected SoCs/boards.
   - Fixed arndale board compile time errors introduced due to
 patch-set v7.
 Changes in V9:
   - Fixed checkpatch errors.
   - Fixed naming error in exynos_gpio_data arrays which could
 be the possible reason behind data abort witnessed over
 Exynos4 boards.
 Changes in v10:
   - Made arch-exynos/gpio.h - arch/gpio.h.
   - Removed dead code.
   - Replaced exynos_gpio_get calls with corresponding pin number.
   - Edited pin numbmers in dts files.
 Changes in v11:
   - Created and fixed pinmux_config for Exynos 4412.
   - Removed dead code.
 Changes in v12:
   - Added change logs in individual patches.
   - Fixed few bugs and typos.
   - Added Acked-by: Przemyslaw Marczak.
 Changes in v13:
   - Fixed typo by replacing EXYNOS5420_GPIO_B10 with
 EXYNOS5420_GPIO_B20 at line # 4149 of patch 1/2.
 
 Akshay Saraswat (2):
   S5P: Exynos: Add GPIO pin numbering and rename definitions
   S5P: Exynos: Config: Enable GPIO CMD config
 
  arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
  arch/arm/dts/exynos4210-origen.dts |4 +-
  arch/arm/dts/exynos4210-trats.dts  |6 +-
  arch/arm/dts/exynos4210-universal_c210.dts |4 +-
  arch/arm/dts/exynos4412-trats2.dts |4 +-
  arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
  arch/arm/include/asm/arch-exynos/gpio.h| 1761 
 +++-
  arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
  board/samsung/arndale/arndale.c|   11 +-
  board/samsung/goni/goni.c  |   32 +-
  board/samsung/smdk5250/exynos5-dt.c|   20 +-
  board/samsung/smdk5250/smdk5250.c  |   19 +-
  board/samsung/smdk5420/smdk5420.c  |   15 +-
  board/samsung/smdkc100/smdkc100.c  |5 +-
  board/samsung/smdkv310/smdkv310.c  |   19 +-
  board/samsung/trats/trats.c|   39 +-
  board/samsung/trats2/trats2.c  |   74 +-
  board/samsung/universal_c210/universal.c   |   51 +-
  drivers/gpio/s5p_gpio.c|  204 +++-
  include/configs/exynos5-dt.h   |2 +
  include/configs/s5p_goni.h |4 +-
  include/configs/s5pc210_universal.h|   16 +-
  include/configs/smdkv310.h |1 +
  include/configs/trats.h|8 +-
  include/configs/trats2.h   |4 +-
  25 files changed, 2918 insertions(+), 911 deletions(-)
 

applied to u-boot-samsung.


Dear SAMSUNG developers,

If you have any problem, please tell us immediately.

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


Re: [U-Boot] net/designware: call phy_connect_dev() to properly setup phylib device

2014-05-13 Thread Ian Campbell
On Mon, 2014-05-12 at 21:53 -0400, Tom Rini wrote:
 On Mon, Apr 28, 2014 at 08:14:05PM +0100, Ian Campbell wrote:
 
  This sets up the linkage from the phydev back to the ethernet device. This
  symptom of not doing this which I noticed was:
  NULL Waiting for PHY auto negotiation to complete
  rather than:
  dwmac.1c5 Waiting for PHY auto negotiation to complete
  
  Signed-off-by: Ian Campbell i...@hellion.org.uk
  Cc: Alexey Brodkin alexey.brod...@synopsys.com
 
 Applied to u-boot/master, thanks!

Thanks, have you also seen [PATCH v3 0/5] net/designware: fixes for
data cache, phylib and burst size which included this and some other
(far more critical, since designware is currently unusable on ARM)
fixes?

Ian.

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


[U-Boot] please pull u-boot-samsung master

2014-05-13 Thread Minkyu Kang
Dear Albert,

The following changes since commit 7904b70885f3c589c239f6ac978f299a6744557f:

  ARM: highbank: use default prompt (2014-05-02 11:43:25 +0200)

are available in the git repository at:

  http://git.denx.de/u-boot-samsung 

for you to fetch changes up to 9b97b727dcfbdc02a0a78e4c1d81670742f28784:

  S5P: Exynos: Config: Enable GPIO CMD config (2014-05-13 15:20:38 +0900)


Akshay Saraswat (3):
  Exynos5: config: Enable FIT
  S5P: Exynos: Add GPIO pin numbering and rename definitions
  S5P: Exynos: Config: Enable GPIO CMD config

Inha Song (1):
  samsung: misc: add env default option to lcd menu

Jaehoon Chung (1):
  ARM: exynos: remove the unused code

Mateusz Zalega (1):
  ARM: Samsung: s5p_goni: maintainer update

Przemyslaw Marczak (4):
  samsung: misc: allows using environmental macros as args in menu commands
  samsung: misc: add gpt restore option to lcd menu
  samsung: misc: menu: increase delay in menu main loop
  samsung: misc: remove download mode info screen

Łukasz Majewski (2):
  trats2: config: fix: Set default console to ttySAC2
  trats: config: fix: Set default console to ttySAC2

 arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
 arch/arm/dts/exynos4210-origen.dts |4 +-
 arch/arm/dts/exynos4210-trats.dts  |6 +-
 arch/arm/dts/exynos4210-universal_c210.dts |4 +-
 arch/arm/dts/exynos4412-trats2.dts |4 +-
 arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
 arch/arm/include/asm/arch-exynos/gpio.h| 1761 +++-
 arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
 board/samsung/arndale/arndale.c|   11 +-
 board/samsung/common/misc.c|  130 +-
 board/samsung/goni/goni.c  |   32 +-
 board/samsung/smdk5250/exynos5-dt.c|   20 +-
 board/samsung/smdk5250/smdk5250.c  |   19 +-
 board/samsung/smdk5420/smdk5420.c  |   15 +-
 board/samsung/smdkc100/smdkc100.c  |5 +-
 board/samsung/smdkv310/smdkv310.c  |   19 +-
 board/samsung/trats/trats.c|   39 +-
 board/samsung/trats2/trats2.c  |   74 +-
 board/samsung/universal_c210/universal.c   |   51 +-
 boards.cfg |2 +-
 drivers/gpio/s5p_gpio.c|  204 +++-
 include/configs/exynos5-dt.h   |2 +
 include/configs/s5p_goni.h |4 +-
 include/configs/s5pc210_universal.h|   23 +-
 include/configs/smdk5250.h |4 +
 include/configs/smdk5420.h |4 +
 include/configs/smdkv310.h |1 +
 include/configs/snow.h |4 +
 include/configs/trats.h|   17 +-
 include/configs/trats2.h   |   17 +-
 include/samsung/misc.h |2 +
 31 files changed, 3002 insertions(+), 1002 deletions(-)

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


Re: [U-Boot] net/designware: call phy_connect_dev() to properly setup phylib device

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 11:19:13AM +0100, Ian Campbell wrote:
 On Mon, 2014-05-12 at 21:53 -0400, Tom Rini wrote:
  On Mon, Apr 28, 2014 at 08:14:05PM +0100, Ian Campbell wrote:
  
   This sets up the linkage from the phydev back to the ethernet device. This
   symptom of not doing this which I noticed was:
   NULL Waiting for PHY auto negotiation to complete
   rather than:
   dwmac.1c5 Waiting for PHY auto negotiation to complete
   
   Signed-off-by: Ian Campbell i...@hellion.org.uk
   Cc: Alexey Brodkin alexey.brod...@synopsys.com
  
  Applied to u-boot/master, thanks!
 
 Thanks, have you also seen [PATCH v3 0/5] net/designware: fixes for
 data cache, phylib and burst size which included this and some other
 (far more critical, since designware is currently unusable on ARM)
 fixes?

For the moment I've tossed all of the rest of them over to Albert to
grab along with the sunxi stuff.  Albert, if you're uncomfortable there
I can pick them up instead, thanks!

-- 
Tom


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] Pull request: u-boot-video/master

2014-05-13 Thread Tom Rini
On Mon, May 12, 2014 at 09:56:41PM +0200, Anatolij Gustschin wrote:

 Hey Tom,
 
 The following changes since commit a405764c1ec835a41ccda943b9156aee25e15d5e:
 
   drivers/i2c/fsl_i2c: modify i2c_read to handle multi-byte write (2014-04-29 
 07:10:58 +0200)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-video.git master
 
 for you to fetch changes up to 1161f98db687a1cb263cbacdc7eb548a0354218d:
 
   at91: video: atmel_hlcdfb.c: fix bad timing configuration (2014-05-05 
 11:50:16 +0200)
 
 
 Eric Nelson (1):
   video: mxc_ipuv3_fb: stash frame buffer pointer in global data.
 
 Stephen Warren (1):
   config: enable CMD_BMP when API+LCD is enabled
 
 Thomas Diener (1):
   logos: Update of the syteco company logo
 
 Wu, Josh (1):
   at91: video: atmel_hlcdfb.c: fix bad timing configuration
 
  drivers/video/atmel_hlcdfb.c |8 
  drivers/video/mxc_ipuv3_fb.c |5 +
  include/config_fallbacks.h   |4 
  tools/logos/syteco.bmp   |  Bin 11414 - 11414 bytes
  4 files changed, 13 insertions(+), 4 deletions(-)

Applied to u-boot/master, thanks!

-- 
Tom


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] [GIT PULL v2 ] fpga changes

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 09:48:07AM +0200, Michal Simek wrote:

 Hi Tom, [cc: Albert]
 
 please pull these fpga changes to your tree.
 I have run buildman to ensure that there are no new compilation warnings.
 As you see from log below there are some warnings for m68k and astro board
 but they are there before my first commit.
 
 When this is in the tree I will send pull request to Albert to get our
 Zynq changes to ARM tree.
 
 Thanks,
 Michal
 
 [u-boot]$ ./tools/buildman/buildman -b fpga apf27 zynq spear astro balloon3 
 pmc440 gen860t mvsmr mt_ventoux -sSed
 Summary of 10 commits for 35 boards (4 threads, 1 job per thread)
 01: kbuild: Fix trailing whitespaces
   m68k: +   astro_mcf5373l
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c: In function 
 'cpu_init_f':
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c:211:10: warning: 
 unused variable 'wdog' [-Wunused-variable]
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c: In function 
 'do_bootm_linux':
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:53:8: warning: unused 
 variable 'rd_len' [-Wunused-variable]
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
 'initrd_start' may be used uninitialized in this function [-Wuninitialized]
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
 'initrd_end' may be used uninitialized in this function [-Wuninitialized]
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_start' 
 may be used uninitialized in this function [-Wuninitialized]
 +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_end' 
 may be used uninitialized in this function [-Wuninitialized]
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c: In function 
 'initdram':
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c:83:5: warning: 
 pointer targets in passing argument 1 of 'get_ram_size' differ in signedness 
 [-Wpointer-sign]
 +/mnt/disk/fpga/.bm-work/01/include/common.h:470:6: note: expected 'long int 
 *' but argument is of type 'long unsigned int *'
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: 
 initialization from incompatible pointer type [enabled by default]
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: (near 
 initialization for 'altera_fns.write') [enabled by default]
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c: In function 
 'astro5373l_altera_load':
 +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:196:20: warning: 
 assignment from incompatible pointer type [enabled by default]
 02: fpga: spartan2: Avoid CamelCase
 03: fpga: spartan3: Avoid CamelCase
 04: fpga: virtex2: Avoid CamelCase
 05: fpga: xilinx: Avoid CamelCase for in Xilinx_desc
 06: fpga: xilinx: Fix the rest of CamelCases
 07: fpga: xilinx: Simplify load/dump/info function handling
arm: (for 30/30 boards)  all -191.0  bss -1.7  data +9.3  rodata -54.9 
  text -143.7
 zynq_zed   :  all -489  bss +24  data +36  rodata -153  text 
 -396
 zynq_zc770_xm013:  all -509  bss +4  data +36  rodata -153  text 
 -396
 zynq_zc770_xm012:  all -509  bss +4  data +36  rodata -153  text 
 -396
 zynq_zc70x :  all -517  bss -4  data +36  rodata -153  text 
 -396
 zynq_zc770_xm010:  all -521  bss -8  data +36  rodata -153  text 
 -396
 zynq_microzed  :  all -521  bss -8  data +36  rodata -153  text 
 -396
 x600   :  all -646  bss +4  data +16  rodata -182  text 
 -484
 apf27  :  all -650  data +16  rodata -182  text -484
 mt_ventoux :  all -654  bss -4  data +16  rodata -182  text 
 -484
 balloon3   :  all -714  bss -64  data +16  rodata -182  text 
 -484
   m68k: (for 1/1 boards)  all -389.0  data -24.0  rodata -149.0  text 
 -216.0
 astro_mcf5373l :  all -389  data -24  rodata -149  text -216
powerpc: (for 4/4 boards)  all -431.0  data +10.0  rodata -133.0  text 
 -308.0
 PMC440 :  all -364  data +40  rodata -148  text -256
 MVSMR  :  all -448  text -448
 GEN860T:  all -456  rodata -192  text -264
 GEN860T_SC :  all -456  rodata -192  text -264
 08: fpga: zynq: Remove sparse warnings
 09: fpga: zynq: Use helper functions for zynq dma
arm: (for 30/30 boards)  all +22.2  rodata +11.8  text +10.4
 zynq_zc770_xm010:  all +119  bss +8  rodata +59  text +52
 zynq_microzed  :  all +119  bss +8  rodata +59  text +52
 zynq_zc70x :  all +119  bss +8  rodata +59  text +52
 zynq_zc770_xm013:  all +111  rodata +59  text +52
 zynq_zc770_xm012:  all +111  rodata +59  text +52
 zynq_zed   :  all +87  bss -24  rodata +59  text +52
 10: fpga: zynq: Use helper function zynq_validate_bitstream
arm: (for 30/30 boards)  all +6.4  rodata +4.8  text +1.6
 

Re: [U-Boot] [PATCH v13 0/2] S5P: Exynos: Add GPIO numbering feature

2014-05-13 Thread Jaehoon Chung
Well..

In my case, exynos4 (trats2) is working fine until now.

On 05/13/2014 05:38 PM, Minkyu Kang wrote:
 Dear Akshay Saraswat,
 
 On 13/05/14 14:00, Akshay Saraswat wrote:
 Used a script to recheck/verify pin mapping and existing mapping
 appears to be fine, returning correct bank and pin values.
 Did ./MAKEALL -a arm and found all Exynos/S5P related boards
 compiled successfully.
 Couldn't test booting over all of them.
 Tested U-Boot bootup over SMDK5420, SMDK5250, Snow.
 Requesting maintaners to please test over affected SoCs/boards.

 Changes in V2:
  - Enabled CMD_GPIO as suggested by Simon Glass
and supported same for EXYNOS5.
 Changes in V3:
  - New patch added to rename S5P GPIO definitions
to S5P_GPIO.
  - GPIO Table added to calculate the base address
of input gpio bank.
 Changes in V4:
  - To have consistent 0..n-1 GPIO numbering the banks
are divided into different parts where ever they
have holes in them.
  - Function and table to support gpio command moved
to s5p-gpio driver.
  - Rebased on latest u-boot-samsung tree.
 Changes in V5:
  - Rebased on latest u-boot-samsung tree.
  - Removed Exynos5 specific code in gpio driver api to
get bank.
  - Added #define HAVE_GENERIC_GPIO in config file
to remove conditinal CPU check in gpio driver.
 Changes in V6:
  - Isolated config changes in a new patch.
  - Updated patches with corresponding changes for Exynos 5420.
 Changes in V7:
  - Added changes for other SoCs like Exynos 4412, 4210 etc.
 Changes in V8:
  - Changed Subject of patch 2/2 to reflect affected SoCs/boards.
  - Fixed arndale board compile time errors introduced due to
patch-set v7.
 Changes in V9:
  - Fixed checkpatch errors.
  - Fixed naming error in exynos_gpio_data arrays which could
be the possible reason behind data abort witnessed over
Exynos4 boards.
 Changes in v10:
  - Made arch-exynos/gpio.h - arch/gpio.h.
  - Removed dead code.
  - Replaced exynos_gpio_get calls with corresponding pin number.
  - Edited pin numbmers in dts files.
 Changes in v11:
  - Created and fixed pinmux_config for Exynos 4412.
  - Removed dead code.
 Changes in v12:
  - Added change logs in individual patches.
  - Fixed few bugs and typos.
  - Added Acked-by: Przemyslaw Marczak.
 Changes in v13:
  - Fixed typo by replacing EXYNOS5420_GPIO_B10 with
EXYNOS5420_GPIO_B20 at line # 4149 of patch 1/2.

 Akshay Saraswat (2):
   S5P: Exynos: Add GPIO pin numbering and rename definitions
   S5P: Exynos: Config: Enable GPIO CMD config

  arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
  arch/arm/dts/exynos4210-origen.dts |4 +-
  arch/arm/dts/exynos4210-trats.dts  |6 +-
  arch/arm/dts/exynos4210-universal_c210.dts |4 +-
  arch/arm/dts/exynos4412-trats2.dts |4 +-
  arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
  arch/arm/include/asm/arch-exynos/gpio.h| 1761 
 +++-
  arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
  board/samsung/arndale/arndale.c|   11 +-
  board/samsung/goni/goni.c  |   32 +-
  board/samsung/smdk5250/exynos5-dt.c|   20 +-
  board/samsung/smdk5250/smdk5250.c  |   19 +-
  board/samsung/smdk5420/smdk5420.c  |   15 +-
  board/samsung/smdkc100/smdkc100.c  |5 +-
  board/samsung/smdkv310/smdkv310.c  |   19 +-
  board/samsung/trats/trats.c|   39 +-
  board/samsung/trats2/trats2.c  |   74 +-
  board/samsung/universal_c210/universal.c   |   51 +-
  drivers/gpio/s5p_gpio.c|  204 +++-
  include/configs/exynos5-dt.h   |2 +
  include/configs/s5p_goni.h |4 +-
  include/configs/s5pc210_universal.h|   16 +-
  include/configs/smdkv310.h |1 +
  include/configs/trats.h|8 +-
  include/configs/trats2.h   |4 +-
  25 files changed, 2918 insertions(+), 911 deletions(-)

 
 applied to u-boot-samsung.
 
 
 Dear SAMSUNG developers,
 
 If you have any problem, please tell us immediately.
 
 Thanks,
 Minkyu Kang.
 

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


Re: [U-Boot] [PATCH] env_eeprom: Assign default environment during board_init_f

2014-05-13 Thread Michal Simek
Hi Tom,

On 05/09/2014 08:08 PM, Tom Rini wrote:
 On Fri, Apr 25, 2014 at 03:49:04PM +0200, Michal Simek wrote:
 
 From: Siva Durga Prasad Paladugu siva.durga.palad...@xilinx.com

 Assign default environment and set env valid during board_init_f
 before relocation as the actual environment will be read from
 eeprom later.

 Signed-off-by: Siva Durga Prasad Paladugu siva...@xilinx.com
 Signed-off-by: Michal Simek michal.si...@xilinx.com
 ---

  common/env_eeprom.c | 5 +
  1 file changed, 5 insertions(+)

 diff --git a/common/env_eeprom.c b/common/env_eeprom.c
 index 0db2bb6..a4020bc 100644
 --- a/common/env_eeprom.c
 +++ b/common/env_eeprom.c
 @@ -224,6 +224,7 @@ int env_init(void)
  #else
  int env_init(void)
  {
 +#ifdef ENV_IS_EMBEDDED
  ulong crc, len, new;
  unsigned off;
  uchar buf[64];
 @@ -256,6 +257,10 @@ int env_init(void)
  gd-env_addr= 0;
  gd-env_valid   = 0;
  }
 +#else
 +gd-env_addr = (ulong)default_environment[0];
 +gd-env_valid = 1;
 +#endif

  return 0;
  }
 
 Sorry for the delay.  This looks fine but doesn't correct the redundant
 env case which has the same problem.  It would be nice if you had time
 to first consolidate env_init for both cases (see how env_nand.c does
 it) as well.  Thanks!

I have asked Siva to look at it but anyway if this is the right fix
it should be applied because it is fixing problem at least for one case.

Thanks,
Michal

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




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


Re: [U-Boot] [PATCH v2] zynq: treat ps7_init.c/h as external files to ignore them

2014-05-13 Thread Michal Simek
On 05/12/2014 05:18 AM, Masahiro Yamada wrote:
 ps7_init.c and ps7_init.h are supposed to be exported by hw project
 and copied to board/xilinx/zynq/ directory.
 
 We want them to be ignored by git.
 So what we should do is to always treat them as external files
 rather than replacing ps7_init.c
 
 This commit does:
 
  - Move a weak function ps7_init() to arch/arm/cpu/armv7/zynq/spl.c
and delete board/xilinx/zynq/ps7_init.c
 
  - Compile board/xilinx/zynq/ps7_init.c only when it exists
 
  - Add .gitignore to ignore ps7_init.c/h
 
 Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
 Signed-off-by: Michal Simek michal.si...@xilinx.com
 ---
 
 Changes in v2:
   (only cosmetic changes in v2)
   - Improve the comment in arch/arm/cpu/armv7/zynq/spl.c
   s/replaced/overridden/
   - Add Signed-off-by credit of Michal Simek
 
  arch/arm/cpu/armv7/zynq/spl.c |  8 
  board/xilinx/zynq/.gitignore  |  1 +
  board/xilinx/zynq/Makefile|  5 -
  board/xilinx/zynq/ps7_init.c  | 12 
  4 files changed, 13 insertions(+), 13 deletions(-)
  create mode 100644 board/xilinx/zynq/.gitignore
  delete mode 100644 board/xilinx/zynq/ps7_init.c

Applied to my zynq branch.

Thanks,
Michal

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




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


Re: [U-Boot] [PATCH] powerpc/mpc85xx: 32bit DDR changes for P1020/P1011

2014-05-13 Thread York Sun
On 05/12/2014 10:50 PM, Prabhakar Kushwaha wrote:
 The P1020/P1011 SOCs support max 32bit DDR width as opposed to P2020/P2010
 where max DDR data width supported is 64bit.
 
 Add dynamic DDR size adjustment in second stage boot loader execution.
 
 Signed-off-by: Prabhakar Kushwaha prabha...@freescale.com
 ---
  arch/powerpc/cpu/mpc85xx/cpu.c |7 ++-
  1 file changed, 6 insertions(+), 1 deletion(-)
 
 diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
 index 12e8e10..9c082b4 100644
 --- a/arch/powerpc/cpu/mpc85xx/cpu.c
 +++ b/arch/powerpc/cpu/mpc85xx/cpu.c
 @@ -347,7 +347,12 @@ phys_size_t initdram(int board_type)
   defined(CONFIG_QEMU_E500)
   return fsl_ddr_sdram_size();
  #else
 - return (phys_size_t)CONFIG_SYS_SDRAM_SIZE * 1024 * 1024;
 + struct cpu_type *cpu = gd-arch.cpu;
 + /* P1020 and it's derivatives support max 32bit DDR width */
 + if (cpu-soc_ver == SVR_P1020 || cpu-soc_ver == SVR_P1011)
 + return CONFIG_SYS_SDRAM_SIZE * 1024 * 1024 / 2;
 + else
 + return CONFIG_SYS_SDRAM_SIZE * 1024 * 1024;
  #endif
  }
  #else /* CONFIG_SYS_RAMBOOT */
 

Prabhakar,

The second stage boot loader doesn't re-initialize DDR, right? The size should
come from either fsl_ddr_sdram_size() or the predefined macro
CONFIG_SYS_SDRAM_SIZE. It looks like you are modifying the predefiend one. Is
this what you want? If so, why don't you defined the macro with correct value?

York


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


Re: [U-Boot] [PATCH] sandbox: move source files from board/ to arch/sandbox/

2014-05-13 Thread Simon Glass
Hi Masahiro,

On 29 April 2014 22:53, Masahiro Yamada yamad...@jp.panasonic.com wrote:

 Hi Simon,

   With this change, you can decrease one level hierarchy.
  
   board/sandbox/sandbox/sandbox.c
   -  board/sandbox/sandbox.c
 
  That seems better to me. Will you update your patch?
 

 Sure.
 Could you check this?
 http://patchwork.ozlabs.org/patch/344051/


Sorry I applied the wrong patch - the other one is not showing up in
patchwork for me. I'll see if I can figure it out.

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


[U-Boot] Please pull mpc85xx

2014-05-13 Thread York Sun
Tom,

The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:

  kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)

are available in the git repository at:

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

for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:

  boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
-0500)


Alexander Graf (4):
  PPC 85xx QEMU: Always assume 1 core
  PPC 85xx QEMU: Don't use HID1
  powerpc/mpc85xx: Update TLB CAMs in relocated mode
  PPC 85xx QEMU: Make a generic board file

Aneesh Bansal (2):
  powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
  powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB

Chunhe Lan (1):
  powerpc/85xx: Add T4240RDB board support

Masahiro Yamada (1):
  boards.cfg: fix a configuration error of ep8248 board

Shaohui Xie (1):
  powerpc/fman/memac: use default MDIO_HOLD value

Shaveta Leekha (2):
  board/freescale/common: ZM7300 driver
  board/b4qds: VID support

Shengzhou Liu (2):
  powerpc/t208x: enable errata A006261, A006593, A006379
  powerpc/85xx: add T4080 SoC support

Stefan Bigler (2):
  kmp204x: Add support for the unit LEDs
  kmp204x: handle dip-switch for factory settings

Valentin Longchamp (6):
  kmp204x: selftest/factory test pin support
  kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
  kmp204x: complete the reset sequence and PRST configuration
  kmp204x: update the RCW
  kmp204x: add workaround for A-004849
  kmp204x: enable the errata command

York Sun (6):
  powerpc/mpc8572ds: Increase u-boot size to 768KB
  powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
  powerpc/freescale: Change the return value of mac_read_from_eeprom()
  powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
  powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
  powerpc/freescale: Convert selected boards to generic board architecture

Zhao Qiang (1):
  qe: disable qe when qe-ucode fails to be uploaded for deep sleep

 arch/powerpc/cpu/mpc85xx/Makefile   |2 +
 arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
 arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
 arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
 arch/powerpc/cpu/mpc85xx/speed.c|3 +-
 arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
 arch/powerpc/cpu/mpc85xx/start.S|4 +-
 arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
 arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
 arch/powerpc/cpu/mpc86xx/start.S|1 +
 arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
 arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
 arch/powerpc/include/asm/fsl_errata.h   |5 +
 arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
 arch/powerpc/include/asm/immap_85xx.h   |6 +-
 arch/powerpc/include/asm/processor.h|1 +
 board/freescale/b4860qds/b4860qds.c |  240 +
 board/freescale/common/Makefile |1 +
 board/freescale/common/sys_eeprom.c |6 +-
 board/freescale/common/zm7300.c |  235 +
 board/freescale/common/zm7300.h |   22 +
 board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
 board/freescale/t4rdb/Makefile  |   12 +
 board/freescale/t4rdb/ddr.c |  118 +
 board/freescale/t4rdb/ddr.h |   78 +++
 board/freescale/t4rdb/eth.c |  146 ++
 board/freescale/t4rdb/law.c |   28 +
 board/freescale/t4rdb/pci.c |   23 +
 board/freescale/t4rdb/t4240rdb.c|  125 +
 board/freescale/t4rdb/t4_pbi.cfg|   31 ++
 board/freescale/t4rdb/t4_rcw.cfg|7 +
 board/freescale/t4rdb/t4rdb.h   |   18 +
 board/freescale/t4rdb/tlb.c |  111 
 board/keymile/kmp204x/kmp204x.c |   75 ++-
 board/keymile/kmp204x/kmp204x.h |3 +
 board/keymile/kmp204x/pbi.cfg   |   43 +-
 board/keymile/kmp204x/pci.c |   17 +-
 board/keymile/kmp204x/qrio.c|   31 +-
 board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
 boards.cfg  |7 +-
 drivers/mmc/fsl_esdhc.c |2 +-
 drivers/net/fm/Makefile |1 +
 drivers/net/fm/memac_phy.c  |8 +-
 drivers/qe/qe.c |9 +-
 include/common.h|5 +-
 include/configs/B4860QDS.h  |   14 +
 include/configs/MPC8536DS.h |2 +
 include/configs/MPC8572DS.h |5 +-
 include/configs/MPC8641HPCN.h   |3 +
 include/configs/T208xRDB.h  |1 +
 

Re: [U-Boot] Please pull mpc85xx

2014-05-13 Thread York Sun
Tom,

I forgot to add my Reviewed-by for this set. Let me know if I have to fix 
them.

York


On 05/13/2014 11:12 AM, York Sun wrote:
 Tom,
 
 The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:
 
   kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-mpc85xx.git master
 
 for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:
 
   boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
 -0500)
 
 
 Alexander Graf (4):
   PPC 85xx QEMU: Always assume 1 core
   PPC 85xx QEMU: Don't use HID1
   powerpc/mpc85xx: Update TLB CAMs in relocated mode
   PPC 85xx QEMU: Make a generic board file
 
 Aneesh Bansal (2):
   powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
   powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB
 
 Chunhe Lan (1):
   powerpc/85xx: Add T4240RDB board support
 
 Masahiro Yamada (1):
   boards.cfg: fix a configuration error of ep8248 board
 
 Shaohui Xie (1):
   powerpc/fman/memac: use default MDIO_HOLD value
 
 Shaveta Leekha (2):
   board/freescale/common: ZM7300 driver
   board/b4qds: VID support
 
 Shengzhou Liu (2):
   powerpc/t208x: enable errata A006261, A006593, A006379
   powerpc/85xx: add T4080 SoC support
 
 Stefan Bigler (2):
   kmp204x: Add support for the unit LEDs
   kmp204x: handle dip-switch for factory settings
 
 Valentin Longchamp (6):
   kmp204x: selftest/factory test pin support
   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
   kmp204x: complete the reset sequence and PRST configuration
   kmp204x: update the RCW
   kmp204x: add workaround for A-004849
   kmp204x: enable the errata command
 
 York Sun (6):
   powerpc/mpc8572ds: Increase u-boot size to 768KB
   powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
   powerpc/freescale: Change the return value of mac_read_from_eeprom()
   powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
   powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
   powerpc/freescale: Convert selected boards to generic board architecture
 
 Zhao Qiang (1):
   qe: disable qe when qe-ucode fails to be uploaded for deep sleep
 
  arch/powerpc/cpu/mpc85xx/Makefile   |2 +
  arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
  arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
  arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
  arch/powerpc/cpu/mpc85xx/speed.c|3 +-
  arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
  arch/powerpc/cpu/mpc85xx/start.S|4 +-
  arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
  arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
  arch/powerpc/cpu/mpc86xx/start.S|1 +
  arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
  arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
  arch/powerpc/include/asm/fsl_errata.h   |5 +
  arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
  arch/powerpc/include/asm/immap_85xx.h   |6 +-
  arch/powerpc/include/asm/processor.h|1 +
  board/freescale/b4860qds/b4860qds.c |  240 +
  board/freescale/common/Makefile |1 +
  board/freescale/common/sys_eeprom.c |6 +-
  board/freescale/common/zm7300.c |  235 +
  board/freescale/common/zm7300.h |   22 +
  board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
  board/freescale/t4rdb/Makefile  |   12 +
  board/freescale/t4rdb/ddr.c |  118 +
  board/freescale/t4rdb/ddr.h |   78 +++
  board/freescale/t4rdb/eth.c |  146 ++
  board/freescale/t4rdb/law.c |   28 +
  board/freescale/t4rdb/pci.c |   23 +
  board/freescale/t4rdb/t4240rdb.c|  125 +
  board/freescale/t4rdb/t4_pbi.cfg|   31 ++
  board/freescale/t4rdb/t4_rcw.cfg|7 +
  board/freescale/t4rdb/t4rdb.h   |   18 +
  board/freescale/t4rdb/tlb.c |  111 
  board/keymile/kmp204x/kmp204x.c |   75 ++-
  board/keymile/kmp204x/kmp204x.h |3 +
  board/keymile/kmp204x/pbi.cfg   |   43 +-
  board/keymile/kmp204x/pci.c |   17 +-
  board/keymile/kmp204x/qrio.c|   31 +-
  board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
  boards.cfg  |7 +-
  drivers/mmc/fsl_esdhc.c |2 +-
  drivers/net/fm/Makefile |1 +
  drivers/net/fm/memac_phy.c  |8 +-
  drivers/qe/qe.c |9 +-
  include/common.h|5 +-
  include/configs/B4860QDS.h  

Re: [U-Boot] powerpc/85xx: Add T4240RDB board support

2014-05-13 Thread York Sun
On 04/14/2014 05:42 AM, Chunhe Lan wrote:
 T4240RDB board Specification
 
 Memory subsystem:
6GB DDR3
128MB NOR flash
2GB NAND flash
 
 Ethernet:
Eight 1G SGMII ports
Four 10Gbps SFP+ ports
 
 PCIe:
Two PCIe slots
 
 USB:
Two USB2.0 Type A ports
 
 SDHC:
One SD-card port
 
 SATA:
One SATA port
 
 UART:
Dual RJ45 ports
 
 Signed-off-by: Chunhe Lan chunhe@freescale.com
 


Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH 1/2] powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb

2014-05-13 Thread York Sun
On 04/22/2014 04:46 AM, Aneesh Bansal wrote:
 T1040RDB.h file is removed and a unified file T104xRDB.h is created.
 Hence macro CONFIG_T1040 is renamed to CONFIG_T104x.
 
 Signed-off-by: Gaurav Kumar Rana gaurav.r...@freescale.com
 Signed-off-by: Aneesh Bansal aneesh.ban...@freescale.com
 ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH] powerpc/fman/memac: use default MDIO_HOLD value

2014-05-13 Thread York Sun
On 04/22/2014 05:21 AM, shh@gmail.com wrote:
 From: Shaohui Xie shaohui@freescale.com
 
 Current driver uses a Maximum value for MDIO_HOLD when doing 10G MDIO
 access, this is due to an errata A-006260 on T4 rev1.0 which is fixed
 on rev2.0, so remove the maximum value to use the default value for rev2.0.
 
 Signed-off-by: Shaohui Xie shaohui@freescale.com
 ---


Applied to u-boot-mpc85xx/master.

Thanks,

York

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


Re: [U-Boot] [PATCH v2] powerpc/85xx: add T4080 SoC support

2014-05-13 Thread York Sun
On 04/25/2014 03:31 AM, Shengzhou Liu wrote:
 The T4080 SoC is a low-power version of the T4160.
 T4080 combines 4 dual-threaded Power Architecture e6500
 cores with single cluster and two memory complexes.
 
 Signed-off-by: Shengzhou Liu shengzhou@freescale.com
 ---
 v2: add more comments.
 

Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH 0/8] kmp204x patches for 2014.07

2014-05-13 Thread York Sun
On 04/30/2014 08:01 AM, Valentin Longchamp wrote:
 Things have stabilized quite a lot on the u-boot front for the Keymile
 kmp204x/kmcoge4 board. There are a few remaining patches that should get
 merged to have a complete support of the architecture/board in u-boot.
 These include:
  - support for some additional functions of the QRIO board reset/control
CPLD (LEDs, Factory Test signal)
  - better reset sequence according to the tests/problems noticed during
the development
  - an update to the RCW (some external interrupts are unused and set as
GPIOs now)
  - an additionnal workaround in the PBL file (with an implementation
enhancement for a previous one).
 
 
 Stefan Bigler (2):
   kmp204x: Add support for the unit LEDs
   kmp204x: handle dip-switch for factory settings
 
 Valentin Longchamp (6):
   kmp204x: selftest/factory test pin support
   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
   kmp204x: complete the reset sequence and PRST configuration
   kmp204x: update the RCW
   kmp204x: add workaround for A-004849
   kmp204x: enable the errata command
 
  board/keymile/kmp204x/kmp204x.c   | 75 
 ++-
  board/keymile/kmp204x/kmp204x.h   |  3 ++
  board/keymile/kmp204x/pbi.cfg | 43 +---
  board/keymile/kmp204x/pci.c   | 17 
  board/keymile/kmp204x/qrio.c  | 29 +-
  board/keymile/kmp204x/rcw_kmp204x.cfg |  2 +-
  include/configs/km/kmp204x-common.h   | 17 
  7 files changed, 151 insertions(+), 35 deletions(-)
 

Applied to u-boot-mpc85xx/master.

Thanks,

York

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


Re: [U-Boot] [PATCH 1/2] board/freescale/common: ZM7300 driver

2014-05-13 Thread York Sun
On 04/11/2014 03:42 AM, Shaveta Leekha wrote:
 Adds Support for PowerOne ZM7300 voltage regulator.
 This device is available on some Freescale Boards like B4860QDS
 and has to be programmed to adjust the voltage on the board.
 
 The device is accessible via I2C interface.
 
 Signed-off-by: Shaveta Leekha shav...@freescale.com
 Signed-off-by: Poonam Aggrwal poonam.aggr...@freescale.com
 ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH 0/5] PPC 85xx QEMU: Additional fixes

2014-05-13 Thread York Sun
On 04/30/2014 12:21 PM, Alexander Graf wrote:
 After the initial round of the e500 QEMU machine patches got merged
 I realized that we were missing 2 pieces of the puzzle to make everything
 great:
 
   - KVM support
   - generic board file
 
 With this patch set I can successfully run u-boot inside of KVM on an e500v2
 as well as an e500mc machine. I also no longer get warnings that I should
 convert the board to the generic board mechanism.
 
 
 Alex
 
 Alexander Graf (5):
   PPC 85xx QEMU: Always assume 1 core
   PPC 85xx QEMU: Don't use HID1
   powerpc/mpc85xx: Update TLB CAMs in relocated mode
   PPC 85xx QEMU: Make a generic board file


Applied to u-boot-mpc85xx/master.

Thanks,

York

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


Re: [U-Boot] [PATCH 2/2] board/b4qds: VID support

2014-05-13 Thread York Sun
On 04/11/2014 03:42 AM, Shaveta Leekha wrote:
 The fuse status register provides the values from on-chip
 voltage ID efuses programmed at the factory.
 These values define the voltage requirements for
 the chip. u-boot reads FUSESR and translates the values
 into the appropriate commands to set the voltage output
 value of an external voltage regulator.
 
 B4860QDS has a PowerOne ZM7300 programmable digital Power
 Manager which is programmed as per the value read from
 the fuses.
 
 Reference for this code is taken from t4qds VID implementation.
 
 Signed-off-by: Shaveta Leekha shav...@freescale.com
 Signed-off-by: Poonam Aggrwal poonam.aggr...@freescale.com
 ---


Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH v4] qe: disable qe when qe-ucode fails to be uploaded for deep sleep

2014-05-13 Thread York Sun
On 04/30/2014 03:45 AM, Zhao Qiang wrote:
 when qe-ucode fails to be uploaded, deep sleep will hang.
 if there is no qe-ucode, disable qe module for platforms
 which support deep sleep
 
 Signed-off-by: Zhao Qiang b45...@freescale.com
 ---
 Changes for v2:
   - add #ifdef CONFIG_MPC85xx
 Changes for v3:
   - modify commit msg
 Changes for v4:
   - modify #ifdef CONFIG_MPC85xx to #ifdef CONFIG_DEEP_SLEEP
 


Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH 2/2] powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB

2014-05-13 Thread York Sun
On 04/22/2014 04:47 AM, Aneesh Bansal wrote:
 Secure Boot Target is added for T2080RDB
 
 Changes:
 For Secure boot, CPC is configured as SRAM and used as house
 keeping area which needs to be disabled.
 So CONFIG_SYS_CPC_REINIT_F is defined for CONFIG_T2080RDB.
 
 Signed-off-by: Aneesh Bansal aneesh.ban...@freescale.com
 ---


Applied to u-boot-mpc85xx/master.

Thanks,

York


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


Re: [U-Boot] [PATCH v2] powerpc/t208x: enable errata A006261, A006593, A006379

2014-05-13 Thread York Sun
On 04/23/2014 10:10 PM, Shengzhou Liu wrote:
 Enable errata A006261, A006593, A006379 for T208x.
 Additionally enable CONFIG_CMD_ERRATA for T2080RDB.
 
 Signed-off-by: Shengzhou Liu shengzhou@freescale.com
 ---
 v2: rebase to latest 'master', and update has_erratum_a006261.
 


Applied to u-boot-mpc85xx/master.

Thanks,

York


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


[U-Boot] [PATCH V2] usb: ci_udc: parse QTD before over-writing it

2014-05-13 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com

ci_udc only allocates a single QTD structure per EP. All data needs to be
extracted from the DTD prior to calling ci_ep_submit_next_request(), since
that fills the QTD with next transaction's parameters. Fix
handle_ep_complete() to extract the transaction (remaining) length before
kicking off the next transaction.

In practice, this only causes writes to UMS devices to fail for me. I may
have tested the final versions of my previous ci_udc patch only with
reads. More recently, I had patches applied locally that allocated a QTD
per USB request rather than per USB EP, although since that doesn't give
any performance benefit, I'm dropping those.

Fixes: 2813006fecda (usb: ci_udc: allow multiple buffer allocs per ep)
Signed-off-by: Stephen Warren swar...@nvidia.com
---
v2: Fix typo in commit description.
---
 drivers/usb/gadget/ci_udc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c
index 290863d61fc9..9cd003636a44 100644
--- a/drivers/usb/gadget/ci_udc.c
+++ b/drivers/usb/gadget/ci_udc.c
@@ -424,6 +424,7 @@ static void handle_ep_complete(struct ci_ep *ep)
item = ci_get_qtd(num, in);
ci_invalidate_qtd(num);
 
+   len = (item-info  16)  0x7fff;
if (item-info  0xff)
printf(EP%d/%s FAIL info=%x pg0=%x\n,
   num, in ? in : out, item-info, item-page0);
@@ -435,7 +436,6 @@ static void handle_ep_complete(struct ci_ep *ep)
if (!list_empty(ep-queue))
ci_ep_submit_next_request(ep);
 
-   len = (item-info  16)  0x7fff;
ci_req-req.actual = ci_req-req.length - len;
ci_debounce(ci_req, in);
 
-- 
1.8.1.5

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


Re: [U-Boot] [PATCH] boards.cfg: fix a configuration error of ep8248 board

2014-05-13 Thread York Sun
On 04/29/2014 10:55 PM, Masahiro Yamada wrote:
 make ep8248_config fails with an error like this:
 
 $ make ep8248_config
 make: *** [ep8248_config] Error 1
 
 Its cause is that there are two entries for ep8248.
 
 The first is around line 652 of boards.cfg. (as Active)
 
 The second appears around line 1230. (as Orphan)
 
 This bug was accidentally introduced by commit e7e90901.
 But it is not the author's fault. He just intended to change
 IDS8247 board.
 
 The commiter added ep8248 entry by mistake when he resolved a conflict.
 
 Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Kim Phillips kim.phill...@linaro.org
 ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


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


[U-Boot] Signed-off-by rules for U-Boot

2014-05-13 Thread Stephen Warren
Does U-Boot follow the same Signed-off-by procedures as the Linux
kernel, or is s-o-b only relevant in U-Boot for the initial patch
submitter, and not maintainers?

I ask because there are a bunch of patches of mine that got applied
without the maintainer adding their S-o-b when applying them, which
would be very unusual for Linux, but perhaps it's normal for U-Boot?

Some likely non-exhaustive examples are:

2813006fecda usb: ci_udc: allow multiple buffer allocs per ep
66b88b07cdab usb: ums: add error handling for failed registration
264e0e591bbd config: enable CMD_BMP when API+LCD is enabled
53eb768dfb97 aes: make apply_cbc_chain_data non-static

Yet others did get a maintainer s-o-b:

d381294aef4a ARM: tegra: pack pinmux data tables tighter
def816a2ba87 mmc: set rca to 1 for MMC cards
91171091c673 ARM: tegra: make use of negative ENV_OFFSET on NVIDIA...
6ab6a650a418 disk: initialize name/part fields when returning a whole...
c75942c7b792 sf: winbond: fix page_size

It seems like it mostly varies by maintainer.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Support for Tegra Jetson TK1

2014-05-13 Thread Hamza Farooq
Hello,

does the current u-boot for Tegra branch fully support Jetson TK1 board?
The recent commits somehow look incomplete (I was not able to find
RAM/Flash configuration). If it is the case, I'll be thankful if I could be
pointed in the right direction. It is obvious that I am not very
experienced with uboot but I am planning to learn/develop it for my Master
thesis.

PS: This is my first post ever in a mailing list. In case I am doing
something wrong, please point me to the right direction.

Thank you all for your patience

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


[U-Boot] gcc 4.3.5 and uboot 1.1.4.54 not working, uboot crashed

2014-05-13 Thread George Wang
Hello.

I just compiled uboot from gcc 4.3.5. When the uboot is running, it crashed
at ath_reg_wr_nf(), which is the SPI register write function. If I comment
it out, the uboot will keep going until it crashed again at
ath_pci_write_config(), which is PCI register write. Both of two call
#define ath_reg_wr(_phys, _val) do {\
ath_reg_wr_nf(_phys, _val); \
ath_reg_rd(_phys);  \


 #define ath_reg_wr_nf(_phys, _val) \
((*(volatile ath_reg_t *)KSEG1ADDR(_phys)) = (_val))


However When I use gcc 4.2.4 to compile the same uboot code, it works fun.
Any idea about what's going on here? I assume it must be related to BUS
configuration issue...

Thanks,

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


[U-Boot] [PATCH v2] Because sandbox is not a real hardware, setting vendor=sandbox is almost meaningless.

2014-05-13 Thread Simon Glass
From: Masahiro Yamada yamad...@jp.panasonic.com

This commit sets sandbox's vendor field to '-'.

It is a good thing that it reduces one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Signed-off-by: Simon Glass s...@chromium.org
(modified to apply to latest upstream)

---

Changes in v2:
- Update to apply on current master

 arch/sandbox/lib/Makefile | 2 +-
 board/sandbox/Makefile| 7 +++
 {doc = board/sandbox}/README.sandbox | 0
 {arch/sandbox/lib = board/sandbox}/sandbox.c | 0
 boards.cfg| 2 +-
 5 files changed, 9 insertions(+), 2 deletions(-)
 create mode 100644 board/sandbox/Makefile
 rename {doc = board/sandbox}/README.sandbox (100%)
 rename {arch/sandbox/lib = board/sandbox}/sandbox.c (100%)

diff --git a/arch/sandbox/lib/Makefile b/arch/sandbox/lib/Makefile
index 6480ebf..4c1a38d 100644
--- a/arch/sandbox/lib/Makefile
+++ b/arch/sandbox/lib/Makefile
@@ -8,4 +8,4 @@
 #
 
 
-obj-y  += interrupts.o sandbox.o
+obj-y  += interrupts.o
diff --git a/board/sandbox/Makefile b/board/sandbox/Makefile
new file mode 100644
index 000..a0b9880
--- /dev/null
+++ b/board/sandbox/Makefile
@@ -0,0 +1,7 @@
+#
+# Copyright (c) 2011 The Chromium OS Authors.
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := sandbox.o
diff --git a/doc/README.sandbox b/board/sandbox/README.sandbox
similarity index 100%
rename from doc/README.sandbox
rename to board/sandbox/README.sandbox
diff --git a/arch/sandbox/lib/sandbox.c b/board/sandbox/sandbox.c
similarity index 100%
rename from arch/sandbox/lib/sandbox.c
rename to board/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 9d90550..59b5c4c 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
-Active  sandbox sandbox-   -   none  
sandbox   - 

Simon Glass s...@chromium.org
+Active  sandbox sandbox-   -   sandbox 
sandbox   - 

Simon Glass s...@chromium.org
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com:Nobuhiro Iwamatsu 
iwama...@nigauri.org
 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy phil.edwor...@renesas.com
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.9.1.423.g4596e3a

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


[U-Boot] [PATCH] patman: Suppress duplicate signoffs only for real patches

2014-05-13 Thread Simon Glass
There is an unfortunate bug in the signoff suppression logic. The first
pass is performed with 'git log', and all signoffs are added to the
supression set, such that the second time (when processing the real
patches) we always suppress the signoffs.

Correct this by only suppressing signoffs in the second pass.

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

 tools/patman/patchstream.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py
index 9f5682c..3228719 100644
--- a/tools/patman/patchstream.py
+++ b/tools/patman/patchstream.py
@@ -275,7 +275,8 @@ class PatchStream:
 
 # Suppress duplicate signoffs
 elif signoff_match:
-if self.commit.CheckDuplicateSignoff(signoff_match.group(1)):
+if (self.is_log or
+self.commit.CheckDuplicateSignoff(signoff_match.group(1))):
 out = [line]
 
 # Well that means this is an ordinary line
-- 
1.9.1.423.g4596e3a

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


Re: [U-Boot] Signed-off-by rules for U-Boot

2014-05-13 Thread Stefano Babic
Hi Stephen,

On 13/05/2014 20:03, Stephen Warren wrote:
 Does U-Boot follow the same Signed-off-by procedures as the Linux
 kernel, or is s-o-b only relevant in U-Boot for the initial patch
 submitter, and not maintainers?
 

Yes, we discussed this in a previous thread, here Wolfgang's answer:

http://lists.denx.de/pipermail/u-boot/2012-October/137248.html


 I ask because there are a bunch of patches of mine that got applied
 without the maintainer adding their S-o-b when applying them, which
 would be very unusual for Linux, but perhaps it's normal for U-Boot?
 

It should be normal for U-Boot. I never add my s-o-b to patches that I
only merge. Only if I change something I add my s-o-b. If this is not
the right rule, I am doing wrong since a lot of time ;-).

 Some likely non-exhaustive examples are:
 
 2813006fecda usb: ci_udc: allow multiple buffer allocs per ep
 66b88b07cdab usb: ums: add error handling for failed registration
 264e0e591bbd config: enable CMD_BMP when API+LCD is enabled
 53eb768dfb97 aes: make apply_cbc_chain_data non-static
 
 Yet others did get a maintainer s-o-b:
 
 d381294aef4a ARM: tegra: pack pinmux data tables tighter
 def816a2ba87 mmc: set rca to 1 for MMC cards
 91171091c673 ARM: tegra: make use of negative ENV_OFFSET on NVIDIA...
 6ab6a650a418 disk: initialize name/part fields when returning a whole...
 c75942c7b792 sf: winbond: fix page_size
 
 It seems like it mostly varies by maintainer.

Agree we should be consistent. I am not aware that the original rule was
changed, even if the behavior is different as the Linux kernel.

Regards,
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] patman: Don't request full names from get_maintainer

2014-05-13 Thread Simon Glass
Hi Doug,

On 18 April 2014 15:32, Doug Anderson diand...@chromium.org wrote:
 Simon,

 On Fri, Apr 18, 2014 at 1:43 PM, Simon Glass s...@chromium.org wrote:
 Hi Doug,

 On 17 April 2014 12:47, Doug Anderson diand...@chromium.org wrote:

 The Linux get_maintainer.pl can often produce a whole lot of results.
 As a result you'll sometimes blow your CC field over 1024 characters
 and that can cause listservs to reject your message.

 As a stopgap, call get_maintainer.pl with --non so it doesn't
 include real names.  This will dramatically reduce the number of
 characters.

 Signed-off-by: Doug Anderson diand...@chromium.org
 ---

  tools/patman/get_maintainer.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/tools/patman/get_maintainer.py b/tools/patman/get_maintainer.py
 index 00b4939..a5160bc 100644
 --- a/tools/patman/get_maintainer.py
 +++ b/tools/patman/get_maintainer.py
 @@ -43,5 +43,5 @@ def GetMaintainer(fname, verbose=False):
  print WARNING: Couldn't find get_maintainer.pl
  return []

 -stdout = command.Output(get_maintainer, '--norolestats', fname)
 +stdout = command.Output(get_maintainer, '--norolestats', '--non', 
 fname)
  return stdout.splitlines()
 --
 1.9.1.423.g4596e3a


 Good to avoid this problem, but I wonder if we should check the size
 and re-run the command if too long? That way we can keep names on the
 thread when it is possible.

 Also, why is there a limit on CC - is that a limitation described in
 the RFC, or just certain mailers? Can we get around it by specifying
 multiple Cc tags?

 I believe that we're actually fighting a heuristic that the mailing
 list servers have to avoid spammers.  In the past I've seen my mail
 get through to list servers that weren't run by kernel.org but _not_
 the the ones at kernel.org.  Kind of like how if you want to write
 code that's for generic exynos 5 hardware you need to use exynos_5yyy
 and _not_ replace the y with x (because that would be adult content!)

 We could possibly only run the heuristic if the CC list was  800
 characters?  I'd hesitate to getting too close to 1024 since I think
 there are instances where git will itself add a CC and that might blow
 us out.

Yes that seems reasonable - that way we get the names most of the time.


 ...another option is to just forget this patch and force people to add
 this to .get_maintainer.conf

I think the previous option is better.

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


Re: [U-Boot] Support for Tegra Jetson TK1

2014-05-13 Thread Stephen Warren
On 05/13/2014 06:06 AM, Hamza Farooq wrote:
 Hello,
 
 does the current u-boot for Tegra branch fully support Jetson TK1 board?

Yes, I believe that what's in u-boot.git branch master should be enough
to boot the board. I've sent various patches since then to clean up the
pinmux configuration etc., but I don't think they're required for U-Boot
to work.

 The recent commits somehow look incomplete (I was not able to find
 RAM/Flash configuration).

I'm not sure what you mean by RAM/Flash configuration. Anyway, it's
likely that the following links will be useful:

Background information on the Tegra boot process, flashing tools, etc.:
http://http.download.nvidia.com/tegra-public-appnotes/

The simplest way to flash a Tegra device for mainline usage:
https://github.com/NVIDIA/tegra-uboot-flasher-scripts
(See README-developer.txt to get the source and build it, then
README-user.txt to run it)

I hope that answers all your questions. Let me know if you have more.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 04/13] Makefile: Support include files for .dts files

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
 Linux supports this, and if we are to have compatible device tree files,
 U-Boot should also.
 
 Avoid giving the device tree files access to U-Boot's include/ directory.
 Only include/dt-bindings is accessible.

Acked-by: Stephen Warren swar...@nvidia.com

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


Re: [U-Boot] [RFC PATCH v2 05/13] tegra: dts: Bring in GPIO bindings from linux

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
 These files are taken from Linux 3.14.

 diff --git a/arch/arm/dts/tegra124.dtsi b/arch/arm/dts/tegra124.dtsi

 - gpio: gpio@6000d000 {
 + gpio: gpio@0,6000d000 {
   compatible = nvidia,tegra124-gpio, nvidia,tegra30-gpio;
 - reg = 0x6000d000 0x1000;
...
 + reg = 0x0 0x6000d000 0x0 0x1000;

That reg value and node name are for #address-cells=2,
#size-cells=2. However, those properties are both =1 in the U-Boot
tegra124.dtsi, and I'm not sure whether U-Boot even supports 64-bit DT
address values yet?

It'd probably be simplest to just use 32-bit address/size values in
tegra124.dtsi now, and import the conversion to 64-bit later?

Aside from that, this patch looks, so assuming the change to those two
lines above is reverted:

Acked-by: Stephen Warren swar...@nvidia.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 12/13] tegra: Enable driver model

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
 Enable driver model for Tegra boards.

Seems fine to me,
Acked-by: Stephen Warren swar...@nvidia.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mx25pdk: Add generic board support

2014-05-13 Thread Fabio Estevam
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
 include/configs/mx25pdk.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/configs/mx25pdk.h b/include/configs/mx25pdk.h
index aff2419..cfaf6ca 100644
--- a/include/configs/mx25pdk.h
+++ b/include/configs/mx25pdk.h
@@ -26,6 +26,8 @@
 #define CONFIG_SETUP_MEMORY_TAGS
 #define CONFIG_INITRD_TAG
 
+#define CONFIG_SYS_GENERIC_BOARD
+
 #define CONFIG_MACH_TYPE   MACH_TYPE_MX25_3DS
 
 /* Size of malloc() pool */
-- 
1.8.3.2

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


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

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass 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.

I still believe that there is a single Tegra GPIO controller, and this
should be represented by a single device in the U-Boot device model.

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


Re: [U-Boot] [U-Boot, 2/2] mkimage: Refactor mxsimage to use common crc32 code

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 07:27:18PM +1200, Charles Manning wrote:
 On Tuesday 13 May 2014 08:26:44 Tom Rini wrote:
  On Tue, May 06, 2014 at 10:46:46AM +1200, Charles Manning wrote:
   mxsimage uses the same crc32 function as pblimage.
  
   Signed-off-by: Charles Manning cdhmann...@gmail.com
   Acked-by: Stefano Babic sba...@denx.de
 
  This introduces warnings:
  tools/mxsimage.c: In function ‘sb_build_command_load’:
  tools/mxsimage.c:979:2: warning: pointer targets in passing argument 2 of
  ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
  note: expected ‘const char *’ but argument is of type ‘uint8_t *’
  u-boot/tools/mxsimage.c: In function ‘sb_verify_command’:
  tools/mxsimage.c:1815:3: warning: pointer targets in passing argument 2 of
  ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
  note: expected ‘const char *’ but argument is of type ‘uint8_t *’
 
 Thanks for the feedback.
 
 Will it be acceptable to do the following:
 
 *Change the called function from uint8_t * to const uint8_t
 *Cast the const char * pointer to const uint8_t *.

Sounds right.

-- 
Tom


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


[U-Boot] pull request for u-boot-tegra/master into ARM/master

2014-05-13 Thread Tom Warren
Albert,

Please pull u-boot-tegra/master into ARM/master. Thanks!

./MAKEALL -s tegra AOK, checkpatch.pl is OK, and ./MAKEALL -a arm only
shows failures that were already present in ARM/master.

The following changes since commit d2a3e911390f9fc4d8c0ee4b3c7fc75f4fd3fd19:

  Merge branch 'u-boot/master' (2014-05-09 11:50:14 +0200)

are available in the git repository at:

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

for you to fetch changes up to 2364e151e432b4ccf32dc9e6147121253d4ff86d:

  ARM: tegra: use a CPU freq that all SKUs can support (2014-05-13 10:41:32
-0700)


Stephen Warren (10):
  ARM: tegra: set CONFIG_SYS_MMC_MAX_DEVICE
  ARM: tegra: fix CPU VDD comment in Tegra30 CPU init code
  ARM: tegra: allow pinmux mux option not to be set by init tables
  ARM: tegra: add GPIO initialization table function
  ARM: tegra: add function to enable input clamping on tristate
  ARM: tegra: make use of GPIO init table on Jetson TK1
  ARM: tegra: clamp inputs on Jetson TK1
  ARM: tegra: update Venice2 pinmux
  ARM: tegra: Venice2 pinmux spreadsheet updates
  ARM: tegra: use a CPU freq that all SKUs can support

 arch/arm/cpu/arm720t/tegra-common/cpu.c|  10 +-
 arch/arm/cpu/arm720t/tegra30/cpu.c |  23 +-
 arch/arm/cpu/tegra-common/pinmux-common.c  |  19 +
 arch/arm/include/asm/arch-tegra/gpio.h |  20 +
 arch/arm/include/asm/arch-tegra/pinmux.h   |   5 +
 arch/arm/include/asm/arch-tegra/tegra_mmc.h|   2 -
 arch/arm/include/asm/arch-tegra114/pinmux.h|   1 +
 arch/arm/include/asm/arch-tegra124/pinmux.h|   1 +
 arch/arm/include/asm/arch-tegra20/pinmux.h |   1 +
 arch/arm/include/asm/arch-tegra30/pinmux.h |   1 +
 board/nvidia/jetson-tk1/jetson-tk1.c   |   6 +
 board/nvidia/jetson-tk1/pinmux-config-jetson-tk1.h | 256 +++---
 board/nvidia/venice2/pinmux-config-venice2.h   | 567
++---
 board/nvidia/venice2/venice2.c |  19 +-
 drivers/gpio/tegra_gpio.c  |  20 +
 drivers/mmc/tegra_mmc.c|  13 +-
 include/configs/beaver.h   |   3 +
 include/configs/cardhu.h   |   3 +
 include/configs/tegra-common.h |   7 +
 19 files changed, 565 insertions(+), 412 deletions(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull mpc85xx

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 11:12:53AM -0500, York Sun wrote:

 Tom,
 
 The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:
 
   kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-mpc85xx.git master
 
 for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:
 
   boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
 -0500)
 
 
 Alexander Graf (4):
   PPC 85xx QEMU: Always assume 1 core
   PPC 85xx QEMU: Don't use HID1
   powerpc/mpc85xx: Update TLB CAMs in relocated mode
   PPC 85xx QEMU: Make a generic board file
 
 Aneesh Bansal (2):
   powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
   powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB
 
 Chunhe Lan (1):
   powerpc/85xx: Add T4240RDB board support
 
 Masahiro Yamada (1):
   boards.cfg: fix a configuration error of ep8248 board
 
 Shaohui Xie (1):
   powerpc/fman/memac: use default MDIO_HOLD value
 
 Shaveta Leekha (2):
   board/freescale/common: ZM7300 driver
   board/b4qds: VID support
 
 Shengzhou Liu (2):
   powerpc/t208x: enable errata A006261, A006593, A006379
   powerpc/85xx: add T4080 SoC support
 
 Stefan Bigler (2):
   kmp204x: Add support for the unit LEDs
   kmp204x: handle dip-switch for factory settings
 
 Valentin Longchamp (6):
   kmp204x: selftest/factory test pin support
   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
   kmp204x: complete the reset sequence and PRST configuration
   kmp204x: update the RCW
   kmp204x: add workaround for A-004849
   kmp204x: enable the errata command
 
 York Sun (6):
   powerpc/mpc8572ds: Increase u-boot size to 768KB
   powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
   powerpc/freescale: Change the return value of mac_read_from_eeprom()
   powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
   powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
   powerpc/freescale: Convert selected boards to generic board architecture
 
 Zhao Qiang (1):
   qe: disable qe when qe-ucode fails to be uploaded for deep sleep
 
  arch/powerpc/cpu/mpc85xx/Makefile   |2 +
  arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
  arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
  arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
  arch/powerpc/cpu/mpc85xx/speed.c|3 +-
  arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
  arch/powerpc/cpu/mpc85xx/start.S|4 +-
  arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
  arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
  arch/powerpc/cpu/mpc86xx/start.S|1 +
  arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
  arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
  arch/powerpc/include/asm/fsl_errata.h   |5 +
  arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
  arch/powerpc/include/asm/immap_85xx.h   |6 +-
  arch/powerpc/include/asm/processor.h|1 +
  board/freescale/b4860qds/b4860qds.c |  240 +
  board/freescale/common/Makefile |1 +
  board/freescale/common/sys_eeprom.c |6 +-
  board/freescale/common/zm7300.c |  235 +
  board/freescale/common/zm7300.h |   22 +
  board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
  board/freescale/t4rdb/Makefile  |   12 +
  board/freescale/t4rdb/ddr.c |  118 +
  board/freescale/t4rdb/ddr.h |   78 +++
  board/freescale/t4rdb/eth.c |  146 ++
  board/freescale/t4rdb/law.c |   28 +
  board/freescale/t4rdb/pci.c |   23 +
  board/freescale/t4rdb/t4240rdb.c|  125 +
  board/freescale/t4rdb/t4_pbi.cfg|   31 ++
  board/freescale/t4rdb/t4_rcw.cfg|7 +
  board/freescale/t4rdb/t4rdb.h   |   18 +
  board/freescale/t4rdb/tlb.c |  111 
  board/keymile/kmp204x/kmp204x.c |   75 ++-
  board/keymile/kmp204x/kmp204x.h |3 +
  board/keymile/kmp204x/pbi.cfg   |   43 +-
  board/keymile/kmp204x/pci.c |   17 +-
  board/keymile/kmp204x/qrio.c|   31 +-
  board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
  boards.cfg  |7 +-
  drivers/mmc/fsl_esdhc.c |2 +-
  drivers/net/fm/Makefile |1 +
  drivers/net/fm/memac_phy.c  |8 +-
  drivers/qe/qe.c |9 +-
  include/common.h|5 +-
  include/configs/B4860QDS.h  |   14 +
  include/configs/MPC8536DS.h |2 +
  

[U-Boot] armv8 relocation questions

2014-05-13 Thread Darwin Rambo

I'm hoping someone can help answer these questions about armv8 relocation.

The CONFIG_SYS_TEXT_BASE seems to be be usually setup to a decent amount 
of alignment. For the purposes of this discussion, let's say it would 
normally be 0x8800 and all is well. The relocation address moves to 
near the end of memory, to say, 0xfffa8000. So far so good.


Now let's say I want to shift the image a bit so that I can add a small 
32-byte header required by a previous bootloader. So I set 
CONFIG_SYS_TEXT_BASE to 0x8820, and the relocated address is still 
0xfffa8000 and the relocated vectors should be at 0xfffa9000. The image 
crashes so after some debugging, I find that the code appears to be 
relocated fine, but some sections have symbols that are not relocated 
properly. The vectors try to relocate to 0xfffa8fe0 and rodata.str1.1 
printf format strings are also 0x20 off. There are likely other offset 
sections with issues as well.


The relocation offset is 0x77fa7fe0 due to the calculations in 
arch/arm/lib/board.c. Simplifying, they look like this:


addr = CONFIG_SYS_SDRAM_BASE + gd-ram_size;

/* round down to next 4 kB limit */
addr = ~(4096 - 1);
debug(Top of RAM usable for U-Boot at: %08lx\n, addr);

/*
 * reserve memory for U-Boot code, data  bss
 * round down to next 4 kB limit
 */
addr -= gd-mon_len;
addr = ~(4096 - 1);

addr += 0x20; // hack to adjust relocaddr to aligned address...

snip

gd-relocaddr = addr;
gd-start_addr_sp = addr_sp;
gd-reloc_off = addr - _TEXT_BASE;
debug(relocation Offset is: %08lx\n, gd-reloc_off);


Since _TEXT_BASE is 0x8820 and addr is 0xfffa8000, the reloc_off is 
a number like 0x77fa7fe0.


Now if I add 0x20 to 'addr' above just before the snip, relocaddr 
becomes 0x77fa8000 and the relocation works perfectly and no more 
crashes happen.


So my question - is the CONFIG_SYS_TEXT_BASE alignment requirement 
related to to any assumptions in the linker itself about image base 
alignment, specifically referring to creation of the rela.dyn sections 
and their use for image relocation?


A related question is if CONFIG_SYS_TEXT_BASE needs to be at a specific 
alignment. The maximum alignment in the armv8 code base is .align 11 
which I believe means 0x800 or 2048.


Note that an armv7 target appears to relocate properly with smaller 
offsets such as 0x20.


Thanks.

Best Regards,
Darwin Rambo


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


Re: [U-Boot] [U-Boot, 1/5] siemens: cosmetic: remove unused and rename defines

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:52PM +0200, Egli, Samuel wrote:

 For dxr2 board DXR2_IOCTRL_VAL is set by data in EEPROM. In pxm2
 board it does not make sense to have dxr2 as prefix. Replace it with
 more meaningful DDR prefix.
 
 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Cc: Pascal Bach pascal.b...@siemens.com
 Cc: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot,2/5] siemens: update DDR3 parameters for dxr2

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:53PM +0200, Egli, Samuel wrote:

 * add parameters for factory and print them at start up to
facilitate control of right DDR3 settings in EEPROM.
 
  * cosmetic changes in a couple of printfs
 
 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Cc: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot,v2] am33xx: add SSC enable macro

2014-05-13 Thread Tom Rini
On Sat, Apr 19, 2014 at 10:12:18PM +0200, Yegor Yefremov wrote:

 Signed-off-by: Yegor Yefremov yegorsli...@googlemail.com

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

-- 
Tom


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] [U-Boot, 4/5] siemens: change LED indication in DFU mode

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:55PM +0200, Egli, Samuel wrote:

 In order to have the same LED indication like in another product
 when ready for updating, enable only red led and disable status
 LED when entering DFU mode.
 
 The status LED is only switched off when defined in board file.
 
 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Cc: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot, 3/5] siemens: add led cmd for flexible LED control

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:54PM +0200, Egli, Samuel wrote:

 * remove setting LED in user button function.
We want to decouple reading user button and setting LED. This
two things need to be done independently.
 
  * led cmd can be used to control LEDs that are defined in board file
having a led cmd, one can easily set LEDs in u-boot shell. For
example bootcmd can be extended to disable status LED before
loading kernel.
 
 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Cc: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot,5/5] siemens: cosmetic: rename project_dir

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:56PM +0200, Egli, Samuel wrote:

 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Cc: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot,1/4] ARM: OMAP5: add UART4 support

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:25PM +0300, Dmitry Lifshitz wrote:

 Add UART4 base address.
 
 Signed-off-by: Dmitry Lifshitz lifsh...@compulab.co.il

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

-- 
Tom


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] [U-Boot, 2/4] ARM: OMAP5: Power: add LDO2 support for Palmas driver

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:26PM +0300, Dmitry Lifshitz wrote:


 Add defines required to turn on LDO2 regulator.
 
 Signed-off-by: Dmitry Lifshitz lifsh...@compulab.co.il

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

-- 
Tom


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] mtd: nand: omap_gpmc: Fix update of read_ecc in oob

2014-05-13 Thread Tom Rini
On Fri, Apr 25, 2014 at 12:00:07PM +0200, Belisko Marek wrote:

 We need to flip only one bit not assign.
 
 Signed-off-by: Marek Belisko marek.beli...@gmail.com
 Acked-by: Pekon Gupta pe...@ti.com

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

-- 
Tom


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] [U-Boot, 1/2] omap3_beagle: use omap3-beagle-xm-ab.dtb for the xM AB revision

2014-05-13 Thread Tom Rini
On Fri, May 02, 2014 at 03:14:39PM -0500, robertcnel...@gmail.com wrote:

 From: Robert Nelson robertcnel...@gmail.com
 
 As of v3.15-rc3, omap3-beagle-xm-ab.dtb now exists with the usb hub (ehci) 
 enabled.
 
 For older kernels versions, cherry pick from mainline:
 ef78f3869c37c480f1d58462a760a40dabc823f4
 
 Signed-off-by: Robert Nelson robertcnel...@gmail.com
 CC: Tom Rini tr...@ti.com
 CC: Nishanth Menon n...@ti.com

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

-- 
Tom


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] [U-Boot,2/2] siemens, draco: add new target

2014-05-13 Thread Tom Rini
On Mon, May 05, 2014 at 04:50:44PM +0200, Egli, Samuel wrote:


 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Reviewed-by: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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] [U-Boot,3/4] ARM: OMAP5: add CKO buffer control mask

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:27PM +0300, Dmitry Lifshitz wrote:

 Add CKOBUFFER_CLK_EN bit mask enabling FREF_XTAL_CLK clock.
 
 Signed-off-by: Dmitry Lifshitz lifsh...@compulab.co.il

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

-- 
Tom


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] [U-Boot, 1/2] siemens:cosmetic, dxr2: rename dxr2 to draco

2014-05-13 Thread Tom Rini
On Mon, May 05, 2014 at 04:50:43PM +0200, Egli, Samuel wrote:

 The actual board name is draco and dxr2 is the target name.
 In the future we'll have different targets based on draco board.
 All changes are purely non-functional and basically rename dxr2
 to draco.
 
 One style fix in board.c that existed already before.
 
 Signed-off-by: Samuel Egli samuel.e...@siemens.com
 Reviewed-by: Roger Meier r.me...@siemens.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Wolfgang Denk w...@denx.de

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

-- 
Tom


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


[U-Boot] Please pull u-boot-ti/master

2014-05-13 Thread Tom Rini
Hey,

The following changes since commit d2a3e911390f9fc4d8c0ee4b3c7fc75f4fd3fd19:

  Merge branch 'u-boot/master' (2014-05-09 11:50:14 +0200)

are available in the git repository at:


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

for you to fetch changes up to 97eeae1a0735fc17351f10b9978e966392badedc:

  mtd: nand: omap_gpmc: Fix update of read_ecc in oob (2014-05-13 19:48:17 
-0400)


Ash Charles (1):
  am335x: pepper: Add Gumstix Pepper AM335x-based machine

Belisko Marek (1):
  mtd: nand: omap_gpmc: Fix update of read_ecc in oob

Christian Riesch (1):
  arm, davinci: Use CONFIG_SPL_PAD_TO for padding the SPL in an ais image

Dmitry Lifshitz (3):
  ARM: OMAP5: add UART4 support
  ARM: OMAP5: Power: add LDO2 support for Palmas driver
  ARM: OMAP5: add CKO buffer control mask

Egli, Samuel (7):
  siemens: cosmetic: remove unused and rename defines
  siemens: update DDR3 parameters for dxr2
  siemens: add led cmd for flexible LED control
  siemens: change LED indication in DFU mode
  siemens: cosmetic: rename project_dir
  siemens:cosmetic, dxr2: rename dxr2 to draco
  siemens, draco: add new target

Khoronzhuk, Ivan (1):
  config: k2hk_evm: Add generic board support

Robert Nelson (2):
  omap3_beagle: use omap3-beagle-xm-ab.dtb for the xM AB revision
  omap3_beagle: xM A/B validate new dtb exits in file system

Yegor Yefremov (1):
  am33xx: add SSC enable macro

 Makefile |2 +-
 arch/arm/include/asm/arch-am33xx/clock.h |2 +
 arch/arm/include/asm/arch-omap5/clock.h  |3 +
 arch/arm/include/asm/arch-omap5/omap.h   |1 +
 board/gumstix/pepper/Makefile|   13 ++
 board/gumstix/pepper/board.c |  226 ++
 board/gumstix/pepper/board.h |   19 +++
 board/gumstix/pepper/mux.c   |   78 +++
 board/siemens/common/board.c |   46 +-
 board/siemens/{dxr2 = draco}/Makefile   |0
 board/siemens/{dxr2 = draco}/board.c|  112 ---
 board/siemens/{dxr2 = draco}/board.h|   26 ++--
 board/siemens/{dxr2 = draco}/mux.c  |2 +-
 board/siemens/pxm2/board.c   |   10 +-
 board/siemens/rut/board.c|2 +-
 boards.cfg   |4 +-
 drivers/mtd/nand/omap_gpmc.c |2 +-
 include/configs/da850evm.h   |1 +
 include/configs/draco.h  |   92 
 include/configs/dxr2.h   |4 +-
 include/configs/k2hk_evm.h   |1 +
 include/configs/omap3_beagle.h   |   10 +-
 include/configs/pepper.h |   99 +
 include/configs/pxm2.h   |2 +-
 include/configs/siemens-am33x-common.h   |8 +-
 include/palmas.h |4 +
 26 files changed, 686 insertions(+), 83 deletions(-)
 create mode 100644 board/gumstix/pepper/Makefile
 create mode 100644 board/gumstix/pepper/board.c
 create mode 100644 board/gumstix/pepper/board.h
 create mode 100644 board/gumstix/pepper/mux.c
 rename board/siemens/{dxr2 = draco}/Makefile (100%)
 rename board/siemens/{dxr2 = draco}/board.c (63%)
 rename board/siemens/{dxr2 = draco}/board.h (71%)
 rename board/siemens/{dxr2 = draco}/mux.c (99%)
 create mode 100644 include/configs/draco.h
 create mode 100644 include/configs/pepper.h

Thanks!

-- 
Tom


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] arm, davinci: Use CONFIG_SPL_PAD_TO for padding the SPL in an ais image

2014-05-13 Thread Tom Rini
On Wed, May 07, 2014 at 10:16:28AM +0200, Christian Riesch wrote:

 The commits
 
 commit b7b5f1a16ca66dfdd817e7339f0e263a5b9f2758
 Author: Albert ARIBAUD albert.u.b...@aribaud.net
 da850evm, da850_am18xxevm: convert to CONFIG_SPL_MAX_FOOTPRINT
 
 and
 
 commit e7497891e34efe5cb2b3a3dc7c6c096c012ede28
 Author: Albert ARIBAUD albert.u.b...@aribaud.net
 cam_enc_4xx: convert to CONFIG_SPL_MAX_FOOTPRINT
 
 replaced CONFIG_SPL_MAX_SIZE by CONFIG_SPL_MAX_FOOTPRINT. However,
 CONFIG_SPL_MAX_SIZE is used in the Makefile for padding the SPL
 when preparing an u-boot.ais image. By removing CONFIG_SPL_MAX_SIZE
 said commits broke the ais image of the da850evm and cam_enc_4xx
 configurations.
 
 This patch converts the u-boot.ais target to use CONFIG_SPL_PAD_TO
 instead of CONFIG_SPL_MAX_SIZE for padding the SPL and adds
 a #define CONFIG_SPL_PAD_TO where it is required.
 
 Signed-off-by: Christian Riesch christian.rie...@omicron.at
 Reported-by: Tom Taylor ttaylor.ta...@gmail.com
 Cc: Sudhakar Rajashekhara sudhakar@ti.com
 Cc: Heiko Schocher h...@denx.de
 Cc: Albert ARIBAUD albert.u.b...@aribaud.net

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

-- 
Tom


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] [U-Boot, 2/2] omap3_beagle: xM A/B validate new dtb exits in file system

2014-05-13 Thread Tom Rini
On Fri, May 02, 2014 at 03:15:03PM -0500, robertcnel...@gmail.com wrote:

 From: Robert Nelson robertcnel...@gmail.com
 
 Fall back to previous dtb used when omap3-beagle-xm-ab.dtb doesn't exist in 
 file system
 
 Signed-off-by: Robert Nelson robertcnel...@gmail.com
 CC: Tom Rini tr...@ti.com
 CC: Nishanth Menon n...@ti.com
 Acked-by: Tom Rini tr...@ti.com

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

-- 
Tom


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] [RFC PATCH v0 3/4] vexpress_aemv8a.h: Clean up the config

2014-05-13 Thread Rob Herring
On Mon, May 5, 2014 at 10:26 AM, Tom Rini tr...@ti.com wrote:
 - Drop DEBUG
 - Drop defines we can use the default of.
 - Drop CONFIG_GICV3 and related defines, we aren't using interrupts and
   do not need to configure the interrupt controller.
 - Provide a larger malloc pool.
 - Correct default locations for kernel / initrd / device tree

 Signed-off-by: Tom Rini tr...@ti.com
 ---

[...]

 -#define CONFIG_GICV3
 -
  /*#define CONFIG_ARMV8_SWITCH_TO_EL1*/

  /*#define CONFIG_SYS_GENERIC_BOARD*/
 @@ -94,20 +90,11 @@
  /* Generic Timer Definitions */
  #define COUNTER_FREQUENCY  (0x180) /* 24MHz */

 -/* Generic Interrupt Controller Definitions */
 -#ifdef CONFIG_GICV3
 -#define GICD_BASE  (0x2f00)
 -#define GICR_BASE  (0x2f10)
 -#else
 -#define GICD_BASE  (0x2C001000)
 -#define GICC_BASE  (0x2C002000)
 -#endif

IIRC, these should be needed if secure to non-secure switching is done.

  #define CONFIG_SYS_MEMTEST_START   V2M_BASE
  #define CONFIG_SYS_MEMTEST_END (V2M_BASE + 0x8000)

  /* Size of malloc() pool */
 -#define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + 128 * 1024)
 +#define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + (8  20))

This seems like another candidate for some sane default value.

  /* SMSC91C111 Ethernet Configuration */
  #define CONFIG_SMC91
 @@ -121,7 +108,6 @@
  #define CONFIG_CONS_INDEX  0

  #define CONFIG_BAUDRATE115200
 -#define CONFIG_SYS_BAUDRATE_TABLE  { 9600, 19200, 38400, 57600, 115200 }
  #define CONFIG_SYS_SERIAL0 V2M_UART0
  #define CONFIG_SYS_SERIAL1 V2M_UART1

 @@ -166,10 +152,9 @@

  /* Initial environment variables */
  #define CONFIG_EXTRA_ENV_SETTINGS  \
 -   kernel_addr=0x20\0\
 -   initrd_addr=0xa0\0\
 -   initrd_size=0x200\0   \
 -   fdt_addr=0x10\0   \
 +   kernel_addr=0x8008\0  \
 +   initrd_addr=0x9000\0  \
 +   fdt_addr=0x8800\0 \

Don't we want the _r variants here?

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


Re: [U-Boot] [RFC PATCH v0 2/4] cmd_bootm.c: Add 'booti' for ARM64 Linux kernel Images

2014-05-13 Thread Rob Herring
On Mon, May 5, 2014 at 10:26 AM, Tom Rini tr...@ti.com wrote:
 The default format for arm64 Linux kernels is the Image format,
 described in Documentation/arm64/booting.txt.  This, along with an
 optional gzip compression on top is all that is generated by default.
 The Image format has a magic number within the header for verification,
 a text_offset where the Image must be run from and reserved fields.

 This does not support automatic detection of a gzip compressed image.

 Signed-off-by: Tom Rini tr...@ti.com
 ---
  README |1 +
  common/cmd_bootm.c |  139 
 
  2 files changed, 140 insertions(+)

 diff --git a/README b/README
 index 12758dc..e7c8b2e 100644
 --- a/README
 +++ b/README
 @@ -946,6 +946,7 @@ The following options need to be configured:
 CONFIG_CMD_BMP  * BMP support
 CONFIG_CMD_BSP  * Board specific commands
 CONFIG_CMD_BOOTD  bootd
 +   CONFIG_CMD_BOOTI* ARM64 Linux kernel Image support

As I mentioned at ELC, I would prefer to see bootz and booti combined
so we have a single boot a plain kernel image command. I'd rather
see the command be smart enough to figure what the image is rather
than the user have to figure out the right command. Of course with
this argument, then everything should be a bootm command. IIRC, there
would be some challenges to add bootz or booti functionality into
bootm.

 CONFIG_CMD_CACHE* icache, dcache
 CONFIG_CMD_CLK  * clock command support
 CONFIG_CMD_CONSOLEconinfo
 diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
 index c243a5b..3b9635c 100644
 --- a/common/cmd_bootm.c
 +++ b/common/cmd_bootm.c
 @@ -1929,3 +1929,142 @@ U_BOOT_CMD(
 boot Linux zImage image from memory, bootz_help_text
  );
  #endif /* CONFIG_CMD_BOOTZ */
 +
 +#ifdef CONFIG_CMD_BOOTI
 +/* See Documentation/arm64/booting.txt in the Linux kernel */
 +struct Image_header {
 +   uint32_tcode0;  /* Executable code */
 +   uint32_tcode1;  /* Executable code */
 +   uint64_ttext_offset;/* Image load offset */
 +   uint64_tres0;   /* reserved */
 +   uint64_tres1;   /* reserved */
 +   uint64_tres2;   /* reserved */
 +   uint64_tres3;   /* reserved */
 +   uint64_tres4;   /* reserved */
 +   uint32_tmagic;  /* Magic number */
 +   uint32_tres5;
 +};
 +
 +#define LINUX_ARM64_IMAGE_MAGIC0x644d5241
 +/* XXX: Hack 16MB image size for now */
 +#define HACK_ARM64_IMAGE_SIZE  (16  20)
 +
 +static int booti_setup(bootm_headers_t *images)
 +{
 +   struct Image_header *ih;
 +   uint64_t dst;
 +
 +   ih = (struct Image_header *)map_sysmem(images-ep, 0);
 +
 +   if (ih-magic != LINUX_ARM64_IMAGE_MAGIC) {
 +   puts(Bad Linux ARM64 Image magic!\n);
 +   return 1;
 +   }
 +
 +   /*
 +* If we are not at the correct run-time location, set the new
 +* correct location and then move the image there.
 +*/
 +   dst = gd-bd-bi_dram[0].start + ih-text_offset;
 +   if (images-ep != dst) {
 +   void *src;
 +
 +   debug(Moving Image from 0x%lx to 0x%llx\n, images-ep, dst);
 +
 +   src = (void *)images-ep;
 +   images-ep = dst;
 +   memmove((void *)dst, src, HACK_ARM64_IMAGE_SIZE);

Don't you at least know the load size or the decompressed size? Only
the .bss size is missing and you only need that to have some checks
for overlapping images.

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


Re: [U-Boot] [PATCH] sandbox: move source files from board/ to arch/sandbox/

2014-05-13 Thread Masahiro Yamada
Hi Simon,

On Tue, 13 May 2014 09:18:22 -0600
Simon Glass s...@chromium.org wrote:

 Hi Masahiro,
 
 On 29 April 2014 22:53, Masahiro Yamada yamad...@jp.panasonic.com wrote:
 
  Hi Simon,
 
With this change, you can decrease one level hierarchy.
   
board/sandbox/sandbox/sandbox.c
-  board/sandbox/sandbox.c
  
   That seems better to me. Will you update your patch?
  
 
  Sure.
  Could you check this?
  http://patchwork.ozlabs.org/patch/344051/
 
 
 Sorry I applied the wrong patch - the other one is not showing up in
 patchwork for me. I'll see if I can figure it out.

After u-boot-x86/tom was merged,
I cleared those patches out.

Please feel free to revert this patch and apply the other one.

Best Regards
Masahiro Yamada

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


[U-Boot] [PATCH v2 0/2] mkimage: Refactor and clean up 32-bit crcs

2014-05-13 Thread Charles Manning
As a step towards adding an image signer for socfpga, this set of patches
breaks out the common crc code that is used in both pblimage and mxsimage.

Not that this is NOT the same algorithm as the crc generated by zlib.

The new function has been named pbl_crc32.

This patch set just changes patch 1/2: casting pointers in mxsimage.c

Charles Manning (2):
  mkimage : Split out and clean pbl_crc32 for use by other image types
  tools: Refactor mxsimage to use pbl_crc32

 tools/Makefile|1 +
 tools/mxsimage.c  |   32 +++---
 tools/pbl_crc32.c |   56 +
 tools/pbl_crc32.h |   13 +
 tools/pblimage.c  |   47 +---
 5 files changed, 78 insertions(+), 71 deletions(-)
 create mode 100644 tools/pbl_crc32.c
 create mode 100644 tools/pbl_crc32.h

-- 
1.7.9.5

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


[U-Boot] [PATCH v2 2/2] tools: Refactor mxsimage to use pbl_crc32

2014-05-13 Thread Charles Manning
Both pblimage and mxsimage use the same crc algorithm, so refactor.

Signed-off-by: Charles Manning cdhmann...@gmail.com
---
Changes for V2:
  - Cast buffer pointers when calling pbl_crc32().

 tools/mxsimage.c |   32 +++-
 1 file changed, 7 insertions(+), 25 deletions(-)

diff --git a/tools/mxsimage.c b/tools/mxsimage.c
index 045b35a..81c7f2d 100644
--- a/tools/mxsimage.c
+++ b/tools/mxsimage.c
@@ -19,6 +19,7 @@
 
 #include imagetool.h
 #include mxsimage.h
+#include pbl_crc32.h
 #include image.h
 
 
@@ -231,29 +232,6 @@ static int sb_aes_reinit(struct sb_image_ctx *ictx, int 
enc)
 }
 
 /*
- * CRC32
- */
-static uint32_t crc32(uint8_t *data, uint32_t len)
-{
-   const uint32_t poly = 0x04c11db7;
-   uint32_t crc32 = 0x;
-   unsigned int byte, bit;
-
-   for (byte = 0; byte  len; byte++) {
-   crc32 ^= data[byte]  24;
-
-   for (bit = 8; bit  0; bit--) {
-   if (crc32  (1UL  31))
-   crc32 = (crc32  1) ^ poly;
-   else
-   crc32 = (crc32  1);
-   }
-   }
-
-   return crc32;
-}
-
-/*
  * Debug
  */
 static void soprintf(struct sb_image_ctx *ictx, const char *fmt, ...)
@@ -998,7 +976,9 @@ static int sb_build_command_load(struct sb_image_ctx *ictx,
 
ccmd-load.address  = dest;
ccmd-load.count= cctx-length;
-   ccmd-load.crc32= crc32(cctx-data, cctx-length);
+   ccmd-load.crc32= pbl_crc32(0,
+   (const char *)cctx-data,
+   cctx-length);
 
cctx-size = sizeof(*ccmd) + cctx-length;
 
@@ -1834,7 +1814,9 @@ static int sb_verify_command(struct sb_image_ctx *ictx,
EVP_DigestUpdate(ictx-md_ctx, cctx-data, asize);
sb_aes_crypt(ictx, cctx-data, cctx-data, asize);
 
-   if (ccmd-load.crc32 != crc32(cctx-data, asize)) {
+   if (ccmd-load.crc32 != pbl_crc32(0,
+ (const char *)cctx-data,
+ asize)) {
fprintf(stderr,
ERR: SB LOAD command payload CRC32 
invalid!\n);
return -EINVAL;
-- 
1.7.9.5

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


[U-Boot] [PATCH v2 1/2] mkimage : Split out and clean pbl_crc32 for use by other image types

2014-05-13 Thread Charles Manning
The crc32 used by pblimgae is NOT the same as zlib crc32.

The pbl_crc32 is useful for other purposes in mkimage so split it out.

While we are about it, clean up redundant and confusing code.

Signed-off-by: Charles Manning cdhmann...@gmail.com
---

Unchanged from V1.

 tools/Makefile|1 +
 tools/pbl_crc32.c |   56 +
 tools/pbl_crc32.h |   13 +
 tools/pblimage.c  |   47 +---
 4 files changed, 71 insertions(+), 46 deletions(-)
 create mode 100644 tools/pbl_crc32.c
 create mode 100644 tools/pbl_crc32.h

diff --git a/tools/Makefile b/tools/Makefile
index c34df4f..2f30749 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -86,6 +86,7 @@ dumpimage-mkimage-objs := aisimage.o \
omapimage.o \
os_support.o \
pblimage.o \
+   pbl_crc32.o \
sha1.o \
sha256.o \
ublimage.o \
diff --git a/tools/pbl_crc32.c b/tools/pbl_crc32.c
new file mode 100644
index 000..6e6735a
--- /dev/null
+++ b/tools/pbl_crc32.c
@@ -0,0 +1,56 @@
+/*
+ * Copyright 2012 Freescale Semiconductor, Inc.
+ *
+ * Cleaned up and refactored by Charles Manning.
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+#include pblimage.h
+
+static uint32_t crc_table[256];
+static int crc_table_valid;
+
+static void make_crc_table(void)
+{
+   uint32_t mask;
+   int i, j;
+   uint32_t poly; /* polynomial exclusive-or pattern */
+
+   if (crc_table_valid)
+   return;
+
+   /*
+* the polynomial used by PBL is 1 + x1 + x2 + x4 + x5 + x7 + x8 + x10
+* + x11 + x12 + x16 + x22 + x23 + x26 + x32.
+*/
+   poly = 0x04c11db7;
+
+   for (i = 0; i  256; i++) {
+   mask = i  24;
+   for (j = 0; j  8; j++) {
+   if (mask  0x8000)
+   mask = (mask  1) ^ poly;
+   else
+   mask = 1;
+   }
+   crc_table[i] = mask;
+   }
+
+   crc_table_valid = 1;
+}
+
+uint32_t pbl_crc32(uint32_t in_crc, const char *buf, uint32_t len)
+{
+   uint32_t crc32_val;
+   int i;
+
+   make_crc_table();
+
+   crc32_val = ~in_crc;
+
+   for (i = 0; i  len; i++)
+   crc32_val = (crc32_val  8) ^
+   crc_table[(crc32_val  24) ^ (*buf++  0xff)];
+
+   return crc32_val;
+}
diff --git a/tools/pbl_crc32.h b/tools/pbl_crc32.h
new file mode 100644
index 000..4ab55ee
--- /dev/null
+++ b/tools/pbl_crc32.h
@@ -0,0 +1,13 @@
+/*
+ * Copyright 2012 Freescale Semiconductor, Inc.
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#ifndef PBLCRC32_H
+#define PBLCRC32_H
+
+#include stdint.h
+uint32_t pbl_crc32(uint32_t in_crc, const char *buf, uint32_t len);
+
+#endif
diff --git a/tools/pblimage.c b/tools/pblimage.c
index ef3d7f6..6e6e801 100644
--- a/tools/pblimage.c
+++ b/tools/pblimage.c
@@ -6,6 +6,7 @@
 #include imagetool.h
 #include image.h
 #include pblimage.h
+#include pbl_crc32.h
 
 /*
  * Initialize to an invalid value.
@@ -137,52 +138,6 @@ static void pbl_parser(char *name)
fclose(fd);
 }
 
-static uint32_t crc_table[256];
-
-static void make_crc_table(void)
-{
-   uint32_t mask;
-   int i, j;
-   uint32_t poly; /* polynomial exclusive-or pattern */
-
-   /*
-* the polynomial used by PBL is 1 + x1 + x2 + x4 + x5 + x7 + x8 + x10
-* + x11 + x12 + x16 + x22 + x23 + x26 + x32.
-*/
-   poly = 0x04c11db7;
-
-   for (i = 0; i  256; i++) {
-   mask = i  24;
-   for (j = 0; j  8; j++) {
-   if (mask  0x8000)
-   mask = (mask  1) ^ poly;
-   else
-   mask = 1;
-   }
-   crc_table[i] = mask;
-   }
-}
-
-unsigned long pbl_crc32(unsigned long crc, const char *buf, uint32_t len)
-{
-   uint32_t crc32_val = 0x;
-   uint32_t xor = 0x0;
-   int i;
-
-   make_crc_table();
-
-   for (i = 0; i  len; i++)
-   crc32_val = (crc32_val  8) ^
-   crc_table[(crc32_val  24) ^ (*buf++  0xff)];
-
-   crc32_val = crc32_val ^ xor;
-   if (crc32_val  0) {
-   crc32_val += 0x;
-   crc32_val += 1;
-   }
-   return crc32_val;
-}
-
 static uint32_t reverse_byte(uint32_t val)
 {
uint32_t temp;
-- 
1.7.9.5

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


[U-Boot] [PATCH] Revert sandbox: move source files from board/ to arch/sandbox/

2014-05-13 Thread Masahiro Yamada
This reverts commit 258060905e04fe2eb509756ef3b37e23e220a2d6.

Conflicts:
boards.cfg

Wrong patch 25806090 was applied by accident. Revert it.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Simon Glass s...@chromium.org
---
 arch/sandbox/lib/Makefile | 2 +-
 board/sandbox/sandbox/Makefile| 7 +++
 {doc = board/sandbox/sandbox}/README.sandbox | 0
 {arch/sandbox/lib = board/sandbox/sandbox}/sandbox.c | 0
 boards.cfg| 2 +-
 5 files changed, 9 insertions(+), 2 deletions(-)
 create mode 100644 board/sandbox/sandbox/Makefile
 rename {doc = board/sandbox/sandbox}/README.sandbox (100%)
 rename {arch/sandbox/lib = board/sandbox/sandbox}/sandbox.c (100%)

diff --git a/arch/sandbox/lib/Makefile b/arch/sandbox/lib/Makefile
index 6480ebf..4c1a38d 100644
--- a/arch/sandbox/lib/Makefile
+++ b/arch/sandbox/lib/Makefile
@@ -8,4 +8,4 @@
 #
 
 
-obj-y  += interrupts.o sandbox.o
+obj-y  += interrupts.o
diff --git a/board/sandbox/sandbox/Makefile b/board/sandbox/sandbox/Makefile
new file mode 100644
index 000..a0b9880
--- /dev/null
+++ b/board/sandbox/sandbox/Makefile
@@ -0,0 +1,7 @@
+#
+# Copyright (c) 2011 The Chromium OS Authors.
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := sandbox.o
diff --git a/doc/README.sandbox b/board/sandbox/sandbox/README.sandbox
similarity index 100%
rename from doc/README.sandbox
rename to board/sandbox/sandbox/README.sandbox
diff --git a/arch/sandbox/lib/sandbox.c b/board/sandbox/sandbox/sandbox.c
similarity index 100%
rename from arch/sandbox/lib/sandbox.c
rename to board/sandbox/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 9d90550..6aea684 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
-Active  sandbox sandbox-   -   none  
sandbox   - 

Simon Glass s...@chromium.org
+Active  sandbox sandbox-   sandbox sandbox 
sandbox   - 

Simon Glass s...@chromium.org
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com:Nobuhiro Iwamatsu 
iwama...@nigauri.org
 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy phil.edwor...@renesas.com
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.8.3.2

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


Re: [U-Boot] [PATCH v2] Because sandbox is not a real hardware, setting vendor=sandbox is almost meaningless.

2014-05-13 Thread Masahiro Yamada
Hi Simon,

On Tue, 13 May 2014 12:13:48 -0600
Simon Glass s...@chromium.org wrote:

 From: Masahiro Yamada yamad...@jp.panasonic.com
 
 This commit sets sandbox's vendor field to '-'.
 
 It is a good thing that it reduces one level directory hierarchy.
 The files board/sandbox/sandbox/* have been moved to board/sandbox/*.
 
 Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
 Signed-off-by: Simon Glass s...@chromium.org
 (modified to apply to latest upstream)


I am not happy about this fix-up approach because the commit description
does not reflect the code any more.



 It is a good thing that it reduces one level directory hierarchy.

This sounds odd.
This patch moves arch/sandbox/lib/sandbox.c
to board/sandbox/sandbox.c,
creating board/sandbox/ directory.


We should revert the wrong commit first,
and then apply the right one.

I will send them.

Best Regards
Masahiro Yamada

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


[U-Boot] [PATCH v2] sandbox: set sandbox's vendor to null

2014-05-13 Thread Masahiro Yamada
Because sandbox is not a real hardware, setting vendor=sandbox is
almost meaningless.

This commit sets sandbox's vendor field to '-'.

It is a good thing that it decreases one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Acked-by: Simon Glass s...@chromium.org
---

Changes in v2:
  - Rebase

 board/sandbox/{sandbox = }/Makefile   | 0
 board/sandbox/{sandbox = }/README.sandbox | 0
 board/sandbox/{sandbox = }/sandbox.c  | 0
 boards.cfg | 2 +-
 4 files changed, 1 insertion(+), 1 deletion(-)
 rename board/sandbox/{sandbox = }/Makefile (100%)
 rename board/sandbox/{sandbox = }/README.sandbox (100%)
 rename board/sandbox/{sandbox = }/sandbox.c (100%)

diff --git a/board/sandbox/sandbox/Makefile b/board/sandbox/Makefile
similarity index 100%
rename from board/sandbox/sandbox/Makefile
rename to board/sandbox/Makefile
diff --git a/board/sandbox/sandbox/README.sandbox b/board/sandbox/README.sandbox
similarity index 100%
rename from board/sandbox/sandbox/README.sandbox
rename to board/sandbox/README.sandbox
diff --git a/board/sandbox/sandbox/sandbox.c b/board/sandbox/sandbox.c
similarity index 100%
rename from board/sandbox/sandbox/sandbox.c
rename to board/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 6aea684..59b5c4c 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda ricardo.riba...@uam.es
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 
ricardo.riba...@uam.es
-Active  sandbox sandbox-   sandbox sandbox 
sandbox   - 

Simon Glass s...@chromium.org
+Active  sandbox sandbox-   -   sandbox 
sandbox   - 

Simon Glass s...@chromium.org
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com:Nobuhiro Iwamatsu 
iwama...@nigauri.org
 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy phil.edwor...@renesas.com
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.8.3.2

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


[U-Boot] [Bug Report] Patman removes author's Signed-off-by credit

2014-05-13 Thread Masahiro Yamada
Hi Simon,

I suspect there is something wrong with Patman
since commit 102061bd (patman: Avoid duplicate sign-offs).


For example, my commit log is like follows

sandbox: set sandbox's vendor to null

Because sandbox is not a real hardware, setting vendor=sandbox is
almost meaningless.

This commit sets sandbox's vendor field to '-'.

It is a good thing that it decreases one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Acked-by: Simon Glass s...@chromium.org
Series-to: uboot
Series-cc: sjg
Series-version: 2
Series-changes: 2
  - Rebase


When I run tools/patman/patman -c 1,
Patman removes all Signed-off-by credits and checkpatch.pl
reports missing Signed-off-by credit error.



1 errors, 0 warnings, 0 checks for 
0001-sandbox-set-sandbox-s-vendor-to-null.patch:
error: unknown,0: Missing Signed-off-by: line(s)

checkpatch.pl found 1 error(s), 0 warning(s), 0 checks(s)


I think this is worse than dupulicated signed-off-by.


Best Regards
Masahiro Yamada

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


Re: [U-Boot] [PATCH v2 0/7] mtd: nand: omap: clean up of omap_elm and omap_gpmc driver

2014-05-13 Thread Gupta, Pekon
From: Gupta, Pekon

*changes v1-v2*
[PATCH 2/7] fix for BCH4 and BCH16 ECC scheme
[PATCH 7/7] new patch
 - white space-clean up


*original v1*
This patch series
 - removes un-wanted  redundant code in omap_elm and omap_gpmc drivers
 - refactors some private struct in omap_gpmc to make them generic across 
 ecc-schemes
 - minor code fixes and clean-ups
This series can be as-it-is applied on u-boot 'v2014.04-rc3'.

*Testing Procedure*
Boot tested on am335x_evm (MLO - U-boot - kernel with filesystem flashed 
from U-boot)
using OMAP_ECC_BCH8_CODE_HW and OMAP_ECC_BCH8_CODE_HW_DETECTION_SW ecc-schemes

Ping .. 
Please review and accept this. These are simple driver clean-ups.


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


Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Wed, May 7, 2014 at 2:29 AM, Stefano Babic sba...@denx.de wrote:
 Hi Tim,

 On 06/05/2014 20:18, Tim Harvey wrote:

 Stefano / York,

 While preparing for a v3 patch series of my IMX6 SPL bootloader, I
 find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks
 the above code because gd is now needed within setup_i2c.

 I've always been a bit fuzzy on the order of the above calls so I dug
 through the code and I think I understand things better. Please
 correct any wrong assumptions I'm making below:
  - assignment to gd should 'always' be first (before anything needs
 it, so why not do it first)
  - arch_cpu_init() should go next as this sets up very low level
 CPU/SoC resources (in this case AIPS config and watchdog disable)
  - board_early_init_f() should be next as that sets up any board-specific 
 iomux
  - any additional iomux necessary for SPL should go next (I take care
 of i2c iomux and setup here)
  - timer_init() next as you need a timer for UART and mxc i2c (for
 delays and busy checks)
  - preloader_console_init() next as we are now able to send something
 over the UART (this gives me early debug for sdram config now too!)
  - sdram setup goes next
  - after sdram is setup, the bss can be cleared
  - board_init_r - pass over to generic SPL code which will load/call
 an image based on boot device

 I think your analyses is correct.


 So, if the above is correct, I should rework the above function as follows:

 void board_init_f(ulong dummy)
 {
 struct ventana_board_info ventana_info;
 int board_model;

 /* Set global data pointer. */
 gd = gdata;

 /* setup AIPS and disable watchdog */
 arch_cpu_init();

 /* iomux and setup of i2c */
 board_early_init_f();
 i2c_setup_iomux();

 /* setup GP timer */
 timer_init();

 /* UART clocks enabled and gd valid - init serial console */
 preloader_console_init();

 /* read/validate EEPROM info to determine board model and SDRAM cfg 
 */
 board_model = read_eeprom(I2C_GSC, ventana_info);

 /* provide some some default: 32bit 128MB */
 if (GW_UNKNOWN == board_model) {
 ventana_info.sdram_width = 2;
 ventana_info.sdram_size = 3;
 }

 /* configure MMDC for SDRAM width/size and per-model calibration */
 spl_dram_init(8  ventana_info.sdram_width,
   16  ventana_info.sdram_size,
   board_model);

 /* Clear the BSS. */
 memset(__bss_start, 0, __bss_end - __bss_start);

 /* load/boot image from boot device */
 board_init_r(NULL, 0);
 }

 It seems reasonable, go on this way.

 Regards,
 Stefano


Stefano,

I've just found that one of my boards fails with the above re-org.
Strangely a board which has the same mem layout, mem width/size, CPU
and nand does not fail.

If I make the following change:
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Tue, May 13, 2014 at 9:58 PM, Tim Harvey thar...@gateworks.com wrote:
 On Wed, May 7, 2014 at 2:29 AM, Stefano Babic sba...@denx.de wrote:
 Hi Tim,

 On 06/05/2014 20:18, Tim Harvey wrote:

 Stefano / York,

 While preparing for a v3 patch series of my IMX6 SPL bootloader, I
 find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks
 the above code because gd is now needed within setup_i2c.

 I've always been a bit fuzzy on the order of the above calls so I dug
 through the code and I think I understand things better. Please
 correct any wrong assumptions I'm making below:
  - assignment to gd should 'always' be first (before anything needs
 it, so why not do it first)
  - arch_cpu_init() should go next as this sets up very low level
 CPU/SoC resources (in this case AIPS config and watchdog disable)
  - board_early_init_f() should be next as that sets up any board-specific 
 iomux
  - any additional iomux necessary for SPL should go next (I take care
 of i2c iomux and setup here)
  - timer_init() next as you need a timer for UART and mxc i2c (for
 delays and busy checks)
  - preloader_console_init() next as we are now able to send something
 over the UART (this gives me early debug for sdram config now too!)
  - sdram setup goes next
  - after sdram is setup, the bss can be cleared
  - board_init_r - pass over to generic SPL code which will load/call
 an image based on boot device

 I think your analyses is correct.


 So, if the above is correct, I should rework the above function as follows:

 void board_init_f(ulong dummy)
 {
 struct ventana_board_info ventana_info;
 int board_model;

 /* Set global data pointer. */
 gd = gdata;

 /* setup AIPS and disable watchdog */
 arch_cpu_init();

 /* iomux and setup of i2c */
 board_early_init_f();
 i2c_setup_iomux();

 /* setup GP timer */
 timer_init();

 /* UART clocks enabled and gd valid - init serial console */
 preloader_console_init();

 /* read/validate EEPROM info to determine board model and SDRAM cfg 
 */
 board_model = read_eeprom(I2C_GSC, ventana_info);

 /* provide some some default: 32bit 128MB */
 if (GW_UNKNOWN == board_model) {
 ventana_info.sdram_width = 2;
 ventana_info.sdram_size = 3;
 }

 /* configure MMDC for SDRAM width/size and per-model calibration */
 spl_dram_init(8  ventana_info.sdram_width,
   16  ventana_info.sdram_size,
   board_model);

 /* Clear the BSS. */
 memset(__bss_start, 0, __bss_end - __bss_start);

 /* load/boot image from boot device */
 board_init_r(NULL, 0);
 }

 It seems reasonable, go on this way.

 Regards,
 Stefano


 Stefano,

 I've just found that one of my boards fails with the above re-org.
 Strangely a board which has the same mem layout, mem width/size, CPU
 and nand does not fail.

 If I make the following change:

(sorry, accidentally fat fingered and sent instead of pasting).  I'll continue:

--- a/board/gateworks/gw_ventana/gw_ventana_spl.c
+++ b/board/gateworks/gw_ventana/gw_ventana_spl.c
@@ -380,9 +380,6 @@ void board_init_f(ulong dummy)
 */
memset((void *)gd, 0, sizeof(struct global_data));

-   /* setup AIPS and disable watchdog */
-   arch_cpu_init();
-
/* iomux and setup of i2c */
board_early_init_f();
i2c_setup_iomux();
@@ -407,6 +404,9 @@ void board_init_f(ulong dummy)
  16  ventana_info.sdram_size,
  board_model);

+   /* setup AIPS and disable watchdog */
+   arch_cpu_init();
+
/* Clear the BSS. */
memset(__bss_start, 0, __bss_end - __bss_start);

Things start to work properly. I took a look at arch_cpu_init() and
the call that really needs to be moved (or just duplicated here) is
mxs_dma_init() to start APBH DMA. The failure mode of the one board is
that apparently it hangs while loading u-boot.img from NAND (which of
course uses mxs_nand and thus APBH DMA).

Anyone know what's going on here and why mxs_dma_init() needs to be
called after MMDC setup, and before clearing the BSS? I don't like
having magic placement of functions without understanding why.

Thanks,

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


Re: [U-Boot] [PATCH v2 0/12] Enable LCD display on snow

2014-05-13 Thread Minkyu Kang
Dear Simon Glass,

On 13/05/14 07:45, Simon Glass wrote:
 Hi,
 
 On 2 April 2014 17:24, Simon Glass s...@chromium.org wrote:
 This series adds a driver for TPS65090 and plumbs it in to get the LCD
 working correctly on snow.

 The display driver is already present, but needs information about the
 display to be provided in the device tree.

 The backlight also needs to be enabled - it is controlled by a FET on
 the TPS65090. Note that the TPS65090 is controlled by the device tree
 so will only be present if the board's device tree file specifies it.
 At present this is only the case for snow, but other exynos5 boards will
 use it (e.g. pit).

 Note: the TPS65090 driver was sent to the list around the middle of last
 year but was never applied.
 
 Is there any news on this series please?
 

I'll check them soon.

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


Re: [U-Boot] [PATCH v2 4/7] mtd: nand: omap_gpmc: remove unused members of 'struct nand_bch_priv'

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch prepares to refactor 'struct nand_bch_priv' - 'struct 
omap_nand_info'
And thus performs following clean-ups:
  - remove nand_bch_priv.type: use nand_bch_priv.ecc_scheme instead
  - remove nand_bch_priv.mode: unused

Signed-off-by: Pekon Gupta pe...@ti.com


Reviewed-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] [PATCH v2 6/7] mtd: nand: omap_gpmc: minor cleanup of omap_correct_data_bch

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch tries to avoid some local pointer dereferences, by using common
local variables in omap_correct_data_bch()

Signed-off-by: Pekon Gupta pe...@ti.com


Reviewed-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] [PATCH v2 7/7] mtd: nand: omap: fix error-codes returned from omap-elm driver

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch
  omap-elm.c: replaces -ve integer value returned during errorneous condition,
  with proper error-codes.
  omap-gpmc.c: updates omap-gpmc driver to pass error-codes returned from
  omap-elm driver to upper layers

Signed-off-by: Pekon Gupta pe...@ti.com


Reviewed-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