Re: [U-Boot] [PATCHv2 02/10] mkenvimage: correct and clarify comments and error messages

2011-12-20 Thread Thomas Petazzoni
Le Wed, 21 Dec 2011 01:58:25 +0100, David Wagner a écrit : > +<<< HEAD > /* >* The right test to do is "=>" (not ">") because of the additional >* ending \0. See below. > @@ -201,6 +196,11 @@ int main(int argc, char **argv) > if (filesize >= envsize) { >

Re: [U-Boot] [PATCH] arm: Tegra: fix undefined instruction hang immediately after reset

2011-12-20 Thread Wolfgang Denk
Dear Tom Warren, In message you wrote: > This is needed for v2011.12-rc2. I'm using my Google email because I > seem to be getting blocked by base64 MIME crap w/Outlook. This has received a number of comments, but not a single ACK so far. Do all agree that his shall go in? Best regards, Wolf

[U-Boot] [PATCH] FAT: update the second FAT when writing a file

2011-12-20 Thread Donggeun Kim
After susccessful write to the FAT partition, fsck program may print warning message due to different FAT, provided that the filesystem supports two FATs. This patch makes the second FAT to be same with the first one when writing a file. Signed-off-by: Donggeun Kim Signed-off-by: Kyungmin Park

Re: [U-Boot] [PATCH] arm: Tegra: fix undefined instruction hang immediately after reset

2011-12-20 Thread Simon Glass
Hi, On Tue, Dec 20, 2011 at 4:27 PM, Tom Warren wrote: > This is needed for v2011.12-rc2. I'm using my Google email because I > seem to be getting blocked by base64 MIME crap w/Outlook. > > Tom Yes please! Tegra is broken in mainline without this patch. Regards, Simon > > On Wed, Dec 14, 2011

Re: [U-Boot] [PATCH v1 1/2] tpm: Rename generic_lpc_tpm to tpm_tis_lpc

2011-12-20 Thread Che-liang Chiou
On Tue, Dec 20, 2011 at 7:43 PM, Wolfgang Denk wrote: > Dear Che-liang Chiou, > > In message > you > wrote: >> >> The board that I test the TPM driver is Seaboard (a tegra2-based >> board). The config settings enabling TPM are literally pasted as >> follows. However, I can't submit the confis s

[U-Boot] [PATCH v3] cmd_fat: add FAT write command

2011-12-20 Thread Donggeun Kim
Once CONFIG_FAT_WRITE is defined, users can invoke 'fatwrite' command that saves data in RAM as a FAT file. By removing variable of 'part_size' in fs/fat.c, compile error occurs when enabling FAT write feature. The variable should be declared only when CONFIG_FAT_WRITE is defined. This patch also

Re: [U-Boot] [PATCH] arm, arm-kirkwood: disable l2c before linux boot

2011-12-20 Thread Marek Vasut
> The decompressor expects the L2 cache to be disabled. This fixes booting > some kernels, which have CONFIG_ARM_PATCH_PHYS_VIRT enabled. > > Signed-off-by: Michael Walle > Cc: Albert ARIBAUD > Cc: Prafulla Wadaskar > --- > arch/arm/cpu/arm926ejs/cache.c | 15 ++ > arch/arm

Re: [U-Boot] [PATCH 0/9] mkenvimage: v10/v11 diff split into several patches

2011-12-20 Thread David Wagner
Le 27/11/2011 20:59, Mike Frysinger a écrit : don't see anything wrong with the latest patchset now, thanks! -mike Hi, I re-sent 2 patches that didn't apply anymore on top of next (v2 2/10 and v2 3/10) ; the others should be fine. I also sent one more patch ([PATCH 09/10] Correctly handle i

[U-Boot] [PATCH 09/10] Correctly handle input files beginning with several newlines

2011-12-20 Thread David Wagner
Also, fix some comments (minor) Signed-off-by: David Wagner --- tools/mkenvimage.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/mkenvimage.c b/tools/mkenvimage.c index 9899631..0c9194b 100644 --- a/tools/mkenvimage.c +++ b/tools/mkenvimage.c @@ -224,18

Re: [U-Boot] [PATCH] overo: add SPL support

2011-12-20 Thread Andreas Müller
On Tuesday, December 20, 2011 02:55:50 PM you wrote: > On Tue, Dec 20, 2011 at 4:20 AM, Igor Grinberg wrote: > > What about forging some very not optimized default DRAM settings, > > that suit any assembled DRAM and then when you have I2C access, > > reconfigure it - is it possible? > > The board

[U-Boot] [PATCHv2 03/10] mkenvimage: Correct the includes and add a missing one

2011-12-20 Thread David Wagner
compiler.h and u-boot/crc.h need to be included from U-Boot's headers. stdlib.h was missing. Signed-off-by: David Wagner --- This version is rebased on top of 'next' (it didn't apply anymore, after Horst Kronstorfer's patches). tools/mkenvimage.c | 15 +++ 1 files changed, 3 ins

[U-Boot] [PATCHv2 02/10] mkenvimage: correct and clarify comments and error messages

2011-12-20 Thread David Wagner
Also, don't split error messages over several lines as per a coding style exception making them easier to grep. Signed-off-by: David Wagner --- This version is rebased on top of 'next' (it didn't apply anymore, after Horst Kronstorfer's patches). tools/mkenvimage.c | 35 +

[U-Boot] [PATCH] USB: move keyboard polling into kbd driver

2011-12-20 Thread Allen Martin
This moves keyboard polling logic from USB HCD drivers into USB keyboard driver. Remove usb_event_poll() as keyboard polling was the only user of this API. With this patch USB keyboard works with EHCI controllers again. Tested on a tegra2 seaboard. Signed-off-by: Allen Martin --- common/usb_k

Re: [U-Boot] [PATCH] arm, arm-kirkwood: disable l2c before linux boot

2011-12-20 Thread Michael Walle
Am Dienstag 20 Dezember 2011, 08:04:23 schrieb Albert ARIBAUD: > I assume the patch was tested against a 3.2-rc1 kernel and also a > pre-3.2-rc1 kernel to avoid regression? Tested with 3.0, 3.1 and 3.2-rc both with CONFIG_ARM_PATCH_PHYS_VIRT enabled and disabled. -- Michael _

[U-Boot] [PATCH] arm, arm-kirkwood: disable l2c before linux boot

2011-12-20 Thread Michael Walle
The decompressor expects the L2 cache to be disabled. This fixes booting some kernels, which have CONFIG_ARM_PATCH_PHYS_VIRT enabled. Signed-off-by: Michael Walle Cc: Albert ARIBAUD Cc: Prafulla Wadaskar --- arch/arm/cpu/arm926ejs/cache.c | 15 ++ arch/arm/cpu/arm926ejs/cpu

Re: [U-Boot] [PATCH] arm: Tegra: fix undefined instruction hang immediately after reset

2011-12-20 Thread Tom Warren
This is needed for v2011.12-rc2. I'm using my Google email because I seem to be getting blocked by base64 MIME crap w/Outlook. Tom On Wed, Dec 14, 2011 at 1:40 PM, Tom Warren wrote: > [Resending due to Outlook Base64 MIME issues. Sorry if you get spammed > w/multiple copies] > > Wolfgang/Albert

Re: [U-Boot] [PATCH v3 1/2] net: add Calxeda xgmac driver

2011-12-20 Thread Rob Herring
Wolfgang, On 12/20/2011 05:09 PM, Wolfgang Denk wrote: > Dear Rob Herring, > > In message <4ef11209.5050...@gmail.com> you wrote: >> >>> Are there going to be any users for this driver? >> >> Yes. Patch 2/2 adds support to Calxeda Highbank. > > I saw this myself 5 seconds later, but my "send" bu

Re: [U-Boot] [PATCH v3 1/2] net: add Calxeda xgmac driver

2011-12-20 Thread Wolfgang Denk
Dear Rob Herring, In message <4ef11209.5050...@gmail.com> you wrote: > > > Are there going to be any users for this driver? > > Yes. Patch 2/2 adds support to Calxeda Highbank. I saw this myself 5 seconds later, but my "send" button has no "undo" function ;-) Sorry... > > Checkpatch says: > >

Re: [U-Boot] [PATCH v3 1/2] net: add Calxeda xgmac driver

2011-12-20 Thread Rob Herring
Wolfgang, On 12/20/2011 04:42 PM, Wolfgang Denk wrote: > Dear Rob Herring, > > In message <1323983750-3399-1-git-send-email-robherri...@gmail.com> you wrote: >> From: Rob Herring >> >> This adds ethernet driver for Calxeda xgmac found on Highbank SOC. >> >> Signed-off-by: Rob Herring >> --- >>

Re: [U-Boot] [PATCH v7] net: ll_temac: Add LL TEMAC driver to u-boot

2011-12-20 Thread Wolfgang Denk
Dear Stephan Linz, In message <1322771978-22008-2-git-send-email-l...@li-pro.net> you wrote: > Xilinx LocalLink Tri-Mode Ether MAC driver can be > used by Xilinx Microblaze or Xilinx ppc405/440 in > SDMA and FIFO mode. DCR or XPS bus can be used. > > The driver uses and requires MII and PHYLIB. >

Re: [U-Boot] [PATCH 4/4] mkenvimage: Add version info switch (-V)

2011-12-20 Thread Wolfgang Denk
Dear Horst Kronstorfer, In message <1323082526-2125-4-git-send-email-hkron...@frequentis.com> you wrote: > Signed-off-by: Horst Kronstorfer > --- > tools/mkenvimage.c |7 ++- > 1 files changed, 6 insertions(+), 1 deletions(-) Applied to "next" branch, thanks. Best regards, Wolfgang De

Re: [U-Boot] [PATCH 3/4] mkenvimage: Print program basename instead of whole path in usage()

2011-12-20 Thread Wolfgang Denk
Dear Horst Kronstorfer, In message <1323082526-2125-3-git-send-email-hkron...@frequentis.com> you wrote: > Signed-off-by: Horst Kronstorfer > --- > tools/mkenvimage.c | 13 + > 1 files changed, 9 insertions(+), 4 deletions(-) > > diff --git a/tools/mkenvimage.c b/tools/mkenvimage.

Re: [U-Boot] [PATCH 2/4] mkenvimage: Fix getopt() error handling

2011-12-20 Thread Wolfgang Denk
Dear Horst Kronstorfer, In message <1323082526-2125-2-git-send-email-hkron...@frequentis.com> you wrote: > Since the original implementation indicates explicit error handling > we turn off getopt()'s internal error messaging to avoid duplicate > error messages. Additionally we add ':' (missing op

Re: [U-Boot] [PATCH 1/4] mkenvimage: Fix some typos

2011-12-20 Thread Wolfgang Denk
Dear Horst Kronstorfer, In message <1323082526-2125-1-git-send-email-hkron...@frequentis.com> you wrote: > Signed-off-by: Horst Kronstorfer > --- > tools/mkenvimage.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) Applied to "next" branch, thanks. Best regards, Wolfgang D

Re: [U-Boot] [PATCH] phy: add Micrel KS8721BL phy definition

2011-12-20 Thread Wolfgang Denk
Dear Vladimir Zapolskiy, In message <1323175677-7654-1-git-send-email...@mleia.com> (sfid-20111206_124928_898952_7AC008A7) you wrote: > This adds support for the Micrel KS8721BL/SL PHY. > > The definition is taken from the correspondent datasheet. > > Signed-off-by: Vladimir Zapolskiy > --- >

Re: [U-Boot] [PATCH] km/common: add toolchain variable

2011-12-20 Thread Wolfgang Denk
Dear Holger Brunck, In message <1323879110-8404-1-git-send-email-holger.bru...@keymile.com> you wrote: > Add a variable "toolchain" and configure the rootpath > for the nfsargs with this variable. > > Signed-off-by: Holger Brunck > cc: Wolfgang Denk > --- > board/keymile/scripts/README

Re: [U-Boot] [PATCH v3 1/2] net: add Calxeda xgmac driver

2011-12-20 Thread Wolfgang Denk
Dear Rob Herring, In message <1323983750-3399-1-git-send-email-robherri...@gmail.com> you wrote: > From: Rob Herring > > This adds ethernet driver for Calxeda xgmac found on Highbank SOC. > > Signed-off-by: Rob Herring > --- > v3: > - whitespace fixes > - move reset to .init function > - fix c

Re: [U-Boot] [PATCH] Makefile: fix distclean and *_config rules to be less noise

2011-12-20 Thread Wolfgang Denk
Dear Lauri Hintsala, In message <1324391037-28128-1-git-send-email-lauri.hints...@bluegiga.com> you wrote: > > Signed-off-by: Lauri Hintsala > --- > Makefile |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Enginee

Re: [U-Boot] [PATCH] board/eltec/elppc/misc.c: Fix GCC 4.6 build warning

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324403469-3085-1-git-send-email-ag...@denx.de> you wrote: > Fix: > misc.c: In function 'misc_init_r': > misc.c:119:3: warning: dereferencing type-punned pointer > will break strict-aliasing rules [-Wstrict-aliasing] > > Signed-off-by: Anatolij Gustschin > --

Re: [U-Boot] [PATCH v2] drivers/net/e1000.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324402599-781-1-git-send-email-ag...@denx.de> you wrote: > Fix: > e1000.c: In function 'e1000_read_mac_addr': > e1000.c:1149:2: warning: dereferencing type-punned pointer > will break strict-aliasing rules [-Wstrict-aliasing] > > e1000.c:1149:2: warning: dere

Re: [U-Boot] [PATCH] board/esd/cpci750/mv_eth.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324394205-3-1-git-send-email-ag...@denx.de> you wrote: > Fix: > mv_eth.c: In function 'mv64360_eth_real_open': > mv_eth.c:425:6: warning: variable 'port_status' set but not used > [-Wunused-but-set-variable] > mv_eth.c: In function 'mv64360_eth_stop': > m

Re: [U-Boot] [PATCH] usb: s3c_udc_otg: fix GCC 4.6 build warnings

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324304435-24076-1-git-send-email-ag...@denx.de> you wrote: > Fix: > In file included from s3c_udc_otg.c:216:0: > s3c_udc_otg_xfer_dma.c: In function 'complete_tx': > s3c_udc_otg_xfer_dma.c:280:33: warning: variable 'is_short' set but not used > s3c_udc_otg_xfe

Re: [U-Boot] [PATCH] board/samsung/universal_c210/universal.c: Fix build warning

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324304404-24044-1-git-send-email-ag...@denx.de> you wrote: > Fix: > universal.c: In function 's5pc210_phy_control': > universal.c:273:7: warning: 'ret' may be used uninitialized > in this function [-Wuninitialized] > > Signed-off-by: Anatolij Gustschin > Cc:

Re: [U-Boot] [PATCH] FAT: fix some issues in FAT write support code

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1323954734-26775-1-git-send-email-ag...@denx.de> you wrote: > Writing a file to the FAT partition didn't work while a > test using a CF card. The test was done on mpc5200 based > board (powerpc). There is a number of problems in FAT > write code: > > Compiler

Re: [U-Boot] Pull request: u-boot-staging

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <20111220223552.2b63881f@wker> you wrote: > Hello Wolfgang, > > Please pull two fixes for inclusion in v2011.12 release. Thanks! > > The following changes since commit 901de79a3792c81aefcbf207a34049e63f21362b: > > Prepare v2011.12-rc2 (2011-12-19 12:12:54 +

Re: [U-Boot] [PATCH] drivers/net/e1000_spi.c: Fix build warnings

2011-12-20 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1324384143-10198-1-git-send-email-ag...@denx.de> you wrote: > Fix: > e1000_spi.c: In function 'spi_free_slave': > e1000_spi.c:115: warning: unused variable 'hw' > e1000_spi.c: In function 'do_e1000_spi': > e1000_spi.c:472: warning: 'checksum' may be used uninit

Re: [U-Boot] [PATCH v2] net: Fix arguments in tftpput command

2011-12-20 Thread Wolfgang Denk
Dear Simon Glass, In message <1324347043-29184-1-git-send-email-...@chromium.org> you wrote: > The switch to strict_strtoul() was not done correctly - this fixes > the compile error when CONFIG_CMD_TFTPPUT is active. > > There appears to very little use of strict_strtoul() in commands, so > it mi

Re: [U-Boot] [STATUS] v2011.12-rc2 is out - finish for release...

2011-12-20 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > Hi Wolfgang, > > On Mon, Dec 19, 2011 at 3:23 AM, Wolfgang Denk wrote: > > Hi everybody, > > > > I've just released v2011.12-rc2; please help testing. > > > > Please respond quickly if you find anything missing for the final > > release, I want to get

Re: [U-Boot] [PATCH v3] AVR32: fix timer_init() function

2011-12-20 Thread Wolfgang Denk
Dear Sven Schnelle, In message <1443988412-13421-1-git-send-email-sv...@stackframe.org> you wrote: > timer_init() now returns an int (the error code) instead of void. > This makes compilation fail with: > > interrupts.c:111: error: conflicting types for 'timer_init' > /home/svens/u-boot/u-boot/in

Re: [U-Boot] [STATUS] v2011.12-rc2 is out - finish for release...

2011-12-20 Thread Wolfgang Denk
Dear "=?ISO-8859-1?Q?Andreas_Bie=DFmann?=", In message <4eef2c5f.8070...@gmail.com> you wrote: > Dear Wolfgang, > > On 19.12.2011 12:23, Wolfgang Denk wrote: > > Hi everybody, > > > > I've just released v2011.12-rc2; please help testing. > > > > Please respond quickly if you find anything missi

Re: [U-Boot] [STATUS] "next" branch started

2011-12-20 Thread Wolfgang Denk
Dear Michael Walle, In message <201112182328.19108.mich...@walle.cc> you wrote: > > Please feel free to send pull-requests for next, or to point me to > > patches that should go into next (or even into the upcoming release). > http://patchwork.ozlabs.org/patch/126328/ > http://patchwork.ozlabs.org

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

2011-12-20 Thread Wolfgang Denk
Dear Michael Walle, In message <1321570349-4224-1-git-send-email-mich...@walle.cc> you wrote: > 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. > > Signed-off-by: Michael Walle > Acke

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

2011-12-20 Thread Wolfgang Denk
Dear Michael Walle, In message <1321570349-4224-1-git-send-email-mich...@walle.cc> you wrote: > 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. > > Signed-off-by: Michael Walle > Acke

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

2011-12-20 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4eef6c88.8090...@aribaud.net> you wrote: > Hi Wolfgang, > > The following changes since commit 901de79a3792c81aefcbf207a34049e63f21362b: > >Prepare v2011.12-rc2 (2011-12-19 12:12:54 +0100) > > are available in the git repository at: >git://git.denx.de/u-

Re: [U-Boot] [PATCH V2] arm: Tegra: Fix Harmony and Ventana builds in u-boot-tegra/master

2011-12-20 Thread Stephen Warren
Tom Warren wrote at Tuesday, December 20, 2011 1:54 PM: > Seaboard changes have removed the need for common/board.o in the > Makefile. Propagate this change to the other Tegra2 builds. > > Signed-off-by: Tom Warren Acked-by: Stephen Warren Tested-by: Stephen Warren -- nvpublic _

Re: [U-Boot] Pull request: u-boot-x86/next

2011-12-20 Thread Wolfgang Denk
Dear Graeme Russ, In message <4eeea032.8050...@gmail.com> you wrote: > Hi Wolfgang: > > The following changes since commit 7b725f8485bcdd0be3cda40e435b783882bf18ec: > > CHECKPATCH: ./board/esd/hh405/logo_320_240_8bpp.c (2011-12-18 00:14:46 > +0100) > > are available in the git repository at:

[U-Boot] Pull request: u-boot-staging

2011-12-20 Thread Anatolij Gustschin
Hello Wolfgang, Please pull two fixes for inclusion in v2011.12 release. Thanks! The following changes since commit 901de79a3792c81aefcbf207a34049e63f21362b: Prepare v2011.12-rc2 (2011-12-19 12:12:54 +0100) are available in the git repository at: git://git.denx.de/u-boot-staging.git ag...@d

Re: [U-Boot] [STATUS] v2011.12-rc2 is out - finish for release...

2011-12-20 Thread Wolfgang Denk
Dear Chris Lalancette, In message you wrote: > > Unfortunately, trying to build v2011.12-rc2 with the cross-compiler > from Ubuntu 11.10 causes an ICE. I'm taking a look at it now to see > if I can figure out the problem, but I thought I would point it out > sooner rather than later. Which sor

Re: [U-Boot] [PATCH] overo: add SPL support

2011-12-20 Thread Wolfgang Denk
Dear Andreas, In message <201112201339.39460.schnitzelt...@gmx.de> you wrote: > > I don't know if I want to jump also into these changes now - especially since > I am quite new here.. > But for my intererst - since it seems more error tolerant: How is SDRAM > timing > set up without exactly kno

Re: [U-Boot] Some thoughts on SPL

2011-12-20 Thread Tom Rini
On Tue, Dec 20, 2011 at 1:48 PM, Scott Wood wrote: > On 12/20/2011 09:38 AM, Tom Rini wrote: >> So, in chasing down another problem I was having (that turned out >> unrelated) I just made SPL use the full NAND stack.  It wouldn't be >> difficult to add CONFIG_SPL_FULL_NAND_SUPPORT > > How about CO

[U-Boot] [PATCH V2] arm: Tegra: Fix Harmony and Ventana builds in u-boot-tegra/master

2011-12-20 Thread Tom Warren
Seaboard changes have removed the need for common/board.o in the Makefile. Propagate this change to the other Tegra2 builds. Signed-off-by: Tom Warren --- V2: Add more info to commit msg about use/source of change board/nvidia/harmony/Makefile |1 - board/nvidia/ventana/Makefile |3 +--

Re: [U-Boot] Some thoughts on SPL

2011-12-20 Thread Scott Wood
On 12/20/2011 09:38 AM, Tom Rini wrote: > So, in chasing down another problem I was having (that turned out > unrelated) I just made SPL use the full NAND stack. It wouldn't be > difficult to add CONFIG_SPL_FULL_NAND_SUPPORT How about CONFIG_SPL_NAND_FULL? > (and either keep the > existing name

Re: [U-Boot] [PATCH v3 0/4] mmc: Implement central card-detection.

2011-12-20 Thread Thierry Reding
* Mike Frysinger wrote: > On Tuesday 20 December 2011 03:52:39 Thierry Reding wrote: > > This series of patches implements central card-detection within the MMC > > framework. Before this patch series, many boards actually did implement > > the board_mmc_getcd() function, but it wasn't used except

Re: [U-Boot] about flashing the u-boot

2011-12-20 Thread Scott Wood
On 12/19/2011 06:43 PM, Érico Porto wrote: > Got usb-tap to work, but ONLY with code warrior - and for some reason > can run it on ubuntu+unity but not on ubuntu+xfce, which loads > everything except the action button in the flash programmer where you > select what you will do: flash dump, erase, w

[U-Boot] [PATCH I2C] fix: error ATMEL_FIO_BASE undeclared, if use I2C_Soft on AT91

2011-12-20 Thread esw
* Since AT91 name schema was changed to ATMEL_BASE_xxx, I2C_SOFT on AT91 devices fails with 'error: ATMEL_FIO_BASE undeclared' * change ATMEL_PIO_BASE to ATMEL_BASE_PIOA will fix this Signed-off-by: Jens Scharsig --- include/i2c.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) d

Re: [U-Boot] [PATCH 3/3] usb_storage: Fix EHCI "out of buffer pointers" with CD-ROM

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 13:24:25 Moffett, Kyle D wrote: > On Dec 20, 2011, at 13:20, Mike Frysinger wrote: > > On Tuesday 20 December 2011 12:41:14 Kyle Moffett wrote: > >> + /* > >> + * The U-Boot EHCI driver cannot handle more than 4096*5 bytes in a > >> + * transfer without running itse

[U-Boot] [PATCH v3 3/4] mmc: fsl_esdhc: Implement card-detect hook.

2011-12-20 Thread Thierry Reding
This card-detect hook probably doesn't work. Perhaps somebody with more knowledge about the hardware can comment on this. I think that perhaps even the complete code from esdhc_init() could go into the getcd() function instead or mmc_getcd() needs to be called at some later time after mmc_init(), w

Re: [U-Boot] [PATCH 3/3] usb_storage: Fix EHCI "out of buffer pointers" with CD-ROM

2011-12-20 Thread Moffett, Kyle D
On Dec 20, 2011, at 13:20, Mike Frysinger wrote: > On Tuesday 20 December 2011 12:41:14 Kyle Moffett wrote: >> +/* >> + * The U-Boot EHCI driver cannot handle more than 4096*5 bytes in a >> + * transfer without running itself out of qt_buffers. >> + */ >> +ss->max_xfer_blk = (40

Re: [U-Boot] [PATCH 1/3] fs/fat: Fix FAT detection to support non-DOS partition tables

2011-12-20 Thread Moffett, Kyle D
On Dec 20, 2011, at 13:20, Mike Frysinger wrote: > On Tuesday 20 December 2011 12:41:12 Kyle Moffett wrote: >> --- a/fs/fat/fat.c >> +++ b/fs/fat/fat.c >> >> +static disk_partition_t cur_part_info = { >> +.start = 0, >> +.size = 0, >> +.blksz = 512, >> +.name = "", >> +.type =

Re: [U-Boot] [PATCH v2] cmd_fat: add FAT write command

2011-12-20 Thread Mike Frysinger
On Monday 19 December 2011 22:48:06 Donggeun Kim wrote: > --- a/common/cmd_fat.c > +++ b/common/cmd_fat.c > > + if (argc < 5) { > + printf("usage: fatwrite " > + " \n"); > + return 1; return cmd_usage(cmdtp); -mike signature.asc Description: Thi

Re: [U-Boot] [PATCH] board/eltec/elppc/misc.c: Fix GCC 4.6 build warning

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 12:51:09 Anatolij Gustschin wrote: > - *(int *) &eerev.res[0] = 0; > + memset(&eerev.res[0], 0, 4); &foo[0] is the same thing as foo -mike signature.asc Description: This is a digitally signed message part. _

Re: [U-Boot] [PATCH 1/3] fs/fat: Fix FAT detection to support non-DOS partition tables

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 12:41:12 Kyle Moffett wrote: > --- a/fs/fat/fat.c > +++ b/fs/fat/fat.c > > +static disk_partition_t cur_part_info = { > + .start = 0, > + .size = 0, > + .blksz = 512, > + .name = "", > + .type = "", > +}; there any way we could delay that initializa

Re: [U-Boot] [PATCH 3/3] usb_storage: Fix EHCI "out of buffer pointers" with CD-ROM

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 12:41:14 Kyle Moffett wrote: > + /* > + * The U-Boot EHCI driver cannot handle more than 4096*5 bytes in a > + * transfer without running itself out of qt_buffers. > + */ > + ss->max_xfer_blk = (4096*5)/dev_desc->blksz; spaces around those operato

[U-Boot] [PATCH] board/eltec/elppc/misc.c: Fix GCC 4.6 build warning

2011-12-20 Thread Anatolij Gustschin
Fix: misc.c: In function 'misc_init_r': misc.c:119:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Anatolij Gustschin --- board/eltec/elppc/misc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/board/elt

Re: [U-Boot] [PATCH v2] drivers/net/e1000.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Moffett, Kyle D
On Dec 20, 2011, at 12:36, Anatolij Gustschin wrote: > Fix: > e1000.c: In function 'e1000_read_mac_addr': > e1000.c:1149:2: warning: dereferencing type-punned pointer > will break strict-aliasing rules [-Wstrict-aliasing] > > e1000.c:1149:2: warning: dereferencing type-punned pointer > will break

[U-Boot] [PATCH 3/3] usb_storage: Fix EHCI "out of buffer pointers" with CD-ROM

2011-12-20 Thread Kyle Moffett
When performing large bulk reads from a CD or DVD using the U-Boot usb_storage driver, it generates requests of up to 20 blocks at a time. With a standard 512-byte block size, that is 10240 bytes and within the limit of U-Boot's EHCI driver (maximum 5 pages at 4k per page). Unfortunately CD-ROM m

[U-Boot] [PATCH 2/3] fs/fat: Improve error handling

2011-12-20 Thread Kyle Moffett
The FAT filesystem fails silently in inexplicable ways when given a filesystem with a block-size that does not match the device sector size. In theory this is not an unsupportable combination but requires a major rewrite of a lot of the filesystem. Until that occurs, the filesystem should detect t

[U-Boot] [PATCH 1/3] fs/fat: Fix FAT detection to support non-DOS partition tables

2011-12-20 Thread Kyle Moffett
The FAT filesystem code currently ends up requiring that the partition table be a DOS MBR, as it checks for the DOS 0x55 0xAA signature on the partition table (which may be Mac, EFI, ISO9660, etc) before actually computing the partition offset. This fixes support for accessing a FAT filesystem in

[U-Boot] Enable access to FAT filesystem in an ISO9660 El-Torito volume

2011-12-20 Thread Kyle Moffett
The included series of 3 patches fixes several bugs in the FAT code and the USB storage stack to make it possible to read from a FAT filesystem image embedded as an "El-Torito boot image" in an ISO9660 volume (IE: a CD or DVD). This has been tested on the eXMeritus HWW-1U-1A.

[U-Boot] [PATCH v2] drivers/net/e1000.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Anatolij Gustschin
Fix: e1000.c: In function 'e1000_read_mac_addr': e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Anatolij

Re: [U-Boot] [PATCH v3 0/4] mmc: Implement central card-detection.

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 03:52:39 Thierry Reding wrote: > This series of patches implements central card-detection within the MMC > framework. Before this patch series, many boards actually did implement > the board_mmc_getcd() function, but it wasn't used except by one driver > (fsl_esdhc). Unf

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

2011-12-20 Thread Anatolij Gustschin
On Tue, 20 Dec 2011 10:07:30 -0600 "Moffett, Kyle D" wrote: > On Dec 20, 2011, at 10:49, Anatolij Gustschin wrote: > > Fix: > > e1000.c: In function 'e1000_read_mac_addr': > > e1000.c:1149:2: warning: dereferencing type-punned pointer will break > > strict-aliasing rules [-Wstrict-aliasing] > >

Re: [U-Boot] [PATCH] arm: Tegra: Fix Harmony and Ventana builds

2011-12-20 Thread Tom Warren
Sorry, folks. Please ignore u-boot-tegra/next for now. It's in transition. u-boot-tegra/master has some new patches (getting ready for a pull request) that changed the Seaboard Makefile but not the Harmony or Ventana ones. The fix-it patch I submitted should've had a line that said it applies ag

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

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 11:07:30 Moffett, Kyle D wrote: > On Dec 20, 2011, at 10:49, Anatolij Gustschin wrote: > > #ifdef CONFIG_E1000_FALLBACK_MAC > > - if ( *(u32*)(nic->enetaddr) == 0 || *(u32*)(nic->enetaddr) == ~0 ) { > > + if (get_unaligned_be32(nic->enetaddr) == 0 || > > + get_

Re: [U-Boot] [PATCH] drivers/net/e1000_spi.c: Fix build warnings

2011-12-20 Thread Mike Frysinger
On Tuesday 20 December 2011 07:29:03 Anatolij Gustschin wrote: > - struct e1000_hw *hw = e1000_hw_from_spi(spi); > + __maybe_unused struct e1000_hw *hw = e1000_hw_from_spi(spi); > E1000_DBG(hw->nic, "EEPROM SPI access released\n"); fix the E1000_DBG() macro to use debug() instead -mi

Re: [U-Boot] [PATCH v8 2/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Marek Vasut
> Let imx_get_mac_from_fuse function be a common function, so that other > mx28 boards can reuse it. > > Signed-off-by: Fabio Estevam > --- > Changes since v7: > - Use unsigned char instead of char > Changes since v6: > - Let mac[5] be set inside mx28_adjust_mac > Changes since v5: > - Allow the

[U-Boot] [PATCH v8 2/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Fabio Estevam
Let imx_get_mac_from_fuse function be a common function, so that other mx28 boards can reuse it. Signed-off-by: Fabio Estevam --- Changes since v7: - Use unsigned char instead of char Changes since v6: - Let mac[5] be set inside mx28_adjust_mac Changes since v5: - Allow the MAC vendor to be overr

Re: [U-Boot] [PATCH] overo: add SPL support

2011-12-20 Thread Aneesh V
On Tuesday 20 December 2011 06:09 PM, Andreas Müller wrote: On Tuesday, December 20, 2011 01:06:07 PM you wrote: Dear Andreas, In message<201112201253.46991.schnitzelt...@gmx.de> you wrote: I agree to your concerns but - as I understood Steve Sakoman - here the situation is slightly differ

Re: [U-Boot] [PATCH] drivers/net/e1000_spi.c: Fix build warnings

2011-12-20 Thread Anatolij Gustschin
On Tue, 20 Dec 2011 10:19:51 -0600 "Moffett, Kyle D" wrote: > On Dec 20, 2011, at 07:29, Anatolij Gustschin wrote: > > Fix: > > e1000_spi.c: In function 'spi_free_slave': > > e1000_spi.c:115: warning: unused variable 'hw' > > e1000_spi.c: In function 'do_e1000_spi': > > e1000_spi.c:472: warning:

[U-Boot] [PATCH] sdhc_boot: Introduce CONFIG_FSL_FIXED_MMC_LOCATION option

2011-12-20 Thread Fabio Estevam
Since commit 97039ab98 (env_mmc: Allow board code to override the environment address) mmc_get_env_addr is a weak-aliased function in common/env_mmc.c The mmc_get_env_addr implementation that exists at board/freescale/common/sdhc_boot.c is meant to be used only for PowerPC boards, but currently i

Re: [U-Boot] [PATCH v7 2/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Marek Vasut
> Let imx_get_mac_from_fuse function be a common function, so that other > mx28 boards can reuse it. > > Signed-off-by: Fabio Estevam > --- > Changes since v6: > - Let mac[5] be set inside mx28_adjust_mac > Changes since v5: > - Allow the MAC vendor to be overriden > Changes since v4: > - No chan

Re: [U-Boot] [PATCH] drivers/net/e1000_spi.c: Fix build warnings

2011-12-20 Thread Moffett, Kyle D
On Dec 20, 2011, at 07:29, Anatolij Gustschin wrote: > Fix: > e1000_spi.c: In function 'spi_free_slave': > e1000_spi.c:115: warning: unused variable 'hw' > e1000_spi.c: In function 'do_e1000_spi': > e1000_spi.c:472: warning: 'checksum' may be used uninitialized in this > function > e1000_spi.c:472

Re: [U-Boot] [PATCH] ehci: speed up initialization

2011-12-20 Thread Wolfgang Grandegger
On 12/19/2011 05:59 PM, Remy Bohmer wrote: > Hi Wolfgang, > > 2011/12/19 Wolfgang Grandegger : >> On 12/19/2011 01:51 PM, Vincent Palatin wrote: >>> On Mon, Dec 19, 2011 at 04:07, Wolfgang Grandegger >>> wrote: I just realized that this patch breaks "usb start" on my mx53loco board: >>

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

2011-12-20 Thread Moffett, Kyle D
On Dec 20, 2011, at 10:49, Anatolij Gustschin wrote: > Fix: > e1000.c: In function 'e1000_read_mac_addr': > e1000.c:1149:2: warning: dereferencing type-punned pointer will break > strict-aliasing rules [-Wstrict-aliasing] > e1000.c:1149:2: warning: dereferencing type-punned pointer will break > s

[U-Boot] [PATCH] drivers/net/e1000.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Anatolij Gustschin
Fix: e1000.c: In function 'e1000_read_mac_addr': e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] e1000.c:1149:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Anatolij

[U-Boot] [PATCH v7 4/4] mx28evk: Add initial support for MX28EVK board

2011-12-20 Thread Fabio Estevam
Add initial support for Freescale MX28EVK board. Tested boot via SD card and by loading a kernel via TFTP through the FEC interface. Signed-off-by: Fabio Estevam Acked-by: Marek Vasut --- - For correct operation of saving environment variables into the SD card, the following patch is needed: ht

[U-Boot] [PATCH v7 3/4] mx28: Let dram_init be common for mx28

2011-12-20 Thread Fabio Estevam
Let dram_init function be a common function, so that other mx28 boards can reuse it. Signed-off-by: Fabio Estevam Acked-by: Marek Vasut --- Changes since v6: - Added Acked-by line Changes since v5: - Change return value on dram_init() Changes since v4: - Call mx28_dram_init from m28evk.c Change

[U-Boot] [PATCH v7 2/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Fabio Estevam
Let imx_get_mac_from_fuse function be a common function, so that other mx28 boards can reuse it. Signed-off-by: Fabio Estevam --- Changes since v6: - Let mac[5] be set inside mx28_adjust_mac Changes since v5: - Allow the MAC vendor to be overriden Changes since v4: - No changes Change since v3: -

[U-Boot] [PATCH v7 1/4] net: imx: Add multi-FEC support for imx_get_mac_from_fuse

2011-12-20 Thread Fabio Estevam
Add multi-FEC support for imx_get_mac_from_fuse by passing dev_id as a parameter. This feature is important on mx28 SoC for example that has two FEC ports. Cc: Ben Warren Signed-off-by: Fabio Estevam --- Change since v6: - No changes. Introduced on this series arch/arm/cpu/arm1136/mx35/gener

Re: [U-Boot] [STATUS] v2011.12-rc2 is out - finish for release...

2011-12-20 Thread Chris Lalancette
On Mon, Dec 19, 2011 at 6:23 AM, Wolfgang Denk wrote: > Hi everybody, > > I've just released v2011.12-rc2; please help testing. > > Please respond quickly if you find anything missing for the final > release, I want to get this out on Friday at latest. I'm testing out the latest u-boot on my pand

Re: [U-Boot] Some thoughts on SPL

2011-12-20 Thread Tom Rini
On Fri, Dec 16, 2011 at 10:20 AM, jonsm...@gmail.com wrote: > The CPU I'm working with, the LPC3130, is kind of an in-between CPU > for SPL. Instead of a tightly constrained RAM of 16KB or so I have > 96KB to work with.  96KB is enough room to support all of the various > boot modes (uart, nand, s

[U-Boot] [PATCH] board/esd/cpci750/mv_eth.c: Fix GCC 4.6 build warnings

2011-12-20 Thread Anatolij Gustschin
Fix: mv_eth.c: In function 'mv64360_eth_real_open': mv_eth.c:425:6: warning: variable 'port_status' set but not used [-Wunused-but-set-variable] mv_eth.c: In function 'mv64360_eth_stop': mv_eth.c:643:15: warning: variable 'port_num' set but not used [-Wunused-but-set-variable] mv_eth.c: In functi

Re: [U-Boot] [PATCH] mx28: remove omap specific config options

2011-12-20 Thread Stefano Babic
On 20/12/2011 13:00, Veli-Pekka Peltola wrote: > Config options for OMAP are not used with i.MX28 so remove dead code. > > Signed-off-by: Veli-Pekka Peltola > Cc: Stefano Babic > Cc: Marek Vasut > Cc: Fabio Estevam > --- > arch/arm/cpu/arm926ejs/mx28/start.S |6 -- > 1 files changed,

Re: [U-Boot] [PATCH v6 1/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Marek Vasut
> On 12/20/2011 03:55 PM, Marek Vasut wrote: > >> Let imx_get_mac_from_fuse function be a common function, so that other > >> mx28 boards can reuse it. > > [snip] > > >> +static void __set_mac_vendor(char *mac) > > [snip] > > > It's good, but why not make it completely generic -- like > > mx28_

Re: [U-Boot] [PATCH v6 1/4] mx28: Let imx_get_mac_from_fuse be common for mx28

2011-12-20 Thread Veli-Pekka Peltola
On 12/20/2011 03:55 PM, Marek Vasut wrote: Let imx_get_mac_from_fuse function be a common function, so that other mx28 boards can reuse it. [snip] +static void __set_mac_vendor(char *mac) [snip] It's good, but why not make it completely generic -- like mx28_adjust_mac() -- so the users ca

Re: [U-Boot] [PATCH v6 2/4] mx28: Allow to set MAC address for the two FECs

2011-12-20 Thread Stefano Babic
On 20/12/2011 14:53, Fabio Estevam wrote: > MX28 has two built-in FECs (Fast Ethernet Controller). > > OCOTP register can be used to only store 4 bytes of MAC addresses. > > In order to workaround this limitation, let the MAC address of FEC1 be > the FEC0 address plus one. > > Signed-off-by: Fab

[U-Boot] [PATCH] Makefile: fix distclean and *_config rules to be less noise

2011-12-20 Thread Lauri Hintsala
Signed-off-by: Lauri Hintsala --- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 05966f4..ba336ed 100644 --- a/Makefile +++ b/Makefile @@ -643,7 +643,7 @@ unconfig: sinclude $(obj).boards.depend $(obj).boards.depend: boards.cfg -

Re: [U-Boot] [PATCH v6 4/4] mx28evk: Add initial support for MX28EVK board

2011-12-20 Thread Marek Vasut
> Add initial support for Freescale MX28EVK board. > > Tested boot via SD card and by loading a kernel via TFTP through > the FEC interface. > > Signed-off-by: Fabio Estevam > --- > - For correct operation of saving environment variables into the SD card, > the following patch is needed: > http:

Re: [U-Boot] [PATCH v6 3/4] mx28: Let dram_init be common for mx28

2011-12-20 Thread Marek Vasut
> Let dram_init function be a common function, so that other mx28 boards > can reuse it. > > Signed-off-by: Fabio Estevam > --- > Changes since v5: > - Change return value on dram_init() > Changes since v4: > - Call mx28_dram_init from m28evk.c > Changes since v3: > - Rename dram_init to mx28_dra

Re: [U-Boot] [PATCH v6 2/4] mx28: Allow to set MAC address for the two FECs

2011-12-20 Thread Marek Vasut
> MX28 has two built-in FECs (Fast Ethernet Controller). > > OCOTP register can be used to only store 4 bytes of MAC addresses. > > In order to workaround this limitation, let the MAC address of FEC1 be > the FEC0 address plus one. > > Signed-off-by: Fabio Estevam > --- > Changes since v5: > N

  1   2   >