staging: vt6655: Type encoding info dropped from variable name "byBBVGANew"
authorPavan Bobba <opensource206@gmail.com>
Sun, 8 Oct 2023 19:01:50 +0000 (00:31 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 15 Oct 2023 16:21:05 +0000 (18:21 +0200)
variable name "byBBVGANew" updated like below:

a.type encoding info dropped from name
b.camelcase name replaced by snakecase

Issue found by checkpatch

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/abb105b822090a82ef081b2e28705bf14b8ce058.1696791459.git.opensource206@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6655/device.h
drivers/staging/vt6655/device_main.c

index 9ae7171d02ee28614ec7e57dc5ba9e0e03b97f89..68bfadacfa7cdd2f7d6bab05bda133b9aa09b7a7 100644 (file)
@@ -248,7 +248,7 @@ struct vnt_private {
        /* For Update BaseBand VGA Gain Offset */
        bool bUpdateBBVGA;
        unsigned int    uBBVGADiffCount;
-       unsigned char byBBVGANew;
+       unsigned char bbvga_new;
        unsigned char bbvga_current;
        unsigned char bbvga[BB_VGA_LEVEL];
        long                    dbm_threshold[BB_VGA_LEVEL];
index 6990129ceb10031d30c18eb5452e7fcf9eb06f79..b08fcf7e6edc296a4f14a0af3ee9eadadeb68727 100644 (file)
@@ -425,7 +425,7 @@ static void device_init_registers(struct vnt_private *priv)
 
        if (priv->bUpdateBBVGA) {
                priv->bbvga_current = priv->bbvga[0];
-               priv->byBBVGANew = priv->bbvga_current;
+               priv->bbvga_new = priv->bbvga_current;
                bb_set_vga_gain_offset(priv, priv->bbvga[0]);
        }
 
@@ -1053,12 +1053,12 @@ static void vnt_check_bb_vga(struct vnt_private *priv)
 
        for (i = 0; i < BB_VGA_LEVEL; i++) {
                if (dbm < priv->dbm_threshold[i]) {
-                       priv->byBBVGANew = priv->bbvga[i];
+                       priv->bbvga_new = priv->bbvga[i];
                        break;
                }
        }
 
-       if (priv->byBBVGANew == priv->bbvga_current) {
+       if (priv->bbvga_new == priv->bbvga_current) {
                priv->uBBVGADiffCount = 1;
                return;
        }
@@ -1067,11 +1067,11 @@ static void vnt_check_bb_vga(struct vnt_private *priv)
 
        if (priv->uBBVGADiffCount == 1) {
                /* first VGA diff gain */
-               bb_set_vga_gain_offset(priv, priv->byBBVGANew);
+               bb_set_vga_gain_offset(priv, priv->bbvga_new);
 
                dev_dbg(&priv->pcid->dev,
                        "First RSSI[%d] NewGain[%d] OldGain[%d] Count[%d]\n",
-                       (int)dbm, priv->byBBVGANew,
+                       (int)dbm, priv->bbvga_new,
                        priv->bbvga_current,
                        (int)priv->uBBVGADiffCount);
        }
@@ -1079,11 +1079,11 @@ static void vnt_check_bb_vga(struct vnt_private *priv)
        if (priv->uBBVGADiffCount >= BB_VGA_CHANGE_THRESHOLD) {
                dev_dbg(&priv->pcid->dev,
                        "RSSI[%d] NewGain[%d] OldGain[%d] Count[%d]\n",
-                       (int)dbm, priv->byBBVGANew,
+                       (int)dbm, priv->bbvga_new,
                        priv->bbvga_current,
                        (int)priv->uBBVGADiffCount);
 
-               bb_set_vga_gain_offset(priv, priv->byBBVGANew);
+               bb_set_vga_gain_offset(priv, priv->bbvga_new);
        }
 }