Re: [PATCH 1/7] qla2xxx: Cleanup NVMe code.

Madhani, Himanshu
2017-07-19T20:58:38Z

Hi Bart,

> On Jul 19, 2017, at 1:44 PM, Bart Van Assche <Bart.VanAssche@wdc.com> wrote:
>
> On Wed, 2017-07-19 at 11:51 -0700, Himanshu Madhani wrote:
>> This patch cleaned up code to make it more readable
>
> Hello Himanshu,
>
> "cleaned up code" is a very vague description. Please be more specific and
> please also mention in the patch description what functionality has been
> changed (if any).
>
> Bart.

There is no functionality change in this patch, it addresses comments to make code readable.

for example,

Changing waitQ -> wait.
Combined multiple debug statements into single
Drop extra parenthesis in if() statements

I’ll add this into commit message for more clarity.

Thanks,
- Himanshu