wifi: ath9k: use clamp() in ar9003_aic_cal_post_process()
authorLi Zetao <lizetao1@huawei.com>
Sat, 14 Sep 2024 09:06:02 +0000 (12:06 +0300)
committerKalle Valo <quic_kvalo@quicinc.com>
Sat, 14 Sep 2024 11:45:57 +0000 (14:45 +0300)
When it needs to get a value within a certain interval, using clamp()
makes the code easier to understand than min(max()).

Signed-off-by: Li Zetao <lizetao1@huawei.com>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://patch.msgid.link/20240830011858.603514-1-lizetao1@huawei.com
drivers/net/wireless/ath/ath9k/ar9003_aic.c

index d0f1e8bcc846b5953634e6e02aca200ba1c57663..45a7ca660f47a11c65a63b666f42fac690ee96f6 100644 (file)
@@ -409,13 +409,11 @@ static bool ar9003_aic_cal_post_process(struct ath_hw *ah)
                sram.valid = true;
 
                sram.rot_dir_att_db =
-                       min(max(rot_dir_path_att_db,
-                               (int16_t)ATH_AIC_MIN_ROT_DIR_ATT_DB),
-                           ATH_AIC_MAX_ROT_DIR_ATT_DB);
+                       clamp(rot_dir_path_att_db, (int16_t)ATH_AIC_MIN_ROT_DIR_ATT_DB,
+                             ATH_AIC_MAX_ROT_DIR_ATT_DB);
                sram.rot_quad_att_db =
-                       min(max(rot_quad_path_att_db,
-                               (int16_t)ATH_AIC_MIN_ROT_QUAD_ATT_DB),
-                           ATH_AIC_MAX_ROT_QUAD_ATT_DB);
+                       clamp(rot_quad_path_att_db, (int16_t)ATH_AIC_MIN_ROT_QUAD_ATT_DB,
+                             ATH_AIC_MAX_ROT_QUAD_ATT_DB);
 
                aic->aic_sram[i] = (SM(sram.vga_dir_sign,
                                       AR_PHY_AIC_SRAM_VGA_DIR_SIGN) |