Re: [U-Boot] [PATCH 37/56] board/genietv/flash.c: Fix flash initialization

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-38-git-send-email...@denx.de> you wrote: > The GENIETV has only one NOR flash bank. Remove all code trying to > deal with a second bank. This also fixes a number of GCC 4.6 build > warnings: > flash.c:65:12: warning: array subscript is above array

Re: [U-Boot] [PATCH 38/56] net/net.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-39-git-send-email...@denx.de> you wrote: > Fix: > net.c: In function 'CDPHandler': > net.c:1083:8: warning: variable 'applid' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > net/net.c |4 +--- > 1 files

Re: [U-Boot] [PATCH 39/56] mpc8xx/spi.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-40-git-send-email...@denx.de> you wrote: > Fix: > spi.c: In function 'spi_init_f': > spi.c:144:21: warning: variable 'iop' set but not used > [-Wunused-but-set-variable] > spi.c:142:22: warning: variable 'cpi' set but not used > [-Wunused-but-set-va

Re: [U-Boot] [PATCH 40/56] board/mbx8xx/mbx8xx.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-41-git-send-email...@denx.de> you wrote: > Fix: > mbx8xx.c: In function 'mbx_init': > mbx8xx.c:127:15: warning: variable 'refclock' set but not used > [-Wunused-but-set-variable] > mbx8xx.c: In function 'board_get_cpufreq': > mbx8xx.c:231:2: warning

Re: [U-Boot] [PATCH 41/56] board/fads/fads.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-42-git-send-email...@denx.de> you wrote: > Fix: > fads.c: In function 'initdram': > fads.c:606:7: warning: variable 'base' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/fads/fads.c |4 +++- > 1

Re: [U-Boot] [PATCH 42/56] board/rbc823/flash.c: CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-43-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch clean (don't convert to use I/O accessors > yet, so there will be "Use of volatile is usually wrong" warnings > left. > > Signed-off-by: Wolfgang Denk > --- > board/rbc823/flash.

Re: [U-Boot] [PATCH 43/56] board/rbc823/flash.c: Fix flash initalization

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-44-git-send-email...@denx.de> you wrote: > The RBC823 has only one NOR flash bank. Remove all code trying to > deal with a second bank. This also fixes a number of GCC 4.6 build > warnings: > flash.c: In function 'flash_init': > flash.c:62:12: war

Re: [U-Boot] [PATCH 44/56] board/svm_sc8xx/svm_sc8xx.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-45-git-send-email...@denx.de> you wrote: > Fix: > svm_sc8xx.c: In function 'checkboard': > svm_sc8xx.c:83:6: warning: variable 'board_type' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/svm_sc8xx/sv

Re: [U-Boot] [PATCH 45/56] board/svm_sc8xx/flash.c: CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-46-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch clean (don't convert to use I/O accessors > yet, so there will be "Use of volatile is usually wrong" warnings > left. > > Signed-off-by: Wolfgang Denk > --- > board/svm_sc8xx/fla

Re: [U-Boot] [PATCH 46/56] board/svm_sc8xx/flash.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-47-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'write_word': > flash.c:542:15: warning: variable 'last' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/svm_sc8xx/flash.c |4

Re: [U-Boot] [PATCH 47/56] TQM850M: Add missing CONFIG_BOOTDELAY

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-48-git-send-email...@denx.de> you wrote: > This also fixes a build warning: > main.c: In function 'main_loop': > main.c:311:16: warning: variable 'bootlimit' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- >

Re: [U-Boot] [PATCH 48/56] mpc8260/speed.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-49-git-send-email...@denx.de> you wrote: > Fix: > speed.c: In function 'get_clocks': > speed.c:113:30: warning: variable 'cpmdf' set but not used > [-Wunused-but-set-variable] > speed.c:113:23: warning: variable 'busdf' set but not used > [-Wunused-

Re: [U-Boot] [PATCH 49/56] board/tqc/tqm8272/tqm8272.c: Fix GCC 4.6 build warning:

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-50-git-send-email...@denx.de> you wrote: > Fix: > tqm8272.c: In function 'initdram': > tqm8272.c:462:14: warning: variable 'lsize' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/tqc/tqm8272/tqm8272.c

Re: [U-Boot] [PATCH 50/56] common/cmd_i2c.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-51-git-send-email...@denx.de> you wrote: > Fix: > cmd_i2c.c: In function 'do_i2c_add_bus': > cmd_i2c.c:1212:19: warning: variable 'dev' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Heiko Schocher > --- >

Re: [U-Boot] [PATCH 51/56] mpc8220/fec.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-52-git-send-email...@denx.de> you wrote: > Fix: > fec.c: In function 'mpc8220_fec_recv': > fec.c:733:8: warning: variable 'frame' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > arch/powerpc/cpu/mpc8220/fec

Re: [U-Boot] [PATCH 52/56] mpc8260/i2c.c: CodingStyle cleanup

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-53-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch clean (don't convert to use I/O accessors > yet, so there will be "Use of volatile is usually wrong" warnings > left. Also accept some other harmless checkpatch warnings. > > Sign

Re: [U-Boot] [PATCH 53/56] mpc8260/i2c.c: replace PRINTD() by debug()

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-54-git-send-email...@denx.de> you wrote: > This also fixes some GCC 4.6 build warnings like: > i2c.c: In function 'i2c_init': > i2c.c:221:26: warning: variable 'txbd' set but not used > [-Wunused-but-set-variable] > i2c.c:221:19: warning: variable '

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

2011-11-07 Thread Stephen Warren
Simon Glass wrote at Monday, November 07, 2011 12:47 PM: > On Mon, Nov 7, 2011 at 9:09 AM, Stephen Warren wrote: > > On 11/07/2011 09:56 AM, Stephen Warren wrote: > >> [Resending in an attempt to avoid base64 encoding] > >> > >> On 11/05/2011 04:20 PM, Wolfgang Denk wrote: > >>> Dear Stephen Warre

Re: [U-Boot] [PATCH 54/56] mpc8220/i2c.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-55-git-send-email...@denx.de> you wrote: > Fix: > i2c.c: In function 'wait_for_bb': > i2c.c:109:16: warning: variable 'temp' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Heiko Schocher > --- > arch/powerp

Re: [U-Boot] [PATCH 55/56] arch/powerpc/lib/bat_rw.c: Fix GCC 4.6 build warning

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-56-git-send-email...@denx.de> you wrote: > Fix: > bat_rw.c: In function 'write_bat': > bat_rw.c:38:6: warning: variable 'batn' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Kumar Gala > Cc: Andy Fleming >

Re: [U-Boot] [PATCH 56/56] board/alaska/flash.c: Fix GCC 4.6 build warnings

2011-11-07 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320458160-23136-57-git-send-email...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_erase': > flash.c:409:21: warning: variable 'last' set but not used > [-Wunused-but-set-variable] > flash.c:408:6: warning: variable 'flag' set but not used > [-Wunused-bu

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

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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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

[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

[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 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 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

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

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

[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

[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 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_

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] 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 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.

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 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 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 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 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 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, 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

[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 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

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 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 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

[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 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

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] [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] 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] [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] 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] 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] [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] 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: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] [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 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 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 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 _

[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] 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

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 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 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] 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 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

[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

[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]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:

<    1   2   3   >