[U-Boot] [PATCH 04/11] MIPS: bootm: refactor initialisation of kernel cmdline
Daniel Schwierzeck
daniel.schwierzeck at gmail.com
Sun Jun 9 13:43:59 CEST 2013
2013/6/9 Wolfgang Denk <wd at denx.de>:
> Dear Daniel Schwierzeck,
>
> In message <1370731370-32767-5-git-send-email-daniel.schwierzeck at gmail.com> you wrote:
>> Move initialisation of Linux command line to separate functions.
>> Also add support for bootm subcommand 'cmdline'.
>
>> +static void linux_cmdline_dump(void)
>> +{
>> +#ifdef DEBUG
>> + int i;
>> +
>> + printf("## cmdline argv at 0x%p, argp at 0x%p\n",
>> + linux_argv, linux_argp);
>> +
>> + for (i = 1; i < linux_argc; i++)
>> + printf(" arg %03d: %s\n", i, linux_argv[i]);
>> +#endif
>> +}
>
> Please remove the #ifdef here, and change printf() into debug()
> instead.
I will change it
>
>> + while (next && quote && quote < next) {
>> + /* we found a left quote before the next blank
>> + * now we have to find the matching right quote
>> + */
>
> Incorrect multiline comment style.
>
a left-over from the original code. I will fix it in patch 01/11.
Thanks,
Daniel
More information about the U-Boot
mailing list