Re: [U-Boot] [PATCH 3/3] am33xx: Add SPI SPL as an option

2012-10-16 Thread Peter Korsgaard
> "Tom" == Tom Rini writes: Tom> Add the required config.mk logic for this SoC as well as the BOOT_DEVICE Tom> define. Finally, enable the options on the am335x_evm. Tom> Signed-off-by: Tom Rini Tom> --- Tom> arch/arm/cpu/armv7/am33xx/config.mk|1 + Tom> arch/arm/include/asm

Re: [U-Boot] [PATCH 2/6 V5] EXYNOS: Add clock for SPI.

2012-10-16 Thread Hatim Rv
Dear Minkyu Kang, On Wed, Oct 17, 2012 at 10:55 AM, Minkyu Kang wrote: > Dear Hatim, > > On 17 October 2012 14:21, Hatim Rv wrote: diff --git a/arch/arm/include/asm/arch-exynos/clk.h b/arch/arm/include/asm/arch-exynos/clk.h index 5529025..4e51402 100644 --- a/arch/arm/includ

Re: [U-Boot] [PATCH] powerpc/fm: fix TBI PHY address settings

2012-10-16 Thread Xie Shaohui-B21989
> Xie Shaohui-B21989 wrote: > > >> Is this still true with Fman v3? > > [S.H] No. Fman V3 does not need to set TBIPA on FM1@DTSEC1. > > Does this patch break Fman V3? [S.H] No. Fman V3 uses following code: static void dtsec_init_phy(struct eth_device *dev) { struct fm_eth *fm_eth = dev

Re: [U-Boot] [PATCH 0/9 V2] EXYNOS5: Add Audio support

2012-10-16 Thread Rajeshwari Birje
Hi All, Please do let me know if any comments on these patch set. Regards, Rajeshwari Shinde. On Thu, Sep 13, 2012 at 11:07 AM, Rajeshwari Birje wrote: > Hi All, > > Do let me know if any more comments on this patchset. > > Regards, > Rajeshwari Shinde. > > > On Wed, Aug 22, 2012 at 12:14 PM,

Re: [U-Boot] [PATCH 2/6 V5] EXYNOS: Add clock for SPI.

2012-10-16 Thread Minkyu Kang
Dear Hatim, On 17 October 2012 14:21, Hatim Rv wrote: >>> diff --git a/arch/arm/include/asm/arch-exynos/clk.h >>> b/arch/arm/include/asm/arch-exynos/clk.h >>> index 5529025..4e51402 100644 >>> --- a/arch/arm/include/asm/arch-exynos/clk.h >>> +++ b/arch/arm/include/asm/arch-exynos/clk.h >>> @@ -2

Re: [U-Boot] [PATCH 2/6 V5] EXYNOS: Add clock for SPI.

2012-10-16 Thread Hatim Rv
Dear Minkyu Kang, On Wed, Oct 17, 2012 at 7:36 AM, Minkyu Kang wrote: > Dear Hatim, > > On 16 October 2012 19:58, Hatim Ali wrote: >> From: Rajeshwari Shinde >> >> This patch adds api to calculate and set the clock for SPI channels >> >> Signed-off-by: James Miller >> Signed-off-by: Simon Glas

[U-Boot] [PATCH 6/6 V6] EXYNOS5: Enable SPI booting.

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde This patch enables SPI Booting for EXYNOS5 Signed-off-by: Rajeshwari Shinde --- Changes since v4: No Change Changes since v5: No Change board/samsung/smdk5250/Makefile |2 +- board/samsung/smdk5250/mmc_boot.c | 58 - boar

[U-Boot] [PATCH 5/6 V6] EXYNOS5: Enable SPI

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde This patch enables SPI driver for EXYNOS5. Signed-off-by: Rajeshwari Shinde Signed-off-by: Hatim Ali --- Changes since v4: - Rebased on u-boot-samsung.git Changes since v5: No change board/samsung/smdk5250/smdk5250.c |4 include/configs/smdk52

[U-Boot] [PATCH 4/6 V6] SPI: Add SPI Driver for EXYNOS.

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde This patch adds SPI driver for EXYNOS. Signed-off-by: Simon Glass Signed-off-by: Padmavathi Venna Signed-off-by: Gabe Black Signed-off-by: Rajeshwari Shinde Signed-off-by: Hatim Ali Acked-by: Mike Frysinger Tested-by: jy0922.s...@samsung.com --- Changes since V4:

[U-Boot] [PATCH 3/6 V6] EXYNOS5: Add base address for SPI.

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde Signed-off-by: Rajeshwari Shinde Acked-by: Simon Glass --- Changes since v4: - Rebased on u-boot-samsung.git - Added Acked-by Simon Glass Changes since v5: No Change arch/arm/include/asm/arch-exynos/cpu.h |6 ++ 1 files changed, 6 insert

[U-Boot] [PATCH 2/6 V6] EXYNOS: Add clock for SPI.

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde This patch adds api to calculate and set the clock for SPI channels Signed-off-by: James Miller Signed-off-by: Simon Glass Signed-off-by: Rajeshwari Shinde Signed-off-by: Hatim Ali --- Changes since v4: Added Signed-off-by of James Miller Changes since v5:

[U-Boot] [PATCH 1/6 V6] EXYNOS5: Add pinmux support for SPI

2012-10-16 Thread Hatim Ali
From: Rajeshwari Shinde This patch adds pinmux support for SPI channels Signed-off-by: Rajeshwari Shinde Signed-off-by: Hatim Ali --- Changes since v4: Fixed minor nits suggested by Simon Glass Changes since v5: No change arch/arm/cpu/armv7/exynos/pinmux.c| 51 +

[U-Boot] [PATCH 0/6 V6] EXYNOS5: Enable SPI support

2012-10-16 Thread Hatim Ali
This patch set adds SPI driver for EXYNOS5 and enables same. This patch set is based on latest Mainline u-boot.git tree. Changes in V2: - Correted the Commit message. Changes in V3: - Removed SPI_SLAVE Flag. - Corrected warning messages. Changes in V4: - Rebased on

Re: [U-Boot] [PATCH 27/32] drivers/i2c/fsl_i2c.c: sparse fix

2012-10-16 Thread Heiko Schocher
Helo Kim, On 17.10.2012 02:28, Kim Phillips wrote: fsl_i2c.c:217:14: warning: symbol 'get_i2c_clock' was not declared. Should it be static? Signed-off-by: Kim Phillips --- drivers/i2c/fsl_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Heiko Schocher Thanks! bye,

[U-Boot] [i2c] Pull request

2012-10-16 Thread Heiko Schocher
Hello Tom, The following changes since commit bd23b22badadcdc414a900828253961fc5ec6c39: Merge branch 'ag...@denx.de-next' of git://git.denx.de/u-boot-staging (2012-10-15 13:37:22 -0700) are available in the git repository at: git://git.denx.de/u-boot-i2c.git master Eric Nelson (1):

Re: [U-Boot] [PATCH 2/6 V5] EXYNOS: Add clock for SPI.

2012-10-16 Thread Minkyu Kang
Dear Hatim, On 16 October 2012 19:58, Hatim Ali wrote: > From: Rajeshwari Shinde > > This patch adds api to calculate and set the clock for SPI channels > > Signed-off-by: James Miller > Signed-off-by: Simon Glass > Signed-off-by: Rajeshwari Shinde > --- > > Changes since v4: > Added

[U-Boot] [PATCH 27/32] drivers/i2c/fsl_i2c.c: sparse fix

2012-10-16 Thread Kim Phillips
fsl_i2c.c:217:14: warning: symbol 'get_i2c_clock' was not declared. Should it be static? Signed-off-by: Kim Phillips --- drivers/i2c/fsl_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/fsl_i2c.c b/drivers/i2c/fsl_i2c.c index 5b017a9..3cb232f 100644 --- a/dr

[U-Boot] [PATCH 10/32] include/command.h: sparse fix

2012-10-16 Thread Kim Phillips
__u_boot_cmd_* should be declared static, e.g.: cmd_mem.c:1142:1: warning: symbol '__u_boot_cmd_md' was not declared. Should it be static? cmd_mem.c:1149:1: warning: symbol '__u_boot_cmd_mm' was not declared. Should it be static? cmd_mem.c:1156:1: warning: symbol '__u_boot_cmd_nm' was not declar

[U-Boot] [PATCH 20/32] arch/powerpc/cpu/mpc8xxx/: sparse fixes

2012-10-16 Thread Kim Phillips
ctrl_regs.c:31:5: warning: symbol 'fsl_ddr_get_version' was not declared. Should it be static? cpu.c:135:14: warning: non-ANSI function declaration of function 'cpu_mask' cpu.c:154:18: warning: non-ANSI function declaration of function 'cpu_numcores' cpu.c:37:17: warning: symbol 'cpu_type_list' wa

[U-Boot] [PATCH 11/32] include/image.h: sparse fixes

2012-10-16 Thread Kim Phillips
include/image.h:378:1: warning: cast to restricted __be32 include/image.h:381:1: warning: cast to restricted __be32 include/image.h:382:1: warning: cast to restricted __be32 include/image.h:383:1: warning: cast to restricted __be32 fix by annotating image header integers big endian. Signed-off-by

[U-Boot] [PATCH 06/32] arch/powerpc/lib/bootm.c: fix noinline attribute

2012-10-16 Thread Kim Phillips
including compiler.h in byteorder changes the 'noinline' definition to expand to __attribute__((noinline)). This fixes: bootm.c:329:16: error: attribute '__attribute__': unknown attribute bootm.c:329:16: error: expected ')' before '__attribute__' bootm.c:329:25: error: expected identifier or '('

[U-Boot] [PATCH 24/32] drivers/block/: sparse fixes

2012-10-16 Thread Kim Phillips
sata_sil.c:371:7: warning: symbol 'sil_sata_rw_lba28' was not declared. Should it be static? sata_sil.c:399:7: warning: symbol 'sil_sata_rw_lba48' was not declared. Should it be static? sata_sil.c:429:6: warning: symbol 'sil_sata_cmd_flush_cache' was not declared. Should it be static? sata_sil.c

[U-Boot] [PATCH 22/32] powerpc/mpc85xx: sparse fixes

2012-10-16 Thread Kim Phillips
fsl_corenet_serdes.c:485:6: warning: symbol '__soc_serdes_init' was not declared. Should it be static? cpu_init.c:185:6: warning: symbol 'invalidate_cpc' was not declared. Should it be static? bcsr.c:28:27: warning: non-ANSI function declaration of function 'enable_8568mds_duart' bcsr.c:39:33: w

[U-Boot] [PATCH 18/32] lib/vsprintf.c: sparse fixes

2012-10-16 Thread Kim Phillips
vsprintf.c:31:12: warning: symbol 'hex_asc' was not declared. Should it be static? vsprintf.c:398:18: warning: Using plain integer as NULL pointer Signed-off-by: Kim Phillips --- lib/vsprintf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/vsprintf.c b/lib/vsprintf

[U-Boot] [PATCH 14/32] common/fdt_support.c: sparse fixes

2012-10-16 Thread Kim Phillips
trivial: fdt_support.c:89:64: warning: Using plain integer as NULL pointer fdt_support.c:325:65: warning: Using plain integer as NULL pointer fdt_support.c:352:65: warning: Using plain integer as NULL pointer For the following bad constant expression, We hardcode the max. number of memory banks t

[U-Boot] [PATCH 28/32] drivers/mmc/mmc.c: sparse fixes

2012-10-16 Thread Kim Phillips
mmc.c:137:5: warning: symbol 'mmc_send_cmd' was not declared. Should it be static? mmc.c:203:5: warning: symbol 'mmc_send_status' was not declared. Should it be static? mmc.c:247:5: warning: symbol 'mmc_set_blocklen' was not declared. Should it be static? mmc.c:440:5: warning: symbol 'mmc_read_b

[U-Boot] [PATCH 26/32] drivers/input/input.c: sparse fix

2012-10-16 Thread Kim Phillips
input.c:97:5: warning: symbol 'input_queue_ascii' was not declared. Should it be TODO: fix: input.c:236:18: error: bad constant expression input.c:344:17: error: bad constant expression Signed-off-by: Kim Phillips --- drivers/input/input.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[U-Boot] [PATCH 32/32] drivers/serial/serial_ns16550.c: sparse fixes

2012-10-16 Thread Kim Phillips
serial_ns16550.c:222:1: warning: symbol 'eserial1_init' was not declared. Should it be static? serial_ns16550.c:222:1: warning: symbol 'eserial1_setbrg' was not declared. Should it be static? serial_ns16550.c:222:1: warning: symbol 'eserial1_getc' was not declared. Should it be static? serial_ns

[U-Boot] [PATCH 13/32] common/misc: sparse fixes

2012-10-16 Thread Kim Phillips
command.c:44:38: error: bad constant expression console.c:537:18: warning: symbol 'search_device' was not declared. Should it be static? dlmalloc.c:1468:2: warning: Using plain integer as NULL pointer dlmalloc.c:1468:5: warning: Using plain integer as NULL pointer dlmalloc.c:2176:12: warning: Usin

[U-Boot] [PATCH 09/32] include/common.h: sparse fix

2012-10-16 Thread Kim Phillips
powerpc sparse builds yield: include/common.h:356:22: error: marked inline, but without a definition the unknown-reason inlining without a definition is considered obsolete given it was part of the 2002 initial commit, and no arm version was 'fixed.' Signed-off-by: Kim Phillips --- include/com

[U-Boot] [PATCH 29/32] drivers/mmc/fsl_esdhc.c: sparse fixes

2012-10-16 Thread Kim Phillips
fsl_esdhc.c:71:6: warning: symbol 'esdhc_xfertyp' was not declared. Should it be static? fsl_esdhc.c:413:6: warning: symbol 'set_sysctl' was not declared. Should it be st Signed-off-by: Kim Phillips --- drivers/mmc/fsl_esdhc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --g

[U-Boot] [PATCH 31/32] drivers/mtd/nand: sparse fixes

2012-10-16 Thread Kim Phillips
nand_ecc.c:82:5: warning: symbol 'nand_calculate_ecc' was not declared. Should it be static? nand_ecc.c:155:5: warning: symbol 'nand_correct_data' was not declared. Should it be static? nand_base.c:2854:1: error: directive in argument list nand_base.c:2856:1: error: directive in argument list Si

[U-Boot] [PATCH 21/32] arch/powerpc/cpu/mpc85xx/fdt.c: sparse fixes

2012-10-16 Thread Kim Phillips
this is mostly about using the proper __be32 type for 32-bit device tree property content variables. includes uncasting fdt_getprop() callsites, enforcing callsite code to use __be32 (identified by casting a 32-bit entity). This allows sparse to continue to find endianness type mismatches at the

[U-Boot] [PATCH 07/32] arch/powerpc/lib/extable.c: sparse fix

2012-10-16 Thread Kim Phillips
extable.c:66:9: warning: symbol 'ex_tab_message' was not declared. Should it be static? Signed-off-by: Kim Phillips --- arch/powerpc/lib/extable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/lib/extable.c b/arch/powerpc/lib/extable.c index 7408d5c..7e74a5d 1

[U-Boot] [PATCH 17/32] lib/zlib: sparse fixes

2012-10-16 Thread Kim Phillips
define Z_NULL to (void *)0 include/u-boot/zlib.h to get rid of most of the NULL pointer warnings. inflate.c:942:1: warning: non-ANSI definition of function 'inflateEnd' inflate.c:9:1: warning: non-ANSI definition of function 'inflateReset' inflate.c:12:17: warning: Using plain integer as NULL poin

[U-Boot] [PATCH 16/32] drivers/net/: sparse fixes

2012-10-16 Thread Kim Phillips
phy.c:46:5: warning: symbol 'genphy_config_advert' was not declared. Should it be static? phy.c:121:5: warning: symbol 'genphy_setup_forced' was not declared. Should it be static? phy.c:468:5: warning: symbol 'phy_probe' was not declared. Should it be static? phy.c:491:19: warning: symbol 'get_ph

[U-Boot] [PATCH 05/32] arch/powerpc/include/asm/io.h: fix warning: preprocessor token __iomem redefined

2012-10-16 Thread Kim Phillips
a fixup __iomem definition in arch code appears to be placed there as a cover up from a code import from linux when u-boot didn't yet have a compiler.h, introduced by commit 812711ce6b3a386125dcf0d6a59588e461abbb87 "Implement __raw_{read,write}[bwl] on all architectures". git show 812711ce6b3a3861

[U-Boot] [PATCH 25/32] drivers/gpio/mpc83xx_gpio.c: sparse fixes

2012-10-16 Thread Kim Phillips
mpc83xx_gpio.c:166:26: warning: non-ANSI function declaration of function 'mpc83xx_gpio_init_f' mpc83xx_gpio.c:190:26: warning: non-ANSI function declaration of function 'mpc83xx_gpio_init_r' Signed-off-by: Kim Phillips --- drivers/gpio/mpc83xx_gpio.c | 4 ++-- 1 file changed, 2 insertions(+),

[U-Boot] [PATCH 19/32] include/fdt.h: sparse fixes

2012-10-16 Thread Kim Phillips
fix endianness warnings such as these: cmd_fdt.c:479:29: warning: cast to restricted __be32 cmd_fdt.c:482:41: warning: cast to restricted __be32 cmd_fdt.c:486:29: warning: cast to restricted __be32 cmd_fdt.c:489:41: warning: cast to restricted __be32 cmd_fdt.c:776:41: warning: cast to restricted _

[U-Boot] [PATCH 04/32] include/linux/stddef.h: avoid 'warning: preprocessor token offsetof redefined'

2012-10-16 Thread Kim Phillips
hack to avoid: /opt/gcc-4.6.3-nolibc/powerpc-linux/bin/../lib/gcc/powerpc-linux/4.6.3/include/stddef.h:413:9: warning: preprocessor token offsetof redefined u-boot/include/linux/stddef.h:20:9: this was the original definition tried protecting with __KERNEL__, and #including compiler.h first. Ev

[U-Boot] [PATCH 30/32] drivers/mtd/cfi_flash.c: sparse fixes

2012-10-16 Thread Kim Phillips
cfi_flash.c:756:43: warning: Using plain integer as NULL pointer cfi_flash.c:1150:52: warning: cast to non-scalar cfi_flash.c:1433:46: warning: cast truncates bits from constant value (890089 becomes 89) cfi_flash.c:1490:61: warning: cast truncates bits from constant value (1f001f becomes 1f) cfi

[U-Boot] [PATCH 23/32] powerpc/mpc83xx: sparse fixes

2012-10-16 Thread Kim Phillips
fdt.c:91:78: warning: Using plain integer as NULL pointer fdt.c:103:78: warning: Using plain integer as NULL pointer speed.c:55:11: warning: symbol 'corecnf_tab' was not declared. Should it be static? speed.c:519:5: warning: symbol 'do_clocks' was not declared. Should it be static? mpc8313erdb.c:

[U-Boot] [PATCH 00/32] Initial sparse fix series

2012-10-16 Thread Kim Phillips
This 32-patch series only begins to address making u-boot source more 'sparseable,' or sparse-clean, ultimately to catch type, address space, and endianness mismatches and generally improve code quality. E.g., in this initial dose whose main purpose is to reduce the output volume to workable levels

[U-Boot] [PATCH 15/32] net/: sparse fixes

2012-10-16 Thread Kim Phillips
bootp.c:44:14: warning: symbol 'dhcp_state' was not declared. Should it be static? bootp.c:45:15: warning: symbol 'dhcp_leasetime' was not declared. Should it be static? bootp.c:46:10: warning: symbol 'NetDHCPServerIP' was not declared. Should it be static? arp.c:30:17: warning: symbol 'NetArpWa

[U-Boot] [PATCH 08/32] arch/powerpc/lib/board.c, *traps.c: sparse fixes

2012-10-16 Thread Kim Phillips
traps.c:*:1: warning: symbol 'print_backtrace' was not declared. Should it be static? traps.c:66:6: warning: symbol 'show_regs' was not declared. Should it be static? traps.c:93:1: warning: symbol '_exception' was not declared. Should it be static? board.c:166:6: warning: symbol '__board_add_ram_

[U-Boot] [PATCH 01/32] include/linux/byteorder: import latest endian definitions from linux

2012-10-16 Thread Kim Phillips
u-boot's byteorder headers did not contain endianness attributions for use with sparse, causing a lot of false positives. Import the kernel's latest definitions, and enable them by including compiler.h and types.h. They come with 'const' added for some swab functions, so fix those up, too: inclu

[U-Boot] [PATCH 02/32] include/linux/compat.h: fix warning: preprocessor token __iomem redefined

2012-10-16 Thread Kim Phillips
include/linux/compat.h:4:9: warning: preprocessor token __user redefined include/linux/compiler.h:7:10: this was the original definition include/linux/compat.h:5:9: warning: preprocessor token __iomem redefined include/linux/compiler.h:12:10: this was the original definition fixup __iomem, __user

[U-Boot] [PATCH 03/32] include/linux/unaligned/generic.h: fix warning: preprocessor token __force redefined

2012-10-16 Thread Kim Phillips
include/linux/unaligned/generic.h:5:9: warning: preprocessor token __force redefined include/linux/compiler.h:10:10: this was the original definition fixup __force definitions in compat.h code appears to be placed there as a cover up from a code import from linux when u-boot didn't yet have a com

[U-Boot] [PATCH] libfdt: Add helper function to create a trivial, empty tree

2012-10-16 Thread Gerald Van Baren
The libfdt read/write functions are now usable enough that it's become a moderately common pattern to use them to build and manipulate a device tree from scratch. For example, we do so ourself in our rw_tree1 testcase, and qemu is starting to use this model when building device trees for some targ

[U-Boot] [PATCH 3/3] am33xx: Add SPI SPL as an option

2012-10-16 Thread Tom Rini
Add the required config.mk logic for this SoC as well as the BOOT_DEVICE define. Finally, enable the options on the am335x_evm. Signed-off-by: Tom Rini --- arch/arm/cpu/armv7/am33xx/config.mk|1 + arch/arm/include/asm/arch-am33xx/spl.h |1 + include/configs/am335x_evm.h |

[U-Boot] [PATCH 1/3] omap3_spi: Undo CONFIG_AM33XX D0/D1 change

2012-10-16 Thread Tom Rini
At some point the am335x evm hardware was updated to previosly used in all other designs layout, so remove the now incorrect code. Signed-off-by: Tom Rini --- drivers/spi/omap3_spi.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/spi/omap3_spi.c b/drivers/spi/omap3_spi.

[U-Boot] [PATCH 2/3] omapimage: Add support for byteswapped SPI images

2012-10-16 Thread Tom Rini
Add MLO.byteswap as a target to spl/Makefile and un-guard the first MLO rule so we don't have to duplicate it. Signed-off-by: Tom Rini --- spl/Makefile |9 ++ tools/omapimage.c | 83 + 2 files changed, 68 insertions(+), 24 deleti

Re: [U-Boot] [PATCH 4/4] ARM: tegra: increase CONFIG_SYS_TEXT_BASE

2012-10-16 Thread Stephen Warren
On 10/16/2012 04:09 PM, Lucas Stach wrote: > Am Dienstag, den 16.10.2012, 15:50 -0600 schrieb Stephen Warren: >> From: Stephen Warren >> >> The SPL has grown. Increase CONFIG_SYS_TEXT_BASE so SPL's BSS does not >> overlap the main U-Boot. > > Is there any specific reason why the SPL is now bigger

Re: [U-Boot] [PATCH] Prevent malloc with size 0

2012-10-16 Thread Graeme Russ
Hi Wolfgang, On Tue, Oct 16, 2012 at 9:43 PM, Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201210160831.20759.ma...@denx.de> you wrote: >> >> > In short, returning non-NULL from malloc(0) and expecting a CPU exception >> > when it is de-referenced is not going to fly. > > We should no

Re: [U-Boot] [PATCH 4/4] ARM: tegra: increase CONFIG_SYS_TEXT_BASE

2012-10-16 Thread Lucas Stach
Am Dienstag, den 16.10.2012, 15:50 -0600 schrieb Stephen Warren: > From: Stephen Warren > > The SPL has grown. Increase CONFIG_SYS_TEXT_BASE so SPL's BSS does not > overlap the main U-Boot. > Is there any specific reason why the SPL is now bigger than before? Or is this just because of the gener

[U-Boot] [PATCH 3/4] ARM: tegra: check for SPL size overflow in makefile

2012-10-16 Thread Stephen Warren
From: Stephen Warren If the SPL extends beyond CONFIG_SYS_TEXT_BASE, then it will likely corrupt the main U-Boot binary during execution, causing the main U-Boot binary to fail. Check for this situation during the build to avoid extremely annoying and hard-to-find bugs. Note that checking the siz

[U-Boot] [PATCH 2/4] ARM: tegra: combine duplicate Makefile rules

2012-10-16 Thread Stephen Warren
From: Stephen Warren The rules to generate u-boot-{no,}dtb-tegra.bin were almost identical. Combine them into a single paremeterized rule. This will allow the next patch to edit a single rule, rather than being cut/paste twice. Signed-off-by: Stephen Warren --- Makefile | 15 ---

[U-Boot] [PATCH 1/4] ARM: fix u-boot.lds for -ffunction-sections/-fdata-sections

2012-10-16 Thread Stephen Warren
From: Stephen Warren When -ffunction-sections or -fdata-section are used, symbols are placed into sections such as .data.eserial1_device and .bss.serial_current. Update the linker script to explicitly include these. Without this change (at least with my gcc-4.5.3 built using crosstool-ng), I see

[U-Boot] [PATCH 4/4] ARM: tegra: increase CONFIG_SYS_TEXT_BASE

2012-10-16 Thread Stephen Warren
From: Stephen Warren The SPL has grown. Increase CONFIG_SYS_TEXT_BASE so SPL's BSS does not overlap the main U-Boot. Signed-off-by: Stephen Warren --- include/configs/tegra20-common.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/configs/tegra20-common.h b/i

Re: [U-Boot] [PATCH 5/5] am33xx: support board specific ddr settings

2012-10-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/16/12 10:43, Peter Korsgaard wrote: >> "Tom" == Tom Rini writes: > > Hi, > > Tom> I would like to see the ddr information kept in Tom> > arch/arm/cpu/armv7/am33xx but prefixed with vendor/part information > so Tom> we can re-use it on othe

Re: [U-Boot] [PATCH] powerpc/fm: fix TBI PHY address settings

2012-10-16 Thread Timur Tabi
Xie Shaohui-B21989 wrote: >> Is this still true with Fman v3? > [S.H] No. Fman V3 does not need to set TBIPA on FM1@DTSEC1. Does this patch break Fman V3? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH 1/9 V2] SOUND: Add I2S driver

2012-10-16 Thread Simon Glass
Hi, On Tue, Aug 21, 2012 at 11:44 PM, Rajeshwari Shinde wrote: > This patch adds driver for I2S interface specific to samsung. > > Signed-off-by: R. Chandrasekar > Signed-off-by: Rajeshwari Shinde > --- > Changes in V2: > - renamed i2s.c to samsung-i2s.c. > Makefile|1 +

Re: [U-Boot] [PATCH v3 13/18] lcd: Add support for flushing LCD fb from dcache after update

2012-10-16 Thread Simon Glass
Hi, On Thu, Aug 9, 2012 at 12:43 AM, Lukasz Majewski wrote: > Hi Simon, > >> This provides an option for the LCD to flush the dcache after each >> update (puts, scroll or clear). >> >> Signed-off-by: Simon Glass >> --- >> Changes in v2: >> - Put the LCD cache flush logic into lcd_putc() instead

[U-Boot] [PATCH 3/5] am33xx/board: use cpu_mmc_init() for default mmc initialization

2012-10-16 Thread Peter Korsgaard
So platforms can override it with board_mmc_init(). Signed-off-by: Peter Korsgaard --- arch/arm/cpu/armv7/am33xx/board.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/cpu/armv7/am33xx/board.c b/arch/arm/cpu/armv7/am33xx/board.c index 8280b35..e4c123c 100644

[U-Boot] [PATCH 5/5] am33xx: support board specific ddr settings

2012-10-16 Thread Peter Korsgaard
Move the hardcoded ddr2/ddr3 settings for the ti boards to board code, so other boards can use different types/timings. Signed-off-by: Peter Korsgaard --- arch/arm/cpu/armv7/am33xx/emif4.c | 114 +++- arch/arm/include/asm/arch-am33xx/ddr_defs.h | 39 + bo

[U-Boot] [PATCH 1/5] am33xx: move ti i2c baseboard header handling to board/ti/am335x/evm.{c, h}

2012-10-16 Thread Peter Korsgaard
The i2c header is specific to ti(-derived) boards, and not generic for all am335x boards. Signed-off-by: Peter Korsgaard --- arch/arm/cpu/armv7/am33xx/board.c | 235 arch/arm/include/asm/arch-am33xx/sys_proto.h | 27 --- board/ti/am335x/Makefile

[U-Boot] [PATCH 2/5] am335x/evm.c: don't make wdtimer/uart_base global

2012-10-16 Thread Peter Korsgaard
Only used here (and for SPL). Signed-off-by: Peter Korsgaard --- board/ti/am335x/evm.c |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/board/ti/am335x/evm.c b/board/ti/am335x/evm.c index a1fa443..9dd372c 100644 --- a/board/ti/am335x/evm.c +++ b/board/ti/am335x/evm.c

[U-Boot] [PATCH 4/5] am33xx: move generic parts of pinmux handling out from board/ti/am335x

2012-10-16 Thread Peter Korsgaard
So they are available for other boards. Signed-off-by: Peter Korsgaard --- arch/arm/cpu/armv7/am33xx/Makefile |1 + arch/arm/cpu/armv7/am33xx/mux.c| 33 arch/arm/include/asm/arch-am33xx/mux.h | 261 board/ti/am335x/mux.c |

[U-Boot] [PATCH 0/5] am33xx: support non-ti boards

2012-10-16 Thread Peter Korsgaard
The am33xx code currently contains a number of details specific to the ti(-derived) boards in the common code. This series restructures the arch code to make it possible to add other boards. This series does not yet any new boards, as the board I'm working on is using nand flash, and omap_gpmc has

Re: [U-Boot] [PATCH 5/7] disk: Allow alternate EFI partition signature

2012-10-16 Thread Stephen Warren
On 10/15/2012 11:17 AM, Stefan Reinauer wrote: > On Mon, Oct 15, 2012 at 8:19 AM, Stephen Warren > wrote: > > On 10/12/2012 06:26 PM, Simon Glass wrote: > > From: Stefan Reinauer > > > > > ChromeOS uses a GPT partiti

Re: [U-Boot] reg: P2020 include config file - u-boot basic doubt

2012-10-16 Thread Scott Wood
On 10/16/2012 07:51:24 AM, vidya Krishnamoorthy wrote: Hi Anybody worked in P2020 ..Please clarify my basic doubt The e500 core datasheet says processor reset vector address is 0xFFC but in p2020 u-boot config file , CONFIG_RESET_VECTOR_ADDRESS is defined as0xEFFC. With boot se

Re: [U-Boot] [PATCH 5/5] am33xx: support board specific ddr settings

2012-10-16 Thread Peter Korsgaard
> "Tom" == Tom Rini writes: Hi, Tom> I would like to see the ddr information kept in Tom> arch/arm/cpu/armv7/am33xx but prefixed with vendor/part information so Tom> we can re-use it on other boards. So something like: /* Micron MT41J128M16JT-125 */ #define MICRON_MT41J128M16JT125_EMIF_

Re: [U-Boot] [PATCH 3/5] am33xx/board: use cpu_mmc_init() for default mmc initialization

2012-10-16 Thread Peter Korsgaard
> "Tom" == Tom Rini writes: Hi, >> #if defined(CONFIG_OMAP_HSMMC) && !defined(CONFIG_SPL_BUILD) -int >> board_mmc_init(bd_t *bis) +int cpu_mmc_init(bd_t *bis) { int ret; - >> + ret = omap_mmc_init(0, 0, 0); if (ret) return ret; >> Tom> What does this look like on your port? I don't h

Re: [U-Boot] [STATUS] v2012.10 released, Merge Window is OPEN

2012-10-16 Thread Simon Glass
Hi, On Tue, Oct 16, 2012 at 7:08 AM, Otavio Salvador wrote: > On Tue, Oct 16, 2012 at 10:43 AM, Simon Glass wrote: >> On Tue, Oct 16, 2012 at 3:35 AM, Wolfgang Denk wrote: >>> In message >>> you >>> wrote: Since you ask, Google, Inc (chromium.org). >>> >>> Can this be made global?

Re: [U-Boot] [PATCH v2] lsxl: unset ncip for rescue mode

2012-10-16 Thread Michael Walle
Hi Prafulla, Am Donnerstag 04 Oktober 2012, 18:54:25 schrieb Michael Walle: > Instead of using the serverip we get from the DHCP server, implicitly use > the broadcast address, which is automatically set when no ncip environment > variable is set. That way it isn't necessary to use a special DHCP

Re: [U-Boot] [PULL] u-boot-pxa/master

2012-10-16 Thread Albert ARIBAUD
Hi Marek, On Tue, 16 Oct 2012 07:16:51 +0200, Marek Vasut wrote: > The following changes since commit bd23b22badadcdc414a900828253961fc5ec6c39: > > Merge branch 'ag...@denx.de-next' of git://git.denx.de/u-boot-staging > (2012-10-15 13:37:22 -0700) > > are available in the git repository at:

Re: [U-Boot] [PATCH v3 5/5] New board support: Nokia RX-51 aka N900

2012-10-16 Thread Pali Rohár
On Tuesday 16 October 2012 17:57:09 Marek Vasut wrote: > > > > > > > + > > > > > > +static unsigned long int twl_wd_time; /* last time > > > > > > of > > > > > > watchdog reset */ +static unsigned long int > > > > > > twl_i2c_lock; > > > > > > > > > > Are you sure you want to use global vars for th

Re: [U-Boot] [PATCH v2 3/5] pmic_fsl: Introduce CONFIG_SYS_FSL_PMIC_I2C_LENGTH

2012-10-16 Thread Lukasz Majewski
Hi Stefano, > Am 16/10/2012 01:32, schrieb Fabio Estevam: > > From: Fabio Estevam > > > > Introduce CONFIG_SYS_FSL_PMIC_I2C_LENGTH to configure the number of > > bytes that are used to communicate with the PMIC via I2C. > > > > Instead of hardcoding the value, pass the number via a config > > o

Re: [U-Boot] [PATCH v3 5/5] New board support: Nokia RX-51 aka N900

2012-10-16 Thread Marek Vasut
Dear Pali Rohár, [...] > > > Reason? ++i and i++ are same (for this situation). > > > > On arm, yes, on intel, no in certain cases. Also, to keep the > > code consistent, go with postdecrement. > > What is different on intel (without assigning value of i)? IIRC the prefetch behaves differently

Re: [U-Boot] [PATCH v3 2/5] arm bootm: Do not append zero ATAG_MEM

2012-10-16 Thread Tom Rini
On Sun, Oct 14, 2012 at 03:08:20AM +0200, Marek Vasut wrote: > Dear Pali Roh?r, > > > On Sunday 14 October 2012 02:27:05 Marek Vasut wrote: > > > Dear Pali Roh?r, > > > > > > > On Sunday 14 October 2012 02:17:01 Marek Vasut wrote: > > > > > Dear Pali Roh?r, > > > > > > > > > > > On Sunday 14 Oct

Re: [U-Boot] [PATCH 2/2] tools/env: Improve debug prints

2012-10-16 Thread Tom Rini
On Mon, Oct 15, 2012 at 08:29:25PM -0500, Joe Hershberger wrote: > Provide more information when using redundant environments > Consistently print debug info to stderr > > Signed-off-by: Joe Hershberger Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature __

Re: [U-Boot] [PATCH 1/2] tools/env: Fix variable delete operation

2012-10-16 Thread Tom Rini
On Mon, Oct 15, 2012 at 08:29:24PM -0500, Joe Hershberger wrote: > Fix crash introduced by a073d63a36524453a817ab029fad5b188f46127e > when attempting to delete a variable. > > Signed-off-by: Joe Hershberger Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signatur

Re: [U-Boot] [PATCH v3 5/5] New board support: Nokia RX-51 aka N900

2012-10-16 Thread Pali Rohár
On Tuesday 16 October 2012 16:55:20 Marek Vasut wrote: > Dear Pali Rohár, > > [...] > > > Nothing. This code must be in assembler because stack is not > > initialized. Also we must be sure that U-Boot will not > > overwrite attached kernel (with can be in U-Boot > > malloc/monitor area...) > Don't

Re: [U-Boot] [PATCH 3/5] am33xx/board: use cpu_mmc_init() for default mmc initialization

2012-10-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/16/12 08:15, Peter Korsgaard wrote: > So platforms can override it with board_mmc_init(). > > Signed-off-by: Peter Korsgaard --- > arch/arm/cpu/armv7/am33xx/board.c |4 ++-- 1 file changed, 2 > insertions(+), 2 deletions(-) > > diff --git

Re: [U-Boot] [PATCH 1/5] am33xx: move ti i2c baseboard header handling to board/ti/am335x/evm.{c, h}

2012-10-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/16/12 08:15, Peter Korsgaard wrote: > The i2c header is specific to ti(-derived) boards, and not generic > for all am335x boards. > > Signed-off-by: Peter Korsgaard --- > arch/arm/cpu/armv7/am33xx/board.c | 235 >

Re: [U-Boot] [PATCH 5/5] am33xx: support board specific ddr settings

2012-10-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/16/12 08:15, Peter Korsgaard wrote: > Move the hardcoded ddr2/ddr3 settings for the ti boards to board > code, so other boards can use different types/timings. > > Signed-off-by: Peter Korsgaard --- > arch/arm/cpu/armv7/am33xx/emif4.c

Re: [U-Boot] USB Host driver for ISP1362?

2012-10-16 Thread Steve Strobel
> Steve Strobel wrote: >> ...I want to access a USB flash drive from U-Boot. But I haven't been >> able to find the drivers for the ISP1362... On Mon, Oct 15, 2012 at 1:04 PM, Wolfgang Denk wrote: > IIRC these boards use only the host mode of the ISP1362, and this is > using just the normal EHCI

Re: [U-Boot] [PATCH v3 5/5] New board support: Nokia RX-51 aka N900

2012-10-16 Thread Marek Vasut
Dear Pali Rohár, [...] > > Nothing. This code must be in assembler because stack is not > initialized. Also we must be sure that U-Boot will not overwrite > attached kernel (with can be in U-Boot malloc/monitor area...) Don't we have something that can reserve a memory area? > > > +static char

Re: [U-Boot] [PATCH v2 3/5] pmic_fsl: Introduce CONFIG_SYS_FSL_PMIC_I2C_LENGTH

2012-10-16 Thread Stefano Babic
Am 16/10/2012 01:32, schrieb Fabio Estevam: > From: Fabio Estevam > > Introduce CONFIG_SYS_FSL_PMIC_I2C_LENGTH to configure the number of bytes > that are used to communicate with the PMIC via I2C. > > Instead of hardcoding the value, pass the number via a config option. > > This will be useful

Re: [U-Boot] [PATCH v3 5/5] New board support: Nokia RX-51 aka N900

2012-10-16 Thread Pali Rohár
On Sunday 14 October 2012 02:06:49 Marek Vasut wrote: > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 971235b..613d8cd 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -1009,6 +1009,10 @@ Nobuhiro Iwamatsu > > > > > armadillo-800evaR8A7740 (RMOBILE SoC) > > > > +Pali R

Re: [U-Boot] [STATUS] v2012.10 released, Merge Window is OPEN

2012-10-16 Thread Rob Herring
On 10/15/2012 04:08 PM, Wolfgang Denk wrote: > Dear Benoît Thébaudeau, > > In message <950821917.6855188.1350329929641.javamail.r...@advansee.com> you > wrote: >> Can you also add National Instruments (ni.com) to your list of employers? >>> >>> Please add O.S. Systems (ossystems.com.br)

Re: [U-Boot] [PATCH v4 3/3] musb_udc : Fix compile warning.

2012-10-16 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/15/12 22:43, Marek Vasut wrote: Lets just drop this patch. The whole series if still in patchwork should be superseded by the work Ilya is doing for MUSB. - -- Tom -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Us

Re: [U-Boot] [PATCH v1 3/4] VIDEO: add macro to set LCD size for DSS driver

2012-10-16 Thread Anatolij Gustschin
On Tue, 16 Oct 2012 16:07:05 +0200 Stefano Babic wrote: > Signed-off-by: Stefano Babic > --- > arch/arm/include/asm/arch-omap3/dss.h |1 + > 1 file changed, 1 insertion(+) Acked-by: Anatolij Gustschin Thanks! ___ U-Boot mailing list U-Boot@list

Re: [U-Boot] [STATUS] v2012.10 released, Merge Window is OPEN

2012-10-16 Thread Otavio Salvador
On Tue, Oct 16, 2012 at 10:43 AM, Simon Glass wrote: > On Tue, Oct 16, 2012 at 3:35 AM, Wolfgang Denk wrote: >> In message >> you >> wrote: >>> >>> Since you ask, Google, Inc (chromium.org). >> >> Can this be made global? At the moment we have special entries for >> Anton Staaf and for you.

[U-Boot] [PATCH v1 4/4] OMAP3: add video support to the mcx board

2012-10-16 Thread Stefano Babic
Add video support to the board with the display focaltech etm070003dh6. Signed-off-by: Stefano Babic --- board/htkw/mcx/mcx.c | 38 ++ board/htkw/mcx/mcx.h |2 ++ include/configs/mcx.h | 16 +++- 3 files changed, 55 insertions(+), 1 delet

[U-Boot] [PATCH v1 3/4] VIDEO: add macro to set LCD size for DSS driver

2012-10-16 Thread Stefano Babic
Signed-off-by: Stefano Babic --- arch/arm/include/asm/arch-omap3/dss.h |1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/include/asm/arch-omap3/dss.h b/arch/arm/include/asm/arch-omap3/dss.h index 54add4b..ffaffbb 100644 --- a/arch/arm/include/asm/arch-omap3/dss.h +++ b/arch/arm/inc

[U-Boot] [PATCH v1 2/4] OMAP3: mcx: updated to new hardware revision

2012-10-16 Thread Stefano Babic
Some GPIOs differ in the new revision board. Previous revision are considered obsolete and they will not anymore supported. Signed-off-by: Stefano Babic --- board/htkw/mcx/mcx.c | 19 +++ board/htkw/mcx/mcx.h |2 +- include/configs/mcx.h |3 +-- 3 files changed, 5 ins

[U-Boot] [PATCH v1 1/4] OMAP3: updated pinmux and environment for new revision of mcx board

2012-10-16 Thread Stefano Babic
The mcx board was slightly modified and the pinmux must be updated. There is no need to support the old board, that becomes obsolete. Signed-off-by: Stefano Babic --- board/htkw/mcx/mcx.h | 26 +- include/configs/mcx.h | 12 +++- 2 files changed, 20 insertion

[U-Boot] [PATCH v1] OMAP3: mt_ventoux: power on USB at startup

2012-10-16 Thread Stefano Babic
Updated revision of the board uses GPIOs to activate the USB ports. Signed-off-by: Stefano Babic --- board/teejet/mt_ventoux/mt_ventoux.c |8 1 file changed, 8 insertions(+) diff --git a/board/teejet/mt_ventoux/mt_ventoux.c b/board/teejet/mt_ventoux/mt_ventoux.c index ecb9b6c..962

Re: [U-Boot] [STATUS] v2012.10 released, Merge Window is OPEN

2012-10-16 Thread Simon Glass
Hi Wolfgang, On Tue, Oct 16, 2012 at 3:35 AM, Wolfgang Denk wrote: > Dear Simon, > > In message > you > wrote: >> >> Since you ask, Google, Inc (chromium.org). > > Can this be made global? At the moment we have special entries for > Anton Staaf and for you. Is all of chromium owned by google

  1   2   >