Re: [U-Boot] [PATCH v2] arm: Correct build error introduced by getenv_ulong() patch

2011-10-24 Thread Albert ARIBAUD
Hi Wolfgang, Le 24/10/2011 21:13, Wolfgang Denk a écrit : > Dear Simon Glass, > > In message<1319427875-29965-1-git-send-email-...@chromium.org> you wrote: >> Commit dc8bbea removed a local variable that is used in most ARM boards. >> >> Since we want to avoid an 'unused variable' warning with la

Re: [U-Boot] [PATCH 0/8] Add tftpput command for uploading files over network

2011-10-24 Thread Albert ARIBAUD
Hi Simon, Le 25/10/2011 06:13, Simon Glass a écrit : Many U-Boot environments use 'tftp' as a shorthand to tftpboot. Did you verify that this is not broken by the introduction of 'tftpput'? > > I forgot to answer this. No this will break 'tftp' as 'tftpserver' > does. I vaguely remember

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Albert ARIBAUD
Hi Simon, Le 25/10/2011 07:02, Simon Glass a écrit : > Hi Albert, > > On Mon, Oct 24, 2011 at 2:21 PM, Albert ARIBAUD > wrote: >> Le 24/10/2011 22:14, Simon Glass a écrit : >> > So how about I create a patch to move the cp15_init() code into > arch/arm/cpu/armv7/lib/lowlevel.S or similar

Re: [U-Boot] [PATCH V2 0/4] add mmc support for pantheon platform

2011-10-24 Thread Albert ARIBAUD
Le 25/10/2011 08:15, Albert ARIBAUD a écrit : > Hi Lei Wen, > > Le 25/10/2011 03:21, Lei Wen a écrit : >> Hi Albert, >> >> On Tue, Oct 25, 2011 at 1:20 AM, Albert ARIBAUD >>wrote: >>> Hi Lei Wen, >>> >>> Le 04/10/2011 08:33, Lei Wen a écrit : This patch seris add the mmc support for the pa

Re: [U-Boot] [PATCH V2 0/4] add mmc support for pantheon platform

2011-10-24 Thread Albert ARIBAUD
Hi Lei Wen, Le 25/10/2011 03:21, Lei Wen a écrit : > Hi Albert, > > On Tue, Oct 25, 2011 at 1:20 AM, Albert ARIBAUD > wrote: >> Hi Lei Wen, >> >> Le 04/10/2011 08:33, Lei Wen a écrit : >>> This patch seris add the mmc support for the pantheon platform. >>> Also give platform like dkb and aspenit

[U-Boot] [PATCH v6 2/6] fdt: Add support for embedded device tree (CONFIG_OF_EMBED)

2011-10-24 Thread Simon Glass
This new option allows U-Boot to embed a binary device tree into its image to allow run-time control of peripherals. This device tree is for U-Boot's own use and is not necessarily the same one as is passed to the kernel. The device tree compiler output should be placed in the $(obj) rooted tree.

[U-Boot] [PATCH v6 5/6] fdt: ARM: Implement and verify embedded and separate device tree

2011-10-24 Thread Simon Glass
This locates the device tree either embedded within U-Boot or attached to the end as a separate binary. When CONFIG_OF_CONTROL is defined, U-Boot requires a valid fdt. A check is provided for this early in initialisation. Signed-off-by: Simon Glass --- Changes in v6: - Move the fdt check into fd

[U-Boot] [PATCH v6 3/6] fdt: Add support for a separate device tree (CONFIG_OF_SEPARATE)

2011-10-24 Thread Simon Glass
This adds support for an FDT to be build as a separate binary file called u-boot.dtb. This can be concatenated with the U-Boot binary to provide a device tree located at run-time by U-Boot. The Makefile is modified to provide this file in u-boot-dtb.bin. Signed-off-by: Simon Glass --- Changes in

[U-Boot] [PATCH v6 4/6] fdt: add decode helper library

2011-10-24 Thread Simon Glass
This library provides useful functions to drivers which want to use the fdt to control their operation. Functions are provided to: - look up and enumerate a device type (for example assigning i2c bus 0, i2c bus 1, etc.) - decode basic types from the fdt, like addresses and integers While thi

[U-Boot] [PATCH v6 1/6] fdt: ARM: Add device tree control of U-Boot (CONFIG_OF_CONTROL)

2011-10-24 Thread Simon Glass
This adds a device tree pointer to the global data. It can be set by board code. A later commit will add support for making a device tree binary blob available to U-Boot for run-time configuration. Signed-off-by: Simon Glass --- Changes in v3: - Rename gd->blob to gd->fdt_blob Changes in v4: - F

[U-Boot] [PATCH v6 6/6] fdt: ARM: Add fdtcontroladdr to set device tree address in environment

2011-10-24 Thread Simon Glass
This adds support for a new environment variable called 'fdtcontroladdr'. If defined, the hex address is used as the address of the control fdt for U-Boot. Note: I have not changed CONFIG_PRAM section as I already have an outstanding patch on that. Signed-off-by: Simon Glass --- Changes in v3: -

[U-Boot] [PATCH v6 0/6] Run-time configuration of U-Boot via a flat device tree (fdt)

2011-10-24 Thread Simon Glass
At present in U-Boot configuration is mostly done using CONFIG options in the board file. This patch set makes it possible for a single U-Boot binary to support multiple boards, with the exact configuration of each board controlled by a flat device tree (fdt). This is the approach recently taken by

Re: [U-Boot] [PATCH v4 0/6] Run-time configuration of U-Boot via a flat device tree (fdt)

2011-10-24 Thread Simon Glass
Hi Wolfgang, On Mon, Oct 24, 2011 at 11:49 AM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message > you > wrote: >> >> There was a request to move the check_fdt function into the fdtdec >> library, and I think it makes sense. So I will do that and send an > > That was my unerstanding, too

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Simon Glass
Hi Albert, On Mon, Oct 24, 2011 at 2:21 PM, Albert ARIBAUD wrote: > Le 24/10/2011 22:14, Simon Glass a écrit : > So how about I create a patch to move the cp15_init() code into arch/arm/cpu/armv7/lib/lowlevel.S or similar so I can call it later? >>> >>> I don't mind moving it to armv7/l

Re: [U-Boot] [PATCH 0/8] Add tftpput command for uploading files over network

2011-10-24 Thread Simon Glass
Hi Albert, On Sun, Oct 23, 2011 at 9:28 PM, Simon Glass wrote: > Hi Albert, > > On Sat, Oct 22, 2011 at 9:15 AM, Simon Glass wrote: >> Hi Albert, >> >> On Sat, Oct 22, 2011 at 1:21 AM, Albert ARIBAUD >> wrote: >>> Le 22/10/2011 06:51, Simon Glass a écrit : The tftpboot command permits

[U-Boot] [PATCH v2 11/11] net: tftpput: add tftpput command

2011-10-24 Thread Simon Glass
This adds the tftpput command to U-Boot. Signed-off-by: Simon Glass --- common/cmd_net.c | 23 +++ 1 files changed, 23 insertions(+), 0 deletions(-) diff --git a/common/cmd_net.c b/common/cmd_net.c index f610385..f89a24b 100644 --- a/common/cmd_net.c +++ b/common/cmd_net.c

[U-Boot] [PATCH v2 04/11] net: tftpput: Add support for receiving ICMP packets

2011-10-24 Thread Simon Glass
ICMP packets can tell you when there is no server at the other end. It is useful for tftp to figure this out, so that a quick error can be displayed, rather than pointlessly retrying. This adds an ICMP packet handler to the net interface. Signed-off-by: Simon Glass --- include/net.h | 19

[U-Boot] [PATCH v2 02/11] Add setenv_ulong() and setenv_addr()

2011-10-24 Thread Simon Glass
It seems we put numbers and addresses into environment variables a lot. We should have some functions to do this. Signed-off-by: Simon Glass --- common/cmd_nvedit.c | 30 ++ include/common.h|2 ++ 2 files changed, 32 insertions(+), 0 deletions(-) diff --git

[U-Boot] [PATCH v2 10/11] net: tftpput: implement tftp logic

2011-10-24 Thread Simon Glass
This adds logic to tftp.c to implement the tftp 'put' command, and updates the README. Signed-off-by: Simon Glass --- Changes in v2: - Cope with block rollover - Don't print amount of data transferred on each block rollover README |2 + net/net.c |4 ++ net/tftp.c | 134 ++

[U-Boot] [PATCH v2 09/11] net: tftpput: add saveaddr and savesize env variables

2011-10-24 Thread Simon Glass
We need something akin to load_addr to handle saving data. Signed-off-by: Simon Glass --- common/cmd_nvedit.c |2 ++ include/common.h|2 ++ 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/common/cmd_nvedit.c b/common/cmd_nvedit.c index 4e27ba0..396a171 100644 --- a/com

[U-Boot] [PATCH v2 07/11] net: tftpput: Factor out start, restart and next block functions

2011-10-24 Thread Simon Glass
This code is required for tftpput, so move it into separate functions. Signed-off-by: Simon Glass --- net/tftp.c | 77 +++ 1 files changed, 51 insertions(+), 26 deletions(-) diff --git a/net/tftp.c b/net/tftp.c index 70daed0..6d46bbb 100

[U-Boot] [PATCH v2 08/11] net: tftpput: Support selecting get/put for tftp

2011-10-24 Thread Simon Glass
TftpStart should support starting either a get or a put. Signed-off-by: Simon Glass --- include/net.h |2 +- net/bootp.c |2 +- net/tftp.c|3 +-- net/tftp.h|2 +- 4 files changed, 4 insertions(+), 5 deletions(-) diff --git a/include/net.h b/include/net.h index 495f60e.

[U-Boot] [PATCH v2 01/11] Move simple_itoa() to vsprintf.c

2011-10-24 Thread Simon Glass
This function is generally useful and shouldn't hide away in hush. It has been moved as is. Signed-off-by: Simon Glass --- common/hush.c| 15 --- include/common.h |1 + lib/vsprintf.c | 16 3 files changed, 17 insertions(+), 15 deletions(-) diff --git

[U-Boot] [PATCH v2 06/11] net: tftpput: move common code into separate functions

2011-10-24 Thread Simon Glass
We want to show block markers on completion of get and put, so move this common code into separate functions. Signed-off-by: Simon Glass --- net/tftp.c | 68 +-- 1 files changed, 38 insertions(+), 30 deletions(-) diff --git a/net/tftp.c

[U-Boot] [PATCH v2 03/11] net: tftpput: Move ICMP code into its own function

2011-10-24 Thread Simon Glass
NetReceive() is a very long function with a lot of indent. Before adding code to the ICMP bit, split it out. Signed-off-by: Simon Glass --- net/net.c | 101 +++-- 1 files changed, 58 insertions(+), 43 deletions(-) diff --git a/net/net.c b

[U-Boot] [PATCH v2 05/11] net: tftpput: Rename TFTP to TFTPGET

2011-10-24 Thread Simon Glass
This is a better name for this protocol. Also remove the typedef to keep checkpatch happy, and move zeroing of NetBootFileXferSize a little earlier since TFTPPUT will need to change this. Signed-off-by: Simon Glass --- board/hymod/bsp.c |2 +- board/hymod/fetch.c |2 +- common/cmd_elf.

[U-Boot] [PATCH v2 0/11] Add tftpput command for uploading files over network

2011-10-24 Thread Simon Glass
The tftpboot command permits reading of files over a network interface using the Trivial FTP protocol. This patch series adds the ability to transfer files the other way. Why is this useful? - Uploading boot time data to a server - Uploading profiling information - Uploading large mounts of data

[U-Boot] [PATCH 2/2] RFC: Add XON/XOFF support

2011-10-24 Thread Simon Glass
This is just for testing - please try it out and report back with results. For me it works on Minicom but not ser2net. This needs to be controlled by an environment variable, CONFIG option or both. We may need a way of specifying flow control on a per-device basis. We may need to do something spec

[U-Boot] [PATCH 1/2] Create a single cmd_call() function to handle command execution

2011-10-24 Thread Simon Glass
We should aim for a single point of entry to the commands, whichever parser is used. Signed-off-by: Simon Glass --- common/command.c | 10 ++ common/hush.c |9 +++-- common/main.c |3 +-- include/command.h | 12 4 files changed, 26 insertions(+), 8 d

[U-Boot] [PATCH] ARM: orion5x: reduce dependence of including platform file

2011-10-24 Thread Lei Wen
For files like the drivers/serial/serial.c, it must include the platform file, as the CONFIG_SYS_NS16550_COM1 must reference to the definition in the platform definition files. Include the platform definition file in the config file, so that it would decouple the dependence for the driver files.

Re: [U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform code

2011-10-24 Thread Zang Roy-R61911
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Tuesday, October 25, 2011 3:05 AM > To: Zang Roy-R61911 > Cc: u-boot@lists.denx.de; Fleming Andy-AFLEMING; Kumar Gala > Subject: Re: [U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform > code > > Dear

Re: [U-Boot] [PATCH 4/5] serial: reduce include platform file for marvell chip

2011-10-24 Thread Lei Wen
Hi Albert, On Tue, Oct 25, 2011 at 12:46 AM, Albert ARIBAUD wrote: > Hi all, > > Le 02/10/2011 16:16, Lei Wen a écrit : >> >> Build pass with following config: >> dkb_config >> aspenite_config >> edminiv2_config >> openrd_ultimate_config >> sheevaplug_config >> mv88f6281gtw_ge_config >> rd6281a_c

Re: [U-Boot] [PATCH V2 0/4] add mmc support for pantheon platform

2011-10-24 Thread Lei Wen
Hi Albert, On Tue, Oct 25, 2011 at 1:20 AM, Albert ARIBAUD wrote: > Hi Lei Wen, > > Le 04/10/2011 08:33, Lei Wen a écrit : >> This patch seris add the mmc support for the pantheon platform. >> Also give platform like dkb and aspenite a workaround when enabling >> the 8bit mode for accessing the m

Re: [U-Boot] [PATCH 01/39] DEBUG: Fix debug macros

2011-10-24 Thread Marek Vasut
> Dear Marek Vasut, > > In message <201110241021.15442.marek.va...@gmail.com> you wrote: > > ... > > > Done, I pushed new patchset right now to > > git://git.denx.de/u-boot-marex.git , "debug" branch. It's around 50 > > patches now. > > Keep in mind that nobody cares about patches anywhere in a

Re: [U-Boot] [PATCH 02/39] GCC4.6: Squash warnings in common/usb.c

2011-10-24 Thread Marek Vasut
> Dear Marek Vasut, > > In message <201110241422.30837.marek.va...@gmail.com> you wrote: > > actually the value is checked by debug_cond(). > > Consider prefixing these names with a double underscore to make more > visibale that these are internally used and not for plain mortal users > to mess w

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread J. William Campbell
On 10/24/2011 3:22 PM, Graeme Russ wrote: > Hi Wolfgang, > > On Tue, Oct 25, 2011 at 8:59 AM, Wolfgang Denk wrote: >> Dear Graeme Russ, >> >> In message<4ea5cd39.2070...@gmail.com> you wrote: Assume we have a simple device with a small Rx FIFO - say, 8 bytes only. Guess what the chances

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Graeme Russ
Hi Wolfgang, On Tue, Oct 25, 2011 at 8:59 AM, Wolfgang Denk wrote: > Dear Graeme Russ, > > In message <4ea5cd39.2070...@gmail.com> you wrote: >> >> > Assume we have a simple device with a small Rx FIFO - say, 8 bytes >> > only. Guess what the chances are that we will overrun this FIFO (and >> > t

Re: [U-Boot] [PATCH v3 0/6] Add cache line alignment support

2011-10-24 Thread Anton Staaf
On Wed, Oct 12, 2011 at 4:55 PM, Anton Staaf wrote: > The cache line alignment issue has gone around a couple of times now.  This > patch set implements all of the details that we have discussed about the > implementation of ALLOC_CACHE_ALIGN_BUFFER.  It also includes patches that use > the macro

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Wolfgang Denk
Dear Graeme Russ, In message <4ea5cd39.2070...@gmail.com> you wrote: > > > Assume we have a simple device with a small Rx FIFO - say, 8 bytes > > only. Guess what the chances are that we will overrun this FIFO (and > > then lose characters) before the other side stops sending? > > Should be zero

Re: [U-Boot] Is there a branch with support for Karo TX51?

2011-10-24 Thread Wolfgang Denk
Dear Andreas =?iso-8859-1?q?M=FCller?=, In message <201110242230.50329.schnitzelt...@gmx.de> you wrote: > > grepping current master for tx51 gives: ... > I think this is not sufficient for building u-boot booting TX51... Correct. > Is there a branch or other source out there supporting TX51? KA

Re: [U-Boot] [PATCH] FIX for dcache_disable() for ARM926ej-s

2011-10-24 Thread Albert ARIBAUD
Hi Bas, Le 21/10/2011 08:44, Bas van den Berg a écrit : > the cache also needs to be invalidated, not just flushed, Since re-enabling > it, > can cause inconsistent data without invalidation. See example below. > > in c-file: > static int num = 1; > > void test() { > num = 1; > dcache_e

Re: [U-Boot] Is there a branch with support for Karo TX51?

2011-10-24 Thread Albert ARIBAUD
Hi Andreas, Le 24/10/2011 22:30, Andreas Müller a écrit : > Hi, > > grepping current master for tx51 gives: > > arch/arm/include/asm/mach-types.h:#define MACH_TYPE_TX51 2529 > arch/arm/include/asm/mach-types.h:#ifdef CONFIG_MACH_TX51 > arch/arm/include/asm/mach-types.h:# define ma

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Albert ARIBAUD
Le 24/10/2011 22:14, Simon Glass a écrit : >>> So how about I create a patch to move the cp15_init() code into >>> arch/arm/cpu/armv7/lib/lowlevel.S or similar so I can call it later? >> >> I don't mind moving it to armv7/lib. I do mind the "later". > > OK, but the AVP cannot execute this code, wh

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Graeme Russ
Hi Wolfgang, On 25/10/11 07:00, Wolfgang Denk wrote: > Dear Graeme Russ, > > In message <4ea5bbef.1050...@gmail.com> you wrote: >> >>> Why would any of the transfer commands actually turn off flow control? >> >> getc() sends an XOFF > > Oops? You got something wrong, I think. > > Receiving a '

[U-Boot] Is there a branch with support for Karo TX51?

2011-10-24 Thread Andreas Müller
Hi, grepping current master for tx51 gives: arch/arm/include/asm/mach-types.h:#define MACH_TYPE_TX51 2529 arch/arm/include/asm/mach-types.h:#ifdef CONFIG_MACH_TX51 arch/arm/include/asm/mach-types.h:# define machine_arch_type MACH_TYPE_TX51 arch/arm/include/asm/mach-types.h:# de

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Simon Glass
Hi Albert, On Mon, Oct 24, 2011 at 1:04 PM, Albert ARIBAUD wrote: > Le 22/10/2011 18:13, Simon Glass a écrit : >> >> Hi Albert, >> >> On Sat, Oct 22, 2011 at 12:56 AM, Albert ARIBAUD >>  wrote: >>> >>> Le 22/10/2011 07:05, Simon Glass a écrit : >>> >> Well I actually haven't moved it! It is

Re: [U-Boot] [PATCH v2 00/11] Define ARCH_DMA_MINALIGN for all architectures

2011-10-24 Thread Anton Staaf
On Mon, Oct 24, 2011 at 12:45 PM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message > you > wrote: >> >> I have now run MAKEALL for both ARMv7a and PowerPC successfully. >> There were a number of >> build failures for "powerpc", but no new ones given my patch.  I'd >> like to land this so

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Albert ARIBAUD
Le 22/10/2011 18:13, Simon Glass a écrit : > Hi Albert, > > On Sat, Oct 22, 2011 at 12:56 AM, Albert ARIBAUD > wrote: >> Le 22/10/2011 07:05, Simon Glass a écrit : >> > Well I actually haven't moved it! It is just that previously it was > impossible to call cp15_init from anywhere later.

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Wolfgang Denk
Dear Graeme Russ, In message <4ea5bbef.1050...@gmail.com> you wrote: > > > Why would any of the transfer commands actually turn off flow control? > > getc() sends an XOFF Oops? You got something wrong, I think. Receiving a '\n' would cause sending XOFF, and getc() would, if state is XOFF, sen

Re: [U-Boot] [PATCH v7] ARM: Add Calxeda Highbank platform

2011-10-24 Thread Albert ARIBAUD
Hi Rob, Le 24/10/2011 20:50, Rob Herring a écrit : > From: Rob Herring > > Add basic support for Calxeda Highbank platform. Only minimal support with > serial and SATA are included. > > Signed-off-by: Jason Hobbs > Signed-off-by: Rob Herring > Cc: Albert ARIBAUD > --- > > changes in v7: > - Reba

Re: [U-Boot] [PATCH v2 00/11] Define ARCH_DMA_MINALIGN for all architectures

2011-10-24 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > I have now run MAKEALL for both ARMv7a and PowerPC successfully. > There were a number of > build failures for "powerpc", but no new ones given my patch. I'd > like to land this soon > so I can get to landing the fixes for the unaligned buffers in U-B

Re: [U-Boot] [PATCH 1/2] fdt: Add new fdt_set_node_status & fdt_set_status_by_alias helpers

2011-10-24 Thread Wolfgang Denk
Dear Kumar Gala, In message <8d0c487d-8573-4556-ad02-60daf1104...@kernel.crashing.org> you wrote: > > > [Building with ELDK 4.2]: ... > Is part of our issue with older gcc-4.2 toolchain that the linker > doesn't garbage collect unused functions? Which binutils version is minimally needed? ElDK

Re: [U-Boot] [PATCH 02/39] GCC4.6: Squash warnings in common/usb.c

2011-10-24 Thread Wolfgang Denk
Dear Marek Vasut, In message <201110241422.30837.marek.va...@gmail.com> you wrote: > > actually the value is checked by debug_cond(). Consider prefixing these names with a double underscore to make more visibale that these are internally used and not for plain mortal users to mess with. Best re

Re: [U-Boot] [PATCH] Update s3c24x0 timer implementation

2011-10-24 Thread Wolfgang Denk
Dear Mark Norman, In message you wrote: > Dear Wolfgang Denk, > > Thank you for your response. > > >> =A0Since the .rel.text section is required by the relocation code, I > >> assume that .bss global variables cannot be used until after > >> relocation? > > > > Why do you have to make such ass

Re: [U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-24 Thread Simon Glass
Hi Albert, On Sat, Oct 22, 2011 at 9:13 AM, Simon Glass wrote: > Hi Albert, > > On Sat, Oct 22, 2011 at 12:56 AM, Albert ARIBAUD > wrote: >> Le 22/10/2011 07:05, Simon Glass a écrit : >> > Well I actually haven't moved it! It is just that previously it was > impossible to call cp15_init

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Graeme Russ
Hi Wolfgang, On 25/10/11 05:46, Wolfgang Denk wrote: > Dear Graeme Russ, > > In message > you > wrote: >> If so, could it not be possible that a Kermit/ymodem command followed by a time consuming command on the same line cause lost input? >>> >>> I don't think so. All serial transfe

Re: [U-Boot] [PATCH] [BUG] arm, lib: fix compile breakage

2011-10-24 Thread Albert ARIBAUD
Hi Wolfgang, Le 24/10/2011 21:12, Wolfgang Denk a écrit : > Dear Albert ARIBAUD, > > In message<4ea59236.4000...@aribaud.net> you wrote: >> >> Le 24/10/2011 07:42, Heiko Schocher a écrit : >>> since commit dc8bbea0170eb2aca428ea221c91fc2e5e11f199 building >>> arch/arm/lib/board.c breaks if CONFIG

Re: [U-Boot] [PATCH] [BUG] arm, lib: fix compile breakage

2011-10-24 Thread Wolfgang Denk
Dear Marek Vasut, In message <201110241115.45602.marek.va...@gmail.com> you wrote: > > I just sent something similar, but please use this one. It's really interesting to see how many people wake up if they are affected - and then hack away and send patches without even checing how many similar pa

Re: [U-Boot] [PATCH] Fix compile issue in arch/arm/lib/board.c

2011-10-24 Thread Wolfgang Denk
Dear Marek Vasut, In message <1319447521-6020-1-git-send-email-marek.va...@gmail.com> you wrote: > The commit dc8bbea0170eb2aca428ea221c91fc2e5e11f199 breaks the build of U-Boot > if CONFIG_CMD_NET is enabled. > > arm: Use getenv_ulong() in place of getenv(), strtoul > > This changes the board c

Re: [U-Boot] [PATCH 01/39] DEBUG: Fix debug macros

2011-10-24 Thread Wolfgang Denk
Dear Marek Vasut, In message <201110241021.15442.marek.va...@gmail.com> you wrote: > ... > Done, I pushed new patchset right now to git://git.denx.de/u-boot-marex.git , > "debug" branch. It's around 50 patches now. Keep in mind that nobody cares about patches anywhere in any repository. What ha

Re: [U-Boot] [PATCH v3 8/8] km_arm: enable POST for these boards

2011-10-24 Thread Wolfgang Denk
Dear Holger Brunck, In message <4ea50ff4.5030...@keymile.com> you wrote: > > you already committed a v4 of this patch together with the patch serie. See: > http://lists.denx.de/pipermail/u-boot/2011-October/104061.html Oops. Sorry... > So could you please revert this one? Because this breaks t

Re: [U-Boot] [PATCH v2] arm: Correct build error introduced by getenv_ulong() patch

2011-10-24 Thread Wolfgang Denk
Dear Simon Glass, In message <1319427875-29965-1-git-send-email-...@chromium.org> you wrote: > Commit dc8bbea removed a local variable that is used in most ARM boards. > > Since we want to avoid an 'unused variable' warning with later compilers, > and the #ifdef logic of whether this variable is

Re: [U-Boot] [PATCH] powerpc: Correct build warning introduced by getenv_ulong() patch

2011-10-24 Thread Wolfgang Denk
Dear Simon Glass, In message <1319427714-29479-1-git-send-email-...@chromium.org> you wrote: > Commit 1272592 introduced a warning since the variable 's' is no longer > always used, depending on the CONFIG options. > > Signed-off-by: Simon Glass > --- > arch/powerpc/lib/board.c | 10 +++--

Re: [U-Boot] [PATCH] [BUG] arm, lib: fix compile breakage

2011-10-24 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4ea59236.4000...@aribaud.net> you wrote: > > Le 24/10/2011 07:42, Heiko Schocher a écrit : > > since commit dc8bbea0170eb2aca428ea221c91fc2e5e11f199 building > > arch/arm/lib/board.c breaks if CONFIG_CMD_NET is defined. > > Fix this. > > > > Signed-off-by: Heiko S

Re: [U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform code

2011-10-24 Thread Wolfgang Denk
Dear Zang Roy-R61911, In message <2239ac579c7d3646a720227a37e02681200...@039-sn1mpn1-004.039d.mgd.msft.net> you wrote: > > How about this way: > Subject: net/frame manager: remove TBI PHY address register setting on plat= > form related code > > Then I add the other more detailed required info

Re: [U-Boot] [PATCH v2 00/11] Define ARCH_DMA_MINALIGN for all architectures

2011-10-24 Thread Anton Staaf
On Mon, Oct 17, 2011 at 4:46 PM, Anton Staaf wrote: > ARCH_DMA_MINALIGN is a new define borrowed from the Linux kernel.  It is > used to define the minimum alignement requirement for a DMA buffer.  This > series of patches ensures that it will always be defined in the arch specific > asm/cache.h h

[U-Boot] [PATCH v7] ARM: Add Calxeda Highbank platform

2011-10-24 Thread Rob Herring
From: Rob Herring Add basic support for Calxeda Highbank platform. Only minimal support with serial and SATA are included. Signed-off-by: Jason Hobbs Signed-off-by: Rob Herring Cc: Albert ARIBAUD --- changes in v7: - Rebase to current master (4962e38 mkimage: adding support for Davinci AIS

Re: [U-Boot] [PATCH v4 0/6] Run-time configuration of U-Boot via a flat device tree (fdt)

2011-10-24 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > There was a request to move the check_fdt function into the fdtdec > library, and I think it makes sense. So I will do that and send an That was my unerstanding, too. > update. I don't have any other changes ATM. And that's why I asked :-) Best re

Re: [U-Boot] [PATCH 2/2] bootstage: Make use of BOOTSTAGE_ID_RUN_OS in show_boot_progress()

2011-10-24 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > IMHO we would be better off with a function call like > >        show_boot_error(BOOTSTAGE_ID_RUN_OS); > > even if this is a macro. It is more explicit about the fact that we > are reporting an error. Does that sound ok? That's fine with me. [Just m

Re: [U-Boot] [PATCH v2] NS16550: buffer reads

2011-10-24 Thread Wolfgang Denk
Dear Graeme Russ, In message you wrote: > > >> If so, could it not be possible that a Kermit/ymodem command followed by a > >> time consuming command on the same line cause lost input? > > > > I don't think so. All serial transfers use a protocol - and when the > > transfer is complete, it does

[U-Boot] [PATCH] mx51evk: Remove unneeded '1' from mx51evk.h

2011-10-24 Thread Fabio Estevam
Remove unneeded '1' from mx51evk.h. Signed-off-by: Fabio Estevam --- include/configs/mx51evk.h | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h index 2c449b8..af9f983 100644 --- a/include/configs/mx51evk.h +++

[U-Boot] [PATCH] mx53: Turn off child clocks before reconfigure perclk_root

2011-10-24 Thread Fabio Estevam
In addition to ensuring that PERCLK remains at least 2.5 times slower than the AHB clock, certain steps need to be followed to ensure robust operation of PERCLK when reconfiguring the PERCLK clock source. To properly configure the PERCLK clock source, the following steps are required: 1.In the CC

Re: [U-Boot] [Patch v2 6/7] powerpc/mpc8349emds: Migrate from spd_sdram to unified DDR driver

2011-10-24 Thread Kumar Gala
On Oct 23, 2011, at 4:53 AM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <2a6300e8-a874-49ac-84cb-be681af1b...@freescale.com> you wrote: >> >> On Aug 26, 2011, at 1:32 PM, York Sun wrote: >> >>> Update MPC8349EMDS to use unified DDR driver instead of spd_sdram.c. >>> The unified dri

Re: [U-Boot] [PATCH 1/2] fdt: Add new fdt_set_node_status & fdt_set_status_by_alias helpers

2011-10-24 Thread Kumar Gala
On Oct 23, 2011, at 3:49 AM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <1318619444-2059-1-git-send-email-ga...@kernel.crashing.org> you > wrote: >> From: Shengzhou Liu >> >> Add common function fdt_set_node_status() to assist in various locations >> that we set a nodes status. T

Re: [U-Boot] [PATCH V2 0/4] add mmc support for pantheon platform

2011-10-24 Thread Albert ARIBAUD
Hi Lei Wen, Le 04/10/2011 08:33, Lei Wen a écrit : > This patch seris add the mmc support for the pantheon platform. > Also give platform like dkb and aspenite a workaround when enabling > the 8bit mode for accessing the mmc. > > Changelog: > V2: remove magic number, and replace it by macro defini

Re: [U-Boot] [PATCH 2/3] nds32: cache: define ARCH_DMA_MINALIGN for DMA buffer alignment

2011-10-24 Thread Anton Staaf
On Mon, Oct 24, 2011 at 2:46 AM, Macpaul Lin wrote: > Add ARCH_DMA_MINALIGN definition to asm/cache.h > > Signed-off-by: Macpaul Lin Acked-by: Anton Staaf > --- >  arch/nds32/include/asm/cache.h |   11 +++ >  1 files changed, 11 insertions(+), 0 deletions(-) > > diff --git a/arch/nds32

Re: [U-Boot] [PATCH 4/5] serial: reduce include platform file for marvell chip

2011-10-24 Thread Albert ARIBAUD
Hi all, Le 02/10/2011 16:16, Lei Wen a écrit : > Build pass with following config: > dkb_config > aspenite_config > edminiv2_config > openrd_ultimate_config > sheevaplug_config > mv88f6281gtw_ge_config > rd6281a_config > guruplug_config > km_kirkwood_config > > Signed-off-by: Lei Wen > --- > dri

[U-Boot] [PATCH] mmc: access mxcmmc from mx31 boards

2011-10-24 Thread Helmut Raiger
This patch modifies mxcmmc.c to be used not only by i.MX27 but also by i.MX31 boards. Both use the same SD controller, but have different clock set-ups. The clock access was renamed to imx_get_mmcclk() to allow both architectures to provide their respective clock. Pins, base address and prototypes

[U-Boot] [PATCH v2 1/2] rtc: Make mc13783-rtc driver generic

2011-10-24 Thread Fabio Estevam
Rename mc13783-rtc so that it can be used for both MC13783 and MC13892 PMICs. efikamx board, for example, does use a MC13892 PMIC, but the RTC selection is currently made as: #define CONFIG_RTC_MC13783 ,which is not very obvious. Let the MC13783 and MC13892 RTC be selected by: #define CONFIG_

[U-Boot] [PATCH v2 2/2] mx51evk: Add RTC support

2011-10-24 Thread Fabio Estevam
MX51EVK has a MC13892 PMIC. Add RTC support. Tested via 'date' command that reads the PMIC RTC registers: MX51EVK U-Boot > date Date: 1970-01-01 (Thursday)Time: 0:40:35 Signed-off-by: Fabio Estevam

[U-Boot] [PATCH 2/2] mx51evk: Add RTC support

2011-10-24 Thread Fabio Estevam
MX51EVK has a MC13892 PMIC. Add RTC support. Tested via 'date' command that reads the PMIC RTC registers: MX51EVK U-Boot > date Date: 1970-01-01 (Thursday)Time: 0:40:35 Signed-off-by: Fabio Estevam

[U-Boot] [PATCH 1/2] rtc: Make mc13783-rtc driver generic

2011-10-24 Thread Fabio Estevam
Rename mc13783-rtc so that it can be used for both MC13783 and MC13892 PMICs. efikamx board, for example, does use a MC13892 PMIC, but the RTC selection is currently made as: #define CONFIG_RTC_MC13783 ,which is not very obvious. Let the MC13783 and MC13892 RTC be selected by: #define CONFIG_

Re: [U-Boot] [PATCH] [BUG] arm, lib: fix compile breakage

2011-10-24 Thread Albert ARIBAUD
Hi Heiko, Le 24/10/2011 07:42, Heiko Schocher a écrit : > since commit dc8bbea0170eb2aca428ea221c91fc2e5e11f199 building > arch/arm/lib/board.c breaks if CONFIG_CMD_NET is defined. > Fix this. > > Signed-off-by: Heiko Schocher > Cc: Albert ARIBAUD > Cc: Simon Glass > --- > arch/arm/lib/board.c |

[U-Boot] [PATCH] mx31pdk: Add MC13783 PMIC support

2011-10-24 Thread Fabio Estevam
Add MC13783 PMIC support. Tested by using the 'date' command, which reads the MC13783 RTC registers: MX31PDK U-Boot > date Date: 1970-01-01 (Thursday)Time: 2:22:35 Signed-off-by: Fabio Estevam --- board/freescale/mx31pdk/mx31pdk

Re: [U-Boot] Accessing MC13783 RTC registers

2011-10-24 Thread Fabio Estevam
Hi Stefano, On Mon, Oct 24, 2011 at 11:48 AM, Stefano Babic wrote: > I did the same on the qong board to check this issue. On the qong board > it works: Thanks for testing it. I managed to get it working on MX31PDK too. Will submit a patch soon. Thanks, Fabio Estevam

[U-Boot] [PATCH] image: Allow images to indicate they're loadable at any address

2011-10-24 Thread Stephen Warren
The legacy uImage format includes an absolute load and entry- point address. When presented with a uImage in memory that isn't loaded at the address in the image's load address, U-Boot will relocate the image to its address in the header. Some payloads can actually be loaded and used at any arbitr

Re: [U-Boot] [PATCH] Update s3c24x0 timer implementation

2011-10-24 Thread Marek Vasut
> Dear Wolfgang Denk, > > Thank you for your response. > > >> Since the .rel.text section is required by the relocation code, I > >> assume that .bss global variables cannot be used until after > >> relocation? > > > > Why do you have to make such assumptions? That's documented > > behaviour.

Re: [U-Boot] [PATCH 3/3] nds32: asm/io.h: add __iormb and __iowmb support

2011-10-24 Thread 馬克泡
Hi Marek, 2011/10/24 Marek Vasut > Hi, > > can you possibly implement those as inline functions ? It'd be also great > if you > could convert the other macros to inline functions. > > Thanks! > Thanks for your comment. I'll try to do it tomorrow (GMT +0800). -- Best regards, Macpaul Lin

Re: [U-Boot] Accessing MC13783 RTC registers

2011-10-24 Thread Stefano Babic
On 10/24/2011 03:18 PM, Fabio Estevam wrote: > Hi, > > Running U-boot from top of tree imx git I get the following when > trying to access the MC13783 RTC registers via 'date' command: > > MX31PDK U-Boot > date > = 22 is invalid. Should be less than 0 > ## Get date failed > > Does anyone else s

Re: [U-Boot] [PATCH 3/3] nds32: asm/io.h: add __iormb and __iowmb support

2011-10-24 Thread Marek Vasut
> This patch add required __iormb and __iowmb to io.h. > This also fix some misbehavior to periphal drivers. > > This io.h has been fixed with referencing arm/include/asm/io.h. > > Signed-off-by: Macpaul Lin Hi, can you possibly implement those as inline functions ? It'd be also great if you

Re: [U-Boot] Accessing MC13783 RTC registers

2011-10-24 Thread Marek Vasut
> Hi, > > Running U-boot from top of tree imx git I get the following when > trying to access the MC13783 RTC registers via 'date' command: > > MX31PDK U-Boot > date > = 22 is invalid. Should be less than 0 > ## Get date failed > Fabio, is it possibly connected via i2c? Cheers __

[U-Boot] Accessing MC13783 RTC registers

2011-10-24 Thread Fabio Estevam
Hi, Running U-boot from top of tree imx git I get the following when trying to access the MC13783 RTC registers via 'date' command: MX31PDK U-Boot > date = 22 is invalid. Should be less than 0 ## Get date failed Does anyone else see this same problem on other boards? Regards, Fabio Estevam __

Re: [U-Boot] [PATCH V3] Ethernut 5 board support

2011-10-24 Thread Igor Grinberg
On 10/24/2011 12:33 PM, Tim Schendekehl wrote: > Add support for the Ethernut 5 open hardware design, based > on Atmel's AT91SAM9XE512 SoC. > > V3 > - Fix issues with latest git. > > Signed-off-by: Tim Schendekehl > --- [...] > diff --git a/board/egnite/ethernut5/Makefile > b/board/egnite/eth

Re: [U-Boot] [RFC] [PATCH 00/39] Rework of the debug() macro

2011-10-24 Thread Marek Vasut
> Hi Marek, Hi Detlev, > > [...] > > > I certainly see your point. Btw. do you feel like help reviewing the PPC > > portion of the patches? That'd be greatly appreciated as I'm not a PPC > > expert. I wanted to avoid resubmission, so I only pushed the new set to > > git://git.denx.de/u-boot- ma

Re: [U-Boot] [PATCH 02/39] GCC4.6: Squash warnings in common/usb.c

2011-10-24 Thread Marek Vasut
Hi Stefano, actually the value is checked by debug_cond(). Cheers ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 02/39] GCC4.6: Squash warnings in common/usb.c

2011-10-24 Thread Stefano Babic
On 10/22/2011 02:16 AM, Marek Vasut wrote: > usb.c: In function ‘usb_parse_config’: > usb.c:331:17: warning: variable ‘ch’ set but not used > [-Wunused-but-set-variable] > usb.c: In function ‘usb_hub_port_connect_change’: > usb.c:1123:29: warning: variable ‘portchange’ set but not used > [-Wunused-

Re: [U-Boot] [RFC] [PATCH 00/39] Rework of the debug() macro

2011-10-24 Thread Detlev Zundel
Hi Marek, [...] > I certainly see your point. Btw. do you feel like help reviewing the PPC > portion > of the patches? That'd be greatly appreciated as I'm not a PPC expert. I > wanted > to avoid resubmission, so I only pushed the new set to > git://git.denx.de/u-boot- > marex.git / debug br

Re: [U-Boot] [PATCH] Update s3c24x0 timer implementation

2011-10-24 Thread Mark Norman
Dear Wolfgang Denk, Thank you for your response. >>  Since the .rel.text section is required by the relocation code, I >> assume that .bss global variables cannot be used until after >> relocation? > > Why do you have to make such assumptions?  That's documented > behaviour.  Didn't you RTFM? >

[U-Boot] [PATCH 1/3] iMX28: Add USB and USB PHY register definitions

2011-10-24 Thread Marek Vasut
Signed-off-by: Marek Vasut Cc: Stefano Babic Cc: Wolfgang Denk Cc: Detlev Zundel Cc: Remy Bohmer --- arch/arm/include/asm/arch-mx28/regs-usb.h| 178 ++ arch/arm/include/asm/arch-mx28/regs-usbphy.h | 151 ++ 2 files changed, 329 insertions(+),

[U-Boot] [PATCH 2/3] iMX28: Add USB HOST driver

2011-10-24 Thread Marek Vasut
This driver supports both EHCI ports on the i.MX28 CPU. Signed-off-by: Marek Vasut Cc: Stefano Babic Cc: Wolfgang Denk Cc: Detlev Zundel Cc: Remy Bohmer --- drivers/usb/host/Makefile |1 + drivers/usb/host/ehci-mxs.c | 154 +++ 2 files changed,

  1   2   >