[U-Boot] [PATCH] Cleanup the comment for m68k linux boot argument passing.
Richard Retanubun
RichardRetanubun at RuggedCom.com
Fri Feb 20 18:15:22 CET 2009
Hi Mike,
wow, that a more rapid response that I am used to :)
Mike Frysinger wrote:
> On Friday 20 February 2009 11:34:06 Richard Retanubun wrote:
>> From e2e085f8b8e638762c59b0bc5376b241b0046132 Mon Sep 17 00:00:00 2001
>> From: Richard Retanubun <RichardRetanubun at RuggedCom.com>
>> Date: Fri, 20 Feb 2009 11:06:36 -0500
>> Subject: [PATCH] Cleanup the comment for m68k linux boot argument passing.
>>
>>
>> Signed-off-by: Richard Retanubun <RichardRetanubun at RuggedCom.com>
>> ---
>> Hi Tsi-Chung,
>>
>> Thanks for the response, I found the solution after using a hw debugger to
>> step through the code. The one gotcha here is that the assembly instruction
>> that the compiler uses to jump to the kernel is 'jsr' which puts the next
>> instruction after the jsr into the stack pointer so there is an
>> 'off-by-one' effect from the stack pointer.
>>
>> This patch cleans up the comment to clarify the boot arg passing.
>> I'll try to push a patch for the kernel as well.
>
> comments for the changelog go above the ---, not below
Yep, I thought this is what I did, the commit message is
"Cleanup the comment for m68k linux boot argument passing."
The stuff under the --- Is just continuing a conversation thread.
I tend to keep the commit message short and be more verbose in the
code comments.
>
> use git and put it into the commit message:
> $ git commit --amend
Thanks for the tip. I'll submit a V2.
> blah blah the patch summary
>
> a paragraph about the changes ....
> -mike
More information about the U-Boot
mailing list