Re: [U-Boot] [PATCH 09/12] OMAP3: Add SPL support to Beagleboard

2011-11-07 Thread Igor Grinberg
On 11/07/11 22:05, Tom Rini wrote: > This introduces 200MHz Micron parts timing information based on x-loader > to . The memory init logic is also based on what > x-loader does in these cases. Note that while previously u-boot would > be flashed in with SW ECC in this case it now must be flashed

Re: [U-Boot] [PATCH 2/2] mvgbe: fix network device indices

2011-11-07 Thread Prafulla Wadaskar
> -Original Message- > From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] > Sent: Saturday, November 05, 2011 3:24 PM > To: Mike Frysinger > Cc: Prafulla Wadaskar; u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH 2/2] mvgbe: fix network device > indices > > Le 05/11/2011 00:06, M

Re: [U-Boot] [PATCH 08/12] OMAP3 SPL: Add identify_pop_memory function

2011-11-07 Thread Igor Grinberg
On 11/07/11 22:05, Tom Rini wrote: > A number of boards are populated with a PoP chip for both DDR and NAND > memory. So to determine DDR timings the NAND chip needs to be probed > and mfr/id returned to the board to make decisions with. All of this > code is put into spl_pop_probe.c and controll

[U-Boot] [PATCH] powerpc/85xx: Fix warning for USB device-fixup

2011-11-07 Thread Ramneek Mehresh
Fix USB device-fixup warning "node not found". This was occuring because of static nature of start_offset variable Static start_offset was storing offset of last node modified, and was becoming issue if node fixup is carried multiple times, resulting in "node not found" warning Signed-off-by: Ram

Re: [U-Boot] [PATCH 2/2] mvgbe: fix network device indices

2011-11-07 Thread Prafulla Wadaskar
> -Original Message- > From: Mike Frysinger [mailto:vap...@gentoo.org] > Sent: Saturday, November 05, 2011 4:37 AM > To: Prafulla Wadaskar > Cc: Michael Walle; u-boot@lists.denx.de; Wolfgang Denk > Subject: Re: [PATCH 2/2] mvgbe: fix network device indices > > On Friday 04 November 2011

Re: [U-Boot] [PATCH] mvgbe: remove setting of ethaddr within the driver

2011-11-07 Thread Prafulla Wadaskar
> -Original Message- > From: Michael Walle [mailto:mich...@walle.cc] > Sent: Tuesday, November 08, 2011 3:38 AM > To: u-boot@lists.denx.de > Cc: Michael Walle; Mike Frysinger; Valentin Longchamp; Eric > Cooper; Jason Cooper; Siddarth Gore; Albert ARIBAUD; Prafulla > Wadaskar; Simon Guinot

[U-Boot] [PATCH v2] [x86] Don't relocate symbols which point to things that aren't relocated

2011-11-07 Thread Gabe Black
This change adds an upper bound for symbols which are fixed up after u-boot is relocated into RAM. This way portions that are left at their original location can be referred to without having to manually fix up any pointers. Signed-off-by: Gabe Black --- Changes in v2: Merge in a fix in a later p

Re: [U-Boot] [PATCH 07/12] OMAP3 SPL: Rework memory initalization and devkit8000 support

2011-11-07 Thread Igor Grinberg
Hi Tom, On 11/07/11 22:05, Tom Rini wrote: > This changes to making the board be responsible for providing the > memory initialization timings in SPL and converts the devkit8000 > to this framework. In SPL we try and initialize both CS0 and CS1. > > Cc: Frederik Kriewitz > Signed-off-by: Tom Ri

Re: [U-Boot] [PATCH] [x86] Don't relocate symbols which point to things that aren't relocated

2011-11-07 Thread Gabe Black
This and my previous patch are quite related and could have been combined into a series or even a single patch. I didn't see this one until I'd already sent the other one, though. Even though keeping them separate was unintentional, these are doing slightly different things. The first one fixes a b

Re: [U-Boot] Code relocation in U-boot.

2011-11-07 Thread Sandeep Sharma
Hi, You are absolutely right. I am talking about ARM code located in start.S file. I am not able to figure out from from the referred link. I also consulted " http://infocenter.arm.com/help/topic/com.arm.doc.ihi0044d/IHI0044D_aaelf.pdf " which talks about arm code relocation but it is also not hel

[U-Boot] [PATCH] [x86] Don't relocate symbols which point to things that aren't relocated

2011-11-07 Thread Gabe Black
This change adds an upper bound for symbols which are fixed up after u-boot is relocated into RAM. This way portions that are left at their original location can be referred to without having to manually fix up any pointers. Signed-off-by: Gabe Black --- arch/x86/lib/bios_setup.c |7 ++-

Re: [U-Boot] [GIT PULL] Please pull u-boot-mpc83xx.git

2011-11-07 Thread Wolfgang Denk
Dear Kim Phillips, In message <2007184242.d07f6073aa57d2b24eece...@freescale.com> you wrote: > Wolfgang, > > Please pull some mpc83xx updates (mostly checkpatch cleanups): > > > The following changes since commit 688d8f33f27ea596efb6632388ee60360996eed0: > > Merge branch 'master' of git:

[U-Boot] [PATCH] [x86] Fix how the location of the realmode and bios blobs are calculated

2011-11-07 Thread Gabe Black
There are two blobs embedded into the u-boot image which are linked to run at an address which is different from where they actually end up in the ROM, one called "realmode" and one called "bios". There are realmode_setup and bios_setup functions which prepare those blobs by copying them into the l

[U-Boot] [PATCH] sh: Add updating method of SPI ROM to README of sh7757lcr

2011-11-07 Thread Nobuhiro Iwamatsu
Signed-off-by: Nobuhiro Iwamatsu --- doc/README.sh7757lcr | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/doc/README.sh7757lcr b/doc/README.sh7757lcr index 49fea50..78a9b8f 100644 --- a/doc/README.sh7757lcr +++ b/doc/README.sh7757lcr @@ -61,3 +61,17 @@ You c

[U-Boot] Starting U-boot from RAM in FPGA custom board

2011-11-07 Thread pcuser p
Hi, I am a newbie to bootloaders, My requirement is to port u-boot + linux to a FPGA board (ARM926EJS + video Coprocessors + basic peripherals (uart,ethernet)) I am basically trying to start u-boot from RAM since flash is already locked with fpga code. I happened to see some of mail chains about q

[U-Boot] [PATCH] arm: a320evb: define mach-type in board config file

2011-11-07 Thread Andrew Chen
From: Yan-Pai Chen MACH_TYPE_FARADAY was dropped from mach-types.h. Add it back to board config file. Signed-off-by: Yan-Pai Chen --- board/faraday/a320evb/a320evb.c |1 - include/configs/a320evb.h |6 ++ 2 files changed, 6 insertions(+), 1 deletions(-) diff --git a/board/fa

Re: [U-Boot] [PATCH] sh: espt: change config for new sh_eth driver

2011-11-07 Thread Nobuhiro Iwamatsu
Applied, thanks. Best regards, Nobuhiro 2011/10/31 Yoshihiro Shimoda : > The new sh_eth driver uses the phy driver. So, this patch enables it. > > Signed-off-by: Yoshihiro Shimoda > --- >  include/configs/espt.h |    4 >  1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/

Re: [U-Boot] [PATCH] sh7763rdp: change config for new sh_eth driver

2011-11-07 Thread Nobuhiro Iwamatsu
Applied, thanks. Best regards, Nobuhiro 2011/10/31 Yoshihiro Shimoda : > The new sh_eth driver uses the phy driver. So, this patch enables it. > > Signed-off-by: Yoshihiro Shimoda > --- >  include/configs/sh7763rdp.h |    4 >  1 files changed, 4 insertions(+), 0 deletions(-) > > diff --g

Re: [U-Boot] [PATCH] sh: avoid multiple definition errors with cache funcs

2011-11-07 Thread Nobuhiro Iwamatsu
Applied, thanks. Best regards, Nobuhiro 2011/10/27 Mike Frysinger : > Recent builds for SH4 boards fail with a lot of errors like: >        cmd_mem.o: In function 'dcache_invalid_range': >        include/asm/cache.h:25: multiple definition of 'dcache_invalid_range' >        include/asm/cache.h:

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

2011-11-07 Thread Simon Glass
Hi Nicolas, On Mon, Nov 7, 2011 at 7:51 PM, Nicolas Pitre wrote: > On Mon, 7 Nov 2011, Simon Glass wrote: > >> On Mon, Nov 7, 2011 at 4:35 PM, Nicolas Pitre wrote: >> >> > On Tue, 8 Nov 2011, Wolfgang Denk wrote: >> > >> >> Dear Nicolas Pitre, >> >> >> >> > We don't want any hardcoded architectu

[U-Boot] [PATCH]powerpc/85xx: Make inclusion of USB device fixup conditional

2011-11-07 Thread Ramneek Mehresh
Include call to usb device-fixup only when CONFIG_HAS_FSL_DR_USB is defined for the platform - P1020RDB, P1010RDB, P1020-PC Signed-off-by: Ramneek Mehresh --- board/freescale/p1010rdb/p1010rdb.c |2 ++ board/freescale/p1_p2_rdb/p1_p2_rdb.c |2 ++ board/freescale/p1_p2_rdb_p

[U-Boot] [PATCH] da8xxevm: fix build error

2011-11-07 Thread prabhakar . csengg
From: Prabhakar Lad This patch fixes following compile error for da8xx evm da830evm.c: In function 'board_init': da830evm.c:222: error: 'DAVINCI_SYSCFG_SUSPSRC_UART2' undeclared (first use in this function) da830evm.c:222: error: (Each undeclared identifier is reported only once da830evm.c:222:

[U-Boot] [PATCH]powerpc/85xx: Make inclusion of USB device fixup conditional

2011-11-07 Thread Ramneek Mehresh
Include call to usb device-fixup only when CONFIG_HAS_FSL_DR_USB is defined for the platform - P1020RDB, P1010RDB, P1020-PC Signed-off-by: Ramneek Mehresh --- board/freescale/p1010rdb/p1010rdb.c |2 ++ board/freescale/p1_p2_rdb/p1_p2_rdb.c |2 ++ board/freescale/p1_p2_rdb_p

[U-Boot] [PATCH] Makefile: Remove mmc_spl related enteries

2011-11-07 Thread Chander Kashyap
As mmc_spl now follows spl infrastructure, removed unwanted enteries in Makefile for mmc_spl related compilation. Signed-off-by: Chander Kashyap --- Makefile |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/Makefile b/Makefile index 294c762..d349c1c 100644 --- a/Ma

[U-Boot] [PATCH v2] add a git mailrc file for maintainers

2011-11-07 Thread Mike Frysinger
This should make sending out e-mails to the right people easier. Signed-off-by: Mike Frysinger --- v2 - update Kumar/Minkyu's entries doc/git-mailrc | 101 1 files changed, 101 insertions(+), 0 deletions(-) create mode 100644 do

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

2011-11-07 Thread Nicolas Pitre
On Mon, 7 Nov 2011, Simon Glass wrote: > On Mon, Nov 7, 2011 at 4:35 PM, Nicolas Pitre wrote: > > > On Tue, 8 Nov 2011, Wolfgang Denk wrote: > > > >> Dear Nicolas Pitre, > >> > >> > We don't want any hardcoded architecture specific address anymore. > >> > This is being removed from the kernel as

Re: [U-Boot] [PATCH] add a git mailrc file for maintainers

2011-11-07 Thread Minkyu Kang
Dear Mike Frysinger, On 8 November 2011 02:43, Mike Frysinger wrote: > On Monday 07 November 2011 10:06:43 Kumar Gala wrote: >> On Nov 6, 2011, at 5:07 PM, Mike Frysinger wrote: >> > +alias kumar          Kumar Gala >> >> Please change to 'galak' > > ah, the crappy patchwork interface made it ha

Re: [U-Boot] [PATCH v3 2/2] mpc83xx: Implement interrupt support for mpc83xx

2011-11-07 Thread Joe Hershberger
Hi Kim, On Thu, Nov 3, 2011 at 3:48 PM, Kim Phillips wrote: > On Tue, 11 Oct 2011 21:46:05 -0500 > Joe Hershberger wrote: > >> Some core ipic code from Linux - commit >> bbf25010f1a6b761914430f5fca081ec8c7accd1 > > commit bbf25010f1a6b761914430f5fca081ec8c7accd1 > Author: Linus Torvalds > Date

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

2011-11-07 Thread Simon Glass
Hi Nicolas, On Mon, Nov 7, 2011 at 4:35 PM, Nicolas Pitre wrote: > On Tue, 8 Nov 2011, Wolfgang Denk wrote: > >> Dear Nicolas Pitre, >> >> In message you wrote: >> > >> > > 1) zImages are are relocatable. They should be loaded and started at >> > >    offsets between 32 KiB and 128 MiB in system

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

2011-11-07 Thread Nicolas Pitre
On Tue, 8 Nov 2011, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <4eb87375.1040...@nvidia.com> you wrote: > > > > The only place that has full knowledge of the board's memory layout is > > the U-Boot environment for that board, and hence I assert that the > > U-Boot environment shou

Re: [U-Boot] [PATCH] matrix vision: fix MVBLM7, MVSMR build error

2011-11-07 Thread Kim Phillips
On Sat, 5 Nov 2011 01:19:02 +0100 Wolfgang Denk wrote: > Dear Kim Phillips, > > In message <2004164356.5ab650e3a7af28de76f55...@freescale.com> you wrote: > > when a mkimage binary isn't present in the default system PATH, we > > get this error: > > Yes - so what? it's an unforced error and

[U-Boot] [GIT PULL] Please pull u-boot-mpc83xx.git

2011-11-07 Thread Kim Phillips
Wolfgang, Please pull some mpc83xx updates (mostly checkpatch cleanups): The following changes since commit 688d8f33f27ea596efb6632388ee60360996eed0: Merge branch 'master' of git://git.denx.de/u-boot-arm (2011-11-08 00:38:52 +0100) are available in the git repository at: git://git.denx.d

Re: [U-Boot] [PATCH v7] powerpc/mpc83xx: Add 33.33MHz support for mpc8360emds

2011-11-07 Thread Kim Phillips
On Mon, 7 Nov 2011 13:20:21 +0800 wrote: > From: Jerry Huang > > The new MPC8360EMDS board changes the oscillator to 33.33MHz > in order to support QE 500MHz since 2008. > > Signed-off-by: Jerry Huang > CC: Kim Phillips > --- Applied to u-boot-mpc83xx/master, thanks. Kim _

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

2011-11-07 Thread Nicolas Pitre
On Tue, 8 Nov 2011, Wolfgang Denk wrote: > Dear Nicolas Pitre, > > In message you wrote: > > > > > 1) zImages are are relocatable. They should be loaded and started at > > >offsets between 32 KiB and 128 MiB in system RAM. > > > > > > 2) Raw images (without the preloader) have to be started

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

2011-11-07 Thread Wolfgang Denk
Dear Stephen Warren, In message <4eb87375.1040...@nvidia.com> you wrote: > > The only place that has full knowledge of the board's memory layout is > the U-Boot environment for that board, and hence I assert that the > U-Boot environment should define where to load the kernel (and initrd > and FDT

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

2011-11-07 Thread Wolfgang Denk
Dear Stephen Warren, In message <4eb87122.3050...@nvidia.com> you wrote: > > The uncompressed image needs to end up at 32K-from-start-of-SDRAM (or > whatever SoC-specific value the kernel defines). If U-Boot puts the > zImage at that same location, the first thing the U-Boot decompressor > must do

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 04:25 PM, Wolfgang Denk wrote: > Dear Nicolas Pitre, > > In message you wrote: >> >>> 1) zImages are are relocatable. They should be loaded and started at >>>offsets between 32 KiB and 128 MiB in system RAM. >>> >>> 2) Raw images (without the preloader) have to be started at a f

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

2011-11-07 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4eb84664.50...@aribaud.net> you wrote: > Hi Wolfgang, > > The following changes since commit fec79acc864bed049b6beae719ccbf2bbec5403a: > >sandbox: drop unused return (2011-11-03 22:35:25 +0100) > > are available in the git repository at: >git://git.denx.

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 04:08 PM, Wolfgang Denk wrote: > In message <4eb85bf3.8030...@nvidia.com> you [Stephen Warren] wrote: ... >> The fundamental problem with uImage having an absolute load address is >> that there may be no single absolute address that is usable as SDRAM >> across all ARM SoCs which may

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Scott Wood
On 11/07/2011 05:32 PM, Gerlando Falauto wrote: > What bothers me more is, for instance, the condition under which my > smartphone will work correctly: > > if (((day_of_week() % 2 == 0) && >(temperature() < 14.4 || temperature() > 15.3)) >|| ((sky_color() == E_BLUE)

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Scott Wood
On 11/07/2011 05:32 PM, Gerlando Falauto wrote: > On 11/08/2011 12:02 AM, Scott Wood wrote: >> Documentation/CodingStyle calls for something like this: >> >> if (himport_ex(&env_htab, (char *)default_environment, >> sizeof(default_environment), '\0', >>

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 04:10 PM, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <4eb85ea6.3000...@nvidia.com> you wrote: >> >>> and we have to add additional configuration information to the boot >>> loader. >> >> Sorry, I'm unclear what "additional configuration information" needs to >> be add

Re: [U-Boot] Pull request for u-boot-coldfire.git

2011-11-07 Thread Wolfgang Denk
Dear Jin Zhengxiong-R64188, In message you wrote: > > Please kindly pull > > The following changes since commit fec79acc864bed049b6beae719ccbf2bbec5403a> : > > sandbox: drop unused return (2011-11-03 22:35:25 +0100) > > are available in the git repository at: > git://git.denx.de/u-boot-

Re: [U-Boot] [i2c] Pull request

2011-11-07 Thread Wolfgang Denk
Dear Heiko Schocher, In message <4eb377a8.9090...@denx.de> you wrote: > Hello Wolfgang, > > The following changes since commit fec79acc864bed049b6beae719ccbf2bbec5403a: > > sandbox: drop unused return (2011-11-03 22:35:25 +0100) > > are available in the git repository at: > git://git.denx.d

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Gerlando Falauto
On 11/08/2011 12:02 AM, Scott Wood wrote: > On 11/07/2011 04:05 PM, Wolfgang Denk wrote: >> Dear Gerlando Falauto, >> >> In message<4eb84859.6000...@keymile.com> you wrote: >>> >>> -int _do_env_set (int flag, int argc, char * const argv[]) >>> +int env_check_apply(const char *name, const char *old

Re: [U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface

2011-11-07 Thread Wolfgang Denk
Dear Gerlando Falauto, In message <4eb86424.7000...@keymile.com> you wrote: > > > http://www.denx.de/wiki/U-Boot/CodingStyle: > > > > Use TAB characters for indentation and vertical alignment, not > > spaces > > That's exactly what you told me in your reply, and doesn't answer my > quest

[U-Boot] Bonsoir Cher(e) Ami(e),

2011-11-07 Thread zoum.abelo
Bonsoir Cher(e) Ami(e), C’est avec tout un grand respect et en toute honnêteté que je vous adresse ce mail. Nous sommes des agents commerciaux évoluant dans le domaine de l’art artistique africain. Nous exposons des objets d’arts traditionnel qaue nous les confectionnons. Nous sommes au Burkina

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

2011-11-07 Thread Wolfgang Denk
Dear Nicolas Pitre, In message you wrote: > > > 1) zImages are are relocatable. They should be loaded and started at > >offsets between 32 KiB and 128 MiB in system RAM. > > > > 2) Raw images (without the preloader) have to be started at a fixed > >address, virt_to_phys(PAGE_OFFSET + TEX

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

2011-11-07 Thread Wolfgang Denk
Dear Stephen Warren, In message <4eb85ea6.3000...@nvidia.com> you wrote: > > > and we have to add additional configuration information to the boot > > loader. > > Sorry, I'm unclear what "additional configuration information" needs to > be added to the boot-loader, and which of cases (1) and (2)

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

2011-11-07 Thread Wolfgang Denk
Dear Stephen Warren, In message <4eb85bf3.8030...@nvidia.com> you wrote: > > I think the difference here is that I get the impression that people > within the U-Boot community would like to do away with zImage in general > and replace it with uImage, which simply isn't plausible, whereas I'm > per

Re: [U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface

2011-11-07 Thread Gerlando Falauto
On 11/07/2011 11:05 PM, Wolfgang Denk wrote: > Dear Gerlando Falauto, > > In message<4eb84859.6000...@keymile.com> you wrote: >> >> -int _do_env_set (int flag, int argc, char * const argv[]) >> +int env_check_apply(const char *name, const char *oldval, >> +const char *newval, int f

[U-Boot] Continuation line alignment

2011-11-07 Thread Scott Wood
On 11/07/2011 04:05 PM, Wolfgang Denk wrote: > Dear Gerlando Falauto, > > In message <4eb84859.6000...@keymile.com> you wrote: >> >> -int _do_env_set (int flag, int argc, char * const argv[]) >> +int env_check_apply(const char *name, const char *oldval, >> +const char *newval, int

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

2011-11-07 Thread Nicolas Pitre
On Mon, 7 Nov 2011, Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <20072204.41980.marek.va...@gmail.com> you wrote: > > > > You have that runtime patching stuff in linux-arm-kernel now, there should > > be no > > problem with that anymore actually. So basically I understood there

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

2011-11-07 Thread Wolfgang Denk
Dear Nicolas, In message you wrote: > > So yes, this is a simplistic solution, but it is damn good, and it > solves the u-Boot restrictions we've been complaining about for at least > two years now. Could you please explain which of these restrictions cannot be solved by using the IH_TYPE_*_R

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

2011-11-07 Thread Nicolas Pitre
On Mon, 7 Nov 2011, Stephen Warren wrote: > (Sigh, resending again to avoid rejected MIME encoding) > > On 11/07/2011 01:26 PM, Wolfgang Denk wrote: > > Dear Stephen Warren, > > > > In message <74cdbe0f657a3d45afbb94109fb122ff173f9a5...@hqmail01.nvidia.com> > > you wrote: > >> Anyway, I have wi

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 03:27 PM, Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <20072204.41980.marek.va...@gmail.com> you wrote: >> >> You have that runtime patching stuff in linux-arm-kernel now, there should >> be no >> problem with that anymore actually. So basically I understood there w

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 03:11 PM, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <74cdbe0f657a3d45afbb94109fb122ff173f9a5...@hqmail01.nvidia.com> > you wrote: >> >> "Stuck with" isn't really a good description. > > It is, IMO. > >> zImage is a way of booting ARM Linux. There may be others(?),

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

2011-11-07 Thread Wolfgang Denk
Dear Marek Vasut, In message <20072204.41980.marek.va...@gmail.com> you wrote: > > You have that runtime patching stuff in linux-arm-kernel now, there should be > no > problem with that anymore actually. So basically I understood there was an > agreement to make special uImage/fitImage whic

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

2011-11-07 Thread Wolfgang Denk
Dear Stephen Warren, In message <74cdbe0f657a3d45afbb94109fb122ff173f9a5...@hqmail01.nvidia.com> you wrote: > > "Stuck with" isn't really a good description. It is, IMO. > zImage is a way of booting ARM Linux. There may be others(?), but zImage > is certainly a valid and popular mechanism. I do

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 02:59 PM, Marek Vasut wrote: >> On 11/07/2011 02:04 PM, Marek Vasut wrote: >> ... >> The problem with this new approach is that Linux kernel images are NOT freely relocatable. They do have a fix entry point, even if this is not an absolute address, but a relative one.

[U-Boot] [PATCH] mvgbe: remove setting of ethaddr within the driver

2011-11-07 Thread Michael Walle
A network driver should not touch the environment at all. This patch fixes this behaviour by removing the code for setting a default/randomized MAC address. Instead a board should either set CONFIG_ETHADDR, CONFIG_ETH1ADDR etc. or use some specific code within the board files, eg. if randomization

Re: [U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface

2011-11-07 Thread Wolfgang Denk
Dear Gerlando Falauto, In message <4eb84859.6000...@keymile.com> you wrote: > > -int _do_env_set (int flag, int argc, char * const argv[]) > +int env_check_apply(const char *name, const char *oldval, > + const char *newval, int flag) > > > Please use only TAB for indentation. Pl

[U-Boot] [PATCH 12/12] AM3517 CraneBoard: Add SPL support

2011-11-07 Thread Tom Rini
The only change of note is that we move from 0x80008000 to 0x8010 for CONFIG_SYS_TEXT_BASE Cc: Nagendra T S Tested-by: Koen Kooi Signed-off-by: Tom Rini --- board/ti/am3517crane/am3517crane.c |2 +- board/ti/am3517crane/config.mk | 29 --- include/configs/am3517_

[U-Boot] [PATCH 11/12] AM3517: Add SPL support

2011-11-07 Thread Tom Rini
The only change of note is that we move from 0x80008000 to 0x8010 for CONFIG_SYS_TEXT_BASE Cc: Vaibhav Hiremath Signed-off-by: Tom Rini --- board/logicpd/am3517evm/am3517evm.c |2 +- board/logicpd/am3517evm/config.mk | 30 --- include/configs/am3517_evm.h|

[U-Boot] [PATCH 10/12] OMAP3: Add SPL support to omap3_evm

2011-11-07 Thread Tom Rini
Add Hynix 200MHz timing information to . We don't calculate the MCFG value here for the Micron parts as the provided one assumes a memory size which is incorrect. Signed-off-by: Tom Rini --- arch/arm/include/asm/arch-omap3/mem.h | 38 + board/ti/evm/config.mk

Re: [U-Boot] [PATCH v0 4/7] x86: Provide more configuration granularity

2011-11-07 Thread Wolfgang Denk
Dear Graeme Russ, In message <1320459711-20257-5-git-send-email-graeme.r...@gmail.com> you wrote: > Planned future ports requires more granularity for some options > > Signed-off-by: Graeme Russ This has a large number of checkpatch problems: total: 2 errors, 27 warnings, 210 lines checked B

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

2011-11-07 Thread Marek Vasut
> On 11/07/2011 02:04 PM, Marek Vasut wrote: > ... > > >> The problem with this new approach is that Linux kernel images are NOT > >> freely relocatable. They do have a fix entry point, even if this is > >> not an absolute address, but a relative one. The natural way to > >> handle this is exact

[U-Boot] [PATCH 0/3 RESEND] devkit3250: add Timll DevKit3250 board initial support

2011-11-07 Thread Vladimir Zapolskiy
This patchset adds generic NXP LPC32XX SoC support and basic support for Embest/Timll DevKit3250 board, which powered by this SoC. Vladimir Zapolskiy (3): arm926ejs: add NXP LPC32x0 cpu series support devkit3250: add Timll DevKit3250 board initial support devkit3250: add DevKit3250 board mai

[U-Boot] [PATCH 3/3 RESEND] devkit3250: add DevKit3250 board maintainer

2011-11-07 Thread Vladimir Zapolskiy
Add Vladimir Zapolskiyf as a DevKit3250 board maintainer in Das U-boot. Signed-off-by: Vladimir Zapolskiy --- MAINTAINERS |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 030fe4a..3da5650 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -90

[U-Boot] [PATCH 1/3 RESEND v4] arm926ejs: add NXP LPC32x0 cpu series support

2011-11-07 Thread Vladimir Zapolskiy
This change adds initial support for NXP LPC32x0 SoC series. Signed-off-by: Vladimir Zapolskiy --- Changes from v3 to v4: * extended memory controller definitions are introduced * added print_cpuinfo() function Changes from v2 to v3: * checkpatch.pl reports zero errors and warnings Changes from

[U-Boot] [PATCH 2/3 RESEND] devkit3250: add Timll DevKit3250 board initial support

2011-11-07 Thread Vladimir Zapolskiy
This change adds a basic support for Embest/Timll DevKit3250 board, NOR and UART are the only supported peripherals for a moment. The board doesn't require low-level init, because the initial SDRAM and GPIO configuration is performed during kickstart bootloader execution. Signed-off-by: Vladimir Z

Re: [U-Boot] [PATCH v0 6/7] x86: Misc PCI touchups

2011-11-07 Thread Wolfgang Denk
Dear Graeme Russ, In message <1320459711-20257-7-git-send-email-graeme.r...@gmail.com> you wrote: > This is not checkpatch clean: ERROR: "foo* bar" should be "foo *bar" #123: FILE: arch/x86/include/asm/pci.h:29: +void pci_setup_type1(struct pci_controller* hose); WARNING: line over 80 characte

Re: [U-Boot] [PATCH v0 4/7] x86: Provide more configuration granularity

2011-11-07 Thread Wolfgang Denk
Dear Graeme Russ, In message <1320459711-20257-5-git-send-email-graeme.r...@gmail.com> you wrote: > Planned future ports requires more granularity for some options ... > +#if defined(CONFIG_CMD_NET) > char *s; > +#endif > +#ifndef CONFIG_SYS_NO_FLASH > ulong size; > +#endif ... > +#ifn

Re: [U-Boot] [PATCH v0 3/7] x86: Add multiboot header

2011-11-07 Thread Wolfgang Denk
Dear Graeme Russ, In message <1320459711-20257-4-git-send-email-graeme.r...@gmail.com> you wrote: > By adding a multiboot heade, U-Boot can be loaded by GRUB2. Using GRUB2 to ^ 'r' missing. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolf

Re: [U-Boot] [PATCH 22/22] board/siemens/SCM/scm.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-23-git-send-email...@denx.de> you wrote: > Fix: > scm.c: In function 'config_scoh_cs': > scm.c:400:16: warning: variable 'tmp' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/siemens/SCM/scm.c |3 +

Re: [U-Boot] [PATCH 21/22] board/funkwerk/vovpn-gw/vovpn-gw.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-22-git-send-email...@denx.de> you wrote: > Fix: > vovpn-gw.c: In function 'misc_init_r': > vovpn-gw.c:266:16: warning: variable 'temp' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/funkwerk/vovpn-gw/

Re: [U-Boot] [PATCH 20/22] board/freescale/mpc8266ads/mpc8266ads.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-21-git-send-email...@denx.de> you wrote: > Fix: > mpc8266ads.c: In function 'initdram': > mpc8266ads.c:278:9: warning: variable 'spd_size' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/freescale/mpc8

Re: [U-Boot] [PATCH 19/22] board/freescale/mpc8266ads/mpc8266ads.c: CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-20-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch clean. > > Signed-off-by: Wolfgang Denk > Cc: Rune Torgersen > --- > board/freescale/mpc8266ads/mpc8266ads.c | 594 > --- > 1 files changed, 304 ins

Re: [U-Boot] [PATCH 18/22] board/sacsng/sacsng.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-19-git-send-email...@denx.de> you wrote: > Fix: > sacsng.c: In function 'initdram': > sacsng.c:180:9: warning: variable 'spd_size' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Jerry Van Baren > --- > board

Re: [U-Boot] [PATCH 17/22] board/sacsng/sacsng.c: CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-18-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch clean. > > Signed-off-by: Wolfgang Denk > Cc: Jerry Van Baren > --- > board/sacsng/sacsng.c | 1329 > - > 1 files changed, 663 in

Re: [U-Boot] [PATCH 16/22] board/rpxsuper/flash.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-17-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_init': > flash.c:52:19: warning: variable 'size' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/rpxsuper/flash.c |5 ++

Re: [U-Boot] [PATCH 15/22] board/rpxsuper/flash.c: minimal CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-16-git-send-email...@denx.de> you wrote: > Really minimal and local, just good enough to make checkpatch not > complain about the changes in the following commit. > > Signed-off-by: Wolfgang Denk > --- > board/rpxsuper/flash.c | 47 +

Re: [U-Boot] [PATCH 14/22] board/ids8247/ids8247.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-15-git-send-email...@denx.de> you wrote: > Fix: > ids8247.c: In function 'initdram': > ids8247.c:284:14: warning: variable 'lsize' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Heiko Schocher > --- > board/

Re: [U-Boot] [PATCH 13/22] board/hymod/input.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-14-git-send-email...@denx.de> you wrote: > Fix: > input.c: In function 'hymod_get_ethaddr': > input.c:79:10: warning: variable 'ea' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Murray Jensen > --- > board/

Re: [U-Boot] [PATCH 12/22] board/ep82xxm/ep82xxm.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-13-git-send-email...@denx.de> you wrote: > Fix: > ep82xxm.c: In function 'initdram': > ep82xxm.c:233:16: warning: variable 'ramtmp' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/ep82xxm/ep82xxm.c |

Re: [U-Boot] [PATCH 10/22] board/gw8260/flash.c: minimal CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-11-git-send-email...@denx.de> you wrote: > Really minimal and local, just good enough to make checkpatch not > complain about the changes in the following commit. > > Signed-off-by: Wolfgang Denk > Cc: Oliver Brown > --- > board/gw8260/flash.c |

Re: [U-Boot] [PATCH 11/22] board/gw8260/flash.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-12-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_init': > flash.c:81:16: warning: variable 'size' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Oliver Brown > --- > board/gw8260

Re: [U-Boot] [PATCH 09/22] board/cogent/flash.c: Fix GCC 4.6 buiild warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-10-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_init': > flash.c:295:16: warning: variable 'fip' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Murray Jensen > --- > board/cogen

Re: [U-Boot] [PATCH 08/22] arch/powerpc/cpu/mpc8260/spi.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-9-git-send-email...@denx.de> you wrote: > Fix: > spi.c: In function 'spi_init_r': > spi.c:279:22: warning: variable 'cp' set but not used > [-Wunused-but-set-variable] > spi.c: In function 'spi_xfer': > spi.c:361:22: warning: variable 'cp' set but no

Re: [U-Boot] [PATCH 07/22] common/cmd_flash.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-8-git-send-email...@denx.de> you wrote: > Fix: > cmd_flash.c:355:32: warning: 'info' may be used uninitialized in this > function [-Wuninitialized] > cmd_flash.c:354:10: warning: 'sect_first' may be used uninitialized in > this function [-Wuninitiali

Re: [U-Boot] [PATCH 06/22] drivers/net/rtl8139.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-7-git-send-email...@denx.de> you wrote: > Fix: > rtl8139.c: In function 'rtl8139_probe': > rtl8139.c:256:15: warning: variable 'fullduplex' set but not used > [-Wunused-but-set-variable] > rtl8139.c:256:6: warning: variable 'speed10' set but not used

Re: [U-Boot] [PATCH 05/22] board/linkstation/ide.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-6-git-send-email...@denx.de> you wrote: > Fix: > ide.c: In function 'ide_preinit': > ide.c:69:21: warning: array subscript is above array bounds > [-Warray-bounds] > ide.c:69:21: warning: array subscript is above array bounds > [-Warray-bounds] > ide

Re: [U-Boot] [PATCH 05/22] board/linkstation/ide.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Guennadi Liakhovetski, In message you wrote: > > > +#if CONFIG_SYS_IDE_MAXBUS > 1 > > I don't have gcc 4.6 installed here unfortunately, but wondering, wouldn't > this > > if (CONFIG_SYS_IDE_MAXBUS > 1) { > ide_bus_offset[1]... > } > > also make the compiler ha

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

2011-11-07 Thread Stephen Warren
On 11/07/2011 02:04 PM, Marek Vasut wrote: ... >> The problem with this new approach is that Linux kernel images are NOT >> freely relocatable. They do have a fix entry point, even if this is >> not an absolute address, but a relative one. The natural way to >> handle this is exactly that: add s

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

2011-11-07 Thread Stephen Warren
(Sigh, resending again to avoid rejected MIME encoding) On 11/07/2011 01:26 PM, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <74cdbe0f657a3d45afbb94109fb122ff173f9a5...@hqmail01.nvidia.com> > you wrote: >> >>> Your own IH_TYPE_*_REL patches are queued and will be merged soon. >> >>

Re: [U-Boot] [PATCH 04/22] drivers/net/rtl8169.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-5-git-send-email...@denx.de> you wrote: > Fix: > rtl8169.c: In function 'rtl_init': > rtl8169.c:742:13: warning: variable 'printed_version' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > drivers/net/rtl8169

Re: [U-Boot] [PATCH 03/22] board/mousse/flash.c: Fix GCC 4.6 buil warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-4-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_erase': > flash.c:780:18: warning: variable 'l_sect' set but not used > [-Wunused-but-set-variable] > flash.c:779:11: warning: variable 'addr' set but not used > [-Wunused-b

Re: [U-Boot] [PATCH 02/22] board/etin/kvme080/multiverse.c: Fix GCC 4.6 build warning.

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-3-git-send-email...@denx.de> you wrote: > Signed-off-by: Wolfgang Denk > Cc: Sangmoon Kim > --- > board/etin/kvme080/multiverse.c |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- D

Re: [U-Boot] [PATCH 01/22] drivers/net/pcnet.c: Change debug code to fix build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320505999-9770-2-git-send-email...@denx.de> you wrote: > Fix: > pcnet.c: In function 'pcnet_probe': > pcnet.c:247:8: warning: variable 'chipname' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Wolfgang Grandegger > --- > d

Re: [U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface

2011-11-07 Thread Gerlando Falauto
On 11/05/2011 05:09 PM, Wolfgang Denk wrote: [NOTE: I removed the quoting from the hunks as it would not make any sense] -int _do_env_set (int flag, int argc, char * const argv[]) +int env_check_apply(const char *name, const char *oldval, + const char *newval, int flag) > Pleas

  1   2   3   >