public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] bootelf and command line arguments
@ 2009-07-18 20:08 Thomas Dörfler
  2009-07-18 20:21 ` Wolfgang Denk
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Dörfler @ 2009-07-18 20:08 UTC (permalink / raw)
  To: u-boot

Hello,

up to now I have worked in private with U-Boot and adapted it to a
user-specific MPC5121e based board. We have worked out how we want to
use U-Boot and soon would like to commit the board specific (and not so
specific) source code changes and additions.

Now we have hit a strange point. the "bootelf" command obviously is
prepared to pass argc/argv to the called program. But in its command
list entry, the number of maximum arguments is limited to 2, which
allows only the command itself and the address of the elf image in
memory to be specified.

I have modified the maxargs field to be CONFIG_SYS_MAXARGS, which works
fine. So my questions are:

- Is there any useful reason why the bootelf command is limited to
maxargs=2?

- Would anybody mind if I commit a patch to change maxargs to
CONFIG_SYS_MAXARGS?

wkr,
Thomas Doerfler.

-- 

--------------------------------------------
Embedded Brains GmbH
Thomas Doerfler        Obere Lagerstrasse 30
D-82178 Puchheim       Germany
email: Thomas.Doerfler at embedded-brains.de
Phone: +49-89-18908079-2
Fax:   +49-89-18908079-9

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [U-Boot] bootelf and command line arguments
  2009-07-18 20:08 [U-Boot] bootelf and command line arguments Thomas Dörfler
@ 2009-07-18 20:21 ` Wolfgang Denk
  0 siblings, 0 replies; 2+ messages in thread
From: Wolfgang Denk @ 2009-07-18 20:21 UTC (permalink / raw)
  To: u-boot

Dear Thomas D?rfler,

In message <4A622BD6.2070303@embedded-brains.de> you wrote:
>
> - Is there any useful reason why the bootelf command is limited to
> maxargs?

No. git blame shows that this entry has never been changed, so I guess
it just went unnoticed so far.

> - Would anybody mind if I commit a patch to change maxargs to
> CONFIG_SYS_MAXARGS?

No, on contrary - I consider this a bug fix.

I recommend to submit a (separate) patch for this ASAP so it can go in
into the upcoming release.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"Anyone attempting to generate random numbers by deterministic  means
is, of course, living in a state of sin."          - John Von Neumann

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-07-18 20:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-18 20:08 [U-Boot] bootelf and command line arguments Thomas Dörfler
2009-07-18 20:21 ` Wolfgang Denk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox