Commit b38892b5 authored by Srinivasa Rao Mandadapu's avatar Srinivasa Rao Mandadapu Committed by Mark Brown

ASoC: codecs: MBHC: Remove useless condition check

Remove redundant conditional check and clean code in special
headset support functions.
Signed-off-by: default avatarSrinivasa Rao Mandadapu <srivasam@codeaurora.org>
Co-developed-by: default avatarVenkata Prasad Potturu <potturu@codeaurora.org>
Signed-off-by: default avatarVenkata Prasad Potturu <potturu@codeaurora.org>
Link: https://lore.kernel.org/r/1637234411-554-1-git-send-email-srivasam@codeaurora.orgSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent b6a5f4f0
...@@ -1055,12 +1055,8 @@ static bool wcd_mbhc_check_for_spl_headset(struct wcd_mbhc *mbhc) ...@@ -1055,12 +1055,8 @@ static bool wcd_mbhc_check_for_spl_headset(struct wcd_mbhc *mbhc)
hs_threshold = wcd_mbhc_get_spl_hs_thres(mbhc); hs_threshold = wcd_mbhc_get_spl_hs_thres(mbhc);
hph_threshold = wcd_mbhc_adc_get_hph_thres(mbhc); hph_threshold = wcd_mbhc_adc_get_hph_thres(mbhc);
if (output_mv > hs_threshold || output_mv < hph_threshold) { if (!(output_mv > hs_threshold || output_mv < hph_threshold))
if (mbhc->force_linein == true)
is_spl_hs = false;
} else {
is_spl_hs = true; is_spl_hs = true;
}
/* Back MIC_BIAS2 to 1.8v if the type is not special headset */ /* Back MIC_BIAS2 to 1.8v if the type is not special headset */
if (!is_spl_hs) { if (!is_spl_hs) {
...@@ -1149,13 +1145,13 @@ static void wcd_correct_swch_plug(struct work_struct *work) ...@@ -1149,13 +1145,13 @@ static void wcd_correct_swch_plug(struct work_struct *work)
plug_type = wcd_mbhc_get_plug_from_adc(mbhc, output_mv); plug_type = wcd_mbhc_get_plug_from_adc(mbhc, output_mv);
is_pa_on = wcd_mbhc_read_field(mbhc, WCD_MBHC_HPH_PA_EN); is_pa_on = wcd_mbhc_read_field(mbhc, WCD_MBHC_HPH_PA_EN);
if ((output_mv > hs_threshold) && (!is_spl_hs)) { if (output_mv > hs_threshold && !is_spl_hs) {
is_spl_hs = wcd_mbhc_check_for_spl_headset(mbhc); is_spl_hs = wcd_mbhc_check_for_spl_headset(mbhc);
output_mv = wcd_measure_adc_once(mbhc, MUX_CTL_IN2P); output_mv = wcd_measure_adc_once(mbhc, MUX_CTL_IN2P);
if (is_spl_hs) { if (is_spl_hs) {
hs_threshold = (hs_threshold * wcd_mbhc_get_micbias(mbhc)) / hs_threshold *= wcd_mbhc_get_micbias(mbhc);
micbias_mv; hs_threshold /= micbias_mv;
} }
} }
...@@ -1185,7 +1181,7 @@ static void wcd_correct_swch_plug(struct work_struct *work) ...@@ -1185,7 +1181,7 @@ static void wcd_correct_swch_plug(struct work_struct *work)
} }
/* cable is extension cable */ /* cable is extension cable */
if (output_mv > hs_threshold || mbhc->force_linein == true) if (output_mv > hs_threshold || mbhc->force_linein)
plug_type = MBHC_PLUG_TYPE_HIGH_HPH; plug_type = MBHC_PLUG_TYPE_HIGH_HPH;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment