* [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons
@ 2016-05-01 15:50 Jacky Boen
2016-05-01 15:50 ` [PATCH 02/11] Staging: drivers: rtl8188eu: fixed typo Jacky Boen
` (9 more replies)
0 siblings, 10 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 07a61b8..c5e5d52 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -62,8 +62,8 @@ static bool HalUsbSetQueuePipeMapping8188EUsb(struct adapter *adapt, u8 NumInPip
_ConfigNormalChipOutEP_8188E(adapt, NumOutPipe);
/* Normal chip with one IN and one OUT doesn't have interrupt IN EP. */
- if (1 == haldata->OutEpNumber) {
- if (1 != NumInPipe)
+ if (haldata->OutEpNumber == 1) {
+ if (NumInPipe != 1)
return result;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 02/11] Staging: drivers: rtl8188eu: fixed typo
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 03/11] Staging: drivers: rtl8188eu: fixed block comments style Jacky Boen
` (8 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed typo
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index c5e5d52..7831f19 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -179,7 +179,7 @@ static void _InitQueueReservedPage(struct adapter *Adapter)
if (haldata->OutEpQueueSel & TX_SELE_LQ)
numLQ = 0x1C;
- /* NOTE: This step shall be proceed before writting REG_RQPN. */
+ /* NOTE: This step shall be proceed before writing REG_RQPN. */
if (haldata->OutEpQueueSel & TX_SELE_NQ)
numNQ = 0x1C;
value8 = (u8)_NPQ(numNQ);
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 03/11] Staging: drivers: rtl8188eu: fixed block comments style
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
2016-05-01 15:50 ` [PATCH 02/11] Staging: drivers: rtl8188eu: fixed typo Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 04/11] Staging: drivers: rtl8188eu: fixed newlines style Jacky Boen
` (7 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 7831f19..507e448 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -457,7 +457,8 @@ static void _InitRetryFunction(struct adapter *Adapter)
* When Who Remark
* 12/10/2010 MHC Separate to smaller function.
*
- *---------------------------------------------------------------------------*/
+ *---------------------------------------------------------------------------
+ */
static void usb_AggSettingTxUpdate(struct adapter *Adapter)
{
struct hal_data_8188e *haldata = GET_HAL_DATA(Adapter);
@@ -489,7 +490,8 @@ static void usb_AggSettingTxUpdate(struct adapter *Adapter)
* When Who Remark
* 12/10/2010 MHC Separate to smaller function.
*
- *---------------------------------------------------------------------------*/
+ *---------------------------------------------------------------------------
+ */
static void
usb_AggSettingRxUpdate(
struct adapter *Adapter
@@ -655,7 +657,8 @@ static void _InitAntenna_Selection(struct adapter *Adapter)
* Revised History:
* When Who Remark
* 08/23/2010 MHC HW suspend mode switch test..
- *---------------------------------------------------------------------------*/
+ *---------------------------------------------------------------------------
+ */
enum rt_rf_power_state RfOnOffDetect(struct adapter *adapt)
{
u8 val8;
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 04/11] Staging: drivers: rtl8188eu: fixed newlines style
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
2016-05-01 15:50 ` [PATCH 02/11] Staging: drivers: rtl8188eu: fixed typo Jacky Boen
2016-05-01 15:50 ` [PATCH 03/11] Staging: drivers: rtl8188eu: fixed block comments style Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 05/11] Staging: drivers: rtl8188eu: fixed brace style Jacky Boen
` (6 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 507e448..58d38d6 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -690,11 +690,9 @@ static u32 rtl8188eu_hal_init(struct adapter *Adapter)
#define HAL_INIT_PROFILE_TAG(stage) do {} while (0)
-
HAL_INIT_PROFILE_TAG(HAL_INIT_STAGES_BEGIN);
if (Adapter->pwrctrlpriv.bkeepfwalive) {
-
if (haldata->odmpriv.RFCalibrateInfo.bIQKInitialized) {
rtl88eu_phy_iq_calibrate(Adapter, true);
} else {
@@ -908,7 +906,6 @@ HAL_INIT_PROFILE_TAG(HAL_INIT_STAGES_END);
DBG_88E("%s in %dms\n", __func__,
jiffies_to_msecs(jiffies - init_start_time));
-
return status;
}
@@ -971,6 +968,7 @@ static void CardDisableRTL8188EU(struct adapter *Adapter)
haldata->bMacPwrCtrlOn = false;
Adapter->bFWReady = false;
}
+
static void rtl8192cu_hw_power_down(struct adapter *adapt)
{
/* 2010/-8/09 MH For power down module, we need to enable register block contrl reg at 0x1c. */
@@ -983,7 +981,6 @@ static void rtl8192cu_hw_power_down(struct adapter *adapt)
static u32 rtl8188eu_hal_deinit(struct adapter *Adapter)
{
-
DBG_88E("==> %s\n", __func__);
usb_write32(Adapter, REG_HIMR_88E, IMR_DISABLED_88E);
@@ -1119,7 +1116,6 @@ readAdapterInfo_8188EU(
Hal_ReadAntennaDiversity88E(adapt, eeprom->efuse_eeprom_data, eeprom->bautoload_fail_flag);
Hal_EfuseParseBoardType88E(adapt, eeprom->efuse_eeprom_data, eeprom->bautoload_fail_flag);
Hal_ReadThermalMeter_88E(adapt, eeprom->efuse_eeprom_data, eeprom->bautoload_fail_flag);
-
}
static void _ReadPROMContent(
@@ -1858,7 +1854,6 @@ static void GetHwReg8188EU(struct adapter *Adapter, u8 variable, u8 *val)
default:
break;
}
-
}
/* */
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 05/11] Staging: drivers: rtl8188eu: fixed brace style
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (2 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 04/11] Staging: drivers: rtl8188eu: fixed newlines style Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 06/11] Staging: drivers: rtl8188eu: fixed unnecessary else branch Jacky Boen
` (5 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 58d38d6..814ecec 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -716,9 +716,8 @@ static u32 rtl8188eu_hal_init(struct adapter *Adapter)
/* Save target channel */
haldata->CurrentChannel = 6;/* default set to 6 */
- if (pwrctrlpriv->reg_rfoff) {
+ if (pwrctrlpriv->reg_rfoff)
pwrctrlpriv->rf_pwrstate = rf_off;
- }
/* 2010/08/09 MH We need to check if we need to turnon or off RF after detecting */
/* HW GPIO pin. Before PHY_RFConfig8192C. */
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 06/11] Staging: drivers: rtl8188eu: fixed unnecessary else branch
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (3 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 05/11] Staging: drivers: rtl8188eu: fixed brace style Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 07/11] Staging: drivers: rtl8188eu: fixed extraneous indentation Jacky Boen
` (4 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 814ecec..c37ba49 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -749,10 +749,9 @@ static u32 rtl8188eu_hal_init(struct adapter *Adapter)
DBG_88E("%s: Download Firmware failed!!\n", __func__);
Adapter->bFWReady = false;
return status;
- } else {
- RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Initializeadapt8192CSdio(): Download Firmware Success!!\n"));
- Adapter->bFWReady = true;
}
+ RT_TRACE(_module_hci_hal_init_c_, _drv_info_, ("Initializeadapt8192CSdio(): Download Firmware Success!!\n"));
+ Adapter->bFWReady = true;
}
rtl8188e_InitializeFirmwareVars(Adapter);
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 07/11] Staging: drivers: rtl8188eu: fixed extraneous indentation
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (4 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 06/11] Staging: drivers: rtl8188eu: fixed unnecessary else branch Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 08/11] Staging: drivers: rtl8188eu: fixed extraneous parentheses Jacky Boen
` (3 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index c37ba49..f5e8ac0 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -877,7 +877,7 @@ HAL_INIT_PROFILE_TAG(HAL_INIT_STAGES_IQK);
/* 2010/08/26 MH Merge from 8192CE. */
if (pwrctrlpriv->rf_pwrstate == rf_on) {
if (haldata->odmpriv.RFCalibrateInfo.bIQKInitialized) {
- rtl88eu_phy_iq_calibrate(Adapter, true);
+ rtl88eu_phy_iq_calibrate(Adapter, true);
} else {
rtl88eu_phy_iq_calibrate(Adapter, false);
haldata->odmpriv.RFCalibrateInfo.bIQKInitialized = true;
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 08/11] Staging: drivers: rtl8188eu: fixed extraneous parentheses
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (5 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 07/11] Staging: drivers: rtl8188eu: fixed extraneous indentation Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 09/11] Staging: drivers: rtl8188eu: fixed extraneous spaces Jacky Boen
` (2 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fix coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 30 ++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index f5e8ac0..a3fffd7 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -1004,7 +1004,7 @@ static unsigned int rtl8188eu_inirp_init(struct adapter *Adapter)
u8 i;
struct recv_buf *precvbuf;
uint status;
- struct recv_priv *precvpriv = &(Adapter->recvpriv);
+ struct recv_priv *precvpriv = &Adapter->recvpriv;
status = _SUCCESS;
@@ -1209,7 +1209,7 @@ static void hw_var_set_opmode(struct adapter *Adapter, u8 variable, u8 *val)
StopTxBeacon(Adapter);
usb_write8(Adapter, REG_BCN_CTRL, 0x19);/* disable atim wnd */
- } else if ((mode == _HW_STATE_ADHOC_)) {
+ } else if (mode == _HW_STATE_ADHOC_) {
ResumeTxBeacon(Adapter);
usb_write8(Adapter, REG_BCN_CTRL, 0x1a);
} else if (mode == _HW_STATE_AP_) {
@@ -1360,7 +1360,7 @@ static void SetHwReg8188EU(struct adapter *Adapter, u8 variable, u8 *val)
{
u64 tsf;
struct mlme_ext_priv *pmlmeext = &Adapter->mlmeextpriv;
- struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+ struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
tsf = pmlmeext->TSFValue - rtw_modular64(pmlmeext->TSFValue, (pmlmeinfo->bcn_interval*1024)) - 1024; /* us */
@@ -1417,7 +1417,7 @@ static void SetHwReg8188EU(struct adapter *Adapter, u8 variable, u8 *val)
usb_write8(Adapter, REG_BCN_CTRL, usb_read8(Adapter, REG_BCN_CTRL) | BIT(4));
} else { /* sitesurvey done */
struct mlme_ext_priv *pmlmeext = &Adapter->mlmeextpriv;
- struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+ struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
if ((is_client_associated_to_ap(Adapter)) ||
((pmlmeinfo->state&0x03) == WIFI_FW_ADHOC_STATE)) {
@@ -1487,7 +1487,7 @@ static void SetHwReg8188EU(struct adapter *Adapter, u8 variable, u8 *val)
{
u8 u1bAIFS, aSifsTime;
struct mlme_ext_priv *pmlmeext = &Adapter->mlmeextpriv;
- struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+ struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
usb_write8(Adapter, REG_SLOT, val[0]);
@@ -1900,19 +1900,19 @@ GetHalDefVar8188EUsb(
case HAL_DEF_RA_DECISION_RATE:
{
u8 MacID = *((u8 *)pValue);
- *((u8 *)pValue) = ODM_RA_GetDecisionRate_8188E(&(haldata->odmpriv), MacID);
+ *((u8 *)pValue) = ODM_RA_GetDecisionRate_8188E(&haldata->odmpriv, MacID);
}
break;
case HAL_DEF_RA_SGI:
{
u8 MacID = *((u8 *)pValue);
- *((u8 *)pValue) = ODM_RA_GetShortGI_8188E(&(haldata->odmpriv), MacID);
+ *((u8 *)pValue) = ODM_RA_GetShortGI_8188E(&haldata->odmpriv, MacID);
}
break;
case HAL_DEF_PT_PWR_STATUS:
{
u8 MacID = *((u8 *)pValue);
- *((u8 *)pValue) = ODM_RA_GetHwPwrStatus_8188E(&(haldata->odmpriv), MacID);
+ *((u8 *)pValue) = ODM_RA_GetHwPwrStatus_8188E(&haldata->odmpriv, MacID);
}
break;
case HW_VAR_MAX_RX_AMPDU_FACTOR:
@@ -1935,7 +1935,7 @@ GetHalDefVar8188EUsb(
break;
case HW_DEF_ODM_DBG_FLAG:
{
- struct odm_dm_struct *dm_ocm = &(haldata->odmpriv);
+ struct odm_dm_struct *dm_ocm = &haldata->odmpriv;
pr_info("dm_ocm->DebugComponents = 0x%llx\n", dm_ocm->DebugComponents);
}
break;
@@ -1963,8 +1963,8 @@ static void UpdateHalRAMask8188EUsb(struct adapter *adapt, u32 mac_id, u8 rssi_l
struct sta_info *psta;
struct hal_data_8188e *haldata = GET_HAL_DATA(adapt);
struct mlme_ext_priv *pmlmeext = &adapt->mlmeextpriv;
- struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
- struct wlan_bssid_ex *cur_network = &(pmlmeinfo->network);
+ struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
+ struct wlan_bssid_ex *cur_network = &pmlmeinfo->network;
if (mac_id >= NUM_STA) /* CAM_SIZE */
return;
@@ -1977,8 +1977,8 @@ static void UpdateHalRAMask8188EUsb(struct adapter *adapt, u32 mac_id, u8 rssi_l
networkType = judge_network_type(adapt, cur_network->SupportedRates, supportRateNum) & 0xf;
raid = networktype_to_raid(networkType);
mask = update_supported_rate(cur_network->SupportedRates, supportRateNum);
- mask |= (pmlmeinfo->HT_enable) ? update_MSC_rate(&(pmlmeinfo->HT_caps)) : 0;
- if (support_short_GI(adapt, &(pmlmeinfo->HT_caps)))
+ mask |= (pmlmeinfo->HT_enable) ? update_MSC_rate(&pmlmeinfo->HT_caps) : 0;
+ if (support_short_GI(adapt, &pmlmeinfo->HT_caps))
shortGIrate = true;
break;
case 1:/* for broadcast/multicast */
@@ -2019,8 +2019,8 @@ static void UpdateHalRAMask8188EUsb(struct adapter *adapt, u32 mac_id, u8 rssi_l
static void SetBeaconRelatedRegisters8188EUsb(struct adapter *adapt)
{
u32 value32;
- struct mlme_ext_priv *pmlmeext = &(adapt->mlmeextpriv);
- struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+ struct mlme_ext_priv *pmlmeext = &adapt->mlmeextpriv;
+ struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
u32 bcn_ctrl_reg = REG_BCN_CTRL;
/* reset TSF, enable update TSF, correcting TSF On Beacon */
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 09/11] Staging: drivers: rtl8188eu: fixed extraneous spaces
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (6 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 08/11] Staging: drivers: rtl8188eu: fixed extraneous parentheses Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 10/11] Staging: drivers: rtl8188eu: fixed if-NULL comparisons style Jacky Boen
2016-05-01 15:50 ` [PATCH 11/11] Staging: drivers: rtl8188eu: use sizeof(*ptr) instead of sizeof(struct) Jacky Boen
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fix coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index a3fffd7..82269eb 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -997,7 +997,7 @@ static u32 rtl8188eu_hal_deinit(struct adapter *Adapter)
}
}
return _SUCCESS;
- }
+}
static unsigned int rtl8188eu_inirp_init(struct adapter *Adapter)
{
@@ -1787,7 +1787,7 @@ static void SetHwReg8188EU(struct adapter *Adapter, u8 variable, u8 *val)
}
break;
case HW_VAR_H2C_MEDIA_STATUS_RPT:
- rtl8188e_set_FwMediaStatus_cmd(Adapter , (*(__le16 *)val));
+ rtl8188e_set_FwMediaStatus_cmd(Adapter, (*(__le16 *)val));
break;
case HW_VAR_BCN_VALID:
/* BCN_VALID, BIT16 of REG_TDECTRL = BIT0 of REG_TDECTRL+2, write 1 to clear, Clear by sw */
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 10/11] Staging: drivers: rtl8188eu: fixed if-NULL comparisons style
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (7 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 09/11] Staging: drivers: rtl8188eu: fixed extraneous spaces Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
2016-05-01 15:50 ` [PATCH 11/11] Staging: drivers: rtl8188eu: use sizeof(*ptr) instead of sizeof(struct) Jacky Boen
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fixed coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index 82269eb..ef04638 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -1969,7 +1969,7 @@ static void UpdateHalRAMask8188EUsb(struct adapter *adapt, u32 mac_id, u8 rssi_l
if (mac_id >= NUM_STA) /* CAM_SIZE */
return;
psta = pmlmeinfo->FW_sta_info[mac_id].psta;
- if (psta == NULL)
+ if (!psta)
return;
switch (mac_id) {
case 0:/* for infra mode */
@@ -2079,7 +2079,7 @@ void rtl8188eu_set_hal_ops(struct adapter *adapt)
adapt->HalData = kzalloc(sizeof(struct hal_data_8188e), GFP_KERNEL);
- if (adapt->HalData == NULL)
+ if (!adapt->HalData)
DBG_88E("cant not alloc memory for HAL DATA\n");
halfunc->hal_power_on = rtl8188eu_InitPowerOn;
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 11/11] Staging: drivers: rtl8188eu: use sizeof(*ptr) instead of sizeof(struct)
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
` (8 preceding siblings ...)
2016-05-01 15:50 ` [PATCH 10/11] Staging: drivers: rtl8188eu: fixed if-NULL comparisons style Jacky Boen
@ 2016-05-01 15:50 ` Jacky Boen
9 siblings, 0 replies; 11+ messages in thread
From: Jacky Boen @ 2016-05-01 15:50 UTC (permalink / raw)
To: gregkh; +Cc: devel, linux-kernel, Jacky Boen
Fix coding style issue
Signed-off-by: Jacky Boen <aqiank@gmail.com>
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
index ef04638..13c1277 100644
--- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
+++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
@@ -2077,8 +2077,7 @@ void rtl8188eu_set_hal_ops(struct adapter *adapt)
{
struct hal_ops *halfunc = &adapt->HalFunc;
-
- adapt->HalData = kzalloc(sizeof(struct hal_data_8188e), GFP_KERNEL);
+ adapt->HalData = kzalloc(sizeof(*adapt->HalData), GFP_KERNEL);
if (!adapt->HalData)
DBG_88E("cant not alloc memory for HAL DATA\n");
--
2.7.4
^ permalink raw reply related [flat|nested] 11+ messages in thread
end of thread, other threads:[~2016-05-01 15:53 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-01 15:50 [PATCH 01/11] Staging: drivers: rtl8188eu: move constants to the right in comparisons Jacky Boen
2016-05-01 15:50 ` [PATCH 02/11] Staging: drivers: rtl8188eu: fixed typo Jacky Boen
2016-05-01 15:50 ` [PATCH 03/11] Staging: drivers: rtl8188eu: fixed block comments style Jacky Boen
2016-05-01 15:50 ` [PATCH 04/11] Staging: drivers: rtl8188eu: fixed newlines style Jacky Boen
2016-05-01 15:50 ` [PATCH 05/11] Staging: drivers: rtl8188eu: fixed brace style Jacky Boen
2016-05-01 15:50 ` [PATCH 06/11] Staging: drivers: rtl8188eu: fixed unnecessary else branch Jacky Boen
2016-05-01 15:50 ` [PATCH 07/11] Staging: drivers: rtl8188eu: fixed extraneous indentation Jacky Boen
2016-05-01 15:50 ` [PATCH 08/11] Staging: drivers: rtl8188eu: fixed extraneous parentheses Jacky Boen
2016-05-01 15:50 ` [PATCH 09/11] Staging: drivers: rtl8188eu: fixed extraneous spaces Jacky Boen
2016-05-01 15:50 ` [PATCH 10/11] Staging: drivers: rtl8188eu: fixed if-NULL comparisons style Jacky Boen
2016-05-01 15:50 ` [PATCH 11/11] Staging: drivers: rtl8188eu: use sizeof(*ptr) instead of sizeof(struct) Jacky Boen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).