diff options
| author | Johan Hovold <johan+linaro@kernel.org> | 2023-06-30 16:27:14 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-06-30 17:55:26 +0200 |
| commit | 5db9fe9562aaefca3c972401a9e3d6454707b1d5 (patch) | |
| tree | 405e06dc37a18b96eb19fe788160f8e2ba913190 | |
| parent | ASoC: codecs: wcd938x: fix mbhc impedance loglevel (diff) | |
| download | linux-5db9fe9562aaefca3c972401a9e3d6454707b1d5.tar.xz linux-5db9fe9562aaefca3c972401a9e3d6454707b1d5.zip | |
ASoC: codecs: wcd938x: drop inline keywords
The compiler is generally better at deciding what functions should be
inlined than driver authors are.
Drop the bogus inline keyword from two functions that were neither very
small or used in a single place.
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
Reviewed-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Link: https://lore.kernel.org/r/20230630142717.5314-3-johan+linaro@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/codecs/wcd938x.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/soc/codecs/wcd938x.c b/sound/soc/codecs/wcd938x.c index 81d865276c9c..63756f0a6a53 100644 --- a/sound/soc/codecs/wcd938x.c +++ b/sound/soc/codecs/wcd938x.c @@ -2124,7 +2124,7 @@ static int wcd938x_mbhc_micb_ctrl_threshold_mic(struct snd_soc_component *compon return wcd938x_mbhc_micb_adjust_voltage(component, micb_mv, MIC_BIAS_2); } -static inline void wcd938x_mbhc_get_result_params(struct wcd938x_priv *wcd938x, +static void wcd938x_mbhc_get_result_params(struct wcd938x_priv *wcd938x, s16 *d1_a, u16 noff, int32_t *zdet) { @@ -2231,8 +2231,8 @@ z_right: *zr = zdet; } -static inline void wcd938x_wcd_mbhc_qfuse_cal(struct snd_soc_component *component, - int32_t *z_val, int flag_l_r) +static void wcd938x_wcd_mbhc_qfuse_cal(struct snd_soc_component *component, + int32_t *z_val, int flag_l_r) { s16 q1; int q1_cal; |