From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932899AbbI1LUN (ORCPT ); Mon, 28 Sep 2015 07:20:13 -0400 Received: from mail-pa0-f43.google.com ([209.85.220.43]:33362 "EHLO mail-pa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932560AbbI1LUM (ORCPT ); Mon, 28 Sep 2015 07:20:12 -0400 Date: Mon, 28 Sep 2015 16:48:28 +0530 From: Sudip Mukherjee To: Larry Finger Cc: Punit Vara , florian.c.schilhabel@googlemail.com, gregkh@linuxfoundation.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, dogukan.ergun@gmail.com, stillcompiling@gmail.com, Joe Perches Subject: Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision Message-ID: <20150928111828.GA10335@sudip-pc> References: <1443286161-3003-1-git-send-email-punitvara@gmail.com> <5606D9CA.1040502@lwfinger.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5606D9CA.1040502@lwfinger.net> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Sep 26, 2015 at 12:45:46PM -0500, Larry Finger wrote: > On 09/26/2015 11:49 AM, Punit Vara wrote: > >This patch is to the rtl871x_ioctl_linux.c that fixes up following > >warning reported by checkpatch.pl : > > > >- Comparisons should place the constant on the right side of the test > > > >Signed-off-by: Punit Vara > > This warning is crap. WTF difference does it make???? The compiler > does not care, and any reader with any piece of a brain is not going > to be confused! > > This patch and all others like it are just meaningless source churning! > > This author has made such a royal mess of his patches that I > recommend that ALL of them be dropped. In addition, we should > continue to drop his changes until he learns how to use git to > generate N/M patches, and until he reads the documentation on patch > submission. Excuse me for my ignorance, but I still can not see what was wrong with his patch. checkpatch is giving warning and he has fixed it. As far as sending in series is concerned, he is a newbie and after telling him how to generate patches in series he has learnt that. I have already told him that his patches might be dropped as they are not in series and he is ready to resend in series as soon as Greg confirms that they are dropped. And as long as the driver is in staging there will be source churning, isn't it? If i remember correctly I was told that for a driver to be moved out of staging the primary thing is that all checkpatch warnings needs to fixed. So if this driver has to move out of staging someday then these warnings also has to be fixed by someone. regards sudip