Re: [PATCH] staging: rtl8188eu: cleanup long line in odm.c
On 2020-03-26 01:08, Joe Perches wrote: On Wed, 2020-03-25 at 22:59 +0100, Michael Straube wrote: Cleanup line over 80 characters by removing unnecessary parentheses. [] diff --git a/drivers/staging/rtl8188eu/hal/odm.c b/drivers/staging/rtl8188eu/hal/odm.c [] @@ -590,7 +590,7 @@ void odm_CCKPacketDetectionThresh(struct odm_dm_struct *pDM_Odm) if (pDM_Odm->bLinked) { if (pDM_Odm->RSSI_Min > 25) { CurCCK_CCAThres = 0xcd; - } else if ((pDM_Odm->RSSI_Min <= 25) && (pDM_Odm->RSSI_Min > 10)) { + } else if (pDM_Odm->RSSI_Min <= 25 && pDM_Odm->RSSI_Min > 10) { The test above this already guarantees pDM_Odm->RSSI_Min <= 25 so the block could be written just } else if (pDM->RSSI_Min > 10) { Ah, yes. Thank you. :) ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: cleanup long line in odm.c
On Wed, 2020-03-25 at 22:59 +0100, Michael Straube wrote: > Cleanup line over 80 characters by removing unnecessary parentheses. [] > diff --git a/drivers/staging/rtl8188eu/hal/odm.c > b/drivers/staging/rtl8188eu/hal/odm.c [] > @@ -590,7 +590,7 @@ void odm_CCKPacketDetectionThresh(struct odm_dm_struct > *pDM_Odm) > if (pDM_Odm->bLinked) { > if (pDM_Odm->RSSI_Min > 25) { > CurCCK_CCAThres = 0xcd; > - } else if ((pDM_Odm->RSSI_Min <= 25) && (pDM_Odm->RSSI_Min > > 10)) { > + } else if (pDM_Odm->RSSI_Min <= 25 && pDM_Odm->RSSI_Min > 10) { The test above this already guarantees pDM_Odm->RSSI_Min <= 25 so the block could be written just } else if (pDM->RSSI_Min > 10) { > CurCCK_CCAThres = 0x83; > } else { > if (FalseAlmCnt->Cnt_Cck_fail > 1000) ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH] staging: rtl8188eu: cleanup long line in odm.c
Cleanup line over 80 characters by removing unnecessary parentheses. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/hal/odm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/hal/odm.c b/drivers/staging/rtl8188eu/hal/odm.c index a6eb9798b6f8..ed1a6ea0eecf 100644 --- a/drivers/staging/rtl8188eu/hal/odm.c +++ b/drivers/staging/rtl8188eu/hal/odm.c @@ -590,7 +590,7 @@ void odm_CCKPacketDetectionThresh(struct odm_dm_struct *pDM_Odm) if (pDM_Odm->bLinked) { if (pDM_Odm->RSSI_Min > 25) { CurCCK_CCAThres = 0xcd; - } else if ((pDM_Odm->RSSI_Min <= 25) && (pDM_Odm->RSSI_Min > 10)) { + } else if (pDM_Odm->RSSI_Min <= 25 && pDM_Odm->RSSI_Min > 10) { CurCCK_CCAThres = 0x83; } else { if (FalseAlmCnt->Cnt_Cck_fail > 1000) -- 2.25.1 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH] staging: rtl8188eu: cleanup long line in odm.c
Align a comment to clear a line over 80 characters checkpatch warning. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/hal/odm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/hal/odm.c b/drivers/staging/rtl8188eu/hal/odm.c index ba3c3e5a8216..95a8fc23e62c 100644 --- a/drivers/staging/rtl8188eu/hal/odm.c +++ b/drivers/staging/rtl8188eu/hal/odm.c @@ -107,7 +107,7 @@ u8 CCKSwingTable_Ch1_Ch13[CCK_TABLE_SIZE][8] = { {0x0a, 0x0a, 0x09, 0x07, 0x05, 0x03, 0x02, 0x01}, /* 29, -14.5dB */ {0x0a, 0x09, 0x08, 0x07, 0x05, 0x03, 0x02, 0x01}, /* 30, -15.0dB */ {0x09, 0x09, 0x08, 0x06, 0x05, 0x03, 0x01, 0x01}, /* 31, -15.5dB */ - {0x09, 0x08, 0x07, 0x06, 0x04, 0x03, 0x01, 0x01}/* 32, -16.0dB */ + {0x09, 0x08, 0x07, 0x06, 0x04, 0x03, 0x01, 0x01} /* 32, -16.0dB */ }; u8 CCKSwingTable_Ch14[CCK_TABLE_SIZE][8] = { -- 2.21.0 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel