From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AA63A7471; Fri, 24 Mar 2023 15:50:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1679673017; x=1711209017; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=akyg3pTXi4IIZBKv/ztYzPKYwRcjG1RkaN9s+21ycb8=; b=TFqRWTfZt5zsvR5aGVkVrSraKSL07CfmrzjKckkieUA7EQ7XSn0YcNDr 3IXsRgNHLqfl7RCPD439kOaMZAwRwinSZ1YL9xxyHIVHJWeKFqiyykuvy oA9Jgxls0RhtabNiFpaeZaz2B2gZNhyjoKZp+V7VXIP4hW6OkDAeOx3sx WRbdbN8DFNJLxouLpfy8OF3c2BXIrFpxeInBxls8rNYn1vELB7yBWC2tJ KeWkEX+pyioH2peFaAygVtOHFo5wuAEd1XEVQUUo1HPy3MShJl+7CX9zq zsx1dftc5bocJnW9O8Chxe7j89HjkO2mEdOrPXvaBdstGPXmX3rRKVoPH g==; X-IronPort-AV: E=McAfee;i="6600,9927,10659"; a="320207224" X-IronPort-AV: E=Sophos;i="5.98,288,1673942400"; d="scan'208";a="320207224" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Mar 2023 08:50:17 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10659"; a="751939797" X-IronPort-AV: E=Sophos;i="5.98,288,1673942400"; d="scan'208";a="751939797" Received: from aschofie-mobl2.amr.corp.intel.com (HELO aschofie-mobl2) ([10.209.62.66]) by fmsmga004-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Mar 2023 08:50:16 -0700 Date: Fri, 24 Mar 2023 08:50:14 -0700 From: Alison Schofield To: Khadija Kamran Cc: outreachy@lists.linux.dev, Greg Kroah-Hartman , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging rtl8192u: fix block comments Message-ID: References: Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Mar 24, 2023 at 05:43:19PM +0500, Khadija Kamran wrote: > Linux kernel coding style for block comments uses a column of '*' on the > left side and ends the comment with '*/' on a separate line. > > Fix block comments by adding '*' on subsequent lines and moving '*/' at > the end of block comments on a separate line. These issues in block > comments are reported by checkpatch.pl script. > > Signed-off-by: Khadija Kamran Hi Khadija, It seems you've gotten feedback suggesting this single patch can be abandoned, and replaced with a patchset, like: Clean up comments in r8192U_dm.c - remove code in comments - remove change history info from comments - reformat block comments In the cover letter, you can refer to this previous patch with a lore link. ie... Reviewer feedback on a previous patch [1] suggested additional cleanups of comments. [1] https://lore.kernel.org/outreachy/ZB2a5zwYpBZ%2F6qwU@khadija-virtual-machine/ Alison > --- > drivers/staging/rtl8192u/r8192U_dm.c | 162 ++++++++++++++------------- > 1 file changed, 86 insertions(+), 76 deletions(-) > > diff --git a/drivers/staging/rtl8192u/r8192U_dm.c b/drivers/staging/rtl8192u/r8192U_dm.c > index 6a33ca02c3dc..a88686be3523 100644 > --- a/drivers/staging/rtl8192u/r8192U_dm.c > +++ b/drivers/staging/rtl8192u/r8192U_dm.c > @@ -1,19 +1,20 @@ > // SPDX-License-Identifier: GPL-2.0 > /*++ > -Copyright-c Realtek Semiconductor Corp. All rights reserved. > - > -Module Name: > - r8192U_dm.c > - > -Abstract: > - HW dynamic mechanism. > - > -Major Change History: > - When Who What > - ---------- --------------- ------------------------------- > - 2008-05-14 amy create version 0 porting from windows code. > - > ---*/ > + * Copyright-c Realtek Semiconductor Corp. All rights reserved. > + * > + * Module Name: > + * r8192U_dm.c > + * > + * Abstract: > + * HW dynamic mechanism. > + * > + * Major Change History: > + * When Who What > + * ---------- --------------- ------------------------------- > + * 2008-05-14 amy create version 0 porting from windows code. > + * > + *-- > + */ > #include "r8192U.h" > #include "r8192U_dm.h" > #include "r8192U_hw.h" > @@ -147,20 +148,20 @@ void dm_CheckRxAggregation(struct net_device *dev) > unsigned long curRxOkCnt = 0; > > /* > - if (pHalData->bForcedUsbRxAggr) { > - if (pHalData->ForcedUsbRxAggrInfo == 0) { > - if (pHalData->bCurrentRxAggrEnable) { > - Adapter->HalFunc.HalUsbRxAggrHandler(Adapter, FALSE); > - } > - } else { > - if (!pHalData->bCurrentRxAggrEnable || (pHalData->ForcedUsbRxAggrInfo != pHalData->LastUsbRxAggrInfoSetting)) { > - Adapter->HalFunc.HalUsbRxAggrHandler(Adapter, TRUE); > - } > - } > - return; > - } > - > -*/ > + * if (pHalData->bForcedUsbRxAggr) { > + * if (pHalData->ForcedUsbRxAggrInfo == 0) { > + * if (pHalData->bCurrentRxAggrEnable) { > + * Adapter->HalFunc.HalUsbRxAggrHandler(Adapter, FALSE); > + * } > + * } else { > + * if (!pHalData->bCurrentRxAggrEnable || (pHalData->ForcedUsbRxAggrInfo != pHalData->LastUsbRxAggrInfoSetting)) { > + * Adapter->HalFunc.HalUsbRxAggrHandler(Adapter, TRUE); > + * } > + * } > + * return; > + * } > + * > + */ > curTxOkCnt = priv->stats.txbytesunicast - lastTxOkCnt; > curRxOkCnt = priv->stats.rxbytesunicast - lastRxOkCnt; > > @@ -279,7 +280,8 @@ void init_rate_adaptive(struct net_device *dev) > * When Who Remark > * 05/26/08 amy Create version 0 porting from windows code. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_check_rate_adaptive(struct net_device *dev) > { > struct r8192_priv *priv = ieee80211_priv(dev); > @@ -377,8 +379,9 @@ static void dm_check_rate_adaptive(struct net_device *dev) > targetRATR = pra->ping_rssi_ratr; > ping_rssi_state = 1; > } > - /*else > - DbgPrint("TestRSSI is between the range.\n");*/ > + /* else > + * DbgPrint("TestRSSI is between the range.\n"); > + */ > } else { > /*DbgPrint("TestRSSI Recover to 0x%x\n", targetRATR);*/ > ping_rssi_state = 0; > @@ -719,9 +722,10 @@ static void dm_TXPowerTrackingCallback_ThermalMeter(struct net_device *dev) > } > tmpCCK40Mindex = 0; > } > - /*DbgPrint("%ddb, tmpOFDMindex = %d, tmpCCK20Mindex = %d, tmpCCK40Mindex = %d", > - ((u1Byte)tmpRegA - pHalData->ThermalMeter[0]), > - tmpOFDMindex, tmpCCK20Mindex, tmpCCK40Mindex);*/ > + /* DbgPrint("%ddb, tmpOFDMindex = %d, tmpCCK20Mindex = %d, tmpCCK40Mindex = %d", > + * ((u1Byte)tmpRegA - pHalData->ThermalMeter[0]), > + * tmpOFDMindex, tmpCCK20Mindex, tmpCCK40Mindex); > + */ > if (priv->CurrentChannelBW != HT_CHANNEL_WIDTH_20) /* 40M */ > tmpCCKindex = tmpCCK40Mindex; > else > @@ -1594,7 +1598,8 @@ static void dm_bb_initialgain_backup(struct net_device *dev) > * When Who Remark > * 05/15/2008 amy Create Version 0 porting from windows code. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_dig_init(struct net_device *dev) > { > struct r8192_priv *priv = ieee80211_priv(dev); > @@ -1638,7 +1643,8 @@ static void dm_dig_init(struct net_device *dev) > * Revised History: > * When Who Remark > * 05/27/2008 amy Create Version 0 porting from windows code. > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_ctrl_initgain_byrssi(struct net_device *dev) > { > if (!dm_digtable.dig_enable_flag) > @@ -1680,8 +1686,9 @@ static void dm_ctrl_initgain_byrssi_by_driverrssi( > else > dm_digtable.cur_connect_state = DIG_DISCONNECT; > > - /*DbgPrint("DM_DigTable.PreConnectState = %d, DM_DigTable.CurConnectState = %d\n", > - DM_DigTable.PreConnectState, DM_DigTable.CurConnectState);*/ > + /* DbgPrint("DM_DigTable.PreConnectState = %d, DM_DigTable.CurConnectState = %d\n", > + * DM_DigTable.PreConnectState, DM_DigTable.CurConnectState); > + */ > > dm_digtable.rssi_val = priv->undecorated_smoothed_pwdb; > /*DbgPrint("DM_DigTable.Rssi_val = %d\n", DM_DigTable.Rssi_val);*/ > @@ -1720,12 +1727,12 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm( > (priv->undecorated_smoothed_pwdb < dm_digtable.rssi_high_thresh)) > return; > > - /*DbgPrint("Dig by Fw False Alarm\n");*/ > - /*if (DM_DigTable.Dig_State == DM_STA_DIG_OFF)*/ > - /*DbgPrint("DIG Check\n\r RSSI=%d LOW=%d HIGH=%d STATE=%d", > - pHalData->UndecoratedSmoothedPWDB, DM_DigTable.RssiLowThresh, > - DM_DigTable.RssiHighThresh, DM_DigTable.Dig_State);*/ > - /* 1. When RSSI decrease, We have to judge if it is smaller than a threshold > + /* DbgPrint("Dig by Fw False Alarm\n"); > + * if (DM_DigTable.Dig_State == DM_STA_DIG_OFF) > + * DbgPrint("DIG Check\n\r RSSI=%d LOW=%d HIGH=%d STATE=%d", > + * pHalData->UndecoratedSmoothedPWDB, DM_DigTable.RssiLowThresh, > + * DM_DigTable.RssiHighThresh, DM_DigTable.Dig_State); > + * 1. When RSSI decrease, We have to judge if it is smaller than a threshold > * and then execute the step below. > */ > if (priv->undecorated_smoothed_pwdb <= dm_digtable.rssi_low_thresh) { > @@ -1757,12 +1764,12 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm( > * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. > */ > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x00); > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(pAdapter, rOFDM0_RxDetector1, 0x40); > - else if (pAdapter->HardwareType == HARDWARE_TYPE_RTL8192E) > - else > - PlatformEFIOWrite1Byte(pAdapter, rOFDM0_RxDetector1, 0x40); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(pAdapter, rOFDM0_RxDetector1, 0x40); > + * else if (pAdapter->HardwareType == HARDWARE_TYPE_RTL8192E) > + * else > + * PlatformEFIOWrite1Byte(pAdapter, rOFDM0_RxDetector1, 0x40); > + */ > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > > @@ -1814,13 +1821,12 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm( > * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. > */ > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x20); > - /* > - else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > - else if (pAdapter->HardwareType == HARDWARE_TYPE_RTL8192E) > - else > - PlatformEFIOWrite1Byte(pAdapter, rOFDM0_RxDetector1, 0x42); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > + * else if (pAdapter->HardwareType == HARDWARE_TYPE_RTL8192E) > + * else > + * PlatformEFIOWrite1Byte(pAdapter, rOFDM0_RxDetector1, 0x42); > + */ > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x44); > > @@ -1855,7 +1861,8 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm( > * When Who Remark > * 05/28/2008 amy Create Version 0 porting from windows code. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_ctrl_initgain_byrssi_highpwr( > struct net_device *dev) > { > @@ -1882,9 +1889,9 @@ static void dm_ctrl_initgain_byrssi_highpwr( > if (priv->CurrentChannelBW != HT_CHANNEL_WIDTH_20) { > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x10); > > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x41); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x41); > + */ > > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x43); > @@ -1899,9 +1906,9 @@ static void dm_ctrl_initgain_byrssi_highpwr( > /* 3.2 Recover PD_TH for OFDM for normal power region. */ > if (priv->CurrentChannelBW != HT_CHANNEL_WIDTH_20) { > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x20); > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > + */ > > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x44); > @@ -2020,9 +2027,9 @@ static void dm_pd_th( > * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. > */ > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x00); > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x40); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x40); > + */ > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > } else if (dm_digtable.curpd_thstate == DIG_PD_AT_NORMAL_POWER) { > @@ -2032,18 +2039,18 @@ static void dm_pd_th( > * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. > */ > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x20); > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x42); > + */ > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x44); > } else if (dm_digtable.curpd_thstate == DIG_PD_AT_HIGH_POWER) { > /* Higher PD_TH for OFDM for high power state. */ > if (priv->CurrentChannelBW != HT_CHANNEL_WIDTH_20) { > write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x10); > - /*else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > - write_nic_byte(dev, rOFDM0_RxDetector1, 0x41); > - */ > + /* else if (priv->card_8192 == HARDWARE_TYPE_RTL8190P) > + * write_nic_byte(dev, rOFDM0_RxDetector1, 0x41); > + */ > } else > write_nic_byte(dev, rOFDM0_RxDetector1, 0x43); > } > @@ -2274,7 +2281,8 @@ static void dm_ctstoself(struct net_device *dev) > * When Who Remark > * 05/28/2008 amy Create Version 0 porting from windows code. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_check_pbc_gpio(struct net_device *dev) > { > struct r8192_priv *priv = ieee80211_priv(dev); > @@ -2308,7 +2316,8 @@ static void dm_check_pbc_gpio(struct net_device *dev) > * When Who Remark > * 01/30/2008 MHC Create Version 0. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > void dm_rf_pathcheck_workitemcallback(struct work_struct *work) > { > struct delayed_work *dwork = to_delayed_work(work); > @@ -2557,7 +2566,8 @@ static void dm_rxpath_sel_byrssi(struct net_device *dev) > * When Who Remark > * 05/28/2008 amy Create Version 0 porting from windows code. > * > - *---------------------------------------------------------------------------*/ > + *--------------------------------------------------------------------------- > + */ > static void dm_check_rx_path_selection(struct net_device *dev) > { > struct r8192_priv *priv = ieee80211_priv(dev); > -- > 2.34.1 > >