[alsa-devel] [PATCH v2] ASoC: mxs: add some error messages to help identifying problems
Lothar Waßmann
LW at KARO-electronics.de
Fri Aug 2 10:30:15 CEST 2013
On Thu, 1 Aug 2013 11:22:19 +0100, Mark Brown wrote:
> On Wed, Jul 31, 2013 at 07:11:48PM -0300, Fabio Estevam wrote:
>> On Wed, Jul 31, 2013 at 11:44 AM, Lothar Waßmann <LW at karo-electronics.de> wrote:
>> > /* Set SGTL5000's SYSCLK (provided by SAIF MCLK) */
>> > ret = snd_soc_dai_set_sysclk(codec_dai, SGTL5000_SYSCLK, mclk, 0);
>> > - if (ret)
>> > + if (ret) {
>> > + dev_err(codec_dai->dev, "Failed to set sysclk to %u kHz\n",
>> > + mclk / 1000);
>
>> Wouldn't it make more sense to express it in MHz instead of kHz?
>
> Well, ideally it'd be fractional MHz (ie, 24.576MHz). I'd either leave
> it as Hz or do the proper pretty print. Also if we're nitpicking the
> format then there's usually no space between the number and units.
>
> Personally I think the bike shed would be most attractive if it were
> pink with yellow spots.
>
Now pink with yellow spots. ;)
Signed-off-by: Lothar Waßmann <LW at KARO-electronics.de>
---
sound/soc/mxs/mxs-sgtl5000.c | 29 +++++++++++++++++++++++------
1 files changed, 23 insertions(+), 6 deletions(-)
diff --git a/sound/soc/mxs/mxs-sgtl5000.c b/sound/soc/mxs/mxs-sgtl5000.c
index b2e372d..ce084eb 100644
--- a/sound/soc/mxs/mxs-sgtl5000.c
+++ b/sound/soc/mxs/mxs-sgtl5000.c
@@ -50,18 +50,27 @@ static int mxs_sgtl5000_hw_params(struct snd_pcm_substream *substream,
}
/* Sgtl5000 sysclk should be >= 8MHz and <= 27M */
- if (mclk < 8000000 || mclk > 27000000)
+ if (mclk < 8000000 || mclk > 27000000) {
+ dev_err(codec_dai->dev, "Invalid mclk frequency: %u.%03uMHz\n",
+ mclk / 1000000, mclk / 1000 % 1000);
return -EINVAL;
+ }
/* Set SGTL5000's SYSCLK (provided by SAIF MCLK) */
ret = snd_soc_dai_set_sysclk(codec_dai, SGTL5000_SYSCLK, mclk, 0);
- if (ret)
+ if (ret) {
+ dev_err(codec_dai->dev, "Failed to set sysclk to %u.%03uMHz\n",
+ mclk / 1000000, mclk / 1000 % 1000);
return ret;
+ }
/* The SAIF MCLK should be the same as SGTL5000_SYSCLK */
ret = snd_soc_dai_set_sysclk(cpu_dai, MXS_SAIF_MCLK, mclk, 0);
- if (ret)
+ if (ret) {
+ dev_err(cpu_dai->dev, "Failed to set sysclk to %u.%03uMHz\n",
+ mclk / 1000000, mclk / 1000 % 1000);
return ret;
+ }
/* set codec to slave mode */
dai_format = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF |
@@ -69,13 +78,19 @@ static int mxs_sgtl5000_hw_params(struct snd_pcm_substream *substream,
/* set codec DAI configuration */
ret = snd_soc_dai_set_fmt(codec_dai, dai_format);
- if (ret)
+ if (ret) {
+ dev_err(codec_dai->dev, "Failed to set dai format to %08x\n",
+ dai_format);
return ret;
+ }
/* set cpu DAI configuration */
ret = snd_soc_dai_set_fmt(cpu_dai, dai_format);
- if (ret)
+ if (ret) {
+ dev_err(cpu_dai->dev, "Failed to set dai format to %08x\n",
+ dai_format);
return ret;
+ }
return 0;
}
@@ -153,8 +168,10 @@ static int mxs_sgtl5000_probe(struct platform_device *pdev)
* should be >= 8MHz and <= 27M.
*/
ret = mxs_saif_get_mclk(0, 44100 * 256, 44100);
- if (ret)
+ if (ret) {
+ dev_err(&pdev->dev, "failed to get mclk\n");
return ret;
+ }
card->dev = &pdev->dev;
platform_set_drvdata(pdev, card);
--
1.7.2.5
More information about the Alsa-devel
mailing list