July 2012 - U-Boot

[U-Boot] [STATUS] v2012.07 released, Merge Window is OPEN
by Wolfgang Denk 30 Jul '12

30 Jul '12

Hello all, U-Boot v2012.07 has been released and is available from the git repository and the FTP server. The Merge Window for the next release (v2012.10) is open until Sat Aug 18, 2012, 23:59:59 CEST = 19 days remaining. Release "v2012.10" is scheduled in 77 days — on October 15, 2012. A little statistics [1] - changes since release v2012.04: Processed 775 csets from 115 developers 14 employers found A total of 55134 lines added, 11790 removed (delta 43344) Developers with the most changesets Joe Hershberger 90 (11.6%) Fabio Estevam 56 (7.2%) Marek Vasut 42 (5.4%) Stephen Warren 35 (4.5%) SRICHARAN R 29 (3.7%) Tom Rini 25 (3.2%) Mike Frysinger 22 (2.8%) Simon Glass 22 (2.8%) Vipin Kumar 21 (2.7%) Anatolij Gustschin 19 (2.5%) ... Developers with the most changed lines Donghwa Lee 9433 (15.7%) Mike Frysinger 3200 (5.3%) Joe Hershberger 2929 (4.9%) Nobuhiro Iwamatsu 2412 (4.0%) Yen Lin 2098 (3.5%) Stefano Babic 2090 (3.5%) Stefan Roese 2001 (3.3%) Lukasz Majewski 1954 (3.3%) Marek Vasut 1815 (3.0%) Macpaul Lin 1690 (2.8%) ... Developers with the most lines removed Tom Rini 831 (7.0%) Jaehoon Chung 353 (3.0%) Enric Balletbò i Serra 262 (2.2%) Amit Virdi 104 (0.9%) York Sun 51 (0.4%) Timur Tabi 51 (0.4%) Andreas Bießmann 43 (0.4%) Wolfgang Denk 22 (0.2%) Simon Guinot 13 (0.1%) Thomas Chou 6 (0.1%) ... Developers with the most signoffs (total 372) Tom Warren 80 (21.5%) Stefan Roese 46 (12.4%) Kyungmin Park 32 (8.6%) Amit Virdi 31 (8.3%) Andy Fleming 16 (4.3%) Kim Phillips 14 (3.8%) Simon Glass 14 (3.8%) Valentin Longchamp 13 (3.5%) Minkyu Kang 11 (3.0%) Anatolij Gustschin 10 (2.7%) ... Developers with the most reviews (total 2) Joe Hershberger 2 (100.0%) Developers with the most test credits (total 25) Anatolij Gustschin 3 (12.0%) Tom Rini 3 (12.0%) Wolfgang Denk 2 (8.0%) Gary Thomas 2 (8.0%) Pali Rohár 2 (8.0%) Stephen Warren 2 (8.0%) Simon Glass 1 (4.0%) Holger Brunck 1 (4.0%) Jaehoon Chung 1 (4.0%) Simon Guinot 1 (4.0%) ... Developers who gave the most tested-by credits (total 25) Anatolij Gustschin 3 (12.0%) Luka Perkov 3 (12.0%) SRICHARAN R 2 (8.0%) Tetsuyuki Kobayashi 2 (8.0%) u-boot(a)lakedaemon.net 2 (8.0%) Grazvydas Ignotas 2 (8.0%) Timo Ketola 2 (8.0%) Stephen Warren 1 (4.0%) amartin(a)nvidia.com 1 (4.0%) Tom Warren 1 (4.0%) ... Developers with the most report credits (total 12) Isabelle Gros 3 (25.0%) Jerome Angeloni 3 (25.0%) Pali Rohár 1 (8.3%) Marek Vasut 1 (8.3%) Deepak Sikri 1 (8.3%) David Jander 1 (8.3%) Armando Visconti 1 (8.3%) Michael Walle 1 (8.3%) Developers who gave the most report credits (total 12) Nishanth Menon 6 (50.0%) Shiraz Hashim 2 (16.7%) Anatolij Gustschin 1 (8.3%) Mike Frysinger 1 (8.3%) Joe Hershberger 1 (8.3%) Stefano Babic 1 (8.3%) Top changeset contributors by employer (Unknown) 321 (41.4%) Freescale 102 (13.2%) DENX Software Engineering 92 (11.9%) Texas Instruments 83 (10.7%) NVidia 51 (6.6%) Samsung 44 (5.7%) ST Microelectronics 41 (5.3%) Renesas Technology 13 (1.7%) Boundary Devices 12 (1.5%) CompuLab 7 (0.9%) ... Top lines changed by employer (Unknown) 21667 (36.1%) Samsung 13488 (22.5%) DENX Software Engineering 6788 (11.3%) NVidia 4584 (7.6%) Freescale 4044 (6.7%) Texas Instruments 3424 (5.7%) Renesas Technology 2803 (4.7%) ST Microelectronics 1752 (2.9%) Atmel 719 (1.2%) Boundary Devices 426 (0.7%) ... Employers with the most signoffs (total 372) NVidia 80 (21.5%) (Unknown) 64 (17.2%) DENX Software Engineering 60 (16.1%) Freescale 59 (15.9%) Samsung 57 (15.3%) ST Microelectronics 35 (9.4%) Texas Instruments 10 (2.7%) CompuLab 5 (1.3%) Marvell 2 (0.5%) Employers with the most hackers (total 119) (Unknown) 59 (49.6%) Freescale 13 (10.9%) Texas Instruments 11 (9.2%) NVidia 9 (7.6%) DENX Software Engineering 7 (5.9%) Samsung 5 (4.2%) ST Microelectronics 4 (3.4%) CompuLab 2 (1.7%) Renesas Technology 2 (1.7%) Boundary Devices 2 (1.7%) ... [1] See http://www.denx.de/wiki/U-Boot/UbootStat_2012_07 for full statistics, and http://www.denx.de/wiki/UBoot/ReleaseCycle for links to statistics for earlier releases. Thanks to anybody who contributed to this project! 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 "Ahead warp factor 1" - Captain Kirk

1 0

[U-Boot] [PATCH] hawkboard: Fix SPL Nand driver misconfiguration
by Linu Cherian 30 Jul '12

30 Jul '12

Hawkboard was using the wrong nand_read_page version for SPL image. As a side effect, the u-boot image loaded by the SPL from nand was getting corrupted. Enable CONFIG_SYS_NAND_HW_ECC_OOBFIRST to select the correct nand_read_page algorithm for SPL. Signed-off-by: Linu Cherian <linucherian(a)gmail.com> --- include/configs/hawkboard.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/configs/hawkboard.h b/include/configs/hawkboard.h index c6e8859..c6e9ce5 100644 --- a/include/configs/hawkboard.h +++ b/include/configs/hawkboard.h @@ -123,6 +123,7 @@ #define CONFIG_SYS_NAND_USE_FLASH_BBT #define CONFIG_NAND_DAVINCI #define CONFIG_SYS_NAND_4BIT_HW_ECC_OOBFIRST +#define CONFIG_SYS_NAND_HW_ECC_OOBFIRST /* SPL nand driver configuration */ #define CFG_DAVINCI_STD_NAND_LAYOUT #define CONFIG_SYS_NAND_CS 3 #define CONFIG_SYS_NAND_PAGE_2K -- 1.7.1

4 4

[U-Boot] [PATCH] km/common: remove printfs for i2c deblocking code
by Gerlando Falauto 30 Jul '12

30 Jul '12

From: Holger Brunck <holger.brunck(a)keymile.com> This code will also be used before reallocation and during this time we are not allowed to do these printings. Signed-off-by: Holger Brunck <holger.brunck(a)keymile.com> --- board/keymile/common/common.c | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c index 1013f42..a90f112 100644 --- a/board/keymile/common/common.c +++ b/board/keymile/common/common.c @@ -151,7 +151,6 @@ int i2c_make_abort(void) sda_state = get_sda(); if (scl_state && sda_state) { ret = 0; - printf("[INFO] i2c abort after %d clocks\n", i); break; } } @@ -159,8 +158,6 @@ int i2c_make_abort(void) if (ret == 0) for (i = 0; i < 5; i++) i2c_write_start_seq(); - else - printf("[ERROR] i2c abort failed\n"); /* respect stop setup time */ udelay(DELAY_ABORT_SEQ); -- 1.7.1

3 3

[U-Boot] [PATCH 1/3] DaVinci DA8xx: tidy up clock ID definition
by Laurence Withers 30 Jul '12

30 Jul '12

Tidy up the clock IDs defined for the DA8xx SOCs. With this new structure in place, it is clear how to define new clock IDs, and how these map to the numbers presented in the technical reference manual. Signed-off-by: Laurence Withers <lwithers(a)guralp.com> Cc: Prabhakar Lad <prabhakar.csengg(a)gmail.com> --- arch/arm/include/asm/arch-davinci/hardware.h | 53 +++++++++++++++++++------- 1 files changed, 39 insertions(+), 14 deletions(-) diff --git a/arch/arm/include/asm/arch-davinci/hardware.h b/arch/arm/include/asm/arch-davinci/hardware.h index b145c6e..dac43bb 100644 --- a/arch/arm/include/asm/arch-davinci/hardware.h +++ b/arch/arm/include/asm/arch-davinci/hardware.h @@ -441,21 +441,46 @@ struct davinci_pllc_regs { #define davinci_pllc1_regs ((struct davinci_pllc_regs *)DAVINCI_PLL_CNTRL1_BASE) #define DAVINCI_PLLC_DIV_MASK 0x1f -#define ASYNC3 get_async3_src() -#define PLL1_SYSCLK2 ((1 << 16) | 0x2) -#define DAVINCI_SPI1_CLKID (cpu_is_da830() ? 2 : ASYNC3) -/* Clock IDs */ +/* + * A clock ID is a 32-bit number where bit 16 represents the PLL controller + * (clear is PLLC0, set is PLLC1) and the low 16 bits represent the divisor, + * counting from 1. Clock IDs may be passed to clk_get(). + */ + +/* flags to select PLL controller */ +#define DAVINCI_PLLC0_FLAG (0) +#define DAVINCI_PLLC1_FLAG (1 << 16) + enum davinci_clk_ids { - DAVINCI_SPI0_CLKID = 2, - DAVINCI_UART2_CLKID = 2, - DAVINCI_MMC_CLKID = 2, - DAVINCI_MDIO_CLKID = 4, - DAVINCI_ARM_CLKID = 6, - DAVINCI_PLLM_CLKID = 0xff, - DAVINCI_PLLC_CLKID = 0x100, - DAVINCI_AUXCLK_CLKID = 0x101 + /* + * Clock IDs for PLL outputs. Each may be switched on/off independently, + * and each may map to one or more peripherals. + */ + DAVINCI_PLL0_SYSCLK2 = DAVINCI_PLLC0_FLAG | 2, + DAVINCI_PLL0_SYSCLK4 = DAVINCI_PLLC0_FLAG | 4, + DAVINCI_PLL0_SYSCLK6 = DAVINCI_PLLC0_FLAG | 6, + DAVINCI_PLL1_SYSCLK2 = DAVINCI_PLLC1_FLAG | 2, + + /* map peripherals to clock IDs */ + DAVINCI_ARM_CLKID = DAVINCI_PLL0_SYSCLK6, + DAVINCI_MDIO_CLKID = DAVINCI_PLL0_SYSCLK4, + DAVINCI_MMC_CLKID = DAVINCI_PLL0_SYSCLK2, + DAVINCI_SPI0_CLKID = DAVINCI_PLL0_SYSCLK2, + DAVINCI_UART2_CLKID = DAVINCI_PLL0_SYSCLK2, + + /* special clock ID - output of PLL multiplier */ + DAVINCI_PLLM_CLKID = 0x0FF, + + /* special clock ID - output of PLL post divisor */ + DAVINCI_PLLC_CLKID = 0x100, + + /* special clock ID - PLL bypass */ + DAVINCI_AUXCLK_CLKID = 0x101, }; +#define DAVINCI_SPI1_CLKID (cpu_is_da830() ? DAVINCI_PLL0_SYSCLK2 \ + : get_async3_src()) + int clk_get(enum davinci_clk_ids id); /* Boot config */ @@ -570,10 +595,10 @@ static inline int cpu_is_da850(void) return ((part_no == 0xb7d1) ? 1 : 0); } -static inline int get_async3_src(void) +static inline enum davinci_clk_ids get_async3_src(void) { return (REG(&davinci_syscfg_regs->cfgchip3) & 0x10) ? - PLL1_SYSCLK2 : 2; + DAVINCI_PLL1_SYSCLK2 : DAVINCI_PLL0_SYSCLK2; } #endif /* CONFIG_SOC_DA8XX */ -- 1.7.2.5

2 4

[U-Boot] multichannel mmc strangeness on omap
by Steve Sakoman 30 Jul '12

30 Jul '12

Does anyone have any experience enabling the second or third mmc channels in u-boot on an omap3 board? I used the feature successfully a couple of years ago on Overo, but of course there have been many changes in the mmc code since then. Here's what I tried for overo with the current head of tree: diff --git a/board/overo/overo.c b/board/overo/overo.c index f973870..51315b3 100644 --- a/board/overo/overo.c +++ b/board/overo/overo.c @@ -404,6 +404,7 @@ int board_eth_init(bd_t *bis) int board_mmc_init(bd_t *bis) { omap_mmc_init(0, 0, 0); + omap_mmc_init(2, 0, 0); return 0; } #endif The result is a hang after printing: MMC: OMAP SD/MMC 0, SD/MMC 1 After adding a few printf's, it seems that the crash/hang is occurring when calling env_relocate_spec in env_nand.c, which of course is completely unrelated code! Just for grins I tried enabling the second mmc channel instead (i.e change omap_mmc_init(2, 0, 0) to omap_mmc_init(1, 0, 0)). This worked as expected. I also tried enabling all three, which resulted in the same crash. So enabling the third mmc channel seems to be the issue. Before I start debugging I thought I would check to see if anyone else has seen this or has any theories as to why it is happening. Steve

2 2

[U-Boot] [PATCH] DaVinci DA8xx: fix set_cpu_clk_info()
by Laurence Withers 30 Jul '12

30 Jul '12

For the DA8xx family of SoCs, the set_cpu_clk_info() function was not initialising the DSP frequency, leading to 'bdinfo' command output such as: [...snip...] ARM frequency = 300 MHz DSP frequency = -536870913 MHz DDR frequency = 300 MHz This commit provides a separate implementation of set_cpu_clk_info() for the DA8xx SoCs that initialises the DSP frequency to zero (since currently the DSP is not enabled by U-Boot on any DA8xx platform). The separate implementation is justified because there is no common code between DA8xx and the other SoC families. It is now much easier to understand the flow of the two separate functions. Signed-off-by: Laurence Withers <lwithers(a)guralp.com> Cc: Tom Rini <trini(a)ti.com> Cc: Hadli, Manjunath <manjunath.hadli(a)ti.com> Cc: Heiko Schocher <hs(a)denx.de> --- arch/arm/cpu/arm926ejs/davinci/cpu.c | 21 +++++++++++++-------- 1 files changed, 13 insertions(+), 8 deletions(-) diff --git a/arch/arm/cpu/arm926ejs/davinci/cpu.c b/arch/arm/cpu/arm926ejs/davinci/cpu.c index 6cb857a..4bdb08b 100644 --- a/arch/arm/cpu/arm926ejs/davinci/cpu.c +++ b/arch/arm/cpu/arm926ejs/davinci/cpu.c @@ -117,6 +117,16 @@ int clk_get(enum davinci_clk_ids id) out: return pll_out; } + +int set_cpu_clk_info(void) +{ + gd->bd->bi_arm_freq = clk_get(DAVINCI_ARM_CLKID) / 1000000; + /* DDR PHY uses an x2 input clock */ + gd->bd->bi_ddr_freq = clk_get(0x10001) / 1000000; + gd->bd->bi_dsp_freq = 0; + return 0; +} + #else /* CONFIG_SOC_DA8XX */ static unsigned pll_div(volatile void *pllbase, unsigned offset) @@ -187,16 +197,9 @@ unsigned int davinci_clk_get(unsigned int div) return pll_sysclk_mhz(DAVINCI_PLL_CNTRL0_BASE, div) * 1000000; } #endif -#endif /* !CONFIG_SOC_DA8XX */ int set_cpu_clk_info(void) { -#ifdef CONFIG_SOC_DA8XX - gd->bd->bi_arm_freq = clk_get(DAVINCI_ARM_CLKID) / 1000000; - /* DDR PHY uses an x2 input clock */ - gd->bd->bi_ddr_freq = clk_get(0x10001) / 1000000; -#else - unsigned int pllbase = DAVINCI_PLL_CNTRL0_BASE; #if defined(CONFIG_SOC_DM365) pllbase = DAVINCI_PLL_CNTRL1_BASE; @@ -215,10 +218,12 @@ int set_cpu_clk_info(void) pllbase = DAVINCI_PLL_CNTRL0_BASE; #endif gd->bd->bi_ddr_freq = pll_sysclk_mhz(pllbase, DDR_PLLDIV) / 2; -#endif + return 0; } +#endif /* !CONFIG_SOC_DA8XX */ + /* * Initializes on-chip ethernet controllers. * to override, implement board_eth_init() -- 1.7.2.5

2 2

[U-Boot] [PATCH 0/8 V7] EXYNOS5: Enable I2C support
by Rajeshwari Shinde 30 Jul '12

30 Jul '12

This patch set enables I2C support for EXYNOS5. This patchset modifies the s3c24x0 I2C driver to use same for EXYNOS5. Multichannel support has been added to the s3c24x0 I2C driver. s3c24x0_i2c struct has been moved to a common place as it can used by different SOC's. Changes in V2: - Incorporated review comments from Simon Glass. - Aligned the pinmux functionality as per latest comments Changes in V3: - Incorporated review comments from Joonyoung Shim for I2C driver. Changes in V4: - Resolved build error for S3C2410. Changes in V5: - Pinmux setting moved to board file. - Multi Bus I2C offset calculation done based on EXYNOS_I2C_SPACE. - Peripheral id calculation removed from i2c driver. Changes in V6: - Incorporated review comments from Simon Glass. Changes in V7: - Incorporated review comments from Joonyoung Shim. Rajeshwari Shinde (8): EXYNOS: CLK: Add i2c clock EXYNOS: Add I2C base address. EXYNOS5: define EXYNOS5_I2C_SPACING EXYNOS: PINMUX: Add pinmux support for I2C I2C: Move struct s3c24x0_i2c to a common place. I2C: Modify the I2C driver for EXYNOS5 I2C: Add support for Multi channel CONFIG: SMDK5250: I2C: Enable I2C arch/arm/cpu/armv7/exynos/clock.c | 33 ++++ arch/arm/cpu/armv7/exynos/pinmux.c | 52 +++++++ arch/arm/include/asm/arch-exynos/clk.h | 1 + arch/arm/include/asm/arch-exynos/cpu.h | 5 + arch/arm/include/asm/arch-exynos/periph.h | 8 + arch/arm/include/asm/arch-s3c24x0/s3c24x0.h | 10 -- board/samsung/smdk5250/smdk5250.c | 30 ++++- drivers/i2c/s3c24x0_i2c.c | 221 +++++++++++++++++--------- drivers/i2c/s3c24x0_i2c.h | 33 ++++ include/configs/smdk5250.h | 10 ++ 10 files changed, 315 insertions(+), 88 deletions(-) create mode 100644 drivers/i2c/s3c24x0_i2c.h -- 1.7.4.4

2 9

[U-Boot] powerdown can't work (mx28)
by alex 30 Jul '12

30 Jul '12

Hi: I find that mx28_powerdown function can't power down mx28, and continue to load kernel.... I don't know whether this issue is on your side. My board is changed frommx28evk. Was mx28_powerdown function tested? Best Regards, Alex

2 3

[U-Boot] Pull request: u-boot-mips/master for v2012.07
by daniel.schwierzeck＠gmail.com 30 Jul '12

30 Jul '12

Dear Wolfgang, please pull a small fix for v2012.07 The following changes since commit c627faf637f5fe091bdb6846a52b16983e97b262: Prepare v2012.07-rc3 (2012-07-27 08:27:18 +0200) are available in the git repository at: git://git.denx.de/u-boot-mips.git master for you to fetch changes up to 8b425b3f8e345b07ce7f0005bb0222970bd3b248: MIPS: fix renaming of inca-swap-bytes to xway-swap-bytes (2012-07-27 23:44:24 +0200) ---------------------------------------------------------------- Daniel Schwierzeck (1): MIPS: fix renaming of inca-swap-bytes to xway-swap-bytes Makefile | 2 +- tools/.gitignore | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

2 1

[U-Boot] [PATCH] dm: ppc: Fixup cpu.c warning in mpc8xx
by Marek Vasut 30 Jul '12

30 Jul '12

cpu.c: In function ‘check_CPU’: cpu.c:256:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Marek Vasut <marex(a)denx.de> Cc: Wolfgang Denk <wd(a)denx.de> --- arch/powerpc/cpu/mpc8xx/cpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/cpu/mpc8xx/cpu.c b/arch/powerpc/cpu/mpc8xx/cpu.c index 5cbf9a6..e453245 100644 --- a/arch/powerpc/cpu/mpc8xx/cpu.c +++ b/arch/powerpc/cpu/mpc8xx/cpu.c @@ -41,6 +41,7 @@ #include <netdev.h> #include <asm/cache.h> #include <linux/compiler.h> +#include <asm/io.h> #if defined(CONFIG_OF_LIBFDT) #include <libfdt.h> @@ -253,7 +254,7 @@ static int check_CPU (long clock, uint pvr, uint immr) if ((pvr >> 16) != 0x0050) return -1; - k = (immr << 16) | *((ushort *) & immap->im_cpm.cp_dparam[0xB0]); + k = (immr << 16) | readw(&immap->im_cpm.cp_dparam[0xB0]); m = 0; switch (k) { -- 1.7.10.4

2 7