Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-09 Thread Steven Stallion
On Sun, Jun 9, 2013 at 12:52 AM, Wolfgang Denk wrote: > This is basically the same code (with only irrelevant differences) as > used by do_bootm_netbsd(). Can you please > > 1) factor out this common code, and > 2) documnt the behaviour > ? Will do. > By the way: this patch still triggers two

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-09 Thread Wolfgang Denk
Dear Steven Stallion, In message <1370562103-92148-1-git-send-email-sstall...@gmail.com> you wrote: > This patch introduces support for command line arguments to Plan 9. > Plan 9 generally dedicates a small region of kernel memory (known > as CONFADDR) for runtime configuration. A new environment

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-09 Thread Wolfgang Denk
Dear Steven Stallion, In message you wrote: > > > "bootm" has a well-known an documented API: it takes up to three > > arguments: kernel address, ramdisk address, and device tree address. > > Nothing else. > > Is there a reason that this should only be used by Linux? The changes > I have submit

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-08 Thread Steven Stallion
On Thu, Jun 6, 2013 at 4:41 PM, Steven Stallion wrote: > This patch introduces support for command line arguments to Plan 9. > Plan 9 generally dedicates a small region of kernel memory (known > as CONFADDR) for runtime configuration. A new environment variable > named confaddr was introduced to

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-07 Thread Steven Stallion
On Fri, Jun 7, 2013 at 2:57 PM, Wolfgang Denk wrote: > Dear Steven, > > In message > you > wrote: >> >> Hmm. Are you arguing against supporting command line arguments to bootm, or >> that bootm should copy these arguments to bootargs prior to boot? This has >> actually been very useful to test

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-07 Thread Wolfgang Denk
Dear Steven, In message you wrote: > > Hmm. Are you arguing against supporting command line arguments to bootm, or > that bootm should copy these arguments to bootargs prior to boot? This has > actually been very useful to test changes without having to update my > bootargs environment variable.

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-07 Thread Steven Stallion
On Fri, Jun 7, 2013 at 1:16 AM, Wolfgang Denk wrote: > In message ceumhu8u4enqyj3wt3rqyvdzatw...@mail.gmail.com> you wrote: > > > > > Please make this code configurable, so that people who never intend to > > > use Plan 9 do not suffer from the increased code size. > > > > This is already done,

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-07 Thread Wolfgang Denk
Dear Steven, please always keep the mailing list on Cc: - this is especially important for patch reviews, where such messages need to be picked up for patchwork, too. In message you wrote: > > > Please make this code configurable, so that people who never intend to > > use Plan 9 do not suffer

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9

2013-06-07 Thread Wolfgang Denk
Dear Steven Stallion, In message <1370563140-31368-1-git-send-email-sstall...@gmail.com> you wrote: > This patch introduces support for command line arguments to Plan 9. > Plan 9 generally dedicates a small region of kernel memory (known > as CONFADDR) for runtime configuration. A new environment