lkml.org 
[lkml]   [2018]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.2 009/104] rtl8192c:dm: Properly initialize local array and set value.
    3.2.101-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Han Shen <shenhan@google.com>

    commit 7c8f0db0d024efda38976fc2acf7743f458e1d96 upstream.

    GCC 4.8 is spitting out uninitialized-variable warnings against
    "drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c". This patch adds
    initialization to the variable and properly sets its value.

    Signed-off-by: Han Shen (shenhan@google.com)
    Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    --- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
    +++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
    @@ -672,7 +672,7 @@ static void rtl92c_dm_txpower_tracking_c
    u8 thermalvalue, delta, delta_lck, delta_iqk;
    long ele_a, ele_d, temp_cck, val_x, value32;
    long val_y, ele_c = 0;
    - u8 ofdm_index[2], ofdm_index_old[2], cck_index_old = 0;
    + u8 ofdm_index[2], ofdm_index_old[2] = {0, 0}, cck_index_old = 0;
    s8 cck_index = 0;
    int i;
    bool is2t = IS_92C_SERIAL(rtlhal->version);
    @@ -722,7 +722,7 @@ static void rtl92c_dm_txpower_tracking_c
    for (i = 0; i < OFDM_TABLE_LENGTH; i++) {
    if (ele_d == (ofdmswing_table[i] &
    MASKOFDM_D)) {
    -
    + ofdm_index_old[1] = (u8) i;
    RT_TRACE(rtlpriv, COMP_POWER_TRACKING,
    DBG_LOUD,
    ("Initial pathB ele_d reg0x%x = "
    \
     
     \ /
      Last update: 2018-03-12 04:53    [W:5.061 / U:0.400 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site