staging: r8188eu: ExternalPA is read-only
authorMichael Straube <straube.linux@gmail.com>
Tue, 1 Feb 2022 08:42:51 +0000 (09:42 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 4 Feb 2022 14:16:33 +0000 (15:16 +0100)
The variable ExternalPA in struct hal_data_8188e is never set.
It stays at the default value 0. Remove it and remove related
dead code.

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Link: https://lore.kernel.org/r/20220201084259.7497-2-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/hal/rtl8188e_rf6052.c
drivers/staging/r8188eu/include/rtl8188e_hal.h

index d10a6cc2d4ab0d78f833f83f12edeabddbef0d61..9b06a23df2ac12319f53ebbdaf827f9eec976d38 100644 (file)
@@ -99,9 +99,6 @@ rtl8188e_PHY_RF6052SetCckTxPower(
                        TxAGC[idx1] =
                                pPowerlevel[idx1] | (pPowerlevel[idx1] << 8) |
                                (pPowerlevel[idx1] << 16) | (pPowerlevel[idx1] << 24);
-                       /*  2010/10/18 MH For external PA module. We need to limit power index to be less than 0x20. */
-                       if (TxAGC[idx1] > 0x20 && pHalData->ExternalPA)
-                               TxAGC[idx1] = 0x20;
                }
        } else {
                for (idx1 = RF_PATH_A; idx1 <= RF_PATH_B; idx1++) {
index a24cdbfa83de50a8b4682ad5cf61647adfa37be1..2de7c0464411e3c4e213f0213902fe6c83896089 100644 (file)
@@ -181,7 +181,6 @@ struct hal_data_8188e {
        u32     MCSTxPowerLevelOriginalOffset[MAX_PG_GROUP][16];
 
        u8      CrystalCap;
-       u8      ExternalPA;
 
        u32     AcParam_BE; /* Original parameter for BE, use for EDCA turbo. */