staging: rtl8723bs: standardize comment style in HAL

The kernel coding style for comments requires a single space after the
"/*". Currently, many files in HAL contain two spaces after the "/*", or
use irregular indentation. The modified files also suffer from this.

Fix line comment style inconsistencies by removing additional space after
"/*" to adhere to kernel coding standards.

Signed-off-by: Mahad Ibrahim <mahad.ibrahim.dev@gmail.com>
Link: https://patch.msgid.link/20260129160426.13737-6-mahad.ibrahim.dev@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/rtl8723bs/hal/HalHWImg8723B_BB.c b/drivers/staging/rtl8723bs/hal/HalHWImg8723B_BB.c
index 988636a1611242f1508a78d244e370b91c271ac1..4666b2ff3157f71607b81c00212eb86b4c04e8f2 100644 (file)
--- a/drivers/staging/rtl8723bs/hal/HalHWImg8723B_BB.c
+++ b/drivers/staging/rtl8723bs/hal/HalHWImg8723B_BB.c
@@ -11,11 +11,11 @@
 static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const u32 Condition2)
 {
        u8 _BoardType =
-               ((pDM_Odm->BoardType & BIT4) >> 4) << 0 | /*  _GLNA */
-               ((pDM_Odm->BoardType & BIT3) >> 3) << 1 | /*  _GPA */
-               ((pDM_Odm->BoardType & BIT7) >> 7) << 2 | /*  _ALNA */
-               ((pDM_Odm->BoardType & BIT6) >> 6) << 3 | /*  _APA */
-               ((pDM_Odm->BoardType & BIT2) >> 2) << 4;  /*  _BT */
+               ((pDM_Odm->BoardType & BIT4) >> 4) << 0 | /* _GLNA */
+               ((pDM_Odm->BoardType & BIT3) >> 3) << 1 | /* _GPA */
+               ((pDM_Odm->BoardType & BIT7) >> 7) << 2 | /* _ALNA */
+               ((pDM_Odm->BoardType & BIT6) >> 6) << 3 | /* _APA */
+               ((pDM_Odm->BoardType & BIT2) >> 2) << 4;  /* _BT */
 
        u32 cond1 = Condition1, cond2 = Condition2;
        u32 driver1 =
@@ -31,7 +31,7 @@ static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const
                pDM_Odm->TypeALNA << 16 |
                pDM_Odm->TypeAPA << 24;
 
-       /*  Value Defined Check =============== */
+       /* Value Defined Check =============== */
        /* QFN Type [15:12] and Cut Version [27:24] need to do value check */
 
        if (((cond1 & 0x0000F000) != 0) && ((cond1 & 0x0000F000) != (driver1 & 0x0000F000)))
@@ -39,16 +39,16 @@ static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const
        if (((cond1 & 0x0F000000) != 0) && ((cond1 & 0x0F000000) != (driver1 & 0x0F000000)))
                return false;
 
-       /*  Bit Defined Check ================ */
-       /*  We don't care [31:28] and [23:20] */
-       /*  */
+       /* Bit Defined Check ================ */
+       /* We don't care [31:28] and [23:20] */
+       /* */
        cond1   &= 0x000F0FFF;
        driver1 &= 0x000F0FFF;
 
        if ((cond1 & driver1) == cond1) {
                u32 bitMask = 0;
 
-               if ((cond1 & 0x0F) == 0) /*  BoardType is DONTCARE */
+               if ((cond1 & 0x0F) == 0) /* BoardType is DONTCARE */
                        return true;
 
                if ((cond1 & BIT0) != 0) /* GLNA */
@@ -60,7 +60,7 @@ static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const
                if ((cond1 & BIT3) != 0) /* APA */
                        bitMask |= 0xFF000000;
 
-               /*  BoardType of each RF path is matched */
+               /* BoardType of each RF path is matched */
                if ((cond2 & bitMask) == (driver2 & bitMask))
                        return true;
        }
@@ -216,16 +216,16 @@ void ODM_ReadAndConfig_MP_8723B_AGC_TAB(struct dm_odm_t *pDM_Odm)
                u32 v1 = Array[i];
                u32 v2 = Array[i + 1];
 
-               /*  This (offset, data) pair doesn't care the condition. */
+               /* This (offset, data) pair doesn't care the condition. */
                if (v1 < 0x40000000) {
                        odm_ConfigBB_AGC_8723B(pDM_Odm, v1, bMaskDWord, v2);
                        continue;
                } else {
-                       /*  This line is the beginning of branch. */
+                       /* This line is the beginning of branch. */
                        bool bMatched = true;
                        u8  cCond  = (u8)((v1 & (BIT29 | BIT28)) >> 28);
 
-                       if (cCond == COND_ELSE) { /*  ELSE, ENDIF */
+                       if (cCond == COND_ELSE) { /* ELSE, ENDIF */
                                bMatched = true;
                                READ_NEXT_PAIR(v1, v2, i);
                        } else if (!CheckPositive(pDM_Odm, v1, v2)) {
@@ -240,21 +240,21 @@ void ODM_ReadAndConfig_MP_8723B_AGC_TAB(struct dm_odm_t *pDM_Odm)
 
                        if (!bMatched) {
                                /*
-                                *   Condition isn't matched.
-                                *   Discard the following (offset, data) pairs.
+                                * Condition isn't matched.
+                                * Discard the following (offset, data) pairs.
                                 */
                                while (v1 < 0x40000000 && i < ArrayLen - 2)
                                        READ_NEXT_PAIR(v1, v2, i);
 
-                               i -= 2; /*  prevent from for-loop += 2 */
+                               i -= 2; /* prevent from for-loop += 2 */
                        } else {
-                               /*  Configure matched pairs and skip to end of if-else. */
+                               /* Configure matched pairs and skip to end of if-else. */
                                while (v1 < 0x40000000 && i < ArrayLen - 2) {
                                        odm_ConfigBB_AGC_8723B(pDM_Odm, v1, bMaskDWord, v2);
                                        READ_NEXT_PAIR(v1, v2, i);
                                }
 
-                               /*  Keeps reading until ENDIF. */
+                               /* Keeps reading until ENDIF. */
                                cCond = (u8)((v1 & (BIT29 | BIT28)) >> 28);
                                while (cCond != COND_ENDIF && i < ArrayLen - 2) {
                                        READ_NEXT_PAIR(v1, v2, i);
@@ -476,16 +476,16 @@ void ODM_ReadAndConfig_MP_8723B_PHY_REG(struct dm_odm_t *pDM_Odm)
                u32 v1 = Array[i];
                u32 v2 = Array[i + 1];
 
-               /*  This (offset, data) pair doesn't care the condition. */
+               /* This (offset, data) pair doesn't care the condition. */
                if (v1 < 0x40000000) {
                        odm_ConfigBB_PHY_8723B(pDM_Odm, v1, bMaskDWord, v2);
                        continue;
                } else {
-                       /*  This line is the beginning of branch. */
+                       /* This line is the beginning of branch. */
                        bool bMatched = true;
                        u8  cCond  = (u8)((v1 & (BIT29 | BIT28)) >> 28);
 
-                       if (cCond == COND_ELSE) { /*  ELSE, ENDIF */
+                       if (cCond == COND_ELSE) { /* ELSE, ENDIF */
                                bMatched = true;
                                READ_NEXT_PAIR(v1, v2, i);
                        } else if (!CheckPositive(pDM_Odm, v1, v2)) {
@@ -500,20 +500,20 @@ void ODM_ReadAndConfig_MP_8723B_PHY_REG(struct dm_odm_t *pDM_Odm)
 
                        if (!bMatched) {
                                /*
-                                *   Condition isn't matched.
-                                *   Discard the following (offset, data) pairs.
+                                * Condition isn't matched.
+                                * Discard the following (offset, data) pairs.
                                 */
                                while (v1 < 0x40000000 && i < ArrayLen - 2)
                                        READ_NEXT_PAIR(v1, v2, i);
 
-                               i -= 2; /*  prevent from for-loop += 2 */
-                       } else { /*  Configure matched pairs and skip to end of if-else. */
+                               i -= 2; /* prevent from for-loop += 2 */
+                       } else { /* Configure matched pairs and skip to end of if-else. */
                                while (v1 < 0x40000000 && i < ArrayLen - 2) {
                                        odm_ConfigBB_PHY_8723B(pDM_Odm, v1, bMaskDWord, v2);
                                        READ_NEXT_PAIR(v1, v2, i);
                                }
 
-                               /*  Keeps reading until ENDIF. */
+                               /* Keeps reading until ENDIF. */
                                cCond = (u8)((v1 & (BIT29 | BIT28)) >> 28);
                                while (cCond != COND_ENDIF && i < ArrayLen - 2) {
                                        READ_NEXT_PAIR(v1, v2, i);

diff --git a/drivers/staging/rtl8723bs/hal/HalHWImg8723B_MAC.c b/drivers/staging/rtl8723bs/hal/HalHWImg8723B_MAC.c
index e428884335a8b8f1d0eb990e70041b46ebcd808b..9a3393f5122b4804ad8bbb17113f7efdce2f3b6a 100644 (file)
--- a/drivers/staging/rtl8723bs/hal/HalHWImg8723B_MAC.c
+++ b/drivers/staging/rtl8723bs/hal/HalHWImg8723B_MAC.c
@@ -11,11 +11,11 @@
 static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const u32 Condition2)
 {
        u8 _BoardType =
-               ((pDM_Odm->BoardType & BIT4) >> 4) << 0 | /*  _GLNA */
-               ((pDM_Odm->BoardType & BIT3) >> 3) << 1 | /*  _GPA */
-               ((pDM_Odm->BoardType & BIT7) >> 7) << 2 | /*  _ALNA */
-               ((pDM_Odm->BoardType & BIT6) >> 6) << 3 | /*  _APA */
-               ((pDM_Odm->BoardType & BIT2) >> 2) << 4;  /*  _BT */
+               ((pDM_Odm->BoardType & BIT4) >> 4) << 0 | /* _GLNA */
+               ((pDM_Odm->BoardType & BIT3) >> 3) << 1 | /* _GPA */
+               ((pDM_Odm->BoardType & BIT7) >> 7) << 2 | /* _ALNA */
+               ((pDM_Odm->BoardType & BIT6) >> 6) << 3 | /* _APA */
+               ((pDM_Odm->BoardType & BIT2) >> 2) << 4;  /* _BT */
 
        u32   cond1   = Condition1, cond2 = Condition2;
        u32    driver1 =
@@ -31,7 +31,7 @@ static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const
                pDM_Odm->TypeALNA << 16 |
                pDM_Odm->TypeAPA  << 24;
 
-       /*  Value Defined Check =============== */
+       /* Value Defined Check =============== */
        /* QFN Type [15:12] and Cut Version [27:24] need to do value check */
 
        if (((cond1 & 0x0000F000) != 0) && ((cond1 & 0x0000F000) != (driver1 & 0x0000F000)))
@@ -39,16 +39,16 @@ static bool CheckPositive(struct dm_odm_t *pDM_Odm, const u32 Condition1, const
        if (((cond1 & 0x0F000000) != 0) && ((cond1 & 0x0F000000) != (driver1 & 0x0F000000)))
                return false;
 
-       /*  Bit Defined Check ================ */
-       /*  We don't care [31:28] and [23:20] */
-       /*  */
+       /* Bit Defined Check ================ */
+       /* We don't care [31:28] and [23:20] */
+       /* */
        cond1   &= 0x000F0FFF;
        driver1 &= 0x000F0FFF;
 
        if ((cond1 & driver1) == cond1) {
                u32 bitMask = 0;
 
-               if ((cond1 & 0x0F) == 0) /*  BoardType is DONTCARE */
+               if ((cond1 & 0x0F) == 0) /* BoardType is DONTCARE */
                        return true;
 
                if ((cond1 & BIT0) != 0) /* GLNA */
@@ -188,16 +188,16 @@ void ODM_ReadAndConfig_MP_8723B_MAC_REG(struct dm_odm_t *pDM_Odm)
                u32 v1 = Array[i];
                u32 v2 = Array[i + 1];
 
-               /*  This (offset, data) pair doesn't care the condition. */
+               /* This (offset, data) pair doesn't care the condition. */
                if (v1 < 0x40000000) {
                        odm_ConfigMAC_8723B(pDM_Odm, v1, (u8)v2);
                        continue;
                } else {
-                       /*  This line is the beginning of branch. */
+                       /* This line is the beginning of branch. */
                        bool bMatched = true;
                        u8  cCond  = (u8)((v1 & (BIT29 | BIT28)) >> 28);
 
-                       if (cCond == COND_ELSE) { /*  ELSE, ENDIF */
+                       if (cCond == COND_ELSE) { /* ELSE, ENDIF */
                                bMatched = true;
                                READ_NEXT_PAIR(v1, v2, i);
                        } else if (!CheckPositive(pDM_Odm, v1, v2)) {
@@ -218,14 +218,14 @@ void ODM_ReadAndConfig_MP_8723B_MAC_REG(struct dm_odm_t *pDM_Odm)
                                while (v1 < 0x40000000 && i < ArrayLen - 2)
                                        READ_NEXT_PAIR(v1, v2, i);
 
-                               i -= 2; /*  prevent from for-loop += 2 */
-                       } else { /*  Configure matched pairs and skip to end of if-else. */
+                               i -= 2; /* prevent from for-loop += 2 */
+                       } else { /* Configure matched pairs and skip to end of if-else. */
                                while (v1 < 0x40000000 && i < ArrayLen - 2) {
                                        odm_ConfigMAC_8723B(pDM_Odm, v1, (u8)v2);
                                        READ_NEXT_PAIR(v1, v2, i);
                                }
 
-                               /*  Keeps reading until ENDIF. */
+                               /* Keeps reading until ENDIF. */
                                cCond = (u8)((v1 & (BIT29 | BIT28)) >> 28);
                                while (cCond != COND_ENDIF && i < ArrayLen - 2) {
                                        READ_NEXT_PAIR(v1, v2, i);