<div dir="ltr"><div>Thanks Julia, I will update commit log message for patch and add version for patch. I will also add series number if there are multiple changes on the same file.<br><br></div>Bhagyashri<br><div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 5, 2018 at 1:36 AM 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, Bhagyashri P. Dighole wrote:<br>
<br>
> This patch includes modified condition to check for bio struct.<br>
<br>
The commit message is sort of wordy and imprecise.  Don't say "This patch<br>
includes".  The commit log message should be in the imperative.  That is,<br>
it should start with a verb.  Here, the first word could perhaps be<br>
Modify.  Afterwards, you should explain what was done and why.  Here you<br>
could say "Use ! for a NULL test rather than explicitly comparing to NULL,<br>
to be more concise and readable".  Then, if you came across this issue due<br>
to checkpatch, it would be nice to acknowledge that - Issue found using<br>
checkpatch.<br>
<br>
Finally, if you are going to send the other changes then they all need to<br>
be in a series.  On the other hand, for your first patch, it is fine to<br>
only send this change.  But in that case you have to work on something<br>
else until Greg picks this up.  If there are multiple changes on one file,<br>
he needs the series number to know in what order to apply them.<br>
<br>
julia<br>
<br>
><br>
> Signed-off-by: Bhagyashri P. Dighole <<a href="mailto:digholebhagyashri@gmail.com" target="_blank">digholebhagyashri@gmail.com</a>><br>
> ---<br>
>  drivers/staging/erofs/data.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/staging/erofs/data.c b/drivers/staging/erofs/data.c<br>
> index 6384f73..a671ad4 100644<br>
> --- a/drivers/staging/erofs/data.c<br>
> +++ b/drivers/staging/erofs/data.c<br>
> @@ -240,7 +240,7 @@ static inline struct bio *erofs_read_raw_page(<br>
>               bio = NULL;<br>
>       }<br>
><br>
> -     if (bio == NULL) {<br>
> +     if (!bio) {<br>
>               struct erofs_map_blocks map = {<br>
>                       .m_la = blknr_to_addr(current_block),<br>
>               };<br>
> --<br>
> 2.7.4<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/20181104195701.GA18201%40bhagyashri-Lenovo-G570" rel="noreferrer" target="_blank">https://groups.google.com/d/msgid/outreachy-kernel/20181104195701.GA18201%40bhagyashri-Lenovo-G570</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></div></div>