Commit 85e59af2 authored by Markus Pargmann's avatar Markus Pargmann Committed by Mark Brown
Browse files

ASoC: fsl-ssi: make fsl,mode property optional 



The simple soundcard binding has its own way for specifying the dai
format. To be able to use this binding we have to make the fsl,mode
property optional. As the property is used in existing devicetrees
keep the option around for compatibility reasons.

Signed-off-by: default avatarMarkus Pargmann <mpa@pengutronix.de>
Tested-By: default avatarMichael Grzeschik <mgr@pengutronix.de>
Signed-off-by: default avatarMark Brown <broonie@linaro.org>
parent fcdbadef

sound/soc/fsl/fsl_ssi.c

+26 −15
Original line number Diff line number Diff line
@@ -660,12 +660,9 @@ static int fsl_ssi_hw_params(struct snd_pcm_substream *substream, 
	return 0;

}



/**

 * fsl_ssi_set_dai_fmt - configure Digital Audio Interface Format.

 */

static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)

static int _fsl_ssi_set_dai_fmt(struct fsl_ssi_private *ssi_private,

		unsigned int fmt)

{

	struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(cpu_dai);

	struct ccsr_ssi __iomem *ssi = ssi_private->ssi;

	u32 strcr = 0, stcr, srcr, scr, mask;

	u8 wm;
@@ -804,6 +801,17 @@ static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt) 
		fsl_ssi_setup_ac97(ssi_private);



	return 0;



}



/**

 * fsl_ssi_set_dai_fmt - configure Digital Audio Interface Format.

 */

static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)

{

	struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(cpu_dai);



	return _fsl_ssi_set_dai_fmt(ssi_private, fmt);

}



/**
@@ -1135,7 +1143,6 @@ static int fsl_ssi_probe(struct platform_device *pdev) 
	const uint32_t *iprop;

	struct resource res;

	char name[64];

	bool ac97 = false;



	/* SSIs that are not connected on the board should have a

	 *      status = "disabled"
@@ -1148,14 +1155,6 @@ static int fsl_ssi_probe(struct platform_device *pdev) 
	if (!of_id || !of_id->data)

		return -EINVAL;



	sprop = of_get_property(np, "fsl,mode", NULL);

	if (!sprop) {

		dev_err(&pdev->dev, "fsl,mode property is necessary\n");

		return -EINVAL;

	}

	if (!strcmp(sprop, "ac97-slave"))

		ac97 = true;



	ssi_private = devm_kzalloc(&pdev->dev, sizeof(*ssi_private),

			GFP_KERNEL);

	if (!ssi_private) {
@@ -1165,10 +1164,19 @@ static int fsl_ssi_probe(struct platform_device *pdev) 


	ssi_private->soc = of_id->data;



	sprop = of_get_property(np, "fsl,mode", NULL);

	if (sprop) {

		if (!strcmp(sprop, "ac97-slave"))

			ssi_private->dai_fmt = SND_SOC_DAIFMT_AC97;

		else if (!strcmp(sprop, "i2s-slave"))

			ssi_private->dai_fmt = SND_SOC_DAIFMT_I2S |

				SND_SOC_DAIFMT_CBM_CFM;

	}



	ssi_private->use_dma = !of_property_read_bool(np,

			"fsl,fiq-stream-filter");



	if (ac97) {

	if (fsl_ssi_is_ac97(ssi_private)) {

		memcpy(&ssi_private->cpu_dai_drv, &fsl_ssi_ac97_dai,

				sizeof(fsl_ssi_ac97_dai));
@@ -1279,6 +1287,9 @@ static int fsl_ssi_probe(struct platform_device *pdev) 
	}



done:

	if (ssi_private->dai_fmt)

		_fsl_ssi_set_dai_fmt(ssi_private, ssi_private->dai_fmt);



	return 0;



error_sound_card:

CRA Git | Maintained and supported by SUSTech CRA and CCSE