Re: linux-4.1-rc1/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c:424: bad if test ?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 27, 2015 at 08:33:30AM +0000, David Binderman wrote:
> Hello there Greg,
> 
> Static analyser cppcheck says
> 
>  [linux-4.1-rc1/drivers/staging/rtl8188eu/hal/Hal8188ERateAdaptive.c:424]: (style) Expression is always false because 'else if' condition matches previous condition at line 422.
> 
> Source code is
> 
>       else if (pRaInfo->HighestRate> 0x0b)
>             pRaInfo->PTModeSS = 2;
>         else if (pRaInfo->HighestRate> 0x0b)
>             pRaInfo->PTModeSS = 1;
> 
> 
> Regards
> 
> David Binderman

Patches always gladly accepted :)

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel





[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux