From: Sumitra Sharma <sumitraartsy@gmail.com>
To: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: rtl8192e: Remove unnecessary blank lines
Date: Sun, 9 Apr 2023 04:27:04 -0700 [thread overview]
Message-ID: <20230409112704.GA264154@sumitra.com> (raw)
In-Reply-To: <052468bd-98db-a564-cf5f-0ea287b1025f@gmail.com>
On Sun, Apr 09, 2023 at 09:05:19AM +0200, Philipp Hortmann wrote:
> On 4/8/23 17:55, Sumitra Sharma wrote:
> > Remove extra blank lines as suggested by the Linux kernel
> > coding-style. These issues were reported by checkpatch.
> >
> > CHECK: Blank lines aren't necessary before a close brace '}'
> > CHECK: Blank lines aren't necessary after an open brace '{'
> > CHECK: Please don't use multiple blank lines
> >
> > Signed-off-by: Sumitra Sharma <sumitraartsy@gmail.com>
> > ---
> > drivers/staging/rtl8192e/rtl8192e/rtl_dm.c | 33 ----------------------
> > 1 file changed, 33 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c b/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
> > index cde41111012a..421265261c73 100644
> > --- a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
> > +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c
> > @@ -140,26 +140,22 @@ const u8 dm_cck_tx_bb_gain_ch14[CCK_TX_BB_GAIN_TABLE_LEN][8] = {
> > /*---------------------------Define Local Constant---------------------------*/
> > -
> > /*------------------------Define global variable-----------------------------*/
> > struct dig_t dm_digtable;
> > struct drx_path_sel dm_rx_path_sel_table;
> > /*------------------------Define global variable-----------------------------*/
> > -
> > /*------------------------Define local variable------------------------------*/
> > /*------------------------Define local variable------------------------------*/
> > -
> > /*---------------------Define local function prototype-----------------------*/
> > static void _rtl92e_dm_check_rate_adaptive(struct net_device *dev);
> > static void _rtl92e_dm_init_bandwidth_autoswitch(struct net_device *dev);
> > static void _rtl92e_dm_bandwidth_autoswitch(struct net_device *dev);
> > -
> > static void _rtl92e_dm_check_tx_power_tracking(struct net_device *dev);
> > static void _rtl92e_dm_bb_initialgain_restore(struct net_device *dev);
> > @@ -180,7 +176,6 @@ static void _rtl92e_dm_check_rx_path_selection(struct net_device *dev);
> > static void _rtl92e_dm_init_rx_path_selection(struct net_device *dev);
> > static void _rtl92e_dm_rx_path_sel_byrssi(struct net_device *dev);
> > -
> > static void _rtl92e_dm_init_fsync(struct net_device *dev);
> > static void _rtl92e_dm_deinit_fsync(struct net_device *dev);
> > @@ -222,9 +217,7 @@ void rtl92e_dm_init(struct net_device *dev)
> > void rtl92e_dm_deinit(struct net_device *dev)
> > {
> > -
> > _rtl92e_dm_deinit_fsync(dev);
> > -
> > }
> > void rtl92e_dm_watchdog(struct net_device *dev)
> > @@ -253,7 +246,6 @@ void rtl92e_dm_watchdog(struct net_device *dev)
> > void rtl92e_init_adaptive_rate(struct net_device *dev)
> > {
> > -
> > struct r8192_priv *priv = rtllib_priv(dev);
> > struct rate_adaptive *pra = &priv->rate_adaptive;
> > @@ -300,7 +292,6 @@ static void _rtl92e_dm_check_rate_adaptive(struct net_device *dev)
> > return;
> > if (priv->rtllib->state == RTLLIB_LINKED) {
> > -
> > bshort_gi_enabled = (ht_info->cur_tx_bw40mhz &&
> > ht_info->bCurShortGI40MHz) ||
> > (!ht_info->cur_tx_bw40mhz &&
> > @@ -527,7 +518,6 @@ static void _rtl92e_dm_tx_power_tracking_callback_tssi(struct net_device *dev)
> > priv->rtllib->bdynamic_txpower_enable = false;
> > for (j = 0; j <= 30; j++) {
> > -
> > tx_cmd.op = TXCMD_SET_TX_PWR_TRACKING;
> > tx_cmd.length = 4;
> > tx_cmd.value = priv->pwr_track >> 24;
> > @@ -759,20 +749,17 @@ void rtl92e_dm_txpower_tracking_wq(void *data)
> > static void _rtl92e_dm_initialize_tx_power_tracking_tssi(struct net_device *dev)
> > {
> > -
> > struct r8192_priv *priv = rtllib_priv(dev);
> > priv->btxpower_tracking = true;
> > priv->txpower_count = 0;
> > priv->tx_pwr_tracking_init = false;
> > -
> > }
> > static void _rtl92e_dm_init_tx_power_tracking_thermal(struct net_device *dev)
> > {
> > struct r8192_priv *priv = rtllib_priv(dev);
> > -
> > if (priv->rtllib->FwRWRF)
> > priv->btxpower_tracking = true;
> > else
> > @@ -802,12 +789,10 @@ static void _rtl92e_dm_check_tx_power_tracking_tssi(struct net_device *dev)
> > return;
> > tx_power_track_counter++;
> > -
> > if (tx_power_track_counter >= 180) {
> > schedule_delayed_work(&priv->txpower_tracking_wq, 0);
> > tx_power_track_counter = 0;
> > }
> > -
> > }
> > static void _rtl92e_dm_check_tx_power_tracking_thermal(struct net_device *dev)
> > @@ -839,7 +824,6 @@ static void _rtl92e_dm_check_tx_power_tracking_thermal(struct net_device *dev)
> > netdev_info(dev, "===============>Schedule TxPowerTrackingWorkItem\n");
> > schedule_delayed_work(&priv->txpower_tracking_wq, 0);
> > TM_Trigger = 0;
> > -
> > }
> > static void _rtl92e_dm_check_tx_power_tracking(struct net_device *dev)
> > @@ -971,7 +955,6 @@ void rtl92e_dm_restore_state(struct net_device *dev)
> > _rtl92e_dm_tx_power_reset_recovery(dev);
> > _rtl92e_dm_bb_initialgain_restore(dev);
> > -
> > }
> > static void _rtl92e_dm_bb_initialgain_restore(struct net_device *dev)
> > @@ -995,7 +978,6 @@ static void _rtl92e_dm_bb_initialgain_restore(struct net_device *dev)
> > rtl92e_set_bb_reg(dev, rCCK0_CCA, bit_mask,
> > (u32)priv->initgain_backup.cca);
> > rtl92e_set_bb_reg(dev, UFWP, bMaskByte1, 0x1);
> > -
> > }
> > void rtl92e_dm_backup_state(struct net_device *dev)
> > @@ -1050,7 +1032,6 @@ static void _rtl92e_dm_dig_init(struct net_device *dev)
> > static void _rtl92e_dm_ctrl_initgain_byrssi(struct net_device *dev)
> > {
> > -
> > if (!dm_digtable.dig_enable_flag)
> > return;
> > @@ -1103,7 +1084,6 @@ static void _rtl92e_dm_ctrl_initgain_byrssi_driver(struct net_device *dev)
> > else
> > dm_digtable.cur_sta_connect_state = DIG_STA_DISCONNECT;
> > -
> > dm_digtable.rssi_val = priv->undecorated_smoothed_pwdb;
> > _rtl92e_dm_initial_gain(dev);
> > _rtl92e_dm_pd_th(dev);
> > @@ -1111,7 +1091,6 @@ static void _rtl92e_dm_ctrl_initgain_byrssi_driver(struct net_device *dev)
> > if (dm_digtable.dig_algorithm_switch)
> > dm_digtable.dig_algorithm_switch = 0;
> > dm_digtable.pre_sta_connect_state = dm_digtable.cur_sta_connect_state;
> > -
> > }
> > static void _rtl92e_dm_ctrl_initgain_byrssi_false_alarm(struct net_device *dev)
> > @@ -1201,7 +1180,6 @@ static void _rtl92e_dm_ctrl_initgain_byrssi_false_alarm(struct net_device *dev)
> > _rtl92e_dm_ctrl_initgain_byrssi_highpwr(dev);
> > }
> > -
> > static void _rtl92e_dm_ctrl_initgain_byrssi_highpwr(struct net_device *dev)
> > {
> > struct r8192_priv *priv = rtllib_priv(dev);
> > @@ -1399,7 +1377,6 @@ static void _rtl92e_dm_cs_ratio(struct net_device *dev)
> > reset_cnt = priv->reset_count;
> > }
> > -
> > if ((dm_digtable.precs_ratio_state != dm_digtable.curcs_ratio_state) ||
> > !initialized || force_write) {
> > if (dm_digtable.curcs_ratio_state == DIG_CS_RATIO_LOWER)
> > @@ -1481,7 +1458,6 @@ static void _rtl92e_dm_check_edca_turbo(struct net_device *dev)
> > edca_setting_UL[ht_info->IOTPeer]);
> > priv->bis_cur_rdlstate = false;
> > }
> > -
> > }
> > priv->bcurrent_turbo_EDCA = true;
> > @@ -1496,7 +1472,6 @@ static void _rtl92e_dm_check_edca_turbo(struct net_device *dev)
> > }
> > }
> > -
> > dm_CheckEdcaTurbo_EXIT:
> > priv->rtllib->bis_any_nonbepkts = false;
> > lastTxOkCnt = priv->stats.txbytesunicast;
> > @@ -1536,7 +1511,6 @@ static void _rtl92e_dm_cts_to_self(struct net_device *dev)
> > }
> > }
> > -
> > static void _rtl92e_dm_init_wa_broadcom_iot(struct net_device *dev)
> > {
> > struct r8192_priv *priv = rtllib_priv((struct net_device *)dev);
> > @@ -1767,7 +1741,6 @@ static void _rtl92e_dm_rx_path_sel_byrssi(struct net_device *dev)
> > tmp_cck_min_pwdb = cur_cck_pwdb;
> > }
> > }
> > -
> > }
> > }
> > }
> > @@ -1833,7 +1806,6 @@ static void _rtl92e_dm_check_rx_path_selection(struct net_device *dev)
> > schedule_delayed_work(&priv->rfpath_check_wq, 0);
> > }
> > -
> > static void _rtl92e_dm_init_fsync(struct net_device *dev)
> > {
> > struct r8192_priv *priv = rtllib_priv(dev);
> > @@ -1851,7 +1823,6 @@ static void _rtl92e_dm_init_fsync(struct net_device *dev)
> > timer_setup(&priv->fsync_timer, _rtl92e_dm_fsync_timer_callback, 0);
> > }
> > -
> > static void _rtl92e_dm_deinit_fsync(struct net_device *dev)
> > {
> > struct r8192_priv *priv = rtllib_priv(dev);
> > @@ -1886,7 +1857,6 @@ static void _rtl92e_dm_fsync_timer_callback(struct timer_list *t)
> > else
> > rate_count_diff = rate_count - priv->rate_record;
> > if (rate_count_diff < priv->rate_count_diff_rec) {
> > -
> > u32 DiffNum = priv->rate_count_diff_rec -
> > rate_count_diff;
> > if (DiffNum >=
> > @@ -2027,7 +1997,6 @@ static void _rtl92e_dm_start_sw_fsync(struct net_device *dev)
> > add_timer(&priv->fsync_timer);
> > rtl92e_writel(dev, rOFDM0_RxDetector2, 0x465c12cd);
> > -
> > }
> > static void _rtl92e_dm_check_fsync(struct net_device *dev)
> > @@ -2070,7 +2039,6 @@ static void _rtl92e_dm_check_fsync(struct net_device *dev)
> > case SW_Fsync:
> > default:
> > break;
> > -
> > }
> > }
> > if (priv->frame_sync_monitor) {
> > @@ -2198,7 +2166,6 @@ static void _rtl92e_dm_dynamic_tx_power(struct net_device *dev)
> > }
> > priv->last_dtp_flag_high = priv->dynamic_tx_high_pwr;
> > priv->last_dtp_flag_low = priv->dynamic_tx_low_pwr;
> > -
> > }
> > static void _rtl92e_dm_check_txrateandretrycount(struct net_device *dev)
>
> Hi,
>
> Please do not send in patches at almost the same time with exactly the same
> name. I need to look into the patch to find out which one is which.
>
Hi Philipp
I am sorry. I will take care of it.
> This patch interfears with one of my patches. So only one can be accepted
> and the otherone has to rebase. Will see what happens.
>
Okay Thank you.
Regards
Sumitra
> Bye Philipp
next prev parent reply other threads:[~2023-04-09 11:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-08 15:55 [PATCH] staging: rtl8192e: Remove unnecessary blank lines Sumitra Sharma
2023-04-09 7:05 ` Philipp Hortmann
2023-04-09 11:27 ` Sumitra Sharma [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-04-08 17:56 Sumitra Sharma
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230409112704.GA264154@sumitra.com \
--to=sumitraartsy@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=philipp.g.hortmann@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).