Unverified Commit 2ba90789 authored by Charles Keepax's avatar Charles Keepax Committed by Mark Brown
Browse files

ASoC: wm_adsp: Remove pointless string comparison 



The control fw_name is always directly assigned from the wm_adsp_fw_text
array, so it isn't necessary to compare the actual strings just the
pointer values.

Signed-off-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210626155941.12251-3-ckeepax@opensource.cirrus.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 4d0b79ec

sound/soc/codecs/wm_adsp.c

+3 −4
Original line number Diff line number Diff line
@@ -2030,8 +2030,7 @@ static struct wm_coeff_ctl *wm_adsp_get_ctl(struct wm_adsp *dsp, 
		if (!pos->subname)

			continue;

		if (strncmp(pos->subname, name, pos->subname_len) == 0 &&

		    strncmp(pos->fw_name, fw_txt,

			    SNDRV_CTL_ELEM_ID_NAME_MAXLEN) == 0 &&

		    pos->fw_name == fw_txt &&

		    pos->alg_region.alg == alg &&

		    pos->alg_region.type == type) {

			rslt = pos;