mirror of
https://github.com/torvalds/linux.git
synced 2026-07-27 09:36:22 +02:00
wifi: rtw89: switch to using FIELD_GET_SIGNED()
Switch from sign_extend32(FIELD_GET()) to the dedicated FIELD_GET_SIGNED() and don't calculate the fields length explicitly. Acked-by: Ping-Ke Shih <pkshih@realtek.com> Signed-off-by: Yury Norov <ynorov@nvidia.com>
This commit is contained in:
parent
1af735749f
commit
115e1f213c
|
|
@ -334,8 +334,8 @@ static void _check_addc(struct rtw89_dev *rtwdev, enum rtw89_rf_path path)
|
|||
|
||||
for (i = 0; i < ADDC_T_AVG; i++) {
|
||||
tmp = rtw89_phy_read32_mask(rtwdev, R_DBG32_D, MASKDWORD);
|
||||
dc_re += sign_extend32(FIELD_GET(0xfff000, tmp), 11);
|
||||
dc_im += sign_extend32(FIELD_GET(0xfff, tmp), 11);
|
||||
dc_re += FIELD_GET_SIGNED(0xfff000, tmp);
|
||||
dc_im += FIELD_GET_SIGNED(0xfff, tmp);
|
||||
}
|
||||
|
||||
dc_re /= ADDC_T_AVG;
|
||||
|
|
|
|||
|
|
@ -206,9 +206,9 @@ static void rtw8852bx_efuse_parsing_tssi(struct rtw89_dev *rtwdev,
|
|||
static bool _decode_efuse_gain(u8 data, s8 *high, s8 *low)
|
||||
{
|
||||
if (high)
|
||||
*high = sign_extend32(FIELD_GET(GENMASK(7, 4), data), 3);
|
||||
*high = FIELD_GET_SIGNED(GENMASK(7, 4), data);
|
||||
if (low)
|
||||
*low = sign_extend32(FIELD_GET(GENMASK(3, 0), data), 3);
|
||||
*low = FIELD_GET_SIGNED(GENMASK(3, 0), data);
|
||||
|
||||
return data != 0xff;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -497,8 +497,8 @@ static void _check_addc(struct rtw89_dev *rtwdev, enum rtw89_rf_path path)
|
|||
|
||||
for (i = 0; i < ADDC_T_AVG; i++) {
|
||||
tmp = rtw89_phy_read32_mask(rtwdev, R_DBG32_D, MASKDWORD);
|
||||
dc_re += sign_extend32(FIELD_GET(0xfff000, tmp), 11);
|
||||
dc_im += sign_extend32(FIELD_GET(0xfff, tmp), 11);
|
||||
dc_re += FIELD_GET_SIGNED(0xfff000, tmp);
|
||||
dc_im += FIELD_GET_SIGNED(0xfff, tmp);
|
||||
}
|
||||
|
||||
dc_re /= ADDC_T_AVG;
|
||||
|
|
|
|||
|
|
@ -517,9 +517,9 @@ static void rtw8852c_efuse_parsing_tssi(struct rtw89_dev *rtwdev,
|
|||
static bool _decode_efuse_gain(u8 data, s8 *high, s8 *low)
|
||||
{
|
||||
if (high)
|
||||
*high = sign_extend32(FIELD_GET(GENMASK(7, 4), data), 3);
|
||||
*high = FIELD_GET_SIGNED(GENMASK(7, 4), data);
|
||||
if (low)
|
||||
*low = sign_extend32(FIELD_GET(GENMASK(3, 0), data), 3);
|
||||
*low = FIELD_GET_SIGNED(GENMASK(3, 0), data);
|
||||
|
||||
return data != 0xff;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user