<div dir="ltr">Hi Gedare,<div><br></div><div class="gmail_extra"><div class="gmail_quote">On 20 March 2014 04:48, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi James,<br>
<br>
Thanks for the patches! I have a couple of comments.<br>
<br>
First, the commit message for this patch should be divided into two<br>
pieces, a short message on the first line, and then a blank line, and<br>
then a longer detailed explanation. This way, the email subject and<br>
the log will be much easier to read.<br></blockquote><div><br></div><div>Can do - that was my first attempt at using git send-email and it didn't turn out quite how I expected!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Comments about the patch in particular are inline below. Most of these<br>
are style nits that are advised but not required. We don't normally<br>
apply the same rigor to BSPs as to the rest of RTEMS, but appreciate<br>
if you do. :)</blockquote><div><br></div><div>Of course, no problem. I should have checked for a style guide before I submitted my patch.</div><div><br></div><div>I'll make the appropriate revisions and submit a revised patch in a few days as I'm about to head off for a short break. </div>
<div><br></div><div>Cheers,</div><div>James</div><div><br></div></div></div></div>