Re: [PATCH 2/2] soundwire: Testing the wrong variable

Dan Carpenter
2018-01-10T12:25:44Z

On Wed, Jan 10, 2018 at 04:25:44PM +0530, Vinod Koul wrote:
> On Tue, Jan 09, 2018 at 12:37:44PM +0300, Dan Carpenter wrote:
> > We should be using "status2" here instead of "status".
>
> This is already fixed by commit:
>
> Author: Wei Yongjun <weiyongjun1@huawei.com>
> Date: Mon Jan 8 22:22:44 2018 +0530
>
> soundwire: Fix typo in return value check of sdw_read()
>
> Fix the typo, 'status' should be instead of 'status2'.
>

Hi Wei,

It looks like you're fixing static checker warnings? Could you CC
kernel-janitors@vger.kernel.org for those fixes? Otherwise we end up
sending duplicates a lot.

Colin King, Julia Lawall, Christophe JAILLET, and I all CC our static
checker fixes to kernel-janitors. I still end up sending duplicates
sometimes... Which is fine.

regards,
dan carpenter