Unverified Commit 316cd941 authored by Mark Brown's avatar Mark Brown
Browse files

ASoC: es7241: Use modern ASoC DAI format terminology 



As part of moving to remove the old style defines for the bus clocks update
the es7241 driver to use more modern terminology for clocking.

Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20220222223300.3120298-1-broonie@kernel.org


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

sound/soc/codecs/es7241.c

+15 −15
Original line number Diff line number Diff line
@@ -29,7 +29,7 @@ struct es7241_data { 
	struct gpio_desc *m1;

	unsigned int fmt;

	unsigned int mclk;

	bool is_slave;

	bool is_consumer;

	const struct es7241_chip *chip;

};
@@ -46,7 +46,7 @@ static void es7241_set_mode(struct es7241_data *priv, int m0, int m1) 
	gpiod_set_value_cansleep(priv->reset, 1);

}



static int es7241_set_slave_mode(struct es7241_data *priv,

static int es7241_set_consumer_mode(struct es7241_data *priv,

				    const struct es7241_clock_mode *mode,

				    unsigned int mfs)

{
@@ -67,7 +67,7 @@ static int es7241_set_slave_mode(struct es7241_data *priv, 
	return 0;

}



static int es7241_set_master_mode(struct es7241_data *priv,

static int es7241_set_provider_mode(struct es7241_data *priv,

				    const struct es7241_clock_mode *mode,

				    unsigned int mfs)

{
@@ -98,10 +98,10 @@ static int es7241_hw_params(struct snd_pcm_substream *substream, 
		if (rate < mode->rate_min || rate >= mode->rate_max)

			continue;



		if (priv->is_slave)

			return es7241_set_slave_mode(priv, mode, mfs);

		if (priv->is_consumer)

			return es7241_set_consumer_mode(priv, mode, mfs);

		else

			return es7241_set_master_mode(priv, mode, mfs);

			return es7241_set_provider_mode(priv, mode, mfs);

	}



	/* should not happen */
@@ -136,12 +136,12 @@ static int es7241_set_fmt(struct snd_soc_dai *dai, unsigned int fmt) 
		return -EINVAL;

	}



	switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) {

	case SND_SOC_DAIFMT_CBS_CFS:

		priv->is_slave = true;

	switch (fmt & SND_SOC_DAIFMT_CLOCK_PROVIDER_MASK) {

	case SND_SOC_DAIFMT_CBC_CFC:

		priv->is_consumer = true;

		break;

	case SND_SOC_DAIFMT_CBM_CFM:

		priv->is_slave = false;

	case SND_SOC_DAIFMT_CBP_CFP:

		priv->is_consumer = false;

		break;



	default: