[PATCH v3 1/8] staging: rtl8192u: change block comments to prefered style - Coding Style

2018-06-24 Thread John Whitmore
Some of the comment blocks are commening out code so have been left for the
moment.

Signed-off-by: John Whitmore 
---
 .../rtl8192u/ieee80211/rtl819x_HTProc.c   | 366 ++
 1 file changed, 195 insertions(+), 171 deletions(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c 
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index 5a48693e2cf3..54c48747f5fa 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -41,17 +41,18 @@ static u8 EDIMAX_RALINK[3] = {0x00, 0x0e, 0x2e};
 static u8 AIRLINK_RALINK[3] = {0x00, 0x18, 0x02};
 //static u8 DLINK_ATHEROS[3] = {0x00, 0x1c, 0xf0};
 static u8 CISCO_BROADCOM[3] = {0x00, 0x17, 0x94};
-
-// 2008/04/01 MH For Cisco G mode RX TP We need to change FW duration. Should 
we put the
-// code in other place??
-//static u8 WIFI_CISCO_G_AP[3] = {0x00, 0x40, 0x96};
-/
+/*
+ * 2008/04/01 MH For Cisco G mode RX TP We need to change FW duration. Should 
we put the
+ * code in other place??
+ * static u8 WIFI_CISCO_G_AP[3] = {0x00, 0x40, 0x96};
+ */
+/*
  *function:  This function update default settings in pHTInfo structure
  *   input:  PRT_HIGH_THROUGHPUT   pHTInfo
  *  output:  none
  *  return:  none
  *  notice:  These value need be modified if any changes.
- * 
*/
+ */
 void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
 {
PRT_HIGH_THROUGHPUT pHTInfo = ieee->pHTInfo;
@@ -93,8 +94,10 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
// 8190 only. Assign duration operation mode to firmware
pMgntInfo->bTxEnableFwCalcDur = 
(BOOLEAN)pNdisCommon->bRegTxEnableFwCalcDur;
 #endif
-   // 8190 only, Realtek proprietary aggregation mode
-   // Set MPDUDensity=2,   1: Set MPDUDensity=2(32k)  for Realtek AP and 
set MPDUDensity=0(8k) for others
+   /*
+* 8190 only, Realtek proprietary aggregation mode
+* Set MPDUDensity=2,   1: Set MPDUDensity=2(32k)  for Realtek AP and 
set MPDUDensity=0(8k) for others
+*/
pHTInfo->bRegRT2RTAggregation = 1;//0: Set MPDUDensity=2,   1: Set 
MPDUDensity=2(32k)  for Realtek AP and set MPDUDensity=0(8k) for others
 
// For Rx Reorder Control
@@ -113,14 +116,14 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
 #endif
 }
 
-/
+/*
  *function:  This function print out each field on HT capability IE mainly 
from (Beacon/ProbeRsp/AssocReq)
  *   input:  u8*   CapIE   //Capability IE to be printed out
  *  u8*TitleString //mainly print out caller function
  *  output:  none
  *  return:  none
  *  notice:  Driver should not print out this message by default.
- * 
*/
+ */
 void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)
 {
static u8   EWC11NHTCap[] = {0x00, 0x90, 0x4c, 0x33};   // For 
11n EWC definition, 2007.07.17, by Emily
@@ -147,14 +150,14 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)
pCapELE->MCS[1], pCapELE->MCS[2], 
pCapELE->MCS[3], pCapELE->MCS[4]);
 }
 
-/
+/*
  *function:  This function print out each field on HT Information IE mainly 
from (Beacon/ProbeRsp)
  *   input:  u8*   InfoIE   //Capability IE to be printed out
  *  u8*TitleString //mainly print out caller function
  *  output:  none
  *  return:  none
  *  notice:  Driver should not print out this message by default.
- * 
*/
+ */
 void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)
 {
static u8   EWC11NHTInfo[] = {0x00, 0x90, 0x4c, 0x34};  // For 
11n EWC definition, 2007.07.17, by Emily
@@ -209,8 +212,8 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)
 }
 
 /*
-*  Return: true if station in half n mode and AP supports 40 bw
-*/
+ * Return: true if station in half n mode and AP supports 40 bw
+ */
 static bool IsHTHalfNmode40Bandwidth(struct ieee80211_device *ieee)
 {
boolretValue = false;
@@ -276,14 +279,14 @@ u16 HTMcsToDataRate(struct ieee80211_device *ieee, u8 
nMcsRate)
return MCS_DATA_RATE[is40MHz][isShortGI][(nMcsRate & 0x7f)];
 }
 
-/
+/*
  *function:  This function returns current datarate.
  

[PATCH v3 1/8] staging: rtl8192u: change block comments to prefered style - Coding Style

2018-06-24 Thread John Whitmore
Some of the comment blocks are commening out code so have been left for the
moment.

Signed-off-by: John Whitmore 
---
 .../rtl8192u/ieee80211/rtl819x_HTProc.c   | 366 ++
 1 file changed, 195 insertions(+), 171 deletions(-)

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c 
b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
index 5a48693e2cf3..54c48747f5fa 100644
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c
@@ -41,17 +41,18 @@ static u8 EDIMAX_RALINK[3] = {0x00, 0x0e, 0x2e};
 static u8 AIRLINK_RALINK[3] = {0x00, 0x18, 0x02};
 //static u8 DLINK_ATHEROS[3] = {0x00, 0x1c, 0xf0};
 static u8 CISCO_BROADCOM[3] = {0x00, 0x17, 0x94};
-
-// 2008/04/01 MH For Cisco G mode RX TP We need to change FW duration. Should 
we put the
-// code in other place??
-//static u8 WIFI_CISCO_G_AP[3] = {0x00, 0x40, 0x96};
-/
+/*
+ * 2008/04/01 MH For Cisco G mode RX TP We need to change FW duration. Should 
we put the
+ * code in other place??
+ * static u8 WIFI_CISCO_G_AP[3] = {0x00, 0x40, 0x96};
+ */
+/*
  *function:  This function update default settings in pHTInfo structure
  *   input:  PRT_HIGH_THROUGHPUT   pHTInfo
  *  output:  none
  *  return:  none
  *  notice:  These value need be modified if any changes.
- * 
*/
+ */
 void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
 {
PRT_HIGH_THROUGHPUT pHTInfo = ieee->pHTInfo;
@@ -93,8 +94,10 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
// 8190 only. Assign duration operation mode to firmware
pMgntInfo->bTxEnableFwCalcDur = 
(BOOLEAN)pNdisCommon->bRegTxEnableFwCalcDur;
 #endif
-   // 8190 only, Realtek proprietary aggregation mode
-   // Set MPDUDensity=2,   1: Set MPDUDensity=2(32k)  for Realtek AP and 
set MPDUDensity=0(8k) for others
+   /*
+* 8190 only, Realtek proprietary aggregation mode
+* Set MPDUDensity=2,   1: Set MPDUDensity=2(32k)  for Realtek AP and 
set MPDUDensity=0(8k) for others
+*/
pHTInfo->bRegRT2RTAggregation = 1;//0: Set MPDUDensity=2,   1: Set 
MPDUDensity=2(32k)  for Realtek AP and set MPDUDensity=0(8k) for others
 
// For Rx Reorder Control
@@ -113,14 +116,14 @@ void HTUpdateDefaultSetting(struct ieee80211_device *ieee)
 #endif
 }
 
-/
+/*
  *function:  This function print out each field on HT capability IE mainly 
from (Beacon/ProbeRsp/AssocReq)
  *   input:  u8*   CapIE   //Capability IE to be printed out
  *  u8*TitleString //mainly print out caller function
  *  output:  none
  *  return:  none
  *  notice:  Driver should not print out this message by default.
- * 
*/
+ */
 void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)
 {
static u8   EWC11NHTCap[] = {0x00, 0x90, 0x4c, 0x33};   // For 
11n EWC definition, 2007.07.17, by Emily
@@ -147,14 +150,14 @@ void HTDebugHTCapability(u8 *CapIE, u8 *TitleString)
pCapELE->MCS[1], pCapELE->MCS[2], 
pCapELE->MCS[3], pCapELE->MCS[4]);
 }
 
-/
+/*
  *function:  This function print out each field on HT Information IE mainly 
from (Beacon/ProbeRsp)
  *   input:  u8*   InfoIE   //Capability IE to be printed out
  *  u8*TitleString //mainly print out caller function
  *  output:  none
  *  return:  none
  *  notice:  Driver should not print out this message by default.
- * 
*/
+ */
 void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)
 {
static u8   EWC11NHTInfo[] = {0x00, 0x90, 0x4c, 0x34};  // For 
11n EWC definition, 2007.07.17, by Emily
@@ -209,8 +212,8 @@ void HTDebugHTInfo(u8 *InfoIE, u8 *TitleString)
 }
 
 /*
-*  Return: true if station in half n mode and AP supports 40 bw
-*/
+ * Return: true if station in half n mode and AP supports 40 bw
+ */
 static bool IsHTHalfNmode40Bandwidth(struct ieee80211_device *ieee)
 {
boolretValue = false;
@@ -276,14 +279,14 @@ u16 HTMcsToDataRate(struct ieee80211_device *ieee, u8 
nMcsRate)
return MCS_DATA_RATE[is40MHz][isShortGI][(nMcsRate & 0x7f)];
 }
 
-/
+/*
  *function:  This function returns current datarate.