]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
staging: rtl8723bs: remove dump_chip_info() function
authorNikolay Kulikov <nikolayof23@gmail.com>
Tue, 21 Apr 2026 05:15:15 +0000 (08:15 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 27 Apr 2026 11:01:56 +0000 (05:01 -0600)
The dump_chip_info() function formats chip version information into a
local 128-byte buffer using the scnprintf(). This buffer was previously
passed to the DBG_871X macro.
Commit 968b15adb0ea ("staging: rtl8723bs: remove all DBG_871X logs")
removed the macro, leaving the buffer formatted but never used or output
anywhere.

dump_chip_info() is now useless, so remove it and its call from
ReadChipVersion8723B().

Signed-off-by: Nikolay Kulikov <nikolayof23@gmail.com>
Reviewed-by: Luka Gejak <luka.gejak@linux.dev>
Link: https://patch.msgid.link/20260421051551.1694-1-nikolayof23@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/hal/hal_com.c
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
drivers/staging/rtl8723bs/include/hal_com.h

index b99dee6a7ed37b5b4569c66cc53b7c5a82c56137..b4b135a9ca22675ba6e8103dc51ab9406a8d955a 100644 (file)
@@ -30,48 +30,6 @@ void rtw_hal_data_deinit(struct adapter *padapter)
        }
 }
 
-
-void dump_chip_info(struct hal_version chip_version)
-{
-       char buf[128];
-       size_t cnt = 0;
-
-       cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "Chip Version Info: CHIP_8723B_%s_",
-                       IS_NORMAL_CHIP(chip_version) ? "Normal_Chip" : "Test_Chip");
-
-       if (IS_CHIP_VENDOR_TSMC(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "TSMC_");
-       else if (IS_CHIP_VENDOR_UMC(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "UMC_");
-       else if (IS_CHIP_VENDOR_SMIC(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "SMIC_");
-
-       if (IS_A_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "A_CUT_");
-       else if (IS_B_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "B_CUT_");
-       else if (IS_C_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "C_CUT_");
-       else if (IS_D_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "D_CUT_");
-       else if (IS_E_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "E_CUT_");
-       else if (IS_I_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "I_CUT_");
-       else if (IS_J_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "J_CUT_");
-       else if (IS_K_CUT(chip_version))
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "K_CUT_");
-       else
-               cnt += scnprintf(buf + cnt, sizeof(buf) - cnt,
-                               "UNKNOWN_CUT(%d)_", chip_version.CUTVersion);
-
-       cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "1T1R_");
-
-       cnt += scnprintf(buf + cnt, sizeof(buf) - cnt, "RomVer(%d)\n", chip_version.ROMVer);
-}
-
-
 #define        EEPROM_CHANNEL_PLAN_BY_HW_MASK  0x80
 
 /*
index f20f0a60fec135082fc0236d8274b52e2a9059d5..74cc2de963f74bc38040294c6499cd9fefedec37 100644 (file)
@@ -856,8 +856,6 @@ static struct hal_version ReadChipVersion8723B(struct adapter *padapter)
        pHalData->MultiFunc |= ((value32 & GPS_FUNC_EN) ? RT_MULTI_FUNC_GPS : 0);
        pHalData->PolarityCtl = ((value32 & WL_HWPDN_SL) ? RT_POLARITY_HIGH_ACT : RT_POLARITY_LOW_ACT);
 
-       dump_chip_info(ChipVersion);
-
        pHalData->VersionID = ChipVersion;
 
        return ChipVersion;
index 483f0390addcaec58aede39094f26c8cae9b3fcd..cf775585c8e228e68cf1261c92f4965c88bcb7af 100644 (file)
@@ -92,8 +92,6 @@ enum rt_media_status {
 u8 rtw_hal_data_init(struct adapter *padapter);
 void rtw_hal_data_deinit(struct adapter *padapter);
 
-void dump_chip_info(struct hal_version ChipVersion);
-
 u8 /* return the final channel plan decision */
 hal_com_config_channel_plan(
 struct adapter *padapter,