lkml.org 
[lkml]   [2022]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.16 0327/1039] iwlwifi: mvm: fix 32-bit build in FTM
    Date
    From: Johannes Berg <johannes.berg@intel.com>

    [ Upstream commit 8b0f92549f2c2458200935c12a2e2a6e80234cf5 ]

    On a 32-bit build, the division here needs to be done
    using do_div(), otherwise the compiler will try to call
    a function that doesn't exist, thus failing to build.

    Fixes: b68bd2e3143a ("iwlwifi: mvm: Add FTM initiator RTT smoothing logic")
    Signed-off-by: Johannes Berg <johannes.berg@intel.com>
    Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
    Link: https://lore.kernel.org/r/iwlwifi.20211219111352.e56cbf614a4d.Ib98004ccd2c7a55fd883a8ea7eebd810f406dec6@changeid
    Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/wireless/intel/iwlwifi/mvm/ftm-initiator.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/ftm-initiator.c b/drivers/net/wireless/intel/iwlwifi/mvm/ftm-initiator.c
    index 949fb790f8fb7..3e6c13fc74eb0 100644
    --- a/drivers/net/wireless/intel/iwlwifi/mvm/ftm-initiator.c
    +++ b/drivers/net/wireless/intel/iwlwifi/mvm/ftm-initiator.c
    @@ -1066,7 +1066,8 @@ static void iwl_mvm_ftm_rtt_smoothing(struct iwl_mvm *mvm,
    overshoot = IWL_MVM_FTM_INITIATOR_SMOOTH_OVERSHOOT;
    alpha = IWL_MVM_FTM_INITIATOR_SMOOTH_ALPHA;

    - rtt_avg = (alpha * rtt + (100 - alpha) * resp->rtt_avg) / 100;
    + rtt_avg = alpha * rtt + (100 - alpha) * resp->rtt_avg;
    + do_div(rtt_avg, 100);

    IWL_DEBUG_INFO(mvm,
    "%pM: prev rtt_avg=%lld, new rtt_avg=%lld, rtt=%lld\n",
    --
    2.34.1


    \
     
     \ /
      Last update: 2022-09-17 16:12    [W:2.355 / U:0.204 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site