Re: [U-Boot-Users] Delete all env vars except read onlys

Hi Stefan, what about a 'scrubenv' or 'cleanenv' command that tries to unset all variables leaving readonlys untouched. The command could request a confirmation: => scrubenv Do you really want to do this ('yes' or 'no')? yes Can't overwrite "serial#" Can't overwrite "ethaddr" => saveenv The ab

Re: [U-Boot-Users] Delete all env vars except read onlys

Hi Matthias, On Wednesday 13 February 2008, Matthias Fuchs wrote: > what about a 'scrubenv' or 'cleanenv' command that tries to unset all > variables leaving readonlys untouched. Or 'resetenv'? > The command could request a confirmation: > > => scrubenv > Do you really want to do this ('yes' or

Re: [U-Boot-Users] [new uImage] Code available on new-image branch of u-boot-testing repo

Kumar Gala wrote: [...] >> On Feb 7, 2008, at 10:15 AM, Bartlomiej Sieka wrote: >>> Kumar Gala wrote: Here is a rough patch to common/cmd_bootm.c that passed load_end down to the do_bootm_* commands. I haven't updated the lib_*/bootm.c at this point. [...] >>> In our developmen

Re: [U-Boot-Users] Delete all env vars except read onlys

Hi Stefan, On Wednesday 13 February 2008 09:56, Stefan Roese wrote: > Hi Matthias, > > On Wednesday 13 February 2008, Matthias Fuchs wrote: > > what about a 'scrubenv' or 'cleanenv' command that tries to unset all > > variables leaving readonlys untouched. > > Or 'resetenv'? All the same to me w

[U-Boot-Users] UNCACHED_SDRAM macro issue

Hi list, I'm working on the porting of u-boot (1.3.1 from git) to a mips based SoC. The port seems to work fine unless the UNCACHED_SDRAM macro (used by do_bootm_linux). Using the version from commit de9a738faa7c2f47286119c3bfebc3dfbfe7d86d (that maps the addresses on KSEG1 instead on physical ad

[U-Boot-Users] [PATCH] Fix return value of mtest when CFG_ALT_MEMTEST set

Fix a missing return statement from a non-void function. Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]> --- diff --git a/common/cmd_mem.c b/common/cmd_mem.c index f6bff54..c905304 100644 --- a/common/cmd_mem.c +++ b/common/cmd_mem.c @@ -695,6 +695,7 @@ int do_mem_mtest (cmd_tbl_t *cmdt

Re: [U-Boot-Users] Delete all env vars except read onlys

On Wednesday 13 February 2008, Markus Klotzbücher wrote: > > => scrubenv > > Do you really want to do this ('yes' or 'no')? yes > > Such a confirmation is not very Unix-like and will prevent this command > to be used in scripts. I like the idea of a scrubenv/resetenv command > but vote for leaving

Re: [U-Boot-Users] Delete all env vars except read onlys

Matthias Fuchs <[EMAIL PROTECTED]> writes: > what about a 'scrubenv' or 'cleanenv' command that tries to unset all > variables leaving readonlys untouched. > > The command could request a confirmation: > > => scrubenv > Do you really want to do this ('yes' or 'no')? yes Such a confirmation is no

Re: [U-Boot-Users] [PATCH] Add 'imload' command

Kumar Gala wrote: > 'imload' provides a more direct means to load from an image file. > > Also created a load_image routine out of the code in do_bootm() that > is shared between do_bootm() and do_imload(). > > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> > --- > > Note, this is against the u-b

Re: [U-Boot-Users] Delete all env vars except read onlys

Stefan Roese <[EMAIL PROTECTED]> writes: > On Wednesday 13 February 2008, Markus Klotzbücher wrote: >> > => scrubenv >> > Do you really want to do this ('yes' or 'no')? yes >> >> Such a confirmation is not very Unix-like and will prevent this command >> to be used in scripts. I like the idea of a

Re: [U-Boot-Users] ramdisk problem (once again?)

Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: > >>> The kernel.org tree does not recognize a ramdisk image in flash. >>> Patches have been posted here since 2.4.17 times, but never made it >>> upstream. See the archives ... >>> >>> >> If you want to use it, pu

Re: [U-Boot-Users] Delete all env vars except read onlys

Hi, > Stefan Roese <[EMAIL PROTECTED]> writes: > >> On Wednesday 13 February 2008, Markus Klotzbücher wrote: >>> > => scrubenv >>> > Do you really want to do this ('yes' or 'no')? yes >>> >>> Such a confirmation is not very Unix-like and will prevent this command >>> to be used in scripts. I like

Re: [U-Boot-Users] [PATCH] add support for Toradex Colibri module

Hi Wolfgang, thanks for your response. On Mon, 2008-02-04 at 00:07 +0100, Wolfgang Denk wrote: > There are coding style issues with your patch: trailing white space, > C++ comments, indentation not by multiples of 8 characters. > > Please clean up and resubmit. > > And while you are at it - ple

Re: [U-Boot-Users] Delete all env vars except read onlys

Detlev Zundel <[EMAIL PROTECTED]> writes: >> Stefan Roese <[EMAIL PROTECTED]> writes: >> >>> On Wednesday 13 February 2008, Markus Klotzbücher wrote: > => scrubenv > Do you really want to do this ('yes' or 'no')? yes Such a confirmation is not very Unix-like and will prevent t

Re: [U-Boot-Users] Delete all env vars except read onlys

On Wednesday 13 February 2008, Markus Klotzbücher wrote: > > Why not do both? Unix untilities do it sometimes this way: Use an > > interactive check for "scrubenv" but allow "scrubenv -force" not to > > ask. As many of the U-Boot users know Unix, this would follow from the > > "principle of least

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: > Kumar Gala wrote: >> 'imload' provides a more direct means to load from an image file. >> Also created a load_image routine out of the code in do_bootm() that >> is shared between do_bootm() and do_imload(). >> Signed-off-by: Kumar Gala <[EMA

[U-Boot-Users] [RFC/PATCH] Add expr command

Add a simple expr command that will set an env variable as the result of the command. This allows us to do simple math in shell. The following operations are supported: &, |, +, -, *, /. --- I'd like to add this to the default command set in include/config_cmd_default.h but wanted to get feedba

Re: [U-Boot-Users] [RFC/PATCH] Add expr command

Hello! Kumar Gala schrieb: > Add a simple expr command that will set an env variable as the result > of the command. Good idea! I have been missing this for a long time but I also was too lazy to implement it. > + /* Validate arguments */ > + if ((argc != 5)) { > + printf("Us

Re: [U-Boot-Users] sam9263ek not booting uboot dataflash

On Tue, 2008-02-12 at 23:38 +0100, michael wrote: > Ok > E Robertson wrote: > >> Can you send me your diff? > >> > > > > The diff I'm using is attached. > > Thanks. > > > I'm talking about your adding code. Don't use the mailing list for send > code. I've narrowed down the problem to the

[U-Boot-Users] [PATCH][ARM] Rationalize ARM compiler options

Signed-off-by: Peter Pearse <[EMAIL PROTECTED]> --- diff --git a/cpu/arm1136/config.mk b/cpu/arm1136/config.mk old mode 100644 new mode 100755 index 6ab0dd3..5c64aa7 --- a/cpu/arm1136/config.mk +++ b/cpu/arm1136/config.mk @@ -2,6 +2,7 @@ # (C) Copyright 2002 # Gary Jennejohn, DENX Software Engine

[U-Boot-Users] [PATCH v2] Add expr command

Add a simple expr command that will set an env variable as the result of the command. This allows us to do simple math in shell. The following operations are supported: &, |, ^, +, -, *, /. Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> --- Added xor (^) and check that the op arg is only one cha

Re: [U-Boot-Users] [RFC/PATCH] Add expr command

On Wed, Feb 13, 2008 at 03:20:56PM +0100, Andreas Schweigstill wrote: > + /* Validate arguments */ > + if ((argc != 5) || (strlen(argv[3]) != 1)) { > + printf("Usage:\n%s\n", cmdtp->usage); > + return 1; > + } > > May we rely on the compiler optimizing the above

Re: [U-Boot-Users] [PATCH v2] Add expr command

On Wed, 13 Feb 2008 09:54:31 -0600 (CST) Kumar Gala <[EMAIL PROTECTED]> wrote: > +} > + > +U_BOOT_CMD( > + expr, 5, 0, do_expr, > + "expr- set environment variable as the result of eval expression\n", > + "name value1 value2\n" > + "- set environment variable 'name' to the

Re: [U-Boot-Users] [PATCH v2] Add expr command

Hello! Kim Phillips schrieb: > can we at least change the name to "setexpr" or something, so as not > break forward compatibility with any future GNU syntax expr > implementation? +1 Regards Andreas Schweigstill -- Dipl.-Phys. Andreas Schweigstill Schweigstill IT | Embedded Systems Schauenbur

Re: [U-Boot-Users] [PATCH v2] Add expr command

On Feb 13, 2008, at 10:35 AM, Kim Phillips wrote: > On Wed, 13 Feb 2008 09:54:31 -0600 (CST) > Kumar Gala <[EMAIL PROTECTED]> wrote: > >> +} >> + >> +U_BOOT_CMD( >> +expr, 5, 0, do_expr, >> +"expr- set environment variable as the result of eval >> expression\n", >> +"name value1

Re: [U-Boot-Users] [PATCH] Add 'imload' command

Kumar Gala wrote: > > On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: > >> Kumar Gala wrote: >>> 'imload' provides a more direct means to load from an image file. >>> Also created a load_image routine out of the code in do_bootm() that >>> is shared between do_bootm() and do_imload(). >>> Si

Re: [U-Boot-Users] u-boot patch submission process

Wolfgang Denk wrote: [snip] > When I scan patches, this requires that I not only check the patch, > but all the following thread(s) it spawns. This is not only > time-consuming, but has also good potential to lose track of what's > been done and what not (yes, we are missing a patc

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 1:55 PM, Bartlomiej Sieka wrote: > Kumar Gala wrote: >> On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: >>> Kumar Gala wrote: 'imload' provides a more direct means to load from an image file. Also created a load_image routine out of the code in do_bootm()

Re: [U-Boot-Users] [PATCH] Add 'imload' command

Kumar Gala wrote: [...] - The load_image routine (and consequently imload commad) will not work when the image is stored in Data Flash. >>> what's the issue here? >> >> Please have a look at code under CONFIG_HAS_DATAFLASH in get_kernel() >> (formerly in do_bootm()), especially the read_

[U-Boot-Users] tftp warning/error?

'TFTP: add host ip addr support' commit added this bit of code which seems bogus: } else { char *p = strchr (p, ':'); Is this really correct? - k - This SF.net email is sponsored by: Microsoft De

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 2:22 PM, Bartlomiej Sieka wrote: > Kumar Gala wrote: > [...] > - The load_image routine (and consequently imload commad) will > not work when the image is stored in Data Flash. what's the issue here? >>> >>> Please have a look at code under CONFIG_HAS_DATAFLASH

[U-Boot-Users] AT91 Bootstrap in U-boot

Hi, Is their a reason why the AT91 Bootstrap code was never ported to U-boot? Thanks. signature.asc Description: This is a digitally signed message part - This SF.net email is sponsored by: Microsoft Defy all challenges. Mic

[U-Boot-Users] [PATCH] Fix building of fdt_support.c if DEBUG set

Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> --- Wolfgang, would like to see this go into mainline as it fixes building. common/fdt_support.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/common/fdt_support.c b/common/fdt_support.c index 92f1c7f..69eb667 100644 -

Re: [U-Boot-Users] [PATCH] Fix building of fdt_support.c if DEBUG set

Jon Loeliger wrote: > Kumar Gala wrote: >> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> >> --- > > Kumar, Thanks for getting this one! > > Acked-by: Jon Loeliger <[EMAIL PROTECTED]> Ditto. That is on my list to check and add to u-boot-fdt, but if Wolfgang can apply this directly, it will heal

Re: [U-Boot-Users] [PATCH] Fix building of fdt_support.c if DEBUG set

Kumar Gala wrote: > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> > --- Kumar, Thanks for getting this one! Acked-by: Jon Loeliger <[EMAIL PROTECTED]> - This SF.net email is sponsored by: Microsoft Defy all challenges. Micro

[U-Boot-Users] [PATCH] Add CFG_MPC86xx_DDR_ADDR and CFG_MPC86xx_DDR2_ADDR symbols

These replace direct structure references for IMMR sections. Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]> --- include/asm-ppc/immap_86xx.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/include/asm-ppc/immap_86xx.h b/include/asm-ppc/immap_86xx.h index 169725b..a15

Re: [U-Boot-Users] [new uImage] Code available on new-image branch of u-boot-testing repo

In message <[EMAIL PROTECTED]> you wrote: > > >> For example, I don't believe we have a way today to just decompress a > >> uImage file. > > > > Um... it you try hard enough there is one. See > > http://www.denx.de/wiki/view/DULG/HowCanILoadAndUncompressACompressedImage > > Hmm, do you have objec

Re: [U-Boot-Users] [PATCH] Add 'imload' command

In message <[EMAIL PROTECTED]> you wrote: > 'imload' provides a more direct means to load from an image file. What exactly is an "image file" here (which sorts of images are allowed?) and what does "load" mean (which actions are performed on the file)? > Also created a load_image routine out of t

Re: [U-Boot-Users] ability to do simple math in u-boot shell?

In message <[EMAIL PROTECTED]> you wrote: > > btw, u-boot users and developers should start prepending new hex values > with "0x", btw. Nope. Please don't. U-Boot standard input base is hex. Has always been that way. 0x is just a waste of key presses and memory footprint. Best regards, Wolf

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > what about a 'scrubenv' or 'cleanenv' command that tries to unset all > variables leaving readonlys untouched. Make this "clearenv", please. > The command could request a confirmation: No. > => scrubenv > Do you really want to do this ('yes' or 'n

Re: [U-Boot-Users] Delete all env vars except read onlys

Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: >> Why not do both? Unix untilities do it sometimes this way: Use an >> interactive check for "scrubenv" but allow "scrubenv -force" not to >> ask. As many of the U-Boot users know Unix, this would follow from >> the "principle of lea

Re: [U-Boot-Users] Delete all env vars except read onlys

Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: >>> U-Boot provides many, much more serious ways to blow you foot off which >>> don't ask such questions. >> Sure. But not by "just" typing one command. Or at least I don't know those >> kind of destructive commands. > > => prot off

Re: [U-Boot-Users] [PATCH] Fix building of fdt_support.c if DEBUG set

In message <[EMAIL PROTECTED]> you wrote: > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> > --- > > Wolfgang, would like to see this go into mainline as it fixes building. Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB

Re: [U-Boot-Users] [PATCH] Add CFG_MPC86xx_DDR_ADDR and CFG_MPC86xx_DDR2_ADDR symbols

In message <[EMAIL PROTECTED]> you wrote: > These replace direct structure references for IMMR sections. Bug fix? Pull request? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Ger

Re: [U-Boot-Users] [PATCH] Add 'imload' command

In message <[EMAIL PROTECTED]> you wrote: > > > What exactly is an "image file" here (which sorts of images are > > allowed?) and what does "load" mean (which actions are performed on > > the file)? > > its suppose to be a 'uImage' not sure what the base way to refer to > that is. Why just uIm

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > > That would be exactly the other way round, i. e. provide an optional > > "-i" argument like "rm *" is doing in Unix. > > > > The default shall be not to ask any questions. > > rm defaults to not asking questions in *some* cases, but to asking in >

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 4:31 PM, Wolfgang Denk wrote: > In message [EMAIL PROTECTED]> you wrote: >> 'imload' provides a more direct means to load from an image file. > > What exactly is an "image file" here (which sorts of images are > allowed?) and what does "load" mean (which actions are performed

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > > U-Boot provides many, much more serious ways to blow you foot off which > > don't ask such questions. > > Sure. But not by "just" typing one command. Or at least I don't know those > kind of destructive commands. => prot off all; era all > So I sti

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > Why not do both? Unix untilities do it sometimes this way: Use an > interactive check for "scrubenv" but allow "scrubenv -force" not to > ask. As many of the U-Boot users know Unix, this would follow from > the "principle of least surprise"... That wo

Re: [U-Boot-Users] [PATCH][ARM] Rationalize ARM compiler options

In message <[EMAIL PROTECTED]> you wrote: > Signed-off-by: Peter Pearse <[EMAIL PROTECTED]> > --- > diff --git a/cpu/arm1136/config.mk b/cpu/arm1136/config.mk > old mode 100644 > new mode 100755 > index 6ab0dd3..5c64aa7 > --- a/cpu/arm1136/config.mk > +++ b/cpu/arm1136/config.mk > @@ -2,6 +2,7 @@ >

Re: [U-Boot-Users] [PATCH v2] Add expr command

In message <[EMAIL PROTECTED]> you wrote: > > > can we at least change the name to "setexpr" or something, so as not > > break forward compatibility with any future GNU syntax expr > > implementation? > > ok. Waiting to here on wd's feeling about having this as part of the > standard command s

Re: [U-Boot-Users] u-boot patch submission process

In message <[EMAIL PROTECTED]> you wrote: > > Jeremy Kerr has a perl script that picks out patches: > > - try before you buy Yes, I'm aware of this. We're hacking gnats to do what we need :-) Best regards, Wolfgang

[U-Boot-Users] [PATCH] Add setexpr command

Add a simple expr style command that will set an env variable as the result of the command. This allows us to do simple math in shell. The following operations are supported: &, |, ^, +, -, *, /. Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> --- this was the expr patch, but renamed based on com

Re: [U-Boot-Users] Delete all env vars except read onlys

Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: >>> That would be exactly the other way round, i. e. provide an optional >>> "-i" argument like "rm *" is doing in Unix. >>> >>> The default shall be not to ask any questions. >> rm defaults to not asking questions in *some* cases, bu

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > > => prot off all; era all > > There's a slight difference between two commands with arguments versus > one command with none. Well, as Markus already wote, there are so many ways to kill yourself. If you don't accept the line above, just

Re: [U-Boot-Users] [PATCH] Add setexpr command

In message <[EMAIL PROTECTED]> you wrote: > Add a simple expr style command that will set an env variable as the result > of the command. This allows us to do simple math in shell. The following > operations are supported: &, |, ^, +, -, *, /. > > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> >

Re: [U-Boot-Users] [PATCH] Add CFG_MPC86xx_DDR_ADDR and CFG_MPC86xx_DDR2_ADDR symbols

Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: >> These replace direct structure references for IMMR sections. > > Bug fix? Pull request? > > Best regards, > > Wolfgang Denk > Posting of patches to the list! :-) I place it into the mpc86xx repository for a later pull request

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > > Oops??? "rm" never asks unless you ask it to ask. > > Not true. It will ask if you don't have write permission to the file, > even if you are able to delete because you have write permission to the > directory. Does it? Indeed. Must be a GNUism.

Re: [U-Boot-Users] 440EPX is not booting from 1Gb/128MB NAND flash

Hi Stefan Thanks for all the help in solving the problem. I have some questions about submitting the patch and testing of the patch. First of all I am working on proprietary board based on AMCC sequoia board. Hence some of the files I changed are module specific like config.mk in nand_spl/board/.

Re: [U-Boot-Users] [PATCH] ppc4xx: Fix problem with init-ram bigger than 4k on 440 platforms

Dear Stefan, in message <[EMAIL PROTECTED]> you wrote: > Signed-off-by: Stefan Roese <[EMAIL PROTECTED]> > --- > cpu/ppc4xx/start.S |4 > include/configs/pcs440ep.h |2 +- > include/configs/yosemite.h |2 +- > 3 files changed, 6 insertions(+), 2 deletions(-) Do you inten

Re: [U-Boot-Users] Bug in cpu/74xx_7xx/speed.c ?

In message <[EMAIL PROTECTED]> you wrote: > > during test with the 1.3.1 Version of u-boot, i have noticed, that the > calculation of the cpu speed for a 1GHz 750GX is wrong. > Here is a patch to fix the problem, it is done the same way as for the > other supported cpu's. Please resubmit as p

Re: [U-Boot-Users] [PATCH] Add setexpr command

On Feb 13, 2008, at 5:07 PM, Wolfgang Denk wrote: > In message [EMAIL PROTECTED]> you wrote: >> Add a simple expr style command that will set an env variable as >> the result >> of the command. This allows us to do simple math in shell. The >> following >> operations are supported: &, |, ^

Re: [U-Boot-Users] [PATCH] Remove duplicate defines for ARRAY_SIZE

In message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> > --- > cpu/arm926ejs/cpuinfo.c |2 -- > drivers/mtd/jedec_flash.c |4 > drivers/net/rtl8169.c |2 -- > include/common.h |2 ++ > 4 files changed, 2 insertions(+), 8 del

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 5:01 PM, Wolfgang Denk wrote: > In message <7DB626E7-34A4-4EAA-B470- > [EMAIL PROTECTED]> you wrote: >> >>> What exactly is an "image file" here (which sorts of images are >>> allowed?) and what does "load" mean (which actions are performed on >>> the file)? >> >> its suppose

Re: [U-Boot-Users] [PATCH] Extend ATI Radeon driver to support more video modes

Hello, Rodolfo Giometti wrote: > On Mon, Feb 11, 2008 at 08:42:08PM +0100, Anatolij Gustschin wrote: >> Adds ATI Radeon 9200 support for 1280x1024, 1024x768, >> 800x600, 640x480 at 24, 16 and 8 bpp. Hope this patch >> won't screw up X300 and X700 support. > > Can someone test/check this issue? :)

Re: [U-Boot-Users] [PATCH] Add *~ to .gitignore

In message <[EMAIL PROTECTED]> you wrote: > One should never add a backup file ending in with ~ to the git repository. > > Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB

[U-Boot-Users] [RFC] raw go command

This patch is just a quick work up to see how to handle a particular problem I've got. The idea is that I want finer control over what happens when I 'go' to piece of code. The idea is it looks more like a native function call than the current go which is more main() like. The calling convention

Re: [U-Boot-Users] [PATCH] Fix incorrect address test in AT91F_DataflashSelect

In message <[EMAIL PROTECTED]> you wrote: > Fix incorrect address test in AT91F_DataflashSelect(). > > Signed-off-by: Stelian Pop <[EMAIL PROTECTED]> Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office

Re: [U-Boot-Users] [PATCH] Fix remaining CONFIG_COMMANDS

In message <[EMAIL PROTECTED]> you wrote: > update comments > Fix coding style > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Offic

Re: [U-Boot-Users] [PATCH] Mark board_init_[fr] as noreturn

In message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> > --- > include/common.h |4 ++-- Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5,

Re: [U-Boot-Users] [PATCH] Do not specify a CROSS_COMPILE default when executing size

In message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> > --- > MAKEALL |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/MAKEALL b/MAKEALL > index d66a5fa..2aff4da 100755 > --- a/MAKEALL > +++ b/MAKEALL > @@ -721,7 +721,7 @@ b

Re: [U-Boot-Users] [PATCH] Use `ln -sf` rather than `rm -f && ln -s`

In message <[EMAIL PROTECTED]> you wrote: > On Mon, Jan 28, 2008 at 06:38:58AM -0500, Mike Frysinger wrote: > > The -f option to `ln` should give the same behavior as the -f option to the > > `rm` command. It is better to do this in one shot so as to avoid race > > conditions when building in para

Re: [U-Boot-Users] AT91 Bootstrap in U-boot

> Hi, > Is their a reason why the AT91 Bootstrap code was never ported to > U-boot? Yes, Integrating AT91Bootstrap only makes sense if you have a parallel flash. There are no Atmel AT91SAM9 dev boards which uses a parallel flash. The AT91RM9200 boards uses parallel flash, but they do not use AT91

Re: [U-Boot-Users] [PATCH] NFS Timeout with large files.

In message <[EMAIL PROTECTED]> you wrote: > Retry to send NFS packet before reaching timeout. > > Signed-off-by: Hiroshi Ito <[EMAIL PROTECTED]> Applied, thanks a lot. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Offic

Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function only if CONFIG_SKIP_LOWLEVEL_INIT is not defined

Hi, I went through "cpu/arm920t/start.S" file and found that this is what this patch is doing. This patch places the entire cpu_init_crit function definition inside ifndef same as in arm920t/start.S. Sorry if i havent understood you. Present cpu/arm926ejs/start.S _start_armboot: .wor

Re: [U-Boot-Users] [PATCH] Only use TEXT_BASE if defined by the board

In message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+

Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function only if CONFIG_SKIP_LOWLEVEL_INIT is not defined

In message <[EMAIL PROTECTED]> you wrote: > > At Present the call to "cpu_init_crit" in uboot/cpu/arm926ejs/start.S (142) > is under ifndef > > #ifndef CONFIG_SKIP_LOWLEVEL_INIT > bl cpu_init_crit > #endif > > But the function definition (203) isn't under this ifndef. So irrespectiv

Re: [U-Boot-Users] [PATCH] Add setexpr command

In message <[EMAIL PROTECTED]> you wrote: > > > Well, *my* command was *not* to rename it. Can't even type any more. I intended to write "comment". > > At least, please don't make it a default command, then. > > If you are good with calling it expr so am I. I'd prefer not > confusing 'setenv

Re: [U-Boot-Users] [PATCH] Add 'imload' command

In message <[EMAIL PROTECTED]> you wrote: > > > Why just uImage? > > what other image formats do we support? I assume we'll extend it for > the new format style once that exists. uImage means "OS Kernel Image" (IH_TYPE_KERNEL in include/image.h). > I feel like I'm missing something here. I

Re: [U-Boot-Users] [PATCH] Extend ATI Radeon driver to support more video modes

In message <[EMAIL PROTECTED]> you wrote: > > >> +#if 1 /* @ 60 Hz */ > >> + mode->crtc_h_total_disp = 0x009f00d2; > >> + mode->crtc_h_sync_strt_wid = 0x000e0528; > >> + mode->crtc_v_total_disp = 0x03ff0429; > >> + mode->crtc_v_sync_strt_wid = 0x00030400; > >> +

Re: [U-Boot-Users] [RFC] raw go command

In message <[EMAIL PROTECTED]> you wrote: > This patch is just a quick work up to see how to handle a particular > problem I've got. The idea is that I want finer control over what happens > when I 'go' to piece of code. The idea is it looks more like a native > function call than the current go

Re: [U-Boot-Users] [RFC] raw go command

On Feb 13, 2008, at 6:40 PM, Wolfgang Denk wrote: > In message [EMAIL PROTECTED]> you wrote: >> This patch is just a quick work up to see how to handle a particular >> problem I've got. The idea is that I want finer control over what >> happens >> when I 'go' to piece of code. The idea is it

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 6:37 PM, Wolfgang Denk wrote: > In message <3192541E-1FBD-4638- > [EMAIL PROTECTED]> you wrote: >> >>> Why just uImage? >> >> what other image formats do we support? I assume we'll extend it for >> the new format style once that exists. > > uImage means "OS Kernel Image" (IH

Re: [U-Boot-Users] [PATCH] Do not specify a CROSS_COMPILE default when executing size

On Wednesday 13 February 2008, Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> > > --- > > MAKEALL |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/MAKEALL b/MAKEALL > > index d66a5fa..2aff4d

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008 12:55 PM, Bartlomiej Sieka <[EMAIL PROTECTED]> wrote: > Kumar Gala wrote: > > > > On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: > > > >> Kumar Gala wrote: > >>> 'imload' provides a more direct means to load from an image file. > >>> Also created a load_image routine out of th

Re: [U-Boot-Users] UNCACHED_SDRAM macro issue

Luigi 'Comio' Mantellini wrote: > I'm working on the porting of u-boot (1.3.1 from git) to a mips based > SoC. The port seems to work fine unless the UNCACHED_SDRAM macro (used > by do_bootm_linux). > > Using the version from commit de9a738faa7c2f47286119c3bfebc3dfbfe7d86d > (that maps the address

Re: [U-Boot-Users] Delete all env vars except read onlys

On Wednesday 13 February 2008, Scott Wood wrote: > Wolfgang Denk wrote: > > In message <[EMAIL PROTECTED]> you wrote: > >>> U-Boot provides many, much more serious ways to blow you foot off which > >>> don't ask such questions. > >> > >> Sure. But not by "just" typing one command. Or at least I don

Re: [U-Boot-Users] [PATCH] ppc4xx: Fix problem with init-ram bigger than 4k on 440 platforms

Hi Wolfgang, On Thursday 14 February 2008, Wolfgang Denk wrote: > in message <[EMAIL PROTECTED]> you wrote: > > Signed-off-by: Stefan Roese <[EMAIL PROTECTED]> > > --- > > cpu/ppc4xx/start.S |4 > > include/configs/pcs440ep.h |2 +- > > include/configs/yosemite.h |2 +- >

Re: [U-Boot-Users] [PATCH] Add 'imload' command

>> +switch (image_get_comp (hdr)) { >> +case IH_COMP_NONE: >> +if (dst == (ulong)hdr) { >> +printf (" XIP %s ... ", type_name); > > This is an ugly hack in the old code that should be removed by the new > implementation. can you explain this further, I'm h

Re: [U-Boot-Users] 440EPX is not booting from 1Gb/128MB NAND flash

Hi Nikhil. On Thursday 14 February 2008, Nikhil Gautam wrote: > Thanks for all the help in solving the problem. I have some questions > about submitting the patch and testing of the patch. > > First of all I am working on proprietary board based on AMCC sequoia > board. You need to create a new b

Re: [U-Boot-Users] [PATCH] Add 'imload' command

On Feb 13, 2008, at 7:13 PM, Grant Likely wrote: > On Feb 13, 2008 12:55 PM, Bartlomiej Sieka <[EMAIL PROTECTED]> wrote: >> Kumar Gala wrote: >>> >>> On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: >>> Kumar Gala wrote: > 'imload' provides a more direct means to load from an image f

[U-Boot-Users] USB ethernet for OMAP850

Hi, I'm porting Linux kernel to OMAP850. I loaded kernel via JTAG, now kernel is booting and can output message to UART console. Now I'm trying to boot kernel from u-boot. I would like to load kernel via ethernet, but my OMAP850 board has no ethernet chip on it. Is there any way to use usb port a

Re: [U-Boot-Users] [PATCH] ppc4xx: Add CONFIG_4xx_DCACHE compile switch to Denali-core SPD code

On Tuesday 22 January 2008, Larry Johnson wrote: > Signed-off-by: Larry Johnson <[EMAIL PROTECTED]> Added to for-1.3.2 branch. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel

Re: [U-Boot-Users] [PATCH 2/4: resubmit] ppc4xx: HCU4/5. Fix make O=../xx

On Tuesday 05 February 2008, Niklaus Giger wrote: > Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> Added to for-1.3.2 branch. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel

Re: [U-Boot-Users] tftp warning/error?

On 14:57 Wed 13 Feb , Kumar Gala wrote: > 'TFTP: add host ip addr support' commit added this bit of code which seems > bogus: > > } else { > char *p = strchr (p, ':'); > > Is this really correct? No, it this - char *p = strchr (p, ':'); + char

[U-Boot-Users] [PATCH] TFTP: fix search of ':' in BootFile

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> diff --git a/net/tftp.c b/net/tftp.c index 3dd2b06..ea8fea2 100644 --- a/net/tftp.c +++ b/net/tftp.c @@ -474,7 +474,7 @@ TftpStart (void) printf ("*** Warning: no boot file name; using '%s'\n",

Re: [U-Boot-Users] [PATCH] TFTP: fix search of ':' in BootFile

Wolfgang, Can you pick this up as its a clear bug fix. - k On Feb 14, 2008, at 1:02 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > <[EMAIL PROTECTED]> > > diff --git a/net/tftp.c b/net/tftp.c > index 3dd2b06..ea8fea2 100644 > --- a/net/tftp.c >

Re: [U-Boot-Users] [PATCH] Add 'imload' command

Grant Likely wrote: > On Feb 13, 2008 12:55 PM, Bartlomiej Sieka <[EMAIL PROTECTED]> wrote: >> Kumar Gala wrote: >>> On Feb 13, 2008, at 4:11 AM, Bartlomiej Sieka wrote: >>> Kumar Gala wrote: > 'imload' provides a more direct means to load from an image file. > Also created a load_imag

Re: [U-Boot-Users] Delete all env vars except read onlys

In message <[EMAIL PROTECTED]> you wrote: > > I would still prefer the version with the '--force' parameter. Just my 2 > cents. I really don't want to have this, for two reasons: 1) U-Boot doesn't do "Do you really want to" confirmations, because these don't fit into the Unix design philisophy; 2