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@xxxxxxxxxx> > 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@xxxxxxxxxxxxxxx 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 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html