From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E2815C43613 for ; Mon, 24 Jun 2019 09:47:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A2A8F208E4 for ; Mon, 24 Jun 2019 09:47:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=web.de header.i=@web.de header.b="ab1LyQP1" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728568AbfFXJrY (ORCPT ); Mon, 24 Jun 2019 05:47:24 -0400 Received: from mout.web.de ([212.227.17.11]:48341 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726481AbfFXJrV (ORCPT ); Mon, 24 Jun 2019 05:47:21 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=dbaedf251592; t=1561369631; bh=hGUoea4WxnOa2zEqej2yKQwh4CIA2+8vTaZIIteiUwE=; h=X-UI-Sender-Class:From:To:Cc:Subject:Date:In-Reply-To:References; b=ab1LyQP1eGYBP3BmT/yBzrKkVFsJX8YJmnWZ+cdcd/jZNLTZnJoaetjSS9MFmwaRV pTK3TIY5Ij/ch97oxQF/a3w6DeyzjJEF9C+MM5RMo5jPbnCMuIxbwGNQlbjJjRTy// nesotbZceoCBDKQA5Yjop1xUohN08r0XUrnZLWr4= X-UI-Sender-Class: c548c8c5-30a9-4db5-a2e7-cb6cb037b8f9 Received: from lab-pc08.sra.uni-hannover.de ([130.75.33.87]) by smtp.web.de (mrweb101 [213.165.67.124]) with ESMTPSA (Nemesis) id 0MOzwh-1hcqzJ3ZNH-006LDX; Mon, 24 Jun 2019 11:47:10 +0200 From: =?UTF-8?q?Christian=20M=C3=BCller?= To: gregkh@linuxfoundation.org Cc: johnfwhitmore@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, linux-kernel@i4.cs.fau.de, felix.trommer@hotmail.de, =?UTF-8?q?Christian=20M=C3=BCller?= Subject: [PATCH 1/2] drivers/staging/rtl8192u: adjust block comments Date: Mon, 24 Jun 2019 11:46:39 +0200 Message-Id: <20190624094640.5459-2-muellerch-privat@web.de> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190624094640.5459-1-muellerch-privat@web.de> References: <20190620113308.GA16195@kroah.com> <20190624094640.5459-1-muellerch-privat@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:4E8viVun25OEewxatvlbgby5vTQE2AFbXqO36Z1IEMD/86u6BZE t3ELcJZJFIifBG2uv/VwFatjcdEIFORPo/rGu6/Fu0unsKddXaZLXZWGklDSen1PhS7xG+x e6qrWNWa3+QIlgvZ2qSSieg66VclIyfL9dy0iP71aD5LrNUsH3OTOTjMVnxmkuAreQDZeRV sO/wmCBqQ0VxlhpOOV4WQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:MEIjjWhFgtE=:0+/hWEWP2hb5G2u9J/OE39 Dz7lK76tQKtUenmEWHkXmK+LZR0QfZCtjPHYCB79/eM0WHTD+2TTjXnFu4Lp92iQm+kiLmhAy +1f5BW3PUnYMk66w9na27xHaM89b7TneoyzqfLJrmQ7H455l114F2u+Ji6NYxBuN4DYCPuAX4 vZazQfrkQx1bBa8lQWP1vKsBjSMiGa8G7LQdl2U7rUYBClnAa3IVQ+LGlcLS4GFgcbrYL/QG6 QdjtQmKAdhPsG+CfaBj/vvDFAQvBusv+6+Ew5aCc+aQ+bIL9yNkGQImQ1gnhzZW4r6HVxyEFl 0mFmVeIIYZZjGSfdd/9i2tWjgWjx+PJAm5Ty8M8bNnFP3Fg3UOI2ovtjCRPuhBPNk0cGA92S+ b90g39mwHRX0igWOC5Mdz0db/g9DGE5zJqgVgdJ/QMTfRcE6zhUuz/KelC11fB67U967zRZOb 7C61pAnQymflwqHSYmTtEXTwQuluRF+uotd/lAIKO2wVLWzIe5z4ObSVNussTjhBfZY6iTZZw Woy4ufuqooWyaq66hIJR8+DIxwuHgr7ZPuZMRJfpXUdfWIDZBV6pUq806ZfV94OWRKec7il1v yB6WaW7L6+1hxCX71OUL+GaAfcqiWtkKmM0PAyPVLngvVgYYuQV/ERmtTs8YCgy4QqeAiWttn ejhrdwumW2e7uj4Vx4jUopt6SzailklCICj/46afvqQLZxEGuQpQdjUeJW46JSu7pdcTZ67c5 fu36yAbggIRpVUyAN2iyuNtYOE022wM4T0w8thpLQ7iUUVS4YX/MPSZOSRPTOUhbLMXXslp7/ ZT8tBAkGxlNSmSObCvmF4h0JzBDieucIPnQepKilAVHHcoAxedeb+Hy/XX9/bloQxBfq2pXvC zMpTt3j7FhLd0eFiCagClNcA3XA6tbPXbocB+Bl4BLB8tkQlQrqgvKjd2q9ruOm+uo0cvHFgY zV2WyWJGAeI5alVWLaXNmNj+ui6Jl0wweYX86XmBGPRM2ZIvsKTtR Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As stated in coding-styles.rst multiline comments should be structured in = a way, that the actual comment starts on the second line of the commented portion= . E.g: /* * Multiline comments * should look like * this. */ However, there is an exception to files in drivers/net/ and net/, where multiline comments are prefered to look like this: /* Mutliline comments for * drivers/net/ should look * like this. */ The comments in this file initially looked like the first example. But since this file is part of a networking driver and thus should be moved to drivers/net/ one day, this patch adjusts the comments such that they are fitting to the style imposed for drivers/net/. Signed-off-by: Christian M=C3=BCller Signed-off-by: Felix Trommer =2D-- drivers/staging/rtl8192u/r8192U_dm.c | 69 ++++++++++------------------ 1 file changed, 23 insertions(+), 46 deletions(-) diff --git a/drivers/staging/rtl8192u/r8192U_dm.c b/drivers/staging/rtl819= 2u/r8192U_dm.c index 2ba01041406b..86215fee8f0b 100644 =2D-- a/drivers/staging/rtl8192u/r8192U_dm.c +++ b/drivers/staging/rtl8192u/r8192U_dm.c @@ -99,8 +99,7 @@ static void dm_dynamic_txpower(struct net_device *dev); static void dm_send_rssi_tofw(struct net_device *dev); static void dm_ctstoself(struct net_device *dev); /*---------------------------Define function prototype-------------------= -----*/ -/* - * =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D +/* =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D * HW Dynamic mechanism interface. * =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D * @@ -178,8 +177,7 @@ void dm_CheckRxAggregation(struct net_device *dev) ulValue =3D (pHTInfo->UsbRxFwAggrEn<<24) | (pHTInfo->UsbRxFwAggrPageNu= m<<16) | (pHTInfo->UsbRxFwAggrPacketNum<<8) | (pHTInfo->UsbRxFwAggrTimeout); - /* - * If usb rx firmware aggregation is enabled, + /* If usb rx firmware aggregation is enabled, * when anyone of three threshold conditions above is reached, * firmware will send aggregated packet to driver. */ @@ -219,8 +217,7 @@ void hal_dm_watchdog(struct net_device *dev) #endif } /* HalDmWatchDog */ -/* - * Decide Rate Adaptive Set according to distance (signal strength) +/* Decide Rate Adaptive Set according to distance (signal strength) * 01/11/2008 MHC Modify input arguments and RATR table level. * 01/16/2008 MHC RF_Type is assigned in ReadAdapterInfo(). We must call * the function after making sure RF_Type. @@ -246,8 +243,7 @@ void init_rate_adaptive(struct net_device *dev) pra->ping_rssi_thresh_for_ra =3D 15; if (priv->rf_type =3D=3D RF_2T4R) { - /* - * 07/10/08 MH Modify for RA smooth scheme. + /* 07/10/08 MH Modify for RA smooth scheme. * 2008/01/11 MH Modify 2T RATR table for different RSSI. 080515 portin= g by amy from windows code. */ pra->upper_rssi_threshold_ratr =3D 0x8f0f0000; @@ -387,8 +383,7 @@ static void dm_check_rate_adaptive(struct net_device *= dev) } } - /* - * 2008.04.01 + /* 2008.04.01 * For RTL819X, if pairwisekey =3D wep/tkip, we support only MCS0~7. */ if (priv->ieee80211->GetHalfNmodeSupportByAPsHandler(dev)) @@ -683,8 +678,7 @@ static void dm_TXPowerTrackingCallback_ThermalMeter(st= ruct net_device *dev) return; } - /* - * =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D + /* =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D * this is only for test, should be masked * =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D */ @@ -850,8 +844,7 @@ static void dm_InitializeTXPowerTracking_TSSI(struct n= et_device *dev) priv->txbbgain_table[36].txbb_iq_amplifygain =3D -24; priv->txbbgain_table[36].txbbgain_value =3D 0x10000040; - /* - * ccktxbb_valuearray[0] is 0xA22 [1] is 0xA24 ...[7] is 0xA29 + /* ccktxbb_valuearray[0] is 0xA22 [1] is 0xA24 ...[7] is 0xA29 * This Table is for CH1~CH13 */ priv->cck_txbbgain_table[0].ccktxbb_valuearray[0] =3D 0x36; @@ -1061,8 +1054,7 @@ static void dm_InitializeTXPowerTracking_TSSI(struct= net_device *dev) priv->cck_txbbgain_table[22].ccktxbb_valuearray[6] =3D 0x03; priv->cck_txbbgain_table[22].ccktxbb_valuearray[7] =3D 0x01; - /* - * ccktxbb_valuearray[0] is 0xA22 [1] is 0xA24 ...[7] is 0xA29 + /* ccktxbb_valuearray[0] is 0xA22 [1] is 0xA24 ...[7] is 0xA29 * This Table is for CH14 */ priv->cck_txbbgain_ch14_table[0].ccktxbb_valuearray[0] =3D 0x36; @@ -1282,8 +1274,7 @@ static void dm_InitializeTXPowerTracking_ThermalMete= r(struct net_device *dev) { struct r8192_priv *priv =3D ieee80211_priv(dev); - /* - * Tx Power tracking by Thermal Meter requires Firmware R/W 3-wire. This= mechanism + /* Tx Power tracking by Thermal Meter requires Firmware R/W 3-wire. This= mechanism * can be enabled only when Firmware R/W 3-wire is enabled. Otherwise, f= requent r/w * 3-wire by driver causes RF to go into a wrong state. */ @@ -1330,8 +1321,7 @@ static void dm_CheckTXPowerTracking_ThermalMeter(str= uct net_device *dev) } if (!TM_Trigger) { - /* - * Attention!! You have to write all 12bits of data to RF, or it may ca= use RF to crash + /* Attention!! You have to write all 12bits of data to RF, or it may ca= use RF to crash * actually write reg0x02 bit1=3D0, then bit1=3D1. * DbgPrint("Trigger ThermalMeter, write RF reg0x2 =3D 0x4d to 0x4f\n")= ; */ @@ -1773,8 +1763,7 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm= ( /* 1.3 Lower PD_TH for OFDM. */ if (priv->CurrentChannelBW !=3D HT_CHANNEL_WIDTH_20) { - /* - * 2008/01/11 MH 40MHZ 90/92 register are not the same. + /* 2008/01/11 MH 40MHZ 90/92 register are not the same. * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. */ write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x00); @@ -1814,8 +1803,7 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm= ( dm_digtable.dig_state =3D DM_STA_DIG_ON; /*DbgPrint("DIG ON\n\r");*/ - /* - * 2.1 Set initial gain. + /* 2.1 Set initial gain. * 2008/02/26 MH SD3-Jerry suggest to prevent dirty environment. */ if (reset_flag =3D=3D 1) { @@ -1832,8 +1820,7 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm= ( /* 2.2 Higher PD_TH for OFDM. */ if (priv->CurrentChannelBW !=3D HT_CHANNEL_WIDTH_20) { - /* - * 2008/01/11 MH 40MHZ 90/92 register are not the same. + /* 2008/01/11 MH 40MHZ 90/92 register are not the same. * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. */ write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x20); @@ -1850,8 +1837,7 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm= ( /* 2.3 Higher CS ratio for CCK. */ write_nic_byte(dev, 0xa0a, 0xcd); - /* - * 2.4 Lower EDCCA. + /* 2.4 Lower EDCCA. * 2008/01/11 MH 90/92 series are the same. */ /*PlatformEFIOWrite4Byte(pAdapter, rOFDM0_ECCAThreshold, 0x346);*/ @@ -1892,8 +1878,7 @@ static void dm_ctrl_initgain_byrssi_highpwr( (priv->undecorated_smoothed_pwdb < dm_digtable.rssi_high_power_highthre= sh)) return; - /* - * 3. When RSSI >75% or <70%, it is a high power issue. We have to judge= if + /* 3. When RSSI >75% or <70%, it is a high power issue. We have to judge= if * it is larger than a threshold and then execute the step below. * * 2008/02/05 MH SD3-Jerry Modify PD_TH for high power issue. @@ -2042,8 +2027,7 @@ static void dm_pd_th( if (dm_digtable.curpd_thstate =3D=3D DIG_PD_AT_LOW_POWER) { /* Lower PD_TH for OFDM. */ if (priv->CurrentChannelBW !=3D HT_CHANNEL_WIDTH_20) { - /* - * 2008/01/11 MH 40MHZ 90/92 register are not the same. + /* 2008/01/11 MH 40MHZ 90/92 register are not the same. * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. */ write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x00); @@ -2055,8 +2039,7 @@ static void dm_pd_th( } else if (dm_digtable.curpd_thstate =3D=3D DIG_PD_AT_NORMAL_POWER) { /* Higher PD_TH for OFDM. */ if (priv->CurrentChannelBW !=3D HT_CHANNEL_WIDTH_20) { - /* - * 2008/01/11 MH 40MHZ 90/92 register are not the same. + /* 2008/01/11 MH 40MHZ 90/92 register are not the same. * 2008/02/05 MH SD3-Jerry 92U/92E PD_TH are the same. */ write_nic_byte(dev, (rOFDM0_XATxAFE+3), 0x20); @@ -2155,8 +2138,7 @@ static void dm_check_edca_turbo( unsigned long curTxOkCnt =3D 0; unsigned long curRxOkCnt =3D 0; - /* - * Do not be Turbo if it's under WiFi config and Qos Enabled, because th= e EDCA parameters + /* Do not be Turbo if it's under WiFi config and Qos Enabled, because th= e EDCA parameters * should follow the settings from QAP. By Bruce, 2007-12-07. */ if (priv->ieee80211->state !=3D IEEE80211_LINKED) @@ -2188,8 +2170,7 @@ static void dm_check_edca_turbo( priv->bcurrent_turbo_EDCA =3D true; } else { - /* - * Turn Off EDCA turbo here. + /* Turn Off EDCA turbo here. * Restore original EDCA according to the declaration of AP. */ if (priv->bcurrent_turbo_EDCA) { @@ -2219,8 +2200,7 @@ static void dm_check_edca_turbo( write_nic_dword(dev, EDCAPARA_BE, u4bAcParam); - /* - * Check ACM bit. + /* Check ACM bit. * If it is set, immediately set ACM control bit to downgrading AC for= passing WMM testplan. Annie, 2005-12-13. */ { @@ -2319,8 +2299,7 @@ static void dm_check_pbc_gpio(struct net_device *dev= ) return; if (tmp1byte & BIT(6) || tmp1byte & BIT(0)) { - /* - * Here we only set bPbcPressed to TRUE + /* Here we only set bPbcPressed to TRUE * After trigger PBC, the variable will be set to FALSE */ RT_TRACE(COMP_IO, "CheckPbcGPIO - PBC is pressed\n"); @@ -2529,8 +2508,7 @@ static void dm_rxpath_sel_byrssi(struct net_device *= dev) } } - /* - * Set CCK Rx path + /* Set CCK Rx path * reg0xA07[3:2]=3Dcck default rx path, reg0xa07[1:0]=3Dcck optional rx = path. */ update_cck_rx_path =3D 0; @@ -3049,8 +3027,7 @@ static void dm_send_rssi_tofw(struct net_device *dev= ) { struct r8192_priv *priv =3D ieee80211_priv(dev); - /* - * If we test chariot, we should stop the TX command ? + /* If we test chariot, we should stop the TX command ? * Because 92E will always silent reset when we send tx command. We use = register * 0x1e0(byte) to notify driver. */ =2D- 2.17.1