* [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
@ 2021-04-08 8:07 Fabio M. De Francesco
2021-04-08 8:36 ` Greg KH
2021-04-08 8:53 ` Julia Lawall
0 siblings, 2 replies; 7+ messages in thread
From: Fabio M. De Francesco @ 2021-04-08 8:07 UTC (permalink / raw)
To: gregkh, outreachy-kernel, linux-staging, linux-kernel
Cc: Fabio M. De Francesco
Remove camelcase in a symbol that is used by several files.
Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
---
drivers/staging/rtl8723bs/core/rtw_cmd.c | 2 +-
drivers/staging/rtl8723bs/core/rtw_mlme.c | 2 +-
drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 18 +++++++++---------
drivers/staging/rtl8723bs/hal/hal_intf.c | 2 +-
drivers/staging/rtl8723bs/hal/rtl8723b_dm.c | 6 +++---
.../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 2 +-
drivers/staging/rtl8723bs/hal/sdio_ops.c | 14 +++++++-------
.../staging/rtl8723bs/include/rtw_pwrctrl.h | 2 +-
8 files changed, 24 insertions(+), 24 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl8723bs/core/rtw_cmd.c
index baf8b1e0f43c..a08f22b53592 100644
--- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
+++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
@@ -1510,7 +1510,7 @@ static void rtw_lps_change_dtim_hdl(struct adapter *padapter, u8 dtim)
if (pwrpriv->dtim != dtim)
pwrpriv->dtim = dtim;
- if ((pwrpriv->bFwCurrentInPSMode == true) && (pwrpriv->pwr_mode > PS_MODE_ACTIVE)) {
+ if ((pwrpriv->b_fw_current_in_ps_mode == true) && (pwrpriv->pwr_mode > PS_MODE_ACTIVE)) {
u8 ps_mode = pwrpriv->pwr_mode;
rtw_hal_set_hwreg(padapter, HW_VAR_H2C_FW_PWRMODE, (u8 *)(&ps_mode));
diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
index a7e40aaae2d9..51cea6cf46e7 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
@@ -1684,7 +1684,7 @@ void rtw_dynamic_check_timer_handler(struct adapter *adapter)
if (adapter->net_closed)
return;
- if ((adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ if ((adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
&& !(hal_btcoex_IsBtControlLps(adapter))
) {
u8 bEnterPS;
diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
index f7465cf90c46..21e7a847866f 100644
--- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
+++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
@@ -365,7 +365,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode, u8 smart_ps, u8 bcn_a
rtw_set_rpwm(padapter, PS_STATE_S4);
rtw_hal_set_hwreg(padapter, HW_VAR_H2C_FW_PWRMODE, (u8 *)(&ps_mode));
- pwrpriv->bFwCurrentInPSMode = false;
+ pwrpriv->b_fw_current_in_ps_mode = false;
hal_btcoex_LpsNotify(padapter, ps_mode);
}
@@ -377,7 +377,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode, u8 smart_ps, u8 bcn_a
hal_btcoex_LpsNotify(padapter, ps_mode);
- pwrpriv->bFwCurrentInPSMode = true;
+ pwrpriv->b_fw_current_in_ps_mode = true;
pwrpriv->pwr_mode = ps_mode;
pwrpriv->smart_ps = smart_ps;
pwrpriv->bcn_ant_mode = bcn_ant_mode;
@@ -734,7 +734,7 @@ s32 rtw_register_task_alive(struct adapter *padapter, u32 task)
register_task_alive(pwrctrl, task);
- if (pwrctrl->bFwCurrentInPSMode) {
+ if (pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm < pslv) {
if (pwrctrl->cpwm < PS_STATE_S2)
res = _FAIL;
@@ -782,7 +782,7 @@ void rtw_unregister_task_alive(struct adapter *padapter, u32 task)
unregister_task_alive(pwrctrl, task);
- if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
+ if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm > pslv)
if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
rtw_set_rpwm(padapter, pslv);
@@ -819,7 +819,7 @@ s32 rtw_register_tx_alive(struct adapter *padapter)
register_task_alive(pwrctrl, XMIT_ALIVE);
- if (pwrctrl->bFwCurrentInPSMode) {
+ if (pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm < pslv) {
if (pwrctrl->cpwm < PS_STATE_S2)
res = _FAIL;
@@ -864,7 +864,7 @@ s32 rtw_register_cmd_alive(struct adapter *padapter)
register_task_alive(pwrctrl, CMD_ALIVE);
- if (pwrctrl->bFwCurrentInPSMode) {
+ if (pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm < pslv) {
if (pwrctrl->cpwm < PS_STATE_S2)
res = _FAIL;
@@ -909,7 +909,7 @@ void rtw_unregister_tx_alive(struct adapter *padapter)
unregister_task_alive(pwrctrl, XMIT_ALIVE);
- if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
+ if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm > pslv)
if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
rtw_set_rpwm(padapter, pslv);
@@ -945,7 +945,7 @@ void rtw_unregister_cmd_alive(struct adapter *padapter)
unregister_task_alive(pwrctrl, CMD_ALIVE);
- if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
+ if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
if (pwrctrl->cpwm > pslv) {
if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
rtw_set_rpwm(padapter, pslv);
@@ -978,7 +978,7 @@ void rtw_init_pwrctrl_priv(struct adapter *padapter)
pwrctrlpriv->power_mgnt = padapter->registrypriv.power_mgnt;/* PS_MODE_MIN; */
pwrctrlpriv->bLeisurePs = pwrctrlpriv->power_mgnt != PS_MODE_ACTIVE;
- pwrctrlpriv->bFwCurrentInPSMode = false;
+ pwrctrlpriv->b_fw_current_in_ps_mode = false;
pwrctrlpriv->rpwm = 0;
pwrctrlpriv->cpwm = PS_STATE_S4;
diff --git a/drivers/staging/rtl8723bs/hal/hal_intf.c b/drivers/staging/rtl8723bs/hal/hal_intf.c
index bc9ae2088754..a73c2f76628d 100644
--- a/drivers/staging/rtl8723bs/hal/hal_intf.c
+++ b/drivers/staging/rtl8723bs/hal/hal_intf.c
@@ -348,7 +348,7 @@ void rtw_hal_dm_watchdog(struct adapter *padapter)
void rtw_hal_dm_watchdog_in_lps(struct adapter *padapter)
{
- if (adapter_to_pwrctl(padapter)->bFwCurrentInPSMode == true) {
+ if (adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode == true) {
if (padapter->HalFunc.hal_dm_watchdog_in_lps)
padapter->HalFunc.hal_dm_watchdog_in_lps(padapter); /* this function caller is in interrupt context */
}
diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
index c2e9e4a0be22..265db187b8d5 100644
--- a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
+++ b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
@@ -141,7 +141,7 @@ void rtl8723b_InitHalDm(struct adapter *Adapter)
void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
{
- bool bFwCurrentInPSMode = false;
+ bool b_fw_current_in_ps_mode = false;
bool bFwPSAwake = true;
u8 hw_init_completed = false;
struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
@@ -151,12 +151,12 @@ void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
if (hw_init_completed == false)
goto skip_dm;
- bFwCurrentInPSMode = adapter_to_pwrctl(Adapter)->bFwCurrentInPSMode;
+ b_fw_current_in_ps_mode = adapter_to_pwrctl(Adapter)->b_fw_current_in_ps_mode;
rtw_hal_get_hwreg(Adapter, HW_VAR_FWLPS_RF_ON, (u8 *)(&bFwPSAwake));
if (
(hw_init_completed == true) &&
- ((!bFwCurrentInPSMode) && bFwPSAwake)
+ ((!b_fw_current_in_ps_mode) && bFwPSAwake)
) {
/* */
/* Calculate Tx/Rx statistics. */
diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
index e31ad3feed62..7ebc438870fd 100644
--- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
+++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
@@ -430,7 +430,7 @@ void rtl8723b_InitializeFirmwareVars(struct adapter *padapter)
struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
/* Init Fw LPS related. */
- adapter_to_pwrctl(padapter)->bFwCurrentInPSMode = false;
+ adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode = false;
/* Init H2C cmd. */
rtw_write8(padapter, REG_HMETFR, 0x0f);
diff --git a/drivers/staging/rtl8723bs/hal/sdio_ops.c b/drivers/staging/rtl8723bs/hal/sdio_ops.c
index af7f846f90fe..c78a6724fc1c 100644
--- a/drivers/staging/rtl8723bs/hal/sdio_ops.c
+++ b/drivers/staging/rtl8723bs/hal/sdio_ops.c
@@ -173,7 +173,7 @@ static u32 sdio_read32(struct intf_hdl *intfhdl, u32 addr)
if (
((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
) {
err = sd_cmd52_read(intfhdl, ftaddr, 4, (u8 *)&le_tmp);
#ifdef SDIO_DEBUG_IO
@@ -230,7 +230,7 @@ static s32 sdio_readN(struct intf_hdl *intfhdl, u32 addr, u32 cnt, u8 *buf)
if (
((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
)
return sd_cmd52_read(intfhdl, ftaddr, cnt, buf);
@@ -297,7 +297,7 @@ static s32 sdio_write32(struct intf_hdl *intfhdl, u32 addr, u32 val)
if (
((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
) {
le_tmp = cpu_to_le32(val);
@@ -334,7 +334,7 @@ static s32 sdio_writeN(struct intf_hdl *intfhdl, u32 addr, u32 cnt, u8 *buf)
if (
((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
)
return sd_cmd52_write(intfhdl, ftaddr, cnt, buf);
@@ -565,7 +565,7 @@ s32 sdio_local_read(
rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
if (
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
)
return sd_cmd52_read(intfhdl, addr, cnt, buf);
@@ -611,7 +611,7 @@ s32 sdio_local_write(
rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
if (
(!mac_pwr_ctrl_on) ||
- (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
+ (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
)
return sd_cmd52_write(intfhdl, addr, cnt, buf);
@@ -660,7 +660,7 @@ static u32 sdio_local_cmd53_read4byte(struct adapter *adapter, u32 addr)
hal_sdio_get_cmd_addr_8723b(adapter, SDIO_LOCAL_DEVICE_ID, addr, &addr);
rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
- if (!mac_pwr_ctrl_on || adapter_to_pwrctl(adapter)->bFwCurrentInPSMode) {
+ if (!mac_pwr_ctrl_on || adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode) {
sd_cmd52_read(intfhdl, addr, 4, (u8 *)&le_tmp);
val = le32_to_cpu(le_tmp);
} else {
diff --git a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
index 2e739a17dd95..fcb06a95fdf6 100644
--- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
+++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
@@ -203,7 +203,7 @@ struct pwrctrl_priv {
u8 LpsIdleCount;
u8 power_mgnt;
u8 org_power_mgnt;
- u8 bFwCurrentInPSMode;
+ u8 b_fw_current_in_ps_mode;
unsigned long DelayLPSLastTimeStamp;
s32 pnp_current_pwr_state;
u8 pnp_bstop_trx;
--
2.30.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:07 [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files Fabio M. De Francesco
@ 2021-04-08 8:36 ` Greg KH
2021-04-08 8:52 ` Fabio M. De Francesco
2021-04-08 8:53 ` Julia Lawall
1 sibling, 1 reply; 7+ messages in thread
From: Greg KH @ 2021-04-08 8:36 UTC (permalink / raw)
To: Fabio M. De Francesco; +Cc: outreachy-kernel, linux-staging, linux-kernel
On Thu, Apr 08, 2021 at 10:07:14AM +0200, Fabio M. De Francesco wrote:
> Remove camelcase in a symbol that is used by several files.
What symbol?
> --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> u8 LpsIdleCount;
> u8 power_mgnt;
> u8 org_power_mgnt;
> - u8 bFwCurrentInPSMode;
> + u8 b_fw_current_in_ps_mode;
The "b" here means "byte" so you can drop the "b_" as that means
nothing, we do not use this type of notation in the kernel as the
compiler can check it for us.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:36 ` Greg KH
@ 2021-04-08 8:52 ` Fabio M. De Francesco
2021-04-08 8:54 ` Julia Lawall
0 siblings, 1 reply; 7+ messages in thread
From: Fabio M. De Francesco @ 2021-04-08 8:52 UTC (permalink / raw)
To: Greg KH; +Cc: outreachy-kernel, linux-staging, linux-kernel
On Thursday, April 8, 2021 10:36:32 AM CEST Greg KH wrote:
> On Thu, Apr 08, 2021 at 10:07:14AM +0200, Fabio M. De Francesco wrote:
> > Remove camelcase in a symbol that is used by several files.
>
> What symbol?
>
I'm not sure I understand what you are asking. You mean
that I have to specify the name of the variable that I changed?
Or you mean that I shouldn't use the term "symbol" but write
"variable", "function", "macro", and so on?
>
> > --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> > u8 LpsIdleCount;
> > u8 power_mgnt;
> > u8 org_power_mgnt;
> > - u8 bFwCurrentInPSMode;
> > + u8 b_fw_current_in_ps_mode;
>
> The "b" here means "byte" so you can drop the "b_" as that means
> nothing, we do not use this type of notation in the kernel as the
> compiler can check it for us.
>
OK, I didn't know what the 'b' meant in this context. I'll drop it.
Thanks,
Fabio
> thanks,
>
> greg k-h
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:07 [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files Fabio M. De Francesco
2021-04-08 8:36 ` Greg KH
@ 2021-04-08 8:53 ` Julia Lawall
2021-04-08 9:23 ` Fabio M. De Francesco
1 sibling, 1 reply; 7+ messages in thread
From: Julia Lawall @ 2021-04-08 8:53 UTC (permalink / raw)
To: Fabio M. De Francesco
Cc: gregkh, outreachy-kernel, linux-staging, linux-kernel
On Thu, 8 Apr 2021, Fabio M. De Francesco wrote:
> Remove camelcase in a symbol that is used by several files.
In addition to Greg's suggestion about dropping the b, there are some more
things you can do to improve the usage of this field.
First, I noticed that the type is u8. It can be changed to bool.
Second, the == true doesn't seem necessary in the following:
if (adapter_to_pwrctl(padapter)->bFwCurrentInPSMode == true) {
Of course before doing any of these, it would be good to check all of the
uses and be sure that the only possible values in this field are true and
false.
julia
>
> Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
> ---
> drivers/staging/rtl8723bs/core/rtw_cmd.c | 2 +-
> drivers/staging/rtl8723bs/core/rtw_mlme.c | 2 +-
> drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 18 +++++++++---------
> drivers/staging/rtl8723bs/hal/hal_intf.c | 2 +-
> drivers/staging/rtl8723bs/hal/rtl8723b_dm.c | 6 +++---
> .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 2 +-
> drivers/staging/rtl8723bs/hal/sdio_ops.c | 14 +++++++-------
> .../staging/rtl8723bs/include/rtw_pwrctrl.h | 2 +-
> 8 files changed, 24 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> index baf8b1e0f43c..a08f22b53592 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> @@ -1510,7 +1510,7 @@ static void rtw_lps_change_dtim_hdl(struct adapter *padapter, u8 dtim)
> if (pwrpriv->dtim != dtim)
> pwrpriv->dtim = dtim;
>
> - if ((pwrpriv->bFwCurrentInPSMode == true) && (pwrpriv->pwr_mode > PS_MODE_ACTIVE)) {
> + if ((pwrpriv->b_fw_current_in_ps_mode == true) && (pwrpriv->pwr_mode > PS_MODE_ACTIVE)) {
> u8 ps_mode = pwrpriv->pwr_mode;
>
> rtw_hal_set_hwreg(padapter, HW_VAR_H2C_FW_PWRMODE, (u8 *)(&ps_mode));
> diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> index a7e40aaae2d9..51cea6cf46e7 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> @@ -1684,7 +1684,7 @@ void rtw_dynamic_check_timer_handler(struct adapter *adapter)
> if (adapter->net_closed)
> return;
>
> - if ((adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + if ((adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> && !(hal_btcoex_IsBtControlLps(adapter))
> ) {
> u8 bEnterPS;
> diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> index f7465cf90c46..21e7a847866f 100644
> --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> @@ -365,7 +365,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode, u8 smart_ps, u8 bcn_a
> rtw_set_rpwm(padapter, PS_STATE_S4);
>
> rtw_hal_set_hwreg(padapter, HW_VAR_H2C_FW_PWRMODE, (u8 *)(&ps_mode));
> - pwrpriv->bFwCurrentInPSMode = false;
> + pwrpriv->b_fw_current_in_ps_mode = false;
>
> hal_btcoex_LpsNotify(padapter, ps_mode);
> }
> @@ -377,7 +377,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8 ps_mode, u8 smart_ps, u8 bcn_a
>
> hal_btcoex_LpsNotify(padapter, ps_mode);
>
> - pwrpriv->bFwCurrentInPSMode = true;
> + pwrpriv->b_fw_current_in_ps_mode = true;
> pwrpriv->pwr_mode = ps_mode;
> pwrpriv->smart_ps = smart_ps;
> pwrpriv->bcn_ant_mode = bcn_ant_mode;
> @@ -734,7 +734,7 @@ s32 rtw_register_task_alive(struct adapter *padapter, u32 task)
>
> register_task_alive(pwrctrl, task);
>
> - if (pwrctrl->bFwCurrentInPSMode) {
> + if (pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm < pslv) {
> if (pwrctrl->cpwm < PS_STATE_S2)
> res = _FAIL;
> @@ -782,7 +782,7 @@ void rtw_unregister_task_alive(struct adapter *padapter, u32 task)
>
> unregister_task_alive(pwrctrl, task);
>
> - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> + if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm > pslv)
> if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
> rtw_set_rpwm(padapter, pslv);
> @@ -819,7 +819,7 @@ s32 rtw_register_tx_alive(struct adapter *padapter)
>
> register_task_alive(pwrctrl, XMIT_ALIVE);
>
> - if (pwrctrl->bFwCurrentInPSMode) {
> + if (pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm < pslv) {
> if (pwrctrl->cpwm < PS_STATE_S2)
> res = _FAIL;
> @@ -864,7 +864,7 @@ s32 rtw_register_cmd_alive(struct adapter *padapter)
>
> register_task_alive(pwrctrl, CMD_ALIVE);
>
> - if (pwrctrl->bFwCurrentInPSMode) {
> + if (pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm < pslv) {
> if (pwrctrl->cpwm < PS_STATE_S2)
> res = _FAIL;
> @@ -909,7 +909,7 @@ void rtw_unregister_tx_alive(struct adapter *padapter)
>
> unregister_task_alive(pwrctrl, XMIT_ALIVE);
>
> - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> + if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm > pslv)
> if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
> rtw_set_rpwm(padapter, pslv);
> @@ -945,7 +945,7 @@ void rtw_unregister_cmd_alive(struct adapter *padapter)
>
> unregister_task_alive(pwrctrl, CMD_ALIVE);
>
> - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->bFwCurrentInPSMode) {
> + if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {
> if (pwrctrl->cpwm > pslv) {
> if ((pslv >= PS_STATE_S2) || (pwrctrl->alives == 0))
> rtw_set_rpwm(padapter, pslv);
> @@ -978,7 +978,7 @@ void rtw_init_pwrctrl_priv(struct adapter *padapter)
> pwrctrlpriv->power_mgnt = padapter->registrypriv.power_mgnt;/* PS_MODE_MIN; */
> pwrctrlpriv->bLeisurePs = pwrctrlpriv->power_mgnt != PS_MODE_ACTIVE;
>
> - pwrctrlpriv->bFwCurrentInPSMode = false;
> + pwrctrlpriv->b_fw_current_in_ps_mode = false;
>
> pwrctrlpriv->rpwm = 0;
> pwrctrlpriv->cpwm = PS_STATE_S4;
> diff --git a/drivers/staging/rtl8723bs/hal/hal_intf.c b/drivers/staging/rtl8723bs/hal/hal_intf.c
> index bc9ae2088754..a73c2f76628d 100644
> --- a/drivers/staging/rtl8723bs/hal/hal_intf.c
> +++ b/drivers/staging/rtl8723bs/hal/hal_intf.c
> @@ -348,7 +348,7 @@ void rtw_hal_dm_watchdog(struct adapter *padapter)
>
> void rtw_hal_dm_watchdog_in_lps(struct adapter *padapter)
> {
> - if (adapter_to_pwrctl(padapter)->bFwCurrentInPSMode == true) {
> + if (adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode == true) {
> if (padapter->HalFunc.hal_dm_watchdog_in_lps)
> padapter->HalFunc.hal_dm_watchdog_in_lps(padapter); /* this function caller is in interrupt context */
> }
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> index c2e9e4a0be22..265db187b8d5 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> @@ -141,7 +141,7 @@ void rtl8723b_InitHalDm(struct adapter *Adapter)
>
> void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
> {
> - bool bFwCurrentInPSMode = false;
> + bool b_fw_current_in_ps_mode = false;
> bool bFwPSAwake = true;
> u8 hw_init_completed = false;
> struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
> @@ -151,12 +151,12 @@ void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
> if (hw_init_completed == false)
> goto skip_dm;
>
> - bFwCurrentInPSMode = adapter_to_pwrctl(Adapter)->bFwCurrentInPSMode;
> + b_fw_current_in_ps_mode = adapter_to_pwrctl(Adapter)->b_fw_current_in_ps_mode;
> rtw_hal_get_hwreg(Adapter, HW_VAR_FWLPS_RF_ON, (u8 *)(&bFwPSAwake));
>
> if (
> (hw_init_completed == true) &&
> - ((!bFwCurrentInPSMode) && bFwPSAwake)
> + ((!b_fw_current_in_ps_mode) && bFwPSAwake)
> ) {
> /* */
> /* Calculate Tx/Rx statistics. */
> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> index e31ad3feed62..7ebc438870fd 100644
> --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> @@ -430,7 +430,7 @@ void rtl8723b_InitializeFirmwareVars(struct adapter *padapter)
> struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
>
> /* Init Fw LPS related. */
> - adapter_to_pwrctl(padapter)->bFwCurrentInPSMode = false;
> + adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode = false;
>
> /* Init H2C cmd. */
> rtw_write8(padapter, REG_HMETFR, 0x0f);
> diff --git a/drivers/staging/rtl8723bs/hal/sdio_ops.c b/drivers/staging/rtl8723bs/hal/sdio_ops.c
> index af7f846f90fe..c78a6724fc1c 100644
> --- a/drivers/staging/rtl8723bs/hal/sdio_ops.c
> +++ b/drivers/staging/rtl8723bs/hal/sdio_ops.c
> @@ -173,7 +173,7 @@ static u32 sdio_read32(struct intf_hdl *intfhdl, u32 addr)
> if (
> ((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> ) {
> err = sd_cmd52_read(intfhdl, ftaddr, 4, (u8 *)&le_tmp);
> #ifdef SDIO_DEBUG_IO
> @@ -230,7 +230,7 @@ static s32 sdio_readN(struct intf_hdl *intfhdl, u32 addr, u32 cnt, u8 *buf)
> if (
> ((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> )
> return sd_cmd52_read(intfhdl, ftaddr, cnt, buf);
>
> @@ -297,7 +297,7 @@ static s32 sdio_write32(struct intf_hdl *intfhdl, u32 addr, u32 val)
> if (
> ((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> ) {
> le_tmp = cpu_to_le32(val);
>
> @@ -334,7 +334,7 @@ static s32 sdio_writeN(struct intf_hdl *intfhdl, u32 addr, u32 cnt, u8 *buf)
> if (
> ((device_id == WLAN_IOREG_DEVICE_ID) && (offset < 0x100)) ||
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> )
> return sd_cmd52_write(intfhdl, ftaddr, cnt, buf);
>
> @@ -565,7 +565,7 @@ s32 sdio_local_read(
> rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
> if (
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> )
> return sd_cmd52_read(intfhdl, addr, cnt, buf);
>
> @@ -611,7 +611,7 @@ s32 sdio_local_write(
> rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
> if (
> (!mac_pwr_ctrl_on) ||
> - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> + (adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> )
> return sd_cmd52_write(intfhdl, addr, cnt, buf);
>
> @@ -660,7 +660,7 @@ static u32 sdio_local_cmd53_read4byte(struct adapter *adapter, u32 addr)
>
> hal_sdio_get_cmd_addr_8723b(adapter, SDIO_LOCAL_DEVICE_ID, addr, &addr);
> rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC, &mac_pwr_ctrl_on);
> - if (!mac_pwr_ctrl_on || adapter_to_pwrctl(adapter)->bFwCurrentInPSMode) {
> + if (!mac_pwr_ctrl_on || adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode) {
> sd_cmd52_read(intfhdl, addr, 4, (u8 *)&le_tmp);
> val = le32_to_cpu(le_tmp);
> } else {
> diff --git a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> index 2e739a17dd95..fcb06a95fdf6 100644
> --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> u8 LpsIdleCount;
> u8 power_mgnt;
> u8 org_power_mgnt;
> - u8 bFwCurrentInPSMode;
> + u8 b_fw_current_in_ps_mode;
> unsigned long DelayLPSLastTimeStamp;
> s32 pnp_current_pwr_state;
> u8 pnp_bstop_trx;
> --
> 2.30.2
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20210408080714.29481-1-fmdefrancesco%40gmail.com.
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:52 ` Fabio M. De Francesco
@ 2021-04-08 8:54 ` Julia Lawall
2021-04-08 9:10 ` Fabio M. De Francesco
0 siblings, 1 reply; 7+ messages in thread
From: Julia Lawall @ 2021-04-08 8:54 UTC (permalink / raw)
To: Fabio M. De Francesco
Cc: Greg KH, outreachy-kernel, linux-staging, linux-kernel
On Thu, 8 Apr 2021, Fabio M. De Francesco wrote:
> On Thursday, April 8, 2021 10:36:32 AM CEST Greg KH wrote:
> > On Thu, Apr 08, 2021 at 10:07:14AM +0200, Fabio M. De Francesco wrote:
> > > Remove camelcase in a symbol that is used by several files.
> >
> > What symbol?
> >
>
> I'm not sure I understand what you are asking. You mean
> that I have to specify the name of the variable that I changed?
> Or you mean that I shouldn't use the term "symbol" but write
> "variable", "function", "macro", and so on?
He wants the name of the symbol. Each changed line includes lots of
symbols, so it is a significant effort to scan the patch to see what
symbol is being changed.
julia
>
> >
> > > --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > > +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > > @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> > > u8 LpsIdleCount;
> > > u8 power_mgnt;
> > > u8 org_power_mgnt;
> > > - u8 bFwCurrentInPSMode;
> > > + u8 b_fw_current_in_ps_mode;
> >
> > The "b" here means "byte" so you can drop the "b_" as that means
> > nothing, we do not use this type of notation in the kernel as the
> > compiler can check it for us.
> >
>
> OK, I didn't know what the 'b' meant in this context. I'll drop it.
>
> Thanks,
>
> Fabio
>
> > thanks,
> >
> > greg k-h
> >
>
>
>
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/2284292.R3b5UFg5HO%40localhost.localdomain.
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:54 ` Julia Lawall
@ 2021-04-08 9:10 ` Fabio M. De Francesco
0 siblings, 0 replies; 7+ messages in thread
From: Fabio M. De Francesco @ 2021-04-08 9:10 UTC (permalink / raw)
To: Julia Lawall; +Cc: Greg KH, outreachy-kernel, linux-staging, linux-kernel
On Thursday, April 8, 2021 10:54:09 AM CEST Julia Lawall wrote:
> On Thu, 8 Apr 2021, Fabio M. De Francesco wrote:
> > On Thursday, April 8, 2021 10:36:32 AM CEST Greg KH wrote:
> > > On Thu, Apr 08, 2021 at 10:07:14AM +0200, Fabio M. De Francesco
wrote:
> > > > Remove camelcase in a symbol that is used by several files.
> > >
> > > What symbol?
> >
> > I'm not sure I understand what you are asking. You mean
> > that I have to specify the name of the variable that I changed?
> > Or you mean that I shouldn't use the term "symbol" but write
> > "variable", "function", "macro", and so on?
>
> He wants the name of the symbol. Each changed line includes lots of
> symbols, so it is a significant effort to scan the patch to see what
> symbol is being changed.
>
> julia
>
Thank you for clarifying this topic,
Fabio
> > > > --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > > > +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > > > @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> > > >
> > > > u8 LpsIdleCount;
> > > > u8 power_mgnt;
> > > > u8 org_power_mgnt;
> > > >
> > > > - u8 bFwCurrentInPSMode;
> > > > + u8 b_fw_current_in_ps_mode;
> > >
> > > The "b" here means "byte" so you can drop the "b_" as that means
> > > nothing, we do not use this type of notation in the kernel as the
> > > compiler can check it for us.
> >
> > OK, I didn't know what the 'b' meant in this context. I'll drop it.
> >
> > Thanks,
> >
> > Fabio
> >
> > > thanks,
> > >
> > > greg k-h
> >
> > --
> > You received this message because you are subscribed to the Google
> > Groups "outreachy-kernel" group. To unsubscribe from this group and
> > stop receiving emails from it, send an email to
> > outreachy-kernel+unsubscribe@googlegroups.com. To view this discussion
> > on the web visit
> > https://groups.google.com/d/msgid/outreachy-kernel/2284292.R3b5UFg5HO%
> > 40localhost.localdomain.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files
2021-04-08 8:53 ` Julia Lawall
@ 2021-04-08 9:23 ` Fabio M. De Francesco
0 siblings, 0 replies; 7+ messages in thread
From: Fabio M. De Francesco @ 2021-04-08 9:23 UTC (permalink / raw)
To: Julia Lawall; +Cc: gregkh, outreachy-kernel, linux-staging, linux-kernel
On Thursday, April 8, 2021 10:53:01 AM CEST Julia Lawall wrote:
> On Thu, 8 Apr 2021, Fabio M. De Francesco wrote:
> > Remove camelcase in a symbol that is used by several files.
>
> In addition to Greg's suggestion about dropping the b, there are some
> more things you can do to improve the usage of this field.
>
> First, I noticed that the type is u8. It can be changed to bool.
>
> Second, the == true doesn't seem necessary in the following:
>
> if (adapter_to_pwrctl(padapter)->bFwCurrentInPSMode == true) {
>
> Of course before doing any of these, it would be good to check all of the
> uses and be sure that the only possible values in this field are true
> and false.
>
> julia
>
I too had noticed the above-mentioned issues. However I thought I couldn't
fix them as they are logically unrelated to removing the camelcase. Am I
wrong on this subject?
Thanks a lot,
Fabio
> > Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
> > ---
> >
> > drivers/staging/rtl8723bs/core/rtw_cmd.c | 2 +-
> > drivers/staging/rtl8723bs/core/rtw_mlme.c | 2 +-
> > drivers/staging/rtl8723bs/core/rtw_pwrctrl.c | 18 +++++++++---------
> > drivers/staging/rtl8723bs/hal/hal_intf.c | 2 +-
> > drivers/staging/rtl8723bs/hal/rtl8723b_dm.c | 6 +++---
> > .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 2 +-
> > drivers/staging/rtl8723bs/hal/sdio_ops.c | 14 +++++++-------
> > .../staging/rtl8723bs/include/rtw_pwrctrl.h | 2 +-
> > 8 files changed, 24 insertions(+), 24 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c
> > b/drivers/staging/rtl8723bs/core/rtw_cmd.c index
> > baf8b1e0f43c..a08f22b53592 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> > @@ -1510,7 +1510,7 @@ static void rtw_lps_change_dtim_hdl(struct
> > adapter *padapter, u8 dtim)>
> > if (pwrpriv->dtim != dtim)
> >
> > pwrpriv->dtim = dtim;
> >
> > - if ((pwrpriv->bFwCurrentInPSMode == true) && (pwrpriv->pwr_mode
>
> > PS_MODE_ACTIVE)) { + if ((pwrpriv->b_fw_current_in_ps_mode == true) &&
> > (pwrpriv->pwr_mode > PS_MODE_ACTIVE)) {>
> > u8 ps_mode = pwrpriv->pwr_mode;
> >
> > rtw_hal_set_hwreg(padapter, HW_VAR_H2C_FW_PWRMODE,
(u8
> > *)(&ps_mode));
> >
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > b/drivers/staging/rtl8723bs/core/rtw_mlme.c index
> > a7e40aaae2d9..51cea6cf46e7 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
> > @@ -1684,7 +1684,7 @@ void rtw_dynamic_check_timer_handler(struct
> > adapter *adapter)>
> > if (adapter->net_closed)
> >
> > return;
> >
> > - if ((adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + if ((adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode)
> >
> > && !(hal_btcoex_IsBtControlLps(adapter))
> > ) {
> > u8 bEnterPS;
> >
> > diff --git a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c index
> > f7465cf90c46..21e7a847866f 100644
> > --- a/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_pwrctrl.c
> > @@ -365,7 +365,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8
> > ps_mode, u8 smart_ps, u8 bcn_a>
> > rtw_set_rpwm(padapter, PS_STATE_S4);
> >
> > rtw_hal_set_hwreg(padapter,
HW_VAR_H2C_FW_PWRMODE, (u8
> > *)(&ps_mode));
> >
> > - pwrpriv->bFwCurrentInPSMode = false;
> > + pwrpriv->b_fw_current_in_ps_mode = false;
> >
> > hal_btcoex_LpsNotify(padapter, ps_mode);
> >
> > }
> >
> > @@ -377,7 +377,7 @@ void rtw_set_ps_mode(struct adapter *padapter, u8
> > ps_mode, u8 smart_ps, u8 bcn_a>
> > hal_btcoex_LpsNotify(padapter, ps_mode);
> >
> > - pwrpriv->bFwCurrentInPSMode = true;
> > + pwrpriv->b_fw_current_in_ps_mode = true;
> >
> > pwrpriv->pwr_mode = ps_mode;
> > pwrpriv->smart_ps = smart_ps;
> > pwrpriv->bcn_ant_mode = bcn_ant_mode;
> >
> > @@ -734,7 +734,7 @@ s32 rtw_register_task_alive(struct adapter
> > *padapter, u32 task)>
> > register_task_alive(pwrctrl, task);
> >
> > - if (pwrctrl->bFwCurrentInPSMode) {
> > + if (pwrctrl->b_fw_current_in_ps_mode) {
> >
> > if (pwrctrl->cpwm < pslv) {
> >
> > if (pwrctrl->cpwm < PS_STATE_S2)
> >
> > res = _FAIL;
> >
> > @@ -782,7 +782,7 @@ void rtw_unregister_task_alive(struct adapter
> > *padapter, u32 task)>
> > unregister_task_alive(pwrctrl, task);
> >
> > - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) &&
> > pwrctrl->bFwCurrentInPSMode) { + if ((pwrctrl->pwr_mode !=
> > PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {>
> > if (pwrctrl->cpwm > pslv)
> >
> > if ((pslv >= PS_STATE_S2) || (pwrctrl-
>alives == 0))
> >
> > rtw_set_rpwm(padapter, pslv);
> >
> > @@ -819,7 +819,7 @@ s32 rtw_register_tx_alive(struct adapter *padapter)
> >
> > register_task_alive(pwrctrl, XMIT_ALIVE);
> >
> > - if (pwrctrl->bFwCurrentInPSMode) {
> > + if (pwrctrl->b_fw_current_in_ps_mode) {
> >
> > if (pwrctrl->cpwm < pslv) {
> >
> > if (pwrctrl->cpwm < PS_STATE_S2)
> >
> > res = _FAIL;
> >
> > @@ -864,7 +864,7 @@ s32 rtw_register_cmd_alive(struct adapter
> > *padapter)
> >
> > register_task_alive(pwrctrl, CMD_ALIVE);
> >
> > - if (pwrctrl->bFwCurrentInPSMode) {
> > + if (pwrctrl->b_fw_current_in_ps_mode) {
> >
> > if (pwrctrl->cpwm < pslv) {
> >
> > if (pwrctrl->cpwm < PS_STATE_S2)
> >
> > res = _FAIL;
> >
> > @@ -909,7 +909,7 @@ void rtw_unregister_tx_alive(struct adapter
> > *padapter)>
> > unregister_task_alive(pwrctrl, XMIT_ALIVE);
> >
> > - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) &&
> > pwrctrl->bFwCurrentInPSMode) { + if ((pwrctrl->pwr_mode !=
> > PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {>
> > if (pwrctrl->cpwm > pslv)
> >
> > if ((pslv >= PS_STATE_S2) || (pwrctrl-
>alives == 0))
> >
> > rtw_set_rpwm(padapter, pslv);
> >
> > @@ -945,7 +945,7 @@ void rtw_unregister_cmd_alive(struct adapter
> > *padapter)>
> > unregister_task_alive(pwrctrl, CMD_ALIVE);
> >
> > - if ((pwrctrl->pwr_mode != PS_MODE_ACTIVE) &&
> > pwrctrl->bFwCurrentInPSMode) { + if ((pwrctrl->pwr_mode !=
> > PS_MODE_ACTIVE) && pwrctrl->b_fw_current_in_ps_mode) {>
> > if (pwrctrl->cpwm > pslv) {
> >
> > if ((pslv >= PS_STATE_S2) || (pwrctrl-
>alives == 0))
> >
> > rtw_set_rpwm(padapter, pslv);
> >
> > @@ -978,7 +978,7 @@ void rtw_init_pwrctrl_priv(struct adapter
> > *padapter)
> >
> > pwrctrlpriv->power_mgnt = padapter->registrypriv.power_mgnt;/*
> > PS_MODE_MIN; */ pwrctrlpriv->bLeisurePs = pwrctrlpriv-
>power_mgnt !=
> > PS_MODE_ACTIVE;>
> > - pwrctrlpriv->bFwCurrentInPSMode = false;
> > + pwrctrlpriv->b_fw_current_in_ps_mode = false;
> >
> > pwrctrlpriv->rpwm = 0;
> > pwrctrlpriv->cpwm = PS_STATE_S4;
> >
> > diff --git a/drivers/staging/rtl8723bs/hal/hal_intf.c
> > b/drivers/staging/rtl8723bs/hal/hal_intf.c index
> > bc9ae2088754..a73c2f76628d 100644
> > --- a/drivers/staging/rtl8723bs/hal/hal_intf.c
> > +++ b/drivers/staging/rtl8723bs/hal/hal_intf.c
> > @@ -348,7 +348,7 @@ void rtw_hal_dm_watchdog(struct adapter *padapter)
> >
> > void rtw_hal_dm_watchdog_in_lps(struct adapter *padapter)
> > {
> >
> > - if (adapter_to_pwrctl(padapter)->bFwCurrentInPSMode == true) {
> > + if (adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode ==
true) {
> >
> > if (padapter->HalFunc.hal_dm_watchdog_in_lps)
> >
> > padapter-
>HalFunc.hal_dm_watchdog_in_lps(padapter); /* this
> > function caller is in interrupt context
*/>
> > }
> >
> > diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> > b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c index
> > c2e9e4a0be22..265db187b8d5 100644
> > --- a/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> > +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
> > @@ -141,7 +141,7 @@ void rtl8723b_InitHalDm(struct adapter *Adapter)
> >
> > void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
> > {
> >
> > - bool bFwCurrentInPSMode = false;
> > + bool b_fw_current_in_ps_mode = false;
> >
> > bool bFwPSAwake = true;
> > u8 hw_init_completed = false;
> > struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
> >
> > @@ -151,12 +151,12 @@ void rtl8723b_HalDmWatchDog(struct adapter
> > *Adapter)>
> > if (hw_init_completed == false)
> >
> > goto skip_dm;
> >
> > - bFwCurrentInPSMode = adapter_to_pwrctl(Adapter)-
>bFwCurrentInPSMode;
> > + b_fw_current_in_ps_mode =
> > adapter_to_pwrctl(Adapter)->b_fw_current_in_ps_mode;>
> > rtw_hal_get_hwreg(Adapter, HW_VAR_FWLPS_RF_ON, (u8 *)
(&bFwPSAwake));
> >
> > if (
> >
> > (hw_init_completed == true) &&
> >
> > - ((!bFwCurrentInPSMode) && bFwPSAwake)
> > + ((!b_fw_current_in_ps_mode) && bFwPSAwake)
> >
> > ) {
> >
> > /* */
> > /* Calculate Tx/Rx statistics. */
> >
> > diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> > b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c index
> > e31ad3feed62..7ebc438870fd 100644
> > --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> > +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
> > @@ -430,7 +430,7 @@ void rtl8723b_InitializeFirmwareVars(struct adapter
> > *padapter)>
> > struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
> >
> > /* Init Fw LPS related. */
> >
> > - adapter_to_pwrctl(padapter)->bFwCurrentInPSMode = false;
> > + adapter_to_pwrctl(padapter)->b_fw_current_in_ps_mode = false;
> >
> > /* Init H2C cmd. */
> > rtw_write8(padapter, REG_HMETFR, 0x0f);
> >
> > diff --git a/drivers/staging/rtl8723bs/hal/sdio_ops.c
> > b/drivers/staging/rtl8723bs/hal/sdio_ops.c index
> > af7f846f90fe..c78a6724fc1c 100644
> > --- a/drivers/staging/rtl8723bs/hal/sdio_ops.c
> > +++ b/drivers/staging/rtl8723bs/hal/sdio_ops.c
> > @@ -173,7 +173,7 @@ static u32 sdio_read32(struct intf_hdl *intfhdl,
> > u32 addr)>
> > if (
> >
> > ((device_id == WLAN_IOREG_DEVICE_ID) && (offset <
0x100)) ||
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > ) {
> >
> > err = sd_cmd52_read(intfhdl, ftaddr, 4, (u8
*)&le_tmp);
> >
> > #ifdef SDIO_DEBUG_IO
> >
> > @@ -230,7 +230,7 @@ static s32 sdio_readN(struct intf_hdl *intfhdl, u32
> > addr, u32 cnt, u8 *buf)>
> > if (
> >
> > ((device_id == WLAN_IOREG_DEVICE_ID) && (offset <
0x100)) ||
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > )
> >
> > return sd_cmd52_read(intfhdl, ftaddr, cnt, buf);
> >
> > @@ -297,7 +297,7 @@ static s32 sdio_write32(struct intf_hdl *intfhdl,
> > u32 addr, u32 val)>
> > if (
> >
> > ((device_id == WLAN_IOREG_DEVICE_ID) && (offset <
0x100)) ||
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > ) {
> >
> > le_tmp = cpu_to_le32(val);
> >
> > @@ -334,7 +334,7 @@ static s32 sdio_writeN(struct intf_hdl *intfhdl,
> > u32 addr, u32 cnt, u8 *buf)>
> > if (
> >
> > ((device_id == WLAN_IOREG_DEVICE_ID) && (offset <
0x100)) ||
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > )
> >
> > return sd_cmd52_write(intfhdl, ftaddr, cnt, buf);
> >
> > @@ -565,7 +565,7 @@ s32 sdio_local_read(
> >
> > rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC,
&mac_pwr_ctrl_on);
> > if (
> >
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > )
> >
> > return sd_cmd52_read(intfhdl, addr, cnt, buf);
> >
> > @@ -611,7 +611,7 @@ s32 sdio_local_write(
> >
> > rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC,
&mac_pwr_ctrl_on);
> > if (
> >
> > (!mac_pwr_ctrl_on) ||
> >
> > - (adapter_to_pwrctl(adapter)->bFwCurrentInPSMode)
> > + (adapter_to_pwrctl(adapter)-
>b_fw_current_in_ps_mode)
> >
> > )
> >
> > return sd_cmd52_write(intfhdl, addr, cnt, buf);
> >
> > @@ -660,7 +660,7 @@ static u32 sdio_local_cmd53_read4byte(struct
> > adapter *adapter, u32 addr)>
> > hal_sdio_get_cmd_addr_8723b(adapter, SDIO_LOCAL_DEVICE_ID,
addr,
> > &addr);
> > rtw_hal_get_hwreg(adapter, HW_VAR_APFM_ON_MAC,
&mac_pwr_ctrl_on);
> >
> > - if (!mac_pwr_ctrl_on ||
> > adapter_to_pwrctl(adapter)->bFwCurrentInPSMode) { + if
> > (!mac_pwr_ctrl_on ||
> > adapter_to_pwrctl(adapter)->b_fw_current_in_ps_mode) {>
> > sd_cmd52_read(intfhdl, addr, 4, (u8 *)&le_tmp);
> > val = le32_to_cpu(le_tmp);
> >
> > } else {
> >
> > diff --git a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h index
> > 2e739a17dd95..fcb06a95fdf6 100644
> > --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h
> > @@ -203,7 +203,7 @@ struct pwrctrl_priv {
> >
> > u8 LpsIdleCount;
> > u8 power_mgnt;
> > u8 org_power_mgnt;
> >
> > - u8 bFwCurrentInPSMode;
> > + u8 b_fw_current_in_ps_mode;
> >
> > unsigned long DelayLPSLastTimeStamp;
> > s32 pnp_current_pwr_state;
> > u8 pnp_bstop_trx;
> >
> > --
> > 2.30.2
> >
> > --
> > You received this message because you are subscribed to the Google
> > Groups "outreachy-kernel" group. To unsubscribe from this group and
> > stop receiving emails from it, send an email to
> > outreachy-kernel+unsubscribe@googlegroups.com. To view this discussion
> > on the web visit
> > https://groups.google.com/d/msgid/outreachy-kernel/20210408080714.2948
> > 1-1-fmdefrancesco%40gmail.com.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-04-08 9:23 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08 8:07 [Outreachy kernel] [PATCH] staging: rtl8723bs: Remove camelcase in several files Fabio M. De Francesco
2021-04-08 8:36 ` Greg KH
2021-04-08 8:52 ` Fabio M. De Francesco
2021-04-08 8:54 ` Julia Lawall
2021-04-08 9:10 ` Fabio M. De Francesco
2021-04-08 8:53 ` Julia Lawall
2021-04-08 9:23 ` Fabio M. De Francesco
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).