Re: [U-Boot] [PATCH 0/3] Bring in new I2C framework

2012-10-29 Thread Heiko Schocher
Hello Stephen, On 29.10.2012 16:34, Stephen Warren wrote: On 10/29/2012 03:47 AM, Heiko Schocher wrote: Hello Stephen, On 26.10.2012 18:07, Stephen Warren wrote: On 10/25/2012 11:48 PM, Heiko Schocher wrote: Hello Simon, On 25.10.2012 23:37, Simon Glass wrote: On Mon, Oct 22, 2012 at 10:40

Re: [U-Boot] [PATCH 0/3] Bring in new I2C framework

2012-10-29 Thread Heiko Schocher
Hello Simon, On 29.10.2012 14:48, Simon Glass wrote: Hi Heiko, On Mon, Oct 29, 2012 at 2:44 AM, Heiko Schocher wrote: Hello Simon, On 26.10.2012 18:08, Simon Glass wrote: On Thu, Oct 25, 2012 at 10:48 PM, Heiko Schocher wrote: Hello Simon, On 25.10.2012 23:37, Simon Glass wrote:

Re: [U-Boot] [PATCH v2 22/22] powerpc/mpc85xx/p2020rdb-pca: Use L2 SRAM for SPL boot

2012-10-29 Thread Tabi Timur-B04825
On Mon, Oct 29, 2012 at 9:05 PM, Scott Wood wrote: > diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h > index ba7a364..f02ba2b 100644 > --- a/include/configs/p1_p2_rdb_pc.h > +++ b/include/configs/p1_p2_rdb_pc.h > @@ -38,6 +38,7 @@ > #define __SW_BOOT_MASK 0x0

[U-Boot] [PATCH v2 13/22] spl/mpc85xx: rename cpu_init_nand.c to spl_minimal.c

2012-10-29 Thread Scott Wood
There is nothing really NAND-specific about this file. Signed-off-by: Scott Wood Cc: Andy Fleming --- arch/powerpc/cpu/mpc85xx/cpu_init_nand.c | 59 arch/powerpc/cpu/mpc85xx/spl_minimal.c | 59 nand_spl/board/freescale/mpc8536d

[U-Boot] [PATCH v2 15/22] spl/powerpc: introduce CONFIG_SPL_INIT_MINIMAL

2012-10-29 Thread Scott Wood
cpu_init_nand.c is renamed to spl_minimal.c as it is not really NAND-specific. Signed-off-by: Scott Wood --- v2: factor out START, and change cpu_init_nand.c to spl_minimal.c Cc: Andy Fleming --- README|3 +++ arch/powerpc/cpu/mpc85xx/Makefile | 19

[U-Boot] [PATCH v2 04/22] powerpc/mpc85xx: add comma before "already enabled"

2012-10-29 Thread Scott Wood
Now outputs like this: L2:512 KB already enabled, moving to 0xf8f8 rather than this: L2:512 KB already enabledmoving to 0xf8f8 Signed-off-by: Scott Wood Cc: Andy Fleming --- arch/powerpc/cpu/mpc85xx/cpu_init.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[U-Boot] [PATCH v2 19/22] spl/nand: config symbol documentation

2012-10-29 Thread Scott Wood
Document parameters used for specifying the NAND image to be loaded. Also fix the definition of CONFIG_SPL_NAND_SIMPLE -- it's only nand_spl_simple.c, not the entire nand directory. The word "simple" is there for a reason. :-) Signed-off-by: Scott Wood --- v2: updated for makefile changes earl

[U-Boot] [PATCH v2 18/22] spl/nand: introduce CONFIG_SPL_NAND_DRIVERS, _BASE, and _ECC.

2012-10-29 Thread Scott Wood
Some small SPLs do not use nand_base.c, and a subset of those also require a special driver. Some SPLs need software ECC but others can't fit it. All existing boards that specify CONFIG_SPL_NAND_SUPPORT have these symbols added to preserve existing behavior. Signed-off-by: Scott Wood -- v2: use

[U-Boot] [PATCH v2 20/22] powerpc/mpc85xx/p1_p2_rdb_pc: convert from nand_spl to new spl

2012-10-29 Thread Scott Wood
Signed-off-by: Scott Wood Cc: Andy Fleming --- include/configs/p1_p2_rdb_pc.h| 54 nand_spl/board/freescale/p1_p2_rdb_pc/Makefile| 142 - nand_spl/board/freescale/p1_p2_rdb_pc/nand_boot.c | 132 --- 3 files changed, 28 inse

[U-Boot] [PATCH v2 21/22] powerpc/mpc85xx/p1_p2_rdb_pc: clean up memory map

2012-10-29 Thread Scott Wood
- Sort by address, and fix column alignment - Don't label things as localbus that aren't. Instead, put chipselect info at the end of the description for localbus windows. Note that NAND/NOR have their chipselects swapped when booting from NAND, and CS2 can be either PMC or VSC7385 dependin

[U-Boot] [PATCH v2 22/22] powerpc/mpc85xx/p2020rdb-pca: Use L2 SRAM for SPL boot

2012-10-29 Thread Scott Wood
This allows DDR configuration to be deferred to the final U-Boot image, which is able to make use of SPD data. The SPL itself cannot use SPD due to code size constraints. It previously used fixed register values for DDR configuration, and those values did not work on the p2020rdb-pca board I test

[U-Boot] [PATCH v2 17/22] powerpc/mpc85xx/p1_p2_rdb_pc: new SPL support

2012-10-29 Thread Scott Wood
Introduces CONFIG_SPL_RELOC_TEXT_BASE and CONFIG_SPL_RELOC_STACK. Signed-off-by: Scott Wood Cc: Andy Fleming --- README |9 ++ board/freescale/p1_p2_rdb_pc/Makefile | 16 board/freescale/p1_p2_rdb_pc/p1_p2_rdb_pc.c |2 +- board/freescale

[U-Boot] [PATCH v2 10/22] powerpc/mpc8xxx: move LAW code into arch/powerpc/cpu/mpc8xxx

2012-10-29 Thread Scott Wood
It's arch code and not a driver, so move it where it belongs. When it originally went into drivers/misc there was no 8xxx CPU directory. This will make new-SPL support a little easier since we can keep the CPU stuff together and not need to pull stuff in from drivers/misc. Signed-off-by: Scott W

[U-Boot] [PATCH v2 16/22] spl/85xx: new SPL support

2012-10-29 Thread Scott Wood
Update CONFIG_RAMBOOT and CONFIG_NAND_SPL references to accept CONFIG_SPL and CONFIG_SPL_BUILD, respectively. CONFIG_NAND_SPL can be removed once the last mpc85xx nand_spl target is gone. CONFIG_RAMBOOT will need to remain for other use cases, but it doesn't seem right to overload it for meaning

[U-Boot] [PATCH v2 12/22] spl: include resetvec and lib8xxx

2012-10-29 Thread Scott Wood
The toplevel makefile hardcodes this stuff, so spl/Makefile needs to as well. Signed-off-by: Scott Wood --- spl/Makefile | 21 + 1 file changed, 21 insertions(+) diff --git a/spl/Makefile b/spl/Makefile index 3195390..eacf4a2 100644 --- a/spl/Makefile +++ b/spl/Makefile @@

[U-Boot] [PATCH v2 14/22] powerpc/mpc85xx: consistently use COBJS-y

2012-10-29 Thread Scott Wood
A subsequent patch will conditionalize some of the files that are currently unconditional. Signed-off-by: Scott Wood Cc: Andy Fleming --- arch/powerpc/cpu/mpc85xx/Makefile | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/arch/powerpc/cpu/mpc85xx/Makefil

[U-Boot] [PATCH v2 09/22] powerpc/mpc85xx: fix TLB alignment

2012-10-29 Thread Scott Wood
In the RAMBOOT/SPL case we were creating a TLB entry starting at CONFIG_SYS_MONITOR_BASE, and just hoping that the base was properly aligned for the TLB entry size. This turned out to not be the case with NAND SPL because the main U-Boot starts at an offset into the image in order to skip the SPL

[U-Boot] [PATCH v2 11/22] spl/mpc85xx: move udelay to cpu code

2012-10-29 Thread Scott Wood
It applies to non-Freescale 85xx boards as well as Freescale boards, so it doesn't belong in board/freescale. Plus, it needs to come out of nand_spl if it's to be used by the new SPL. Signed-off-by: Scott Wood Cc: Andy Fleming --- arch/powerpc/cpu/mpc85xx/cpu_init_nand.c | 17 +

[U-Boot] [PATCH v2 08/22] powerpc: change .fixup test to a GCC version test

2012-10-29 Thread Scott Wood
This was introduced by commit 244615197469dd6fe75ae082f38424b97c79aeaf, but it fails in a minimal SPL build where the only thing in arch/powerpc/lib is cache.c, which apparently doesn't generate any fixup records. The problem is reported to occur with GCC 3.x, so insist on GCC 4.0 or newer. Patter

[U-Boot] [PATCH v2 07/22] spl: introduce CONFIG_SPL_TARGET

2012-10-29 Thread Scott Wood
Currently the SPL target is specified in a CPU-specific makefile fragment. While some targets may need something more complicated than a simple target name, targets which don't need this shouldn't have to provide a makefile fragment just for this. Signed-off-by: Scott Wood --- v2: Removed defau

[U-Boot] [PATCH v2 05/22] Add u-boot-pad.bin target to the Makefile

2012-10-29 Thread Scott Wood
From: José Miguel Gonçalves Samsung's S3C24XX SoCs need this in order to generate a binary image with a padded SPL concatenated with U-Boot. Signed-off-by: José Miguel Gonçalves [scottw...@freescale.com: fixed prereq of u-boot.ubl] Signed-off-by: Scott Wood -- v2: Removed spl/ prefix from u-bo

[U-Boot] [PATCH v2 06/22] spl: rename u-boot-pad.bin to u-boot-with-spl.bin

2012-10-29 Thread Scott Wood
Signed-off-by: Scott Wood --- Makefile |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index 366096f..61f4d69 100644 --- a/Makefile +++ b/Makefile @@ -470,14 +470,14 @@ $(obj)u-boot.sha1:$(obj)u-boot.bin $(obj)u-boot.dis: $(obj)u-bo

[U-Boot] [PATCH v2 03/22] powerpc/mpc85xx: move debug tlb entry after TLB is in known state

2012-10-29 Thread Scott Wood
Previously, in many if not all configs we were creating overlapping TLB entries which is illegal. This caused a crash during boot when moving p2020rdb NAND SPL into L2 SRAM. Signed-off-by: Scott Wood Cc: Prabhakar Kushwaha Cc: Andy Fleming -- Prabhakar, please test that debug still works. ---

[U-Boot] [PATCH v2 01/22] serial/ns16550: don't build serial_ns16550 with MIN_FUNCTIONS

2012-10-29 Thread Scott Wood
CONFIG_NS16550_MIN_FUNCTIONS is used by small SPLs to gain access to basic ns16550 output code without pulling in things not needed by the SPL. This previously only worked with non-MULTI configs. Recently MULTI was made mandatory, and MIN_FUNCTIONS fails like this: drivers/serial/libserial.o: In

[U-Boot] [PATCH v2 00/22] Convert p1_p2_rdb_pc to new spl

2012-10-29 Thread Scott Wood
Tested on p2020rdb-pca and p1020rdb-pc. José Miguel Gonçalves (1): Add u-boot-pad.bin target to the Makefile Scott Wood (21): serial/ns16550: don't build serial_ns16550 with MIN_FUNCTIONS serial/ns16550: wait for TEMT before initializing powerpc/mpc85xx: move debug tlb entry after TLB is

[U-Boot] [PATCH v2 02/22] serial/ns16550: wait for TEMT before initializing

2012-10-29 Thread Scott Wood
TEMT is set when the transmitter is totally empty and all output has finished. This prevents output problems (including a loss of synchronization observed on p2020 that persisted for quite a while) if SPL has output still on its way out. Signed-off-by: Scott Wood -- v2: fixed typo in subject, an

Re: [U-Boot] [PATCH] cmd_fdt.c: Use %p when printing pointers

2012-10-29 Thread Joe Hershberger
Hi Tom, On Mon, Oct 29, 2012 at 7:53 PM, Tom Rini wrote: > When putting pointers into a format string use %p to ensure that they > are printed correctly regardless of bitsize. This fixes warnings on > sandbox on 64bit systems. > > Cc: Joe Hershberger > Cc: Gerald Van Baren > Signed-off-by: Tom

[U-Boot] [PATCH] cmd_fdt.c: Use %p when printing pointers

2012-10-29 Thread Tom Rini
When putting pointers into a format string use %p to ensure that they are printed correctly regardless of bitsize. This fixes warnings on sandbox on 64bit systems. Cc: Joe Hershberger Cc: Gerald Van Baren Signed-off-by: Tom Rini --- common/cmd_fdt.c |6 +++--- 1 file changed, 3 insertions

Re: [U-Boot] [PATCH v2 24/25] drivers/mtd/nand: sparse fixes

2012-10-29 Thread Scott Wood
On 10/29/2012 06:34:46 PM, Kim Phillips wrote: 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_b

Re: [U-Boot] [PATCH v7] qi_lb60: add nand spl support

2012-10-29 Thread Scott Wood
On 10/26/2012 08:07:47 PM, xian...@openmobilefree.net wrote: From: Xiangfu The JZ4740 CPU can load 8KB from two different addresses: 1. the normal area up to 8KB starting from NAND flash address 0x 2. the backup area up to 8KB starting from NAND flash address 0x2000 Si

Re: [U-Boot] [PATCH 0/2] Standardize on run-time board ID variables

2012-10-29 Thread Joe Hershberger
Hi Stephen, On Mon, Oct 29, 2012 at 1:13 PM, Tom Rini wrote: > On Mon, Oct 29, 2012 at 09:15:41AM -0600, Stephen Warren wrote: >> On 10/26/2012 01:45 AM, Joe Hershberger wrote: >> > Hi Tom, >> > >> > On Wed, Oct 24, 2012 at 2:32 PM, Tom Rini wrote: >> >> On Wed, Oct 24, 2012 at 01:05:16PM -0600,

[U-Boot] [PATCH] mv-common.h: increase malloc arena to 4MiB

2012-10-29 Thread Andreas Bießmann
This will fix the following error: ---8<--- UBIFS error (pid 0): ubifs_mount: Error reading superblock on volume 'ubi:root' errno=-12! --->8--- Signed-off-by: Andreas Bießmann Cc: prafu...@marvell.com Cc: dimax.m...@gmail.com --- include/configs/mv-common.h |2 +- 1 file changed, 1 inserti

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Andreas Bießmann
Dear Dimax, On 29.10.2012 20:05, Dimax wrote: U-boot will _not_ recover your UBIFS partition like the Linux kernel. CMIIW. In other words it is not possible to use UBIFS on embedded Linux boards with u-boot unless you can provide unterminated power supply. NAK I invested some time to reprodu

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

2012-10-29 Thread Tom Warren
Albert, Please pull u-boot-tegra/master into ARM/master. Thanks! checkpatch is clean, and ./MAKEALL -a arm is clean (except for previous failures not created by these patches). Note that the serial: patch from Allen is already in u-boot/master, but is needed to get u-boot-tegra binaries to work c

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

2012-10-29 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 v2 00/32] Initial sparse fix series

2012-10-29 Thread Kim Phillips
[v2 summary: zero inter-commit build errors on all power, arm, and mips boards (which means some patches got merged into patch #1), checkpatch fixes, and postponement of the libfdt changes, primarily to get the bulk of these in during this merge window] This 25-patch series only begins to address

[U-Boot] [PATCH v2 15/25] arch/powerpc/cpu/mpc8xxx/: sparse fixes

2012-10-29 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 v2 25/25] drivers/serial/serial_ns16550.c: sparse fixes

2012-10-29 Thread Kim Phillips
Signed-off-by: Kim Phillips --- v2: checkpatch fixes. drivers/serial/serial_ns16550.c | 42 ++--- 1 file changed, 27 insertions(+), 15 deletions(-) diff --git a/drivers/serial/serial_ns16550.c b/drivers/serial/serial_ns16550.c index 4176e25..bf280ba 100644 --

[U-Boot] [PATCH v2 07/25] arch/powerpc/lib/board.c, *traps.c: sparse fixes

2012-10-29 Thread Kim Phillips
traps.c:*:1: warning: symbol 'print_backtrace' 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_info' was not declared. Should it be static? board.c:174:5: warning: symbol '__b

[U-Boot] [PATCH v2 12/25] drivers/net/: sparse fixes

2012-10-29 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 v2 21/25] drivers/mmc/mmc.c: sparse fixes

2012-10-29 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 v2 02/25] include/linux/compat.h: fix warning: preprocessor token {__iomem, __user} redefined

2012-10-29 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 v2 23/25] drivers/mtd/cfi_flash.c: sparse fixes

2012-10-29 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 v2 18/25] drivers/block/: sparse fixes

2012-10-29 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 v2 22/25] drivers/mmc/fsl_esdhc.c: sparse fixes

2012-10-29 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 --- v2: no changes drivers/mmc/fsl_esdhc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

[U-Boot] [PATCH v2 20/25] drivers/input/input.c: sparse fix

2012-10-29 Thread Kim Phillips
input.c:97:5: warning: symbol 'input_queue_ascii' was not declared. Should it be Signed-off-by: Kim Phillips --- v2: no changes drivers/input/input.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/input.c b/drivers/input/input.c index 9800667..04fa5f0 100644 -

[U-Boot] [PATCH v2 11/25] net/: sparse fixes

2012-10-29 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 v2 16/25] powerpc/mpc85xx: sparse fixes

2012-10-29 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 v2 19/25] drivers/gpio/mpc83xx_gpio.c: sparse fixes

2012-10-29 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 --- v2: no changes drivers/gpio/mpc83xx_gpio.c | 4 ++-- 1 file changed,

[U-Boot] [PATCH v2 10/25] common/misc: sparse fixes

2012-10-29 Thread Kim Phillips
command.c:44:38: error: bad constant expression 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: Using plain integer as NULL pointer dlmalloc.c:2179:31: warning: Using plain integer as NULL p

[U-Boot] [PATCH v2 13/25] lib/zlib: sparse fixes

2012-10-29 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 v2 05/25] arch/powerpc/include/asm/io.h: fix warning: preprocessor token __iomem redefined

2012-10-29 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 v2 04/25] include/linux/stddef.h: avoid 'warning: preprocessor token offsetof redefined'

2012-10-29 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 v2 14/25] lib/vsprintf.c: sparse fixes

2012-10-29 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 --- v2: no changes lib/vsprintf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/vsprintf.

[U-Boot] [PATCH v2 06/25] arch/powerpc/lib/extable.c: sparse fix

2012-10-29 Thread Kim Phillips
extable.c:66:9: warning: symbol 'ex_tab_message' was not declared. Should it be static? making it static can produce a new build warning on some boards: extable.c:66:12: warning: 'ex_tab_message' defined but not used [-Wunused-variable] but ex_tab_message doesn't do much even when used, so jus

[U-Boot] [PATCH v2 08/25] include/image.h: sparse fixes

2012-10-29 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 v2 17/25] powerpc/mpc83xx: sparse fixes

2012-10-29 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 v2 03/25] include/linux/unaligned/generic.h: fix warning: preprocessor token __force redefined

2012-10-29 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 v2 24/25] drivers/mtd/nand: sparse fixes

2012-10-29 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

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Andreas Bießmann
Dear Vikram Narayanan, first of all you are right. u-boot ubifs implementation will never recover the ubifs on media, cause it is mounted read only. On 29.10.2012 19:53, Vikram Narayanan wrote: On 10/29/2012 10:55 PM, Dimax wrote: I remember I have unbricked sheevaplug with USB key. But I'm

Re: [U-Boot] [PATCH] ext4: Fix printf() format string error

2012-10-29 Thread Tom Rini
On Tue, Oct 23, 2012 at 04:49:25PM -0700, Simon Glass wrote: > Fix the following error in the ext4 command: > > cmd_ext4.c:110:3: error: format '%lu' expects argument of type > 'long unsigned int', but argument 4 has type 'int' [-Werror=format] > > Signed-off-by: Simon Glass Applied to u-boot/

Re: [U-Boot] [PATCH] FPGA: Cyclon II: Correctly reset the FPGA before configuration

2012-10-29 Thread Tom Rini
On Tue, Oct 23, 2012 at 11:11:41AM +0200, Stefan Roese wrote: > From: Stephan Gatzka > > Deassert the CONFIG pin before asserting it again. This assures that the > FPGA will be resetted and therefore configuration will be correctly > enabled. > > This is also already done on other FPGA's, e.g.

Re: [U-Boot] [PATCH V4 1/3] fs: delete unused Makefile

2012-10-29 Thread Tom Rini
On Mon, Oct 22, 2012 at 10:43:49AM -0600, Stephen Warren wrote: > From: Stephen Warren > > fs/Makefile is unused. The top-level Makefile sets LIBS-y += fs/xxx and > hence causes make to directly descend two directory levels into each > individual filesystem, and it never descends into fs/ itself

Re: [U-Boot] [PATCH] arm720t: add back common.h include

2012-10-29 Thread Tom Warren
Thanks, Allen. Nice work. Applied to /next and testing now. I'll push a new copy to denx.de if it builds/boots OK. > -Original Message- > From: Allen Martin [mailto:amar...@nvidia.com] > Sent: Monday, October 29, 2012 1:48 PM > To: Tom Warren; swar...@wwwdotorg.org; ma...@denx.de > Cc: u-

Re: [U-Boot] [PATCH] arm720t: add back common.h include

2012-10-29 Thread Tom Warren
> -Original Message- > From: Allen Martin [mailto:amar...@nvidia.com] > Sent: Monday, October 29, 2012 1:48 PM > To: Tom Warren; swar...@wwwdotorg.org; ma...@denx.de > Cc: u-boot@lists.denx.de; Allen Martin > Subject: [PATCH] arm720t: add back common.h include > > Add back common.h header

Re: [U-Boot] [PATCH v5 1/7] powerpc: Extract EPAPR_MAGIC constants into processor.h

2012-10-29 Thread Tom Rini
On Tue, Oct 23, 2012 at 10:28:29AM +0200, Stefan Roese wrote: > By extracting these defines into a header, they can be re-used by other > C sources as well. This will be done by the SPL framework OS boot > support. > > Signed-off-by: Stefan Roese release.S:354:0: warning: "EPAPR_MAGIC" redefine

[U-Boot] [PATCH] arm720t: add back common.h include

2012-10-29 Thread Allen Martin
Add back common.h header that was removed in previous patch so that CONFIG_TEGRA can be evaluated correctly. Signed-off-by: Allen Martin --- arch/arm/cpu/arm720t/interrupts.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/cpu/arm720t/interrupts.c b/arch/arm/cpu/arm720t/interr

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Dimax
> U-boot will _not_ recover your UBIFS partition like the Linux kernel. CMIIW. In other words it is not possible to use UBIFS on embedded Linux boards with u-boot unless you can provide unterminated power supply. I'm going to another solution - Split flash in two UBIFS partitions. Make the first o

Re: [U-Boot] [PATCH] common/spl: Mark arguments as unused

2012-10-29 Thread Vikram Narayanan
On 10/29/2012 10:04 PM, Scott Wood wrote: It's not worth arguing over a single line of code that isn't going to cause any significant change. That would save us both some time. So you're withdrawing the patch? Yes. I'm more concerned about the precedent it sets than the one specific line.

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Vikram Narayanan
On 10/29/2012 10:55 PM, Dimax wrote: > I remember I have unbricked sheevaplug with USB key. > But I'm still not sure if effort is not useless as I see no conclusion > about u-boot ability to recover UBIFS partition (at least to try). > Can anybody tell it for sure? This is what I've got from the c

Re: [U-Boot] [PATCH v3 21/22] ide: Correct function signatures for ide_read/write()

2012-10-29 Thread Simon Glass
Hi, On Mon, Oct 29, 2012 at 11:22 AM, Tom Rini wrote: > On Mon, Oct 29, 2012 at 08:24:04AM -0700, Simon Glass wrote: > >> The prototypes in the header were changed by commit 4ac8f8e0 but the >> functions no longer match. Correct this. > > Oops, not sure how I missed that. > >> It seems odd that b

Re: [U-Boot] [PATCH v3 21/22] ide: Correct function signatures for ide_read/write()

2012-10-29 Thread Tom Rini
On Mon, Oct 29, 2012 at 08:24:04AM -0700, Simon Glass wrote: > The prototypes in the header were changed by commit 4ac8f8e0 but the > functions no longer match. Correct this. Oops, not sure how I missed that. > It seems odd that block devices take an lbaint_t for the block count, but > an unsign

Re: [U-Boot] [PATCH 0/2] Standardize on run-time board ID variables

2012-10-29 Thread Tom Rini
On Mon, Oct 29, 2012 at 09:15:41AM -0600, Stephen Warren wrote: > On 10/26/2012 01:45 AM, Joe Hershberger wrote: > > Hi Tom, > > > > On Wed, Oct 24, 2012 at 2:32 PM, Tom Rini wrote: > >> On Wed, Oct 24, 2012 at 01:05:16PM -0600, Stephen Warren wrote: > >>> On 10/24/2012 12:41 PM, Tom Rini wrote:

Re: [U-Boot] Current u-boot memory mapping

2012-10-29 Thread Tom Rini
On Sat, Oct 27, 2012 at 08:17:00PM +0900, RgC wrote: [snip] > My understanding is that after relocation no area between the bottom > and the top of RAM is reserved. We can use it freely. Is this correct? Basically, yes. You can use 'bdinfo' to see what / where things are being used at run-time.

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

2012-10-29 Thread Pali Rohár
On Tuesday 23 October 2012 09:20:39 Igor Grinberg wrote: > > > +/* > > + * Routine: board_init > > + * Description: Early hardware init. > > + */ > > +int board_init(void) > > +{ > > + gpmc_init(); /* in SRAM or SDRAM, finish GPMC */ > > + /* board id for Linux */ > > + gd->bd->bi_arch_number

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

2012-10-29 Thread Tom Rini
On Sat, Oct 27, 2012 at 05:29:32PM +0200, Marek Vasut wrote: > Dear Tom Rini, > > > On Fri, Oct 19, 2012 at 02:00:05PM +0200, Pali Roh??r wrote: > > > If dram bank size is calculated at runtime, it can be zero on some > > > boards. This patch added code which ignore these zero bank size in > > > A

[U-Boot] [PATCH 3/4] apf27: add spl support for the apf27

2012-10-29 Thread Philippe Reynes
Signed-off-by: Philippe Reynes Signed-off-by: Eric Jarrige create mode 100644 board/armadeus/apf27/start.S create mode 100644 board/armadeus/apf27/u-boot-spl.lds diff --git a/board/armadeus/apf27/Makefile b/board/armadeus/apf27/Makefile index 1da9548..f57f405 100644 --- a/board/armadeus/apf27

[U-Boot] [PATCH 4/4] apf27: add FPGA support for the apf27 board

2012-10-29 Thread Philippe Reynes
Signed-off-by: Philippe Reynes Signed-off-by: Eric Jarrige create mode 100644 board/armadeus/apf27/fpga.c create mode 100644 board/armadeus/apf27/fpga.h diff --git a/board/armadeus/apf27/Makefile b/board/armadeus/apf27/Makefile index f57f405..9d356a3 100644 --- a/board/armadeus/apf27/Makefile

[U-Boot] [PATCH 2/4] apf27: add support for the armadeus APF27 board

2012-10-29 Thread Philippe Reynes
Signed-off-by: Philippe Reynes Signed-off-by: Eric Jarrige Signed-off-by: Nicolas Colombain create mode 100644 board/armadeus/apf27/Makefile create mode 100644 board/armadeus/apf27/apf27.c create mode 100644 board/armadeus/apf27/apf27.h create mode 100644 include/configs/apf27.h diff --git

[U-Boot] [PATCH 1/4] apf27: add missing constant to mx27

2012-10-29 Thread Philippe Reynes
Signed-off-by: Philippe Reynes Signed-off-by: Eric Jarrige create mode 100644 arch/arm/include/asm/arch-mx27/mxc_nand.h diff --git a/arch/arm/cpu/arm926ejs/mx27/asm-offsets.c b/arch/arm/cpu/arm926ejs/mx27/asm-offsets.c index f3a8d7b..b95c58c 100644 --- a/arch/arm/cpu/arm926ejs/mx27/asm-offset

[U-Boot] [PATCH 0/4] apf27: add support of armadeus apf27 v2

2012-10-29 Thread Philippe Reynes
Difference between v1 and v2: - use spl instead of nand_spl - use gpio api - remove use of useless CONFIG_SYS - remove use of config.mk - remove dead code Philippe Reynes (4): apf27: add missing constant to mx27 apf27: add support for the armadeus APF27 board apf27: add spl support for the a

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Dimax
I remember I have unbricked sheevaplug with USB key. But I'm still not sure if effort is not useless as I see no conclusion about u-boot ability to recover UBIFS partition (at least to try). Can anybody tell it for sure? On Mon, Oct 29, 2012 at 2:15 PM, Prafulla Wadaskar wrote: > > > > -Orig

Re: [U-Boot] Pull request: u-boot-mpc8xx/master

2012-10-29 Thread Tom Rini
On Sun, Oct 28, 2012 at 08:33:06PM +0100, Wolfgang Denk wrote: > Dear tom, > > the following changes since commit 5bb3505fa867ded03cbee83f7722ab5182930637: > > Merge branch 'master' of git://git.denx.de/u-boot-arm (2012-10-26 15:44:31 > -0700) > > are available in the git repository at: > >

Re: [U-Boot] Pull request:u-boot-coldfire/master

2012-10-29 Thread Tom Rini
On Mon, Oct 29, 2012 at 02:22:52PM +, Jin Zhengxiong-R64188 wrote: > Hi, Tom, > > The following changes since commit 5bb3505fa867ded03cbee83f7722ab5182930637: > > Merge branch 'master' of git://git.denx.de/u-boot-arm (2012-10-26 15:44:31 > -0700) > > are available in the git repository a

Re: [U-Boot] [PATCH] common/spl: Mark arguments as unused

2012-10-29 Thread Scott Wood
On 10/28/2012 12:04:06 PM, Vikram Narayanan wrote: On 10/24/2012 10:41 PM, Scott Wood wrote: On 10/23/2012 11:14:34 PM, Vikram Narayanan wrote: On 10/24/2012 7:22 AM, Scott Wood wrote: On 10/23/2012 12:15:11 PM, Vikram Narayanan wrote: On 10/23/2012 9:15 PM, Tom Rini wrote: On Tue, Oct 23, 2

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Prafulla Wadaskar
> -Original Message- > From: u-boot-boun...@lists.denx.de [mailto:u-boot- > boun...@lists.denx.de] On Behalf Of Andreas Bießmann > Sent: 29 October 2012 14:41 > To: Dimax > Cc: Marek Vasut; u-boot@lists.denx.de > Subject: Re: [U-Boot] UBIFS fails on SheevaPlug > > Dear Dimax, > > please

Re: [U-Boot] UBIFS fails on SheevaPlug

2012-10-29 Thread Vikram Narayanan
Hello Dimax, On 10/29/2012 2:57 PM, Dimax wrote: I also want to look back at my original problem of non-recovered UBIFS. Want to make sure my effort will not be useless. Let's say I've increased the memory allocation size. But are you sure u-boot implementation of ubifs really can recover partit

Re: [U-Boot] [PATCH 0/3] Bring in new I2C framework

2012-10-29 Thread Simon Glass
Hi Stephen, On Mon, Oct 29, 2012 at 8:34 AM, Stephen Warren wrote: > On 10/29/2012 03:47 AM, Heiko Schocher wrote: >> Hello Stephen, >> >> On 26.10.2012 18:07, Stephen Warren wrote: >>> On 10/25/2012 11:48 PM, Heiko Schocher wrote: Hello Simon, On 25.10.2012 23:37, Simon Glass wrot

Re: [U-Boot] [PATCH 0/3] Bring in new I2C framework

2012-10-29 Thread Stephen Warren
On 10/29/2012 03:47 AM, Heiko Schocher wrote: > Hello Stephen, > > On 26.10.2012 18:07, Stephen Warren wrote: >> On 10/25/2012 11:48 PM, Heiko Schocher wrote: >>> Hello Simon, >>> >>> On 25.10.2012 23:37, Simon Glass wrote: On Mon, Oct 22, 2012 at 10:40 AM, Heiko Schocher wrote: > rebas

Re: [U-Boot] ping failed; host 192.168.1.101 is not alive

2012-10-29 Thread akdas75
hi No I dont have any firewall running. I have enabled to receive all packets [] i am using the uboot version 2011.03 with the following changes. But i think it will not affect the ping functionality. Any help. I will try putting more logs and pointing the issue. attached is the patch Ajoy

[U-Boot] [PATCH v3 02/22] scsi: Provide support for a list of AHCI controllers.

2012-10-29 Thread Simon Glass
From: Vadim Bendebury Many AHCI controllers are identical, the main (and often the only) difference being the PCI Vendor ID/Device ID combination reported by the device. This change allows the config file to define a list of PCI vendor ID/device ID pairs. The driver would scan the list and initi

[U-Boot] [PATCH v3 14/22] ahci: Support 64-bit LBA option when reading capacity

2012-10-29 Thread Simon Glass
From: Gabe Black Capacity needs to allow for a 64-bit value. Signed-off-by: Gabe Black Signed-off-by: Simon Glass --- Changes in v3: - Add patch to support 64-bit LBA option when reading capacity common/cmd_scsi.c | 92 ++--- 1 files changed,

[U-Boot] [PATCH v3 22/22] x86: config: Enable AHCI support for coreboot

2012-10-29 Thread Simon Glass
Enable AHCI driver for Intel SATA devices. Signed-off-by: Simon Glass --- include/configs/coreboot.h | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git a/include/configs/coreboot.h b/include/configs/coreboot.h index 3df085b..cc95e2b 100644 --- a/include

[U-Boot] [PATCH v3 11/22] ahci: Make the AHCI code find the capacity of disks > 128 GB properly

2012-10-29 Thread Simon Glass
From: Gabe Black In the structure returned by the ATA identify device command, there are two fields which describe the device capacity. One is a 32 bit data type which reports the number of sectors as a 28 bit LBA, and the other is a 64 bit data type which is for a 48 bit LBA. If the device doesn

[U-Boot] [PATCH v3 06/22] ahci: Optimise AHCI controller reset and start-up

2012-10-29 Thread Simon Glass
From: Stefan Reinauer The existing code waits a whole second for the AHCI controller to reset. Instead, let's poll the status register to see if the reset has succeeded and return earlier if possible. This brings down the time for AHCI probing from 1s to 20ms. Signed-off-by: Stefan Reinauer Sig

[U-Boot] [PATCH v3 09/22] ahci: Make sending the SETFEATURES_XFER command optional

2012-10-29 Thread Simon Glass
From: Gabe Black This command doesn't really do anything when talking to a SATA device, and sending it confuses some of them. This change makes sending the command optional, and defaults to not. The situations where it should be sent are not the common case. With the standard SSD in the machine,

[U-Boot] [PATCH v3 05/22] scsi: Add function and env var to report number of scsi drives

2012-10-29 Thread Simon Glass
From: Stefan Reinauer Add a new function to find out the number of available SCSI disks. Also set the 'scsidevs' environment variable after each scan. Signed-off-by: Stefan Reinauer Signed-off-by: Simon Glass --- Changes in v3: - Fix setenv operation to use correct function Changes in v2: - S

[U-Boot] [PATCH v3 07/22] ahci: Improve AHCI debugging

2012-10-29 Thread Simon Glass
From: Stefan Reinauer - remove unused ssleep macro - add some useful debugging information Signed-off-by: Stefan Reinauer Signed-off-by: Simon Glass --- drivers/block/ahci.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/block/ahci.c b/drivers/block/ahci.c

[U-Boot] [PATCH v3 04/22] Support setenv_ulong() and setenv_addr() for powerpc

2012-10-29 Thread Simon Glass
This includes were outside an #ifdef CONFIG_PPC, but there is not reason to exclude powerpc from using them. Move the declaration outside the #ifdef. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to enable setenv_ulong/addr() for powerpc include/common.h |2 +- 1 files chan

[U-Boot] [PATCH v3 19/22] ahci: Perform SATA flush after disk write.

2012-10-29 Thread Simon Glass
From: Marc Jones Writes in u-boot are so rare, and the logic to know when is the last write and do a flush only there is sufficiently difficult. Just do a flush after every write. This incurs, usually, one extra flush when the rare writes do happen. Signed-off-by: Marc Jones Signed-off-by: Simo

  1   2   >