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 <gr...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to