Re: linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533: pointless test ?

2016-08-22 Thread Greg KH
On Mon, Aug 22, 2016 at 03:07:44PM +0100, David Binderman wrote: > Hello there, > > linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533]: > (style) Condition 'pBtMgnt.ExtConfig.HCIExtensionVer>=1' is always > true > > Source code is > > if

Re: linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533: pointless test ?

2016-08-22 Thread Greg KH
On Mon, Aug 22, 2016 at 03:07:44PM +0100, David Binderman wrote: > Hello there, > > linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533]: > (style) Condition 'pBtMgnt.ExtConfig.HCIExtensionVer>=1' is always > true > > Source code is > > if

linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533: pointless test ?

2016-08-22 Thread David Binderman
Hello there, linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533]: (style) Condition 'pBtMgnt.ExtConfig.HCIExtensionVer>=1' is always true Source code is if (pBtMgnt->ExtConfig.HCIExtensionVer < 1) { } else if (pBtMgnt->ExtConfig.HCIExtensionVer

linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533: pointless test ?

2016-08-22 Thread David Binderman
Hello there, linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533]: (style) Condition 'pBtMgnt.ExtConfig.HCIExtensionVer>=1' is always true Source code is if (pBtMgnt->ExtConfig.HCIExtensionVer < 1) { } else if (pBtMgnt->ExtConfig.HCIExtensionVer