<div dir="auto"><div>Thanks Gao. As per discussion in previous mails, I thought we need to submit one change at a time in a patch that's why I modified accordingly. As you suggested it makes sense to fix all similar issues at once and create a single patch for those issues. I will update patch accordingly.</div><div dir="auto"><br></div><div dir="auto"> Thanks </div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Mon, Nov 5, 2018, 10:58 AM Gao Xiang <<a href="mailto:gaoxiang25@huawei.com">gaoxiang25@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Bhagyashri,<br>
<br>
On 2018/11/5 13:13, Bhagyashri P. Dighole wrote:<br>
> Found issue using checkpatch.<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" rel="noreferrer">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>
<br>
Could you please fix pointer comparisons to NULL all in once?<br>
at least for a specific file (eg. for the staging/erofs/data.c only)?<br>
<br>
It is somewhat weird to just fix one place only of data.c +243 as above,<br>
this is my personal suggestion...<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" rel="noreferrer">outreachy-kernel+unsubscribe@googlegroups.com</a>.<br>
To post to this group, send email to <a href="mailto:outreachy-kernel@googlegroups.com" target="_blank" rel="noreferrer">outreachy-kernel@googlegroups.com</a>.<br>
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/outreachy-kernel/5d7fc290-08ad-0c83-d030-de255de81c47%40huawei.com" rel="noreferrer noreferrer" target="_blank">https://groups.google.com/d/msgid/outreachy-kernel/5d7fc290-08ad-0c83-d030-de255de81c47%40huawei.com</a>.<br>
For more options, visit <a href="https://groups.google.com/d/optout" rel="noreferrer noreferrer" target="_blank">https://groups.google.com/d/optout</a>.<br>
</blockquote></div></div></div>