Unverified Commit 879142be authored by Alexandru Ardelean's avatar Alexandru Ardelean Committed by Mark Brown
Browse files

ASoC: codecs: tas5720: split a tas5720_mute_soc_component() function 



This is to be re-used in tas5720_mute() (which is part of the dai_ops) and
also in the tas5720_fault_check_work() hook.

The benefit here isn't too great (now).
It's only when we add support for a new device with a slightly different
regmap that this becomes more useful.

Signed-off-by: default avatarAlexandru Ardelean <alex@shruggie.ro>
Link: https://lore.kernel.org/r/20230128082744.41849-1-alex@shruggie.ro


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

sound/soc/codecs/tas5720.c

+7 −4
Original line number Diff line number Diff line
@@ -199,9 +199,8 @@ static int tas5720_set_dai_tdm_slot(struct snd_soc_dai *dai, 
	return ret;

}



static int tas5720_mute(struct snd_soc_dai *dai, int mute, int direction)

static int tas5720_mute_soc_component(struct snd_soc_component *component, int mute)

{

	struct snd_soc_component *component = dai->component;

	int ret;



	ret = snd_soc_component_update_bits(component, TAS5720_DIGITAL_CTRL2_REG,
@@ -214,6 +213,11 @@ static int tas5720_mute(struct snd_soc_dai *dai, int mute, int direction) 
	return 0;

}



static int tas5720_mute(struct snd_soc_dai *dai, int mute, int direction)

{

	return tas5720_mute_soc_component(dai->component, mute);

}



static void tas5720_fault_check_work(struct work_struct *work)

{

	struct tas5720_data *tas5720 = container_of(work, struct tas5720_data,
@@ -318,8 +322,7 @@ static int tas5720_codec_probe(struct snd_soc_component *component) 
			 expected_device_id, device_id);



	/* Set device to mute */

	ret = snd_soc_component_update_bits(component, TAS5720_DIGITAL_CTRL2_REG,

				  TAS5720_MUTE, TAS5720_MUTE);

	ret = tas5720_mute_soc_component(component, 1);

	if (ret < 0)

		goto error_snd_soc_component_update_bits;