Re: [PATCH] Staging: rtl8712: rtl871x_ioctl_linux.c Move constant to right side of the comparision

[Joshua Clayton <stillcompiling@gmail.com>]

On Saturday, September 26, 2015 11:24:06 PM punit vara wrote:
> On Sat, Sep 26, 2015 at 11:15 PM, Larry Finger
> <Larry.Finger@lwfinger.net> 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 <punitvara@gmail.com>
> >
> >
> > 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.
> >
> > Larry
> >
> >
> >> ---
> >>   drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 16 ++++++++--------
> >>   1 file changed, 8 insertions(+), 8 deletions(-)
> >>
> >> diff --git a/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
> >> b/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
> >> index 143be0f..6d7f8f7 100644
> >> --- a/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
> >> +++ b/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
> >> @@ -957,7 +957,7 @@ static int r871x_wx_set_priv(struct net_device *dev,
> >>         if (IS_ERR(ext))
> >>                 return PTR_ERR(ext);
> >>
> >> -       if (0 == strcasecmp(ext, "RSSI")) {
> >> +       if (strcasecmp(ext, "RSSI") == 0) {
> >>                 /*Return received signal strength indicator in -db for */
> >>                 /* current AP */
> >>                 /*<ssid> Rssi xx */
> >> @@ -974,7 +974,7 @@ static int r871x_wx_set_priv(struct net_device *dev,
> >>                 } else {
> >>                         sprintf(ext, "OK");
> >>                 }
> >> -       } else if (0 == strcasecmp(ext, "LINKSPEED")) {
> >> +       } else if (strcasecmp(ext, "LINKSPEED") == 0) {
> >>                 /*Return link speed in MBPS */
> >>                 /*LinkSpeed xx */
> >>                 union iwreq_data wrqd;
> >> @@ -982,30 +982,30 @@ static int r871x_wx_set_priv(struct net_device *dev,
> >>                 int mbps;
> >>
> >>                 ret_inner = r8711_wx_get_rate(dev, info, &wrqd, extra);
> >> -               if (0 != ret_inner)
> >> +               if (ret_inner != 0)
> >>                         mbps = 0;
> >>                 else
> >>                         mbps = wrqd.bitrate.value / 1000000;
> >>                 sprintf(ext, "LINKSPEED %d", mbps);
> >> -       } else if (0 == strcasecmp(ext, "MACADDR")) {
> >> +       } else if (strcasecmp(ext, "MACADDR") == 0) {
> >>                 /*Return mac address of the station */
> >>                 /* Macaddr = xx:xx:xx:xx:xx:xx */
> >>                 sprintf(ext, "MACADDR = %pM", dev->dev_addr);
> >> -       } else if (0 == strcasecmp(ext, "SCAN-ACTIVE")) {
> >> +       } else if (strcasecmp(ext, "SCAN-ACTIVE") == 0) {
> >>                 /*Set scan type to active */
> >>                 /*OK if successful */
> >>                 struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
> >>
> >>                 pmlmepriv->passive_mode = 1;
> >>                 sprintf(ext, "OK");
> >> -       } else if (0 == strcasecmp(ext, "SCAN-PASSIVE")) {
> >> +       } else if (strcasecmp(ext, "SCAN-PASSIVE") == 0) {
> >>                 /*Set scan type to passive */
> >>                 /*OK if successful */
> >>                 struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
> >>
> >>                 pmlmepriv->passive_mode = 0;
> >>                 sprintf(ext, "OK");
> >> -       } else if (0 == strncmp(ext, "DCE-E", 5)) {
> >> +       } else if (strncmp(ext, "DCE-E", 5) == 0) {
> >>                 /*Set scan type to passive */
> >>                 /*OK if successful */
> >>                 r8712_disconnectCtrlEx_cmd(padapter
> >> @@ -1015,7 +1015,7 @@ static int r871x_wx_set_priv(struct net_device *dev,
> >>                         , 5000 /*u32 firstStageTO */
> >>                 );
> >>                 sprintf(ext, "OK");
> >> -       } else if (0 == strncmp(ext, "DCE-D", 5)) {
> >> +       } else if (strncmp(ext, "DCE-D", 5) == 0) {
> >>                 /*Set scan type to passive */
> >>                 /*OK if successfu */
> >>                 r8712_disconnectCtrlEx_cmd(padapter
> >>
> >
> 
> Sorry I have disappointed you .I learned now how to generate N/M
> patches .I have recently sent my last 6 patches in that way only.   I
> am extremely sorry for previous patches as I could not able to send in
> series. Please let me allow to send me those patches again in correct
> way. If you would say I will definitely send them in series.Can I send
> all the patches in series for rtl8712 directory if you do not mind ?
I have not seen your other patches, but I agree with Larry. This is a formatting patch
that  does not improve the readability of the code. Should be dropped.
This particular checkpatch warning might be useful in expressing a preference for new code, I guess.