<div dir="ltr">Hi Gedare,<div><br></div><div>I've just submitted a revised patch, I hope it all looks ok now.</div><div><br></div><div>This new patch includes the changes from both the patches I previously submitted - I hope that isn't a problem?</div>
<div><br></div><div>Cheers,</div><div>James </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 20 March 2014 09:06, 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"><div class="HOEnZb"><div class="h5">On Wed, Mar 19, 2014 at 3:11 PM, James Fitzsimons<br>
<<a href="mailto:james.fitzsimons@gmail.com">james.fitzsimons@gmail.com</a>> wrote:<br>
> Hi Gedare,<br>
><br>
> On 20 March 2014 04:48, Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>
>><br>
>> 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>
><br>
><br>
> Can do - that was my first attempt at using git send-email and it didn't<br>
> turn out quite how I expected!<br>
><br>
>><br>
>> 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. :)<br>
><br>
><br>
> Of course, no problem. I should have checked for a style guide before I<br>
> submitted my patch.<br>
><br>
> I'll make the appropriate revisions and submit a revised patch in a few days<br>
> as I'm about to head off for a short break.<br>
><br>
</div></div>Thanks.<br>
Gedare<br>
<br>
> Cheers,<br>
> James<br>
><br>
</blockquote></div><br></div>