Unverified Commit 899b1254 authored by Bard Liao's avatar Bard Liao Committed by Mark Brown
Browse files

ASoC: rt711: add snd_soc_component remove callback 



We do some IO operations in the snd_soc_component_set_jack callback
function and snd_soc_component_set_jack() will be called when soc
component is removed. However, we should not access SoundWire registers
when the bus is suspended.
So set regcache_cache_only(regmap, true) to avoid accessing in the
soc component removal process.

Signed-off-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: default avatarKai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: default avatarRander Wang <rander.wang@intel.com>
Link: https://lore.kernel.org/r/20210316005254.29699-1-yung-chuan.liao@linux.intel.com


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

sound/soc/codecs/rt711.c

+8 −0
Original line number Diff line number Diff line
@@ -895,6 +895,13 @@ static int rt711_probe(struct snd_soc_component *component) 
	return 0;

}



static void rt711_remove(struct snd_soc_component *component)

{

	struct rt711_priv *rt711 = snd_soc_component_get_drvdata(component);



	regcache_cache_only(rt711->regmap, true);

}



static const struct snd_soc_component_driver soc_codec_dev_rt711 = {

	.probe = rt711_probe,

	.set_bias_level = rt711_set_bias_level,
@@ -905,6 +912,7 @@ static const struct snd_soc_component_driver soc_codec_dev_rt711 = { 
	.dapm_routes = rt711_audio_map,

	.num_dapm_routes = ARRAY_SIZE(rt711_audio_map),

	.set_jack = rt711_set_jack_detect,

	.remove = rt711_remove,

};



static int rt711_set_sdw_stream(struct snd_soc_dai *dai, void *sdw_stream,