[U-Boot] bootelf and command line arguments

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.

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
participants (2)
-
Thomas Dörfler
-
Wolfgang Denk