staging: wlan-ng remove redundant conditional
authorTillmann Heidsieck <theidsieck@leenox.de>
Wed, 23 Sep 2015 20:07:54 +0000 (22:07 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 29 Sep 2015 02:19:27 +0000 (04:19 +0200)
We exit the above loop either if curroff >= (HFA384x_PDA_LEN_MAX / 2 - 1)
or if we found the END marker in the element beyond the current one. The
first case is checked for in the preceding if statement, therefore the
second if statement is redundant and can be removed.

Signed-off-by: Tillmann Heidsieck <theidsieck@leenox.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wlan-ng/prism2fw.c

index d357b7ecb1fcdc88773e62b9e1d0964f4cabf819..f5b97273164de5a7f5422e37f81ca47719052e4b 100644 (file)
@@ -631,10 +631,8 @@ static int mkpdrlist(struct pda *pda)
                       curroff, pda->nrec);
                return 1;
        }
-       if (le16_to_cpu(pda16[curroff + 1]) == HFA384x_PDR_END_OF_PDA) {
-               pda->rec[pda->nrec] = (hfa384x_pdrec_t *) &(pda16[curroff]);
-               (pda->nrec)++;
-       }
+       pda->rec[pda->nrec] = (hfa384x_pdrec_t *) &(pda16[curroff]);
+       (pda->nrec)++;
        return result;
 }