[U-Boot] [PATCH v5] serial: uartlite: Support for SERIAL_MULTI

2011-09-26 Thread Michal Simek
Add support for SERIAL MULTI for uartlite. Signed-off-by: Michal Simek --- v2: Use in/out_be32 functions v3: Clean coding style v4: Simplify userial_ports array initialization v5: const userial_ports Do not use u32 Separate declarations and code by a single blank line Update copyr

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201109252341.30328.vap...@gentoo.org> you wrote: > > > The assigned variable can be removed because the pointers are volatile so > > accesses to their addresses are always generated. > > i think the stores to a var were added to avoid a gcc warning, but if newer

[U-Boot] [PATCH] net: dc2114x: check for apropriate command

2011-09-26 Thread Linus Walleij
The code had two paths depending on whether the card was to be accessed from plain memory or the IO region. However the error path checks whether IO region was obtained - twice. Fix up the error path according to the probable intention. Signed-off-by: Linus Walleij --- drivers/net/dc2114x.c |

Re: [U-Boot] [PATCH 6/8] arm, davinci, am1808: add lowlevel functions for booting from NOR

2011-09-26 Thread Christian Riesch
Heiko, I am looking forward to see these patches in mainline u-boot. My custom board also directly boots from NOR, so I think there is a lot I can re-use for my board. I have one question, please see below: On Thu, Sep 15, 2011 at 7:59 AM, Heiko Schocher wrote: > Signed-off-by: Heiko Schocher >

[U-Boot] [PATCH 1/2] VIDEO: MX5: Switch MX5 to CONFIG_VIDEO

2011-09-26 Thread Stefano Babic
The framebuffer driver for MX5 is based on CONFIG_LCD. In the current implementation, there is a serious bug because the required memory is allocated before relocation, but the driver knows only later which is the resolution of the display. The patch switches the driver to CONFIG_VIDEO and the memo

[U-Boot] [PATCH 2/2] MX5: Switch vision2 to CONFIG_VIDEO

2011-09-26 Thread Stefano Babic
Signed-off-by: Stefano Babic CC: Anatolij Gustschin --- board/ttcontrol/vision2/vision2.c | 36 +--- include/configs/vision2.h | 11 +-- 2 files changed, 26 insertions(+), 21 deletions(-) diff --git a/board/ttcontrol/vision2/vision2.c b/board

Re: [U-Boot] mx51+ide+problem

2011-09-26 Thread Georgy Obraztsov
How i cat apply this patch u-boot-v2009.08-imx_10.11.01 for another version u-boot. For example for last version u-boot-2011.09-rc1 ? patch in attach. On Thu, Sep 22, 2011 at 11:32 PM, Georgy Obraztsov wrote: > Because in old version (u-boot-2009.08-rc3) i can apply specific board > patch for u-b

Re: [U-Boot] [PATCH v4 1/4] USB: move ehci.h and ehci-core.h into /include/usb/

2011-09-26 Thread Jana Rapava
Why don´t you just resend my original patch (put my name in the From: > field) as part of this series? > http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/109335 > Thanks, I'll be glad to use it. Regards, Jana Rapava ___ U-Boot mailing list U-Bo

Re: [U-Boot] [PATCH 6/8] arm, davinci, am1808: add lowlevel functions for booting from NOR

2011-09-26 Thread Heiko Schocher
Hello Christian, Christian Riesch wrote: > Heiko, > I am looking forward to see these patches in mainline u-boot. My > custom board also directly boots from NOR, so I think there is a lot I > can re-use for my board. Thats the goal ;-) > I have one question, please see below: [...] >> +void am18

Re: [U-Boot] [PATCH RFC] Add general nand functions

2011-09-26 Thread Simon Schwarz
On 09/23/2011 07:22 PM, Scott Wood wrote: > On 09/23/2011 11:36 AM, Simon Schwarz wrote: >> Added a nand_base.h header that defines all prototypes of nand_base.c >> functions. > > Just add the new prototypes to include/linux/mtd/nand.h (which gets > included by nand.h), that's the header that goes

Re: [U-Boot] Faster boot time

2011-09-26 Thread Simon Schwarz
On 09/25/2011 10:03 PM, dtran11 wrote: > > I am trying to make uboot boot as fast as possible for my omap3 Overo board. > I add the below to my config header file. Does anyone see anything I can add > to it to make the build smaller and faster? Thank you. > > [SNIP] This may be of interest to you

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 09:25:36 AM Wolfgang Denk wrote: > Dear Mike Frysinger, > > In message <201109252341.30328.vap...@gentoo.org> you wrote: > > > The assigned variable can be removed because the pointers are volatile > > > so accesses to their addresses are always generated. > > > > i

Re: [U-Boot] [PATCH 1/7 V2] GCC4.6: Convert various empty macros to inline functions

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 04:39:42 AM Fabio Estevam wrote: > On Sun, Sep 25, 2011 at 9:26 PM, Marek Vasut wrote: > > Fixes problems similar to the following ones: > > > > cmd_date.c: In function ‘do_date’: > > cmd_date.c:50:6: warning: variable ‘old_bus’ set but not used > > [-Wunused-but-set

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Nick Thompson
On 26/09/11 03:06, Marek Vasut wrote: > Signed-off-by: Marek Vasut > --- > arch/arm/include/asm/io.h | 30 ++ > 1 files changed, 18 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/include/asm/io.h b/arch/arm/include/asm/io.h > index 1fbc531..61f4987 100644 >

Re: [U-Boot] [PATCH v3] net: axi_ethernet: Add driver to u-boot

2011-09-26 Thread Michal Simek
Michal Simek wrote: > Add axi_ethernet driver for little-endian Microblaze. > > RX/TX BDs and rxframe buffer are shared among all axi_ethernet MACs. > Only one MAC can work in one time. > > Signed-off-by: Michal Simek > > --- > > v2: Fix cammelcase weirdness > Create mdio_wait function wit

Re: [U-Boot] mx51+ide+problem

2011-09-26 Thread Stefano Babic
On 09/26/2011 10:27 AM, Georgy Obraztsov wrote: > How i cat apply this patch u-boot-v2009.08-imx_10.11.01 for another > version u-boot. For example for last version u-boot-2011.09-rc1 ? Patches are always suitable for one version only - you have to port the patch (I have not seen it, it was missin

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: > On 26/09/11 03:06, Marek Vasut wrote: > > Signed-off-by: Marek Vasut > > --- > > > > arch/arm/include/asm/io.h | 30 ++ > > 1 files changed, 18 insertions(+), 12 deletions(-) > > > > diff --git a/arch

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Nick Thompson
On 26/09/11 10:32, Marek Vasut wrote: > On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: >> On 26/09/11 03:06, Marek Vasut wrote: >>> Signed-off-by: Marek Vasut >>> --- >>> >>> arch/arm/include/asm/io.h | 30 ++ >>> 1 files changed, 18 insertions(+), 12

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 11:44:22 AM Nick Thompson wrote: > On 26/09/11 10:32, Marek Vasut wrote: > > On Monday, September 26, 2011 11:26:51 AM Nick Thompson wrote: > >> On 26/09/11 03:06, Marek Vasut wrote: > >>> Signed-off-by: Marek Vasut > >>> --- > >>> > >>> arch/arm/include/asm/io.h |

Re: [U-Boot] [PATCH 2/2 V2] EfikaSB: Add preliminary EfikaSB support

2011-09-26 Thread Stefano Babic
On 09/25/2011 09:55 PM, Marek Vasut wrote: > Signed-off-by: Marek Vasut > Cc: Stefano Babic > --- > board/efikamx/efikamx.c | 124 > - > include/configs/efikamx.h |4 ++ > 2 files changed, 103 insertions(+), 25 deletions(-) > > V2: Introduce m

Re: [U-Boot] [PATCH 1/2 V2] EfikaMX: Add imximage config for Efika SB

2011-09-26 Thread Stefano Babic
On 09/25/2011 09:52 PM, Marek Vasut wrote: > Signed-off-by: Marek Vasut > Cc: Stefano Babic > --- > MAINTAINERS |1 + > board/efikamx/imximage.cfg| 122 > - > board/efikamx/imximage_mx.cfg | 122 > +

Re: [U-Boot] [PATCH 0/4] Buffer overruns in printf

2011-09-26 Thread Albert ARIBAUD
Hi Simon, Le 25/09/2011 16:50, Simon Glass a écrit : >> Basically, printf family functions which do not have the 'n' are *know* by >> all -- experienced enough :) -- programmers to be *unsafe* (but to require >> less from the caller) and it should remain so: no programmer should ever >> encounter

Re: [U-Boot] mx51+ide+problem

2011-09-26 Thread Stefano Babic
On 09/26/2011 12:15 PM, Georgy Obraztsov wrote: > Ok. I do queston another words. Please stop sending only private e-mails. You must *always* send your question to the ML, too. > I have i.mx51 processor with Compact Flash interface connected. How i > can compile u-boot for this configuration. U-b

Re: [U-Boot] Please pull u-boot-ti/next into arm/next

2011-09-26 Thread Albert ARIBAUD
Hi Sandeep, Le 23/09/2011 22:58, s-paul...@ti.com a écrit : > Albert, > > Please pull u-boot-ti/next into arm/next > I checked all the patches i applied for checkpatch errors and saw 2, both > of which cannot be fixed. Checkpatch complained about the location of the > header file. It expects thing

Re: [U-Boot] [PATCH 1/7 V2] GCC4.6: Convert various empty macros to inline functions

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261104.13082.marek.va...@gmail.com> you wrote: > > > Your commit log does not match your patch. > > "Fixes problems similar to the following ones" ... why not ? Please show _exactly_ which problems the patch fixes. Best regards, Wolfgang Denk -- DENX Softw

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261132.43616.marek.va...@gmail.com> you wrote: > > > Would it be better to change the __arch_putb macro into an extern inline > > function instead which would catch these and future cases? > > Yes, but you'll need to do that on a much larger scale. Is anyone up

[U-Boot] Please pull u-boot-imx/next into arm/next

2011-09-26 Thread Stefano Babic
Hi Albert, please pull u-boot-imx/next (already rebased on u-boot-arm/next after you merged u-boot-ti) into your u-boot-arm/next. Best regards, Stefano The following changes since commit f3340eeaf574b3297e801a563089c90d23e84f8d: arm, davinci, da8xx: add cpuinfo (2011-09-23 15:50:39 -0400) ar

[U-Boot] [PATCH 0/2] powerpc, mpc83xx: mpc83xx.h cleanup and enhancement

2011-09-26 Thread Heiko Schocher
patches are checkpatch clean, and a "MAKEALL -c mpc83xx" compiles clean. Heiko Schocher (2): cosmetic, powerpc, mpc83xx: checkpatch cleanup powerpc, mpc83xx: add DDR SDRAM Timing Configuration 3 definitions Cc: Kim Phillips include/mpc83xx.h | 394 +++--

[U-Boot] [PATCH 2/2] powerpc, mpc83xx: add DDR SDRAM Timing Configuration 3 definitions

2011-09-26 Thread Heiko Schocher
Signed-off-by: Heiko Schocher Cc: Kim Phillips --- include/mpc83xx.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/include/mpc83xx.h b/include/mpc83xx.h index d345c85..881fd43 100644 --- a/include/mpc83xx.h +++ b/include/mpc83xx.h @@ -1054,6 +1054,11 @@ #define TI

[U-Boot] [PATCH 1/2] cosmetic, powerpc, mpc83xx: checkpatch cleanup

2011-09-26 Thread Heiko Schocher
Signed-off-by: Heiko Schocher Cc: Kim Phillips --- include/mpc83xx.h | 389 +++-- 1 files changed, 257 insertions(+), 132 deletions(-) diff --git a/include/mpc83xx.h b/include/mpc83xx.h index 8292018..d345c85 100644 --- a/include/mpc83xx.h +++ b/

Re: [U-Boot] [PATCHv6] new tool mkenvimage: generates an env image from an arbitrary config file

2011-09-26 Thread Thomas Petazzoni
Hello, Le Fri, 2 Sep 2011 10:48:03 +0200, David Wagner a écrit : > +"-s -o \n" ^^ typo Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com __

Re: [U-Boot] Faster boot time

2011-09-26 Thread Dat Tran
What does SPL stand for again? It looks really interesting. Thanks. On Mon, Sep 26, 2011 at 3:56 AM, Simon Schwarz < simonschwarz...@googlemail.com> wrote: > On 09/25/2011 10:03 PM, dtran11 wrote: > >> >> I am trying to make uboot boot as fast as possible for my omap3 Overo >> board. >> I add th

Re: [U-Boot] Faster boot time

2011-09-26 Thread Simon Schwarz
Stands for Second Program Loader. It loads the U-Boot and runs it - or with this patch a Linux Kernel. Regards Simon On 09/26/2011 02:30 PM, Dat Tran wrote: > What does SPL stand for again? It looks really interesting. > > Thanks. > > On Mon, Sep 26, 2011 at 3:56 AM, Simon Schwarz > mailto:simon

[U-Boot] [PATCHv7] new tool mkenvimage: generates an env image from an arbitrary config file

2011-09-26 Thread David Wagner
This tool takes a key=value configuration file (same as would a `printenv' show) and generates the corresponding environment image, ready to be flashed. use case: flash the environment with an external tool Signed-off-by: David Wagner Acked-by; Mike Frysinger Tested-by: Thomas Petazzoni ---

Re: [U-Boot] Please pull u-boot-ti/next into arm/next

2011-09-26 Thread Paulraj, Sandeep
> > Hi Sandeep, > > Le 23/09/2011 22:58, s-paul...@ti.com a écrit : > > Albert, > > > > Please pull u-boot-ti/next into arm/next > > I checked all the patches i applied for checkpatch errors and saw 2, > both > > of which cannot be fixed. Checkpatch complained about the location of > the > > he

Re: [U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

2011-09-26 Thread Philip Balister
On 09/23/2011 08:47 AM, Holger Brunck wrote: > This prevents u-boot to display during bootime: > Out: serial > Err: serial > Net: UEC3Warning: failed to set MAC address > > Now corrected and it will display: > Out: serial > Err: serial > Net: UEC3 > Warning: failed to set MAC address >

Re: [U-Boot] Faster boot time

2011-09-26 Thread Dat Tran
Sorry for my ignorance on this issue. My current system (Overo) does this when it boots: ROM Code -> X-Load (MLO) -> U-Boot -> Linux Kernel -> Root File System Does the SPL patch allow me to do this?: ROM Code -> U-Boot (MLO) -> Linux Kernel -> Root File System Also running the latest

[U-Boot] [PATCH 0/3] misc:pmic: New PMIC generic driver

2011-09-26 Thread Lukasz Majewski
This patch series adds new generic PMIC framework for u-boot. It is supposed to handle various types of pmic IC's (both I2C and SPI). This patch series has been tested on Samsung's GONI reference target. Moreover a special patch for mx51evk target board has been prepared. It is compiling, but I c

[U-Boot] [PATCH 2/3] misc:pmic: Enable PMIC handling at u-boot startup code

2011-09-26 Thread Lukasz Majewski
The pmic_init() initialization handler has been added to board_init_r function. Signed-off-by: Lukasz Majewski Signed-off-by: Kyungmin Park Cc: Stefano Babic --- arch/arm/lib/board.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/arch/arm/lib/board.c b/arch/arm/li

[U-Boot] [PATCH 1/3] misc:pmic New generic pmic driver

2011-09-26 Thread Lukasz Majewski
This commit adds new PMIC core driver. PMIC IC devices connected via I2C or SPI can be used. Separate files: pmic_i2c.c and pmic_spi.c are responsible for handling transmission specific files for I2C or SPI busses. pmic_fsl.c and pmic_max8998.c are PMIC specific files. They define pmic_init() met

[U-Boot] [PATCH 3/3] misc:pmic:mx: Code modification for mx51evk board

2011-09-26 Thread Lukasz Majewski
This is an example code for adjusting the mx51evk example board to new pmic_core driver. Signed-off-by: Lukasz Majewski Signed-off-by: Kyungmin Park Cc: Stefano Babic --- board/freescale/mx51evk/mx51evk.c | 68 + 1 files changed, 46 insertions(+), 22 delet

Re: [U-Boot] TR: u-boot SquashFS

2011-09-26 Thread Jean-Michel Hautbois
Hi List, > > I am looking for a u-boot that handles SquashFS filesystem ? > > Did it exist ? if not can I have some indication or recommendation > > Because I want to try to upgrade u-boot such that it handles SquashFS > > Tuxbox has such an u-boot version, see: > http://cvs.tuxbox.org/cgi-bin/vi

Re: [U-Boot] Faster boot time

2011-09-26 Thread Simon Schwarz
On 09/26/2011 04:30 PM, Dat Tran wrote: > Sorry for my ignorance on this issue. My current system (Overo) does > this when it boots: > > ROM Code -> X-Load (MLO) -> U-Boot -> Linux Kernel -> Root File System > > Does the SPL patch allow me to do this?: > > ROM Code -> U-Boot (MLO) -> Linu

Re: [U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

2011-09-26 Thread Mike Frysinger
On Monday, September 26, 2011 10:24:54 Philip Balister wrote: > On 09/23/2011 08:47 AM, Holger Brunck wrote: > > This prevents u-boot to display during bootime: > > Out: serial > > Err: serial > > Net: UEC3Warning: failed to set MAC address > > > > Now corrected and it will display: > > Out:

Re: [U-Boot] [PATCH] image: push default arch values to arch headers

2011-09-26 Thread Mike Frysinger
On Monday, September 26, 2011 02:08:51 Wolfgang Denk wrote: > Mike Frysinger wrote: > > This pushes the ugly duplicated arch ifdef lists we maintain in various > > image related files out to the arch headers themselves. > > I appreciate this change in general, but ... > > ... > > > -#else > > -#

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Mike Frysinger
On Monday, September 26, 2011 05:03:46 Marek Vasut wrote: > On Monday, September 26, 2011 09:25:36 AM Wolfgang Denk wrote: > > Mike Frysinger wrote: > > > > The assigned variable can be removed because the pointers are > > > > volatile so accesses to their addresses are always generated. > > > > >

Re: [U-Boot] Faster boot time

2011-09-26 Thread Dat Tran
That is amazing. I have been looking for this type of project for a long time. Thank you very much. I will start to look at your patches. Will the patches work well with the newer versions like U-Boot 2011.09 or later, or is there a more specific version you recommend. Thanks. On Mon, Sep 26, 201

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Brian S. Park
Thanks for the reply. I was in the process updating ELDK from 4.1 to 4.2 and must have had something screwed up. I tried again Today and it compiles without error. Thanks for looking into this and helping a newbie. Brian On 9/24/2011 3:05 AM, Anatolij Gustschin wrote: Hi, On Fri, 23 Sep 201

Re: [U-Boot] [PATCH] powerpc/85xx: corenet_ds: increase console buffer size to 1024

2011-09-26 Thread Kim Phillips
On Sun, 25 Sep 2011 22:11:34 +0200 Wolfgang Denk wrote: > Dear Kim Phillips, > > In message <20110923140020.1eac241b.kim.phill...@freescale.com> you wrote: > > > > > Can you explain this in a bit more detail. > > > > if, for whatever reason, the nfsboot env var has been > > modified/deleted, on

Re: [U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

2011-09-26 Thread Holger Brunck
On 09/26/2011 06:07 PM, Mike Frysinger wrote: > On Monday, September 26, 2011 10:24:54 Philip Balister wrote: >> On 09/23/2011 08:47 AM, Holger Brunck wrote: >>> This prevents u-boot to display during bootime: >>> Out: serial >>> Err: serial >>> Net: UEC3Warning: failed to set MAC address >>>

Re: [U-Boot] [RFC PATCH 06/20] sandbox: Allow board_init_f() and board_init_r() to return

2011-09-26 Thread Simon Glass
Hi Mike, On Sun, Sep 25, 2011 at 9:47 PM, Mike Frysinger wrote: > On Friday, September 23, 2011 11:55:11 Simon Glass wrote: >> On Sat, Sep 17, 2011 at 5:05 PM, Mike Frysinger wrote: >> > On Saturday, September 17, 2011 12:48:45 Simon Glass wrote: >> >> Since the sandbox architecture doesn't do re

Re: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

2011-09-26 Thread Simon Glass
Hi Mike, On Sun, Sep 25, 2011 at 9:48 PM, Mike Frysinger wrote: > On Sunday, September 25, 2011 16:18:32 Simon Glass wrote: >> On Sun, Sep 25, 2011 at 12:25 PM, Wolfgang Denk wrote: >> > Simon Glass wrote: >> >> > do_reset() is not supposed to return >> >> >> >> I have adjusted the function meani

Re: [U-Boot] [RFC PATCH v2 02/20] Fix use of int as pointer in image.c

2011-09-26 Thread Simon Glass
Hi, On Sun, Sep 25, 2011 at 10:00 PM, Mike Frysinger wrote: > Acked-by: Mike Frysinger > -mike > Thanks - quick question: I want to move this from RFC to normal patch in the next version - should I start at version 3 or version 1 for the header? Regards, Simon _

Re: [U-Boot] [PATCH v2 0/6] tegra2: Add more clock/pinmux functionality

2011-09-26 Thread Simon Glass
Hi Albert, On Wed, Sep 21, 2011 at 3:40 PM, Simon Glass wrote: > This adds to the basic clock functionality already available. The concept > of a peripheral ID is introduced, and all peripheral clock access is done > using this ID. > > Functions are provided to start, query and adjust peripheral

Re: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

2011-09-26 Thread Anton Staaf
On Mon, Sep 26, 2011 at 9:49 AM, Simon Glass wrote: > Hi Mike, > > On Sun, Sep 25, 2011 at 9:48 PM, Mike Frysinger wrote: >> On Sunday, September 25, 2011 16:18:32 Simon Glass wrote: >>> On Sun, Sep 25, 2011 at 12:25 PM, Wolfgang Denk wrote: >>> > Simon Glass wrote: >>> >> > do_reset() is not sup

Re: [U-Boot] [PATCH 0/4] Buffer overruns in printf

2011-09-26 Thread Simon Glass
Hi Wolfgang, On Sun, Sep 25, 2011 at 1:04 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message <1316799532-20761-1-git-send-email-...@chromium.org> you wrote: >> The printf family of functions in U-Boot cannot deal with a situation where >> the caller provides a buffer which turns out to b

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 06:10:37 PM Mike Frysinger wrote: > On Monday, September 26, 2011 05:03:46 Marek Vasut wrote: > > On Monday, September 26, 2011 09:25:36 AM Wolfgang Denk wrote: > > > Mike Frysinger wrote: > > > > > The assigned variable can be removed because the pointers are > > > >

[U-Boot] [PATCH 1/7 V3] GCC4.6: Convert various empty macros to inline functions

2011-09-26 Thread Marek Vasut
Fix the following gcc4.6 problems: cmd_date.c: In function ‘do_date’: cmd_date.c:50:6: warning: variable ‘old_bus’ set but not used [-Wunused-but-set-variable] asix.c: In function ‘asix_init’: asix.c:317:6: warning: variable ‘rx_ctl’ set but not used [-Wunused-but-set-variable] usb.c: In function

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 01:30:41 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109261132.43616.marek.va...@gmail.com> you wrote: > > > Would it be better to change the __arch_putb macro into an extern > > > inline function instead which would catch these and future cases? >

Re: [U-Boot] [RFC PATCH 06/20] sandbox: Allow board_init_f() and board_init_r() to return

2011-09-26 Thread Anton Staaf
On Mon, Sep 26, 2011 at 9:48 AM, Simon Glass wrote: > Hi Mike, > > On Sun, Sep 25, 2011 at 9:47 PM, Mike Frysinger wrote: >> On Friday, September 23, 2011 11:55:11 Simon Glass wrote: >>> On Sat, Sep 17, 2011 at 5:05 PM, Mike Frysinger wrote: >>> > On Saturday, September 17, 2011 12:48:45 Simon Gl

Re: [U-Boot] [PATCH 0/4] Buffer overruns in printf

2011-09-26 Thread Simon Glass
Hi Albert, On Mon, Sep 26, 2011 at 4:20 AM, Albert ARIBAUD wrote: > Hi Simon, > > Le 25/09/2011 16:50, Simon Glass a écrit : > >>> Basically, printf family functions which do not have the 'n' are *know* >>> by >>> all -- experienced enough :) -- programmers to be *unsafe* (but to >>> require >>>

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Brian S. Park
I found my problem. My board, though based on walnut board, does not have real time clock. So, I have to disable RTC support in u-boot. However, as soon as I comment out #define CONFIG_RTC_DS174x1/* use DS1743 RTC in Walnut*/ in walnut.h, line 68, I get build error. [root@new-li

Re: [U-Boot] Faster boot time

2011-09-26 Thread Wolfgang Denk
Dear Dat Tran, In message you wrote: > > Also running the latest X-Loader 1.50 and U-Boot 2011.09-rc1, it takes about > 2 seconds from power on to starting the Linux Kernel. My Kernel takes 1 > second to load and my file system takes another 1.5 seconds for a total of > 4.5 seconds from power on

Re: [U-Boot] [PATCH] image: push default arch values to arch headers

2011-09-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201109261209.46800.vap...@gentoo.org> you wrote: > > > ...here you change a compile time error check into a runtim error. > > no, it'll still be a build time error as IH_ARCH_DEFAULT will be undefined > > it just might not be as clean an error ... gcc will compla

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261931.24045.marek.va...@gmail.com> you wrote: > > > i think gcc-3.x has been broken for a while but no one has noticed > > I guess we should go the kernel way -- make it 4.2 and be done with it. You are off by one major version. The kernel README says:

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Simon Glass
Hi Marek, On Sun, Sep 25, 2011 at 5:26 PM, Marek Vasut wrote: > cmd_mem.c: In function ‘do_mem_loop’: > cmd_mem.c:474:25: warning: variable ‘junk’ set but not used > [-Wunused-but-set-variable] > > The assigned variable can be removed because the pointers are volatile so > accesses to their addre

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Brian S. Park
Sorry to keep relying to my self. It seems that the tool that generates u-boot.lds does not like // in the header file and putting it in the u-boot.lds file. I started using /* */ instead and it compiles OK now. Thanks. Brian On 9/26/2011 10:54 AM, Brian S. Park wrote: I found my problem.

Re: [U-Boot] [RFC PATCH 18/20] sandbox: Remove unused variable warnings

2011-09-26 Thread Simon Glass
Hi Mike, On Sun, Sep 25, 2011 at 9:49 PM, Mike Frysinger wrote: > On Friday, September 23, 2011 12:01:46 Simon Glass wrote: >> On Sat, Sep 17, 2011 at 5:25 PM, Mike Frysinger wrote: >> > On Saturday, September 17, 2011 12:48:57 Simon Glass wrote: >> >> There are a few variables set but not used -

Re: [U-Boot] [PATCH] powerpc/85xx: corenet_ds: increase console buffer size to 1024

2011-09-26 Thread Wolfgang Denk
Dear Kim Phillips, In message <20110926112756.bb93d41b.kim.phill...@freescale.com> you wrote: > > > Instead of extending buffer sizes just to be able to enter some > > incomrephensible long variable setting you should try and figure > > this variable (which I have been comprehending for years

Re: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

2011-09-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201109260048.41976.vap...@gentoo.org> you wrote: > > i would expect "reset" in the sandbox to "exit(1)". how else would you exit ? "reset" should then use "exit(EXIT_SUCCESS)" as thisis a normal termination of the program, not an error condition. Best regards,

Re: [U-Boot] [PATCH] image: push default arch values to arch headers

2011-09-26 Thread Mike Frysinger
On Monday, September 26, 2011 13:58:59 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > ...here you change a compile time error check into a runtim error. > > > > no, it'll still be a build time error as IH_ARCH_DEFAULT will be > > undefined > > > > it just might not be as clean an error ... gc

Re: [U-Boot] [PATCH 4/7] GCC4.6: Squash warning in cmd_nvedit.c

2011-09-26 Thread Simon Glass
On Sun, Sep 25, 2011 at 5:26 PM, Marek Vasut wrote: > cmd_nvedit.c: In function ‘do_env_edit’: > cmd_nvedit.c:463:6: warning: variable ‘len’ set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Marek Vasut I need this also! Acked-by: Simon Glass > --- >  common/cmd_nvedit.c |    

Re: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

2011-09-26 Thread Mike Frysinger
On Monday, September 26, 2011 14:12:21 Wolfgang Denk wrote: > Mike Frysinger wrote: > > i would expect "reset" in the sandbox to "exit(1)". how else would you > > exit ? > > "reset" should then use "exit(EXIT_SUCCESS)" as thisis a normal > termination of the program, not an error condition. sure

Re: [U-Boot] [RFC PATCH 17/20] sandbox: Add basic config file

2011-09-26 Thread Simon Glass
Hi Mike, On Sun, Sep 25, 2011 at 9:52 PM, Mike Frysinger wrote: > On Friday, September 23, 2011 12:00:15 Simon Glass wrote: >> On Sat, Sep 17, 2011 at 5:22 PM, Mike Frysinger wrote: >> > On Saturday, September 17, 2011 12:48:56 Simon Glass wrote: >> >> +#define CONFIG_DRAM_SIZE     (128 << 20) >>

Re: [U-Boot] [PATCH v2] mkconfig: start deprecating Makefile config targets

2011-09-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201109191036.38248.vap...@gentoo.org> you wrote: > > > Please update doc/feature-removal-schedule.txt as well. > > i didnt have a schedule in mind ... just start scaring people :) > > any target you have in mind ? Next release? Second next release? Best regards

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:05:43 PM Simon Glass wrote: > Hi Marek, > > On Sun, Sep 25, 2011 at 5:26 PM, Marek Vasut wrote: > > cmd_mem.c: In function ‘do_mem_loop’: > > cmd_mem.c:474:25: warning: variable ‘junk’ set but not used > > [-Wunused-but-set-variable] > > > > The assigned variable

Re: [U-Boot] [PATCH 3/4] Make printf and vprintf safe from buffer overruns

2011-09-26 Thread Simon Glass
HI Wolfgang, On Sun, Sep 25, 2011 at 1:14 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message > you > wrote: >> >> Yes, indeed. Could we go as far as removing CONFIG_SYS_PBSIZE, and >> just use a standard value? > > If you can find one that fits for all boards?  Keep in mind that > pri

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Simon Glass
Hi Merek, On Mon, Sep 26, 2011 at 11:24 AM, Marek Vasut wrote: > On Monday, September 26, 2011 08:05:43 PM Simon Glass wrote: >> Hi Marek, >> >> On Sun, Sep 25, 2011 at 5:26 PM, Marek Vasut wrote: >> > cmd_mem.c: In function ‘do_mem_loop’: >> > cmd_mem.c:474:25: warning: variable ‘junk’ set but

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:03:45 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109261931.24045.marek.va...@gmail.com> you wrote: > > > i think gcc-3.x has been broken for a while but no one has noticed > > > > I guess we should go the kernel way -- make it 4.2 and be done w

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109261937.01263.marek.va...@gmail.com> you wrote: > > > What exactly are the advantages, and what the disadvantages? > > > > code size / memory footprint? > > I assume converting those to inline function will yield no memory usage > growth > at all. Please don

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:29:22 PM Simon Glass wrote: > Hi Merek, > > On Mon, Sep 26, 2011 at 11:24 AM, Marek Vasut wrote: > > On Monday, September 26, 2011 08:05:43 PM Simon Glass wrote: > >> Hi Marek, > >> > >> On Sun, Sep 25, 2011 at 5:26 PM, Marek Vasut wrote: > >> > cmd_mem.c: In fu

Re: [U-Boot] [PATCH 3/5] NAND: Allow per-buffer allocation

2011-09-26 Thread Scott Wood
On 09/24/2011 07:37 AM, Marek Vasut wrote: > On Friday, September 23, 2011 07:35:15 PM Scott Wood wrote: >> On 09/22/2011 03:51 AM, Marek Vasut wrote: >>> On Thursday, September 22, 2011 09:41:21 AM Stefano Babic wrote: On 09/21/2011 10:16 PM, Wolfgang Denk wrote: > Dear Stefano & Marek, >

Re: [U-Boot] [PATCH] GCC4.4: Squash multiple warnings due to strict aliasing

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:33:58 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109261937.01263.marek.va...@gmail.com> you wrote: > > > What exactly are the advantages, and what the disadvantages? > > > > > > code size / memory footprint? > > > > I assume converting those t

Re: [U-Boot] [PATCH 0/4] Buffer overruns in printf

2011-09-26 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > For sprintf() I agree - this is well understood and people are aware > of it. For printf() I am not so sure. We are a resource limited boot loader. We got for a small footprint, and accept some resulting restrictions, if they are not really severe.

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Wolfgang Denk
Dear "Brian S. Park", In message <4e80bc71.4000...@corelis.com> you wrote: > > I found my problem. > > My board, though based on walnut board, does not have real time clock. > So, I have to disable RTC support in u-boot. However, as soon as I > comment out > > #define CONFIG_RTC_DS174x1

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Wolfgang Denk
Dear "Brian S. Park", In message <4e80bf3d.8050...@corelis.com> you wrote: > > It seems that the tool that generates u-boot.lds does not like // in the > header file and putting it in the u-boot.lds file. The "tool that generates u-boot.lds" is the standard C preprocessor. Best regards, Wolfga

Re: [U-Boot] [PATCH] image: push default arch values to arch headers

2011-09-26 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201109261415.14398.vap...@gentoo.org> you wrote: > > > Should it not be trivial to add a > > > > #ifndef IH_ARCH_DEFAULT > > # error Unknown CPU type: IH_ARCH_DEFAULT not set > > #endif > > > > or similar to include/image.h ? > > it'd certainly be ea

Re: [U-Boot] [PATCH 3/4] Make printf and vprintf safe from buffer overruns

2011-09-26 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > > If you can find one that fits for all boards? =A0Keep in mind that > > printf() gets used before relocation, when available stack space may > > be _very_ limited. > > Yes that is a problem. Perhaps we could changes things so that this > CONFIG real

[U-Boot] [PATCH] ARM: Convert {in,out}s[bwl] to inline functions

2011-09-26 Thread Marek Vasut
The size of uboot binary grows by a few bytes, but the gain (better type checking) is worth it. Signed-off-by: Marek Vasut Cc: Wolfgang Denk Cc: Nick Thompson Cc: Simon Glass --- arch/arm/include/asm/io.h | 34 -- 1 files changed, 28 insertions(+), 6 deletion

Re: [U-Boot] [PATCH 3/5] NAND: Allow per-buffer allocation

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 08:33:56 PM Scott Wood wrote: > On 09/24/2011 07:37 AM, Marek Vasut wrote: > > On Friday, September 23, 2011 07:35:15 PM Scott Wood wrote: > >> On 09/22/2011 03:51 AM, Marek Vasut wrote: > >>> On Thursday, September 22, 2011 09:41:21 AM Stefano Babic wrote: > On 0

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109262029.24934.marek.va...@gmail.com> you wrote: > > Anyway, is there still anyone using 3.x ? If there isn't any such users, we > should simply discard it, add gcc version check and set the lower bound to > some > not-too-buggy version of gcc4. To be honest:

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Simon Glass
Hi Marek, On Mon, Sep 26, 2011 at 11:34 AM, Marek Vasut wrote: > On Monday, September 26, 2011 08:29:22 PM Simon Glass wrote: >> Hi Merek, >> >> On Mon, Sep 26, 2011 at 11:24 AM, Marek Vasut wrote: >> > On Monday, September 26, 2011 08:05:43 PM Simon Glass wrote: >> >> Hi Marek, >> >> >> >> On S

Re: [U-Boot] [PATCH 3/4] Make printf and vprintf safe from buffer overruns

2011-09-26 Thread Simon Glass
Hi Wolfgang, On Mon, Sep 26, 2011 at 11:47 AM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message > you > wrote: >> >> > If you can find one that fits for all boards? =A0Keep in mind that >> > printf() gets used before relocation, when available stack space may >> > be _very_ limited. >>

Re: [U-Boot] Building u-boot for Walnut board (PPC 405GPr)

2011-09-26 Thread Brian S. Park
Thank you for the information. I'm just re-learning my way around u-boot. I appreciate your comments and pointers. Last time I used u-boot, it did not use the current method for configuration and I was able to us // to comment stuff out in the configuration header file. Thank you and you guys

Re: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

2011-09-26 Thread Simon Glass
Hi Anton, On Mon, Sep 26, 2011 at 10:10 AM, Anton Staaf wrote: > On Mon, Sep 26, 2011 at 9:49 AM, Simon Glass wrote: >> Hi Mike, >> >> On Sun, Sep 25, 2011 at 9:48 PM, Mike Frysinger wrote: >>> On Sunday, September 25, 2011 16:18:32 Simon Glass wrote: On Sun, Sep 25, 2011 at 12:25 PM, Wolf

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Simon Glass
Hi Marek, On Mon, Sep 26, 2011 at 12:10 PM, Marek Vasut wrote: > On Monday, September 26, 2011 09:01:16 PM Simon Glass wrote: >> Hi Marek, >> >> >> The assignment to i I was referring to is here: >> >>       if (length == 1) { >>               if (size == 4) { >>                       longp = (ui

Re: [U-Boot] [PATCH 7/7] GCC4.6: Squash warning in cmd_mem.c

2011-09-26 Thread Marek Vasut
On Monday, September 26, 2011 09:01:16 PM Simon Glass wrote: > Hi Marek, > > On Mon, Sep 26, 2011 at 11:34 AM, Marek Vasut wrote: > > On Monday, September 26, 2011 08:29:22 PM Simon Glass wrote: > >> Hi Merek, > >> > >> On Mon, Sep 26, 2011 at 11:24 AM, Marek Vasut wrote: > >> > On Monday, Sep

Re: [U-Boot] [RFC PATCH 06/20] sandbox: Allow board_init_f() and board_init_r() to return

2011-09-26 Thread Simon Glass
Hi Anton, On Mon, Sep 26, 2011 at 10:49 AM, Anton Staaf wrote: > On Mon, Sep 26, 2011 at 9:48 AM, Simon Glass wrote: >> Hi Mike, >> >> On Sun, Sep 25, 2011 at 9:47 PM, Mike Frysinger wrote: >>> On Friday, September 23, 2011 11:55:11 Simon Glass wrote: On Sat, Sep 17, 2011 at 5:05 PM, Mike

Re: [U-Boot] [PATCH 2/2] phylib: remove a couple of redundant code lines

2011-09-26 Thread Vladimir Zapolskiy
Hello Wolfgang, On 23.09.2011 09:06, Wolfgang Denk wrote: > Dear Andy Fleming, > > In > message > you wrote: >> >> Shoot, I just saw this patch in my tree. It's incorrect. > > Argh... > >> The lines weren't redundant. The logic is (and probably should be >> better commented): >> >> Find the int

Re: [U-Boot] [RFC PATCH 06/20] sandbox: Allow board_init_f() and board_init_r() to return

2011-09-26 Thread Anton Staaf
On Mon, Sep 26, 2011 at 12:22 PM, Simon Glass wrote: > Hi Anton, > > On Mon, Sep 26, 2011 at 10:49 AM, Anton Staaf wrote: >> On Mon, Sep 26, 2011 at 9:48 AM, Simon Glass wrote: >>> Hi Mike, >>> >>> On Sun, Sep 25, 2011 at 9:47 PM, Mike Frysinger wrote: On Friday, September 23, 2011 11:55:1

  1   2   >