<div dir="ltr"><div dir="ltr"><div><div><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">Hi Gao,</span><br style="color:rgb(49,49,49);word-spacing:1px"><br style="color:rgb(49,49,49);word-spacing:1px"><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">In
 PATCH version v1 which is first stage of contribution, I modified 
data.c for all alignment changes and all conditional checks.</span><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(253,253,253)"> </span><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">As per discussion on email for this patch, It was concluded that I need to re-submit patch for single change at a time.</span></div><div dir="auto"><span style="background-color:rgb(255,255,255)"><br style="color:rgb(49,49,49);word-spacing:1px"></span><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">So,
 I created PATCH v2. In this patch, I modified data.c only for single 
conditional check and single alignment change and committed accordingly. 
As per Gao’s suggestion to do all alignment change in a first patch and 
all conditional changes in another patch, I modified and created PATCH 
v3.</span></div><div dir="auto"><br style="color:rgb(49,49,49);word-spacing:1px"><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">PATCH v3 is divided into two patches.</span></div><div dir="auto"><span style="background-color:rgb(255,255,255)">1) PATCH v3 1/2 has all alignment changes.</span></div><div dir="auto"><span style="background-color:rgb(255,255,255)">2) PATCH v3 2/2 has all conditional changes.</span></div></div><div dir="auto"><br></div><div dir="auto"><font color="#313131"><span style="word-spacing:1px">Also, I mentioned what changes each version of patch represents under ‘—-‘ symbol.</span></font></div><div dir="auto"><br style="color:rgb(49,49,49);word-spacing:1px"><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">I am not able to understand, where I am making mistake? </span><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">Can you please help me?</span></div><div dir="auto"><br></div><br style="color:rgb(49,49,49);word-spacing:1px"><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)"><a tabindex="-1" href="mailto:julia.lawall@lip6.fr" id="plusReplyChip-0" class="gmail_plusreply">@Julia Lawall</a>  How can I include Reviewed-by in patch submission? we are allow to do 
that? If yes, then from where I can get that information?</span><br style="color:rgb(49,49,49);word-spacing:1px"><br style="color:rgb(49,49,49);word-spacing:1px"><span style="color:rgb(49,49,49);word-spacing:1px;background-color:rgb(255,255,255)">Regards</span></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 5, 2018 at 9:21 PM Julia Lawall <<a href="mailto:julia.lawall@lip6.fr">julia.lawall@lip6.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On Mon, 5 Nov 2018, Gao Xiang wrote:<br>
<br>
> Hi Bhagyashri,<br>
><br>
> On 2018/11/5 22:50, Bhagyashri P. Dighole wrote:<br>
> > Fix coding style alignment issues detected by <a href="http://checkpatch.pl" rel="noreferrer" target="_blank">checkpatch.pl</a><br>
> > Use ! for NULL test rather than explicitly comparing to NULL.<br>
> ><br>
> > Signed-off-by: Bhagyashri P. Dighole <<a href="mailto:digholebhagyashri@gmail.com" target="_blank">digholebhagyashri@gmail.com</a>><br>
> > ---<br>
> > Changes in v3:<br>
> > - Split patch in two patches. Each patch have similar<br>
> >   kind of changes. One patch for all alignment fixes and<br>
> >     second patch for all conditional check fixes.<br>
> ><br>
> > Changes in v2:<br>
> > - Fix for one alignment issue and one conditional check issue.<br>
> ><br>
> > Changes in v1:<br>
> > - Fix for all issues in data.c<br>
><br>
> As the Greg said before, there are too many patchset v3... :(<br>
> Apart from its content, could you please resend the whole patchset<br>
> by using a proper version number...<br>
><br>
> It is really confusing for me as well...<br>
<br>
Bhagyashri,<br>
<br>
When you resend, you should include the Reviewed-by that you received.<br>
<br>
julia<br>
<br>
<br>
><br>
> Thanks,<br>
> Gao Xiang<br>
><br>
> --<br>
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.<br>
> To unsubscribe from this group and stop receiving emails from it, send an email to <a href="mailto:outreachy-kernel%2Bunsubscribe@googlegroups.com" target="_blank">outreachy-kernel+unsubscribe@googlegroups.com</a>.<br>
> To post to this group, send email to <a href="mailto:outreachy-kernel@googlegroups.com" target="_blank">outreachy-kernel@googlegroups.com</a>.<br>
> To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/outreachy-kernel/3ac63350-6bfc-0dba-2649-ad7f91dd94c9%40huawei.com" rel="noreferrer" target="_blank">https://groups.google.com/d/msgid/outreachy-kernel/3ac63350-6bfc-0dba-2649-ad7f91dd94c9%40huawei.com</a>.<br>
> For more options, visit <a href="https://groups.google.com/d/optout" rel="noreferrer" target="_blank">https://groups.google.com/d/optout</a>.<br>
><br>
</blockquote></div>