December 2009 - U-Boot

[U-Boot] i2c on mx27
by alfred steele 06 Dec '09

06 Dec '09

Hi, Has anyone tried playing with the hardware i2c on the mx27 platform using u-boot. I am trying to enable the second controller and use hardware i2c by doing memory writes to the appropiate registers using "mw". Looks like i am missing out on some register values. I am unable to toggle the signals on the I2C2_CLK and I2C2_data lines. I have modifed the mux registers and the pad registers and the Clock gating registers appropiately. Any hints? Thanks, Alfred.

1 0

[U-Boot] [PATCH] trab: fix warning: implicit declaration of function 'disable_vfd'
by Wolfgang Denk 06 Dec '09

06 Dec '09

Signed-off-by: Wolfgang Denk <wd(a)denx.de> --- cpu/arm920t/s3c24x0/timer.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/cpu/arm920t/s3c24x0/timer.c b/cpu/arm920t/s3c24x0/timer.c index 20cedd4..a27f0e2 100644 --- a/cpu/arm920t/s3c24x0/timer.c +++ b/cpu/arm920t/s3c24x0/timer.c @@ -205,6 +205,8 @@ void reset_cpu(ulong ignored) struct s3c24x0_watchdog *watchdog; #ifdef CONFIG_TRAB + extern void disable_vfd(void); + disable_vfd(); #endif -- 1.6.2.5

1 0

[U-Boot] [PATCH] zlib.c: avoid build conflicts for cradle board
by Wolfgang Denk 06 Dec '09

06 Dec '09

Commit dce3d79710 updated the zlib code to v0.95; this caused conflicts when building for the "cradle" board, because the (pretty generic) preprocessor variable "OFF" was used in multiple files. Make sure to avoid further conflicts by #undef'ing it in zlib.c before redefining it. Signed-off-by: Wolfgang Denk <wd(a)denx.de> cc: Giuseppe Condorelli <giuseppe.condorelli(a)st.com> cc: Angelo Castello <angelo.castello(a)st.com> cc: Alessandro Rubini <rubini-list(a)gnudd.com> --- lib_generic/zlib.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/lib_generic/zlib.c b/lib_generic/zlib.c index 8fe3bd0..ef1fa22 100644 --- a/lib_generic/zlib.c +++ b/lib_generic/zlib.c @@ -28,6 +28,8 @@ #include "u-boot/zlib.h" #include <common.h> +#undef OFF /* avoid conflicts */ + /* To avoid a build time warning */ #ifdef STDC #include <malloc.h> -- 1.6.2.5

1 0

[U-Boot] "MAKEALL: Add summary information" breaks MAKEALL return code
by Wolfgang Denk 06 Dec '09

06 Dec '09

Dear Peter, your commit 40a28f0885 (MAKEALL: Add summary information) breaks MAKEALL because we now don't get any useful return code any more. You may argument that we didn't get one before, which is correct for building multiple boards in one call. But when running something like "MAKEALL <board>" we did get a return code, so wee could use this for example with "git bisect run". One possible fix could look like this: diff --git a/MAKEALL b/MAKEALL index d63c5c2..0cafed8 100755 --- a/MAKEALL +++ b/MAKEALL @@ -952,13 +952,16 @@ build_target() { #----------------------------------------------------------------------- print_stats() { + rc=0 echo "" echo "--------------------- SUMMARY ----------------------------" echo "Boards compiled: ${TOTAL_CNT}" if [ ${ERR_CNT} -gt 0 ] ; then + rc=1 echo "Boards with warnings or errors: ${ERR_CNT} (${ERR_LIST} )" fi echo "----------------------------------------------------------" + exit $rc } #----------------------------------------------------------------------- But I don't really like this approach, as it still changes behaviour: now even warnings get accounted as errors. What do you think? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd(a)denx.de An expert is a person who avoids the small errors while sweeping on to the grand fallacy.

2 1

[U-Boot] [PATCH] Fix out-of-tree building of "apollon" board.
by Wolfgang Denk 06 Dec '09

06 Dec '09

Signed-off-by: Wolfgang Denk <wd(a)denx.de> --- Makefile | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile index bcb3fe9..10abadf 100644 --- a/Makefile +++ b/Makefile @@ -3257,6 +3257,7 @@ zylonite_config : ######################################################################### apollon_config : unconfig + @mkdir -p $(obj)include @echo "#define CONFIG_ONENAND_U_BOOT" > $(obj)include/config.h @$(MKCONFIG) $(@:_config=) arm arm1136 apollon NULL omap24xx @echo "CONFIG_ONENAND_U_BOOT = y" >> $(obj)include/config.mk -- 1.6.2.5

1 0

[U-Boot] Machine check exception on video driver initialization
by Thirumalai 05 Dec '09

05 Dec '09

Hi Stefan, I am getting Machine check exception when my video driver is initializing. When i searched the net i found that you also faced the same kind of problem. I have mentioned the VIDEO_IO_OFFSET as 0xf0000000. This address is nothing but CONFIG_SYS_PCI1_IO_VIRT which is mentioned on my config file. Here the log of my output. ####################### U-Boot 2009.06 (Dec 05 2009 - 12:06:50) Data Patterns DPVPX0689 SBC COTS Module CPU: 8641, Version: 2.1, (0x80900021) Core: E600 Core 0, Version: 2.2, (0x80040202) Clock Configuration: CPU:800 MHz, MPX:400 MHz DDR:200 MHz (400 MT/s data rate), LBC:25 MHz L1: D-cache 32 KB enabled I-cache 32 KB enabled L2: 1024 KB enabled Board: DPVPX0689 SBC I2C: ready DRAM: DDR: 1 GB Now running in RAM - U-Boot at: 3fe70000 FLASH: 64 MB *** Warning - bad CRC, using default environment PCI-EXPRESS 1: Host 0xf6008020=0x00000000 ...PCIE LTSSM=0x16, Negotiated link width=8 Scanning PCI bus 01 03 00 1000 0057 0100 00 02 01 10b5 8648 0604 00 06 00 1002 4c66 0300 ff 05 01 10b5 6540 0604 00 04 00 12d8 e130 0604 00 02 02 10b5 8648 0604 00 02 04 10b5 8648 0604 00 09 01 1131 1561 0c03 00 09 01 1131 1561 0c03 00 09 01 1131 1562 0c03 00 08 00 12d8 e130 0604 00 02 05 10b5 8648 0604 00 0a 00 12d8 e130 0604 00 02 06 10b5 8648 0604 00 02 08 10b5 8648 0604 00 02 09 10b5 8648 0604 00 02 0a 10b5 8648 0604 00 02 0b 10b5 8648 0604 00 01 00 10b5 8648 0604 00 PCI-EXPRESS 1 on bus 00 - 0f SRIO: Disabled Video: ATI Radeon video card (1002, 4c66) found @(6:0:0) rinfo->mmio_base = 0x50000000 rinfo->fb_local_base = 0xefc00000 videoboot: Booting PCI video card bus 6, function 0, device 0 Machine check in kernel mode. Caused by (from msr): regs 3fa6dba8 MSS error. MSSSR0: 00001000 NIP: 3FE81E1C XER: 20000000 LR: 3FE81D14 REGS: 3fa6dba8 TRAP: 0200 DAR: 00000000 MSR: 00101030 EE: 0 PR: 0 FP: 0 ME: 1 IR/DR: 11 GPR00: 00000000 3FA6DC98 3FA6DF68 000003BA 00000005 3FA6DCB4 00000000 00000001 GPR08: 00000006 F00003BA 00000000 00000000 00000000 FFFFFFFF 3FEE5900 3FF70000 GPR16: FFFFFFBF FFBF7BFF FFFFFFEB FFFFDFFF 00001032 00000001 00000000 3FE722C4 GPR24: 3FEFC468 3FEFC468 3FEFC468 3FEFC468 3FEDE704 3FEFC468 3FEE6058 000003BA Call backtrace: 3FE82CCC 3FE900E4 3FE82AF8 3FE80EEC 3FE78998 3FE78DAC 3FE78E88 3FECD684 3FECD730 3FECC290 3FECC434 3FEB36E8 3FE7BDE8 3FE72560 machine check Resetting board.... Thanks.

1 0

[U-Boot] [PATCH] Removes dead code in the file common/cmd_i2c.c
by Pratap Chandu 05 Dec '09

05 Dec '09

There is some dead code enclosed by #if 0 .... #endif in the file common/cmd_i2c.c This patch removes the dead code. Signed-off-by: Pratap Chandu <pratap.rrke(a)gmail.com> --- common/cmd_i2c.c | 8 -------- 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c index 8f0fc9e..8d6feda 100644 --- a/common/cmd_i2c.c +++ b/common/cmd_i2c.c @@ -326,14 +326,6 @@ int do_i2c_mw ( cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) #if !defined(CONFIG_SYS_I2C_FRAM) udelay(11000); #endif - -#if 0 - for (timeout = 0; timeout < 10; timeout++) { - udelay(2000); - if (i2c_probe(chip) == 0) - break; - } -#endif } return (0); -- 1.5.6.5

3 4

[U-Boot] [PATCH] samsung: fix DMC1_MEM_CFG for s3c64xx
by "Seunghyeon Rhee (이승현)" 05 Dec '09

05 Dec '09

The MSB of DMC1_MEM_CFG can be set to '1' for separate CKE control for S3C6400. In the configuration of SMDK6400, however, two 16-bit mDDR (SAMSUNG K4X51163) chips are used in parallel to form 32-bit memory bus and there is no need to contorl CKE for each chip separately. AFAIK, CKE1 is not at all connected. Only CKE0 is used. Futhermore, it should be '0' always for S3C6410. When tested with a board which has a S3C6410 and the same memory configuration, a side effect is obsearved that u-boot command "reset" doesn't work leading to system hang. Leaving the bit clear is safe in most cases. Signed-off-by: Seunghyeon Rhee <seunghyeon(a)lpmtec.com> --- include/s3c6400.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/s3c6400.h b/include/s3c6400.h index e527c08..7229ea6 100644 --- a/include/s3c6400.h +++ b/include/s3c6400.h @@ -817,7 +817,7 @@ /*----------------------------------------------------------------------- * Physical Memory Map */ -#define DMC1_MEM_CFG 0x80010012 /* Chip1, Burst4, Row/Column bit */ +#define DMC1_MEM_CFG 0x00010012 /* Chip1, Burst4, Row/Column bit */ #define DMC1_MEM_CFG2 0xB45 #define DMC1_CHIP0_CFG 0x150F8 /* 0x4000_0000 ~ 0x43ff_ffff (64MB) */ #define DMC_DDR_32_CFG 0x0 /* 32bit, DDR */ -- 1.6.2.5 -- Seunghyeon Rhee, Ph.D. / Director LPM Technology Inc. T +82-70-8255-6007 F +82-2-6442-6462 M +82-10-2790-0657

4 7

[U-Boot] [PATCH] mpc83xx: spd_sdram.c: Disable memory controller before initializing
by Stefan Roese 05 Dec '09

05 Dec '09

The memory controller could already be enabled, when spd_sdram() is called. This could be the case for example, when the SDRAM is initialized by the JTAG debugger. Signed-off-by: Stefan Roese <sr(a)denx.de> Cc: Reinhard Arlt <reinhard.arlt(a)esd.eu> Cc: Kim Phillips <kim.phillips(a)freescale.com> --- cpu/mpc83xx/spd_sdram.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/cpu/mpc83xx/spd_sdram.c b/cpu/mpc83xx/spd_sdram.c index 0f61180..952afeb 100644 --- a/cpu/mpc83xx/spd_sdram.c +++ b/cpu/mpc83xx/spd_sdram.c @@ -150,6 +150,13 @@ long int spd_sdram() unsigned int ddrc_ecc_enable; unsigned int pvr = get_pvr(); + /* + * First disable the memory controller (could be enabled + * by the debugger) + */ + ddr->sdram_cfg &= ~SDRAM_CFG_MEM_EN; + asm("sync;isync"); + /* Read SPD parameters with I2C */ CONFIG_SYS_READ_SPD(SPD_EEPROM_ADDRESS, 0, 1, (uchar *) & spd, sizeof (spd)); #ifdef SPD_DEBUG -- 1.6.6.rc1

3 2

Re: [U-Boot] Clean-up of s3c24x0 header files
by Wolfgang Denk 05 Dec '09

05 Dec '09

Hi Kevin, looking at the code resulting from commit 9ebfdc20 again, I see that you keep a LOT of uppercase variable names, which are not allowed as per coding style. DO you intend to convert these to proper lower-case names any time soon? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd(a)denx.de A father doesn't destroy his children. -- Lt. Carolyn Palamas, "Who Mourns for Adonais?", stardate 3468.1.

2 2