[U-Boot] [PATCH 1/5] Add debug output to mx29 SPL source files
Graeme Russ
gruss at tss-engineering.com
Thu Jan 22 05:24:05 CET 2015
On 22/01/15 10:24, Marek Vasut wrote:
> On Wednesday, January 21, 2015 at 12:55:14 PM, Graeme Russ wrote:
>> Signed-off-by: Graeme Russ <gruss at tss-engineering.com>
>
> Hi!
>
> - Commit message is missing.
> - Some of the debug outputs are missing newline (\n) character
> - You can use __func__ and __LINE__ in the debug output to better
> specify where the debug spit happened.
I saw something earlier on the list about this - I thought the feeling
was to use descriptive strings that are easy to grep rather than
__func__ and __LINE__, or maybe I misread
>
> Oh, and the CC list looks pretty random. You can just add "Cc: " tag
> into the patches so that git handles it correctly.
Cc list was generated by patman (I added Damien as his patch is what
mine are based on)
Regards,
Graeme
More information about the U-Boot
mailing list