From: sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)
To: linux-arm-kernel@lists.infradead.org
Subject: Ask for help. ARM:mvebu (CM-A510), add sound support using TLV320AIC23 audio-codec
Date: Tue, 24 Feb 2015 22:54:31 +0100 [thread overview]
Message-ID: <54ECF317.10603@gmail.com> (raw)
In-Reply-To: <54ECEA1D.6090306@gmail.com>
On 24.02.2015 22:16, Gabriel Dobato wrote:
>> Looking at sound/soc/codecs/tlv320aic23.c:
>>
>> static const struct snd_soc_dapm_route tlv320aic23_intercon[] = {
>> /* Output Mixer */
>> {"Output Mixer", "Line Bypass Switch", "Line Input"},
>> {"Output Mixer", "Playback Switch", "DAC"},
>> {"Output Mixer", "Mic Sidetone Switch", "Mic Input"},
>>
>> /* Outputs */
>> {"RHPOUT", NULL, "Output Mixer"},
>> {"LHPOUT", NULL, "Output Mixer"},
>> {"LOUT", NULL, "Output Mixer"},
>> {"ROUT", NULL, "Output Mixer"},
>>
>> /* Inputs */
>> {"Line Input", "NULL", "LLINEIN"},
>> {"Line Input", "NULL", "RLINEIN"},
>>
>> {"Mic Input", "NULL", "MICIN"},
>>
>> /* input mux */
>> {"Capture Source", "Line", "Line Input"},
>> {"Capture Source", "Mic", "Mic Input"},
>> {"ADC", NULL, "Capture Source"},
>>
>> };
>>
>> The lines with the failing routes really look suspicious, i.e.
>> there is strings "NULL" where I'd expect plain NULL.
>>
> You were right, as always :) , I have changed them and now errors are
> not reported. Now, in the boot trace I find:
Mind to send a proper patch fixing the above? Make sure you run
./scripts/checkpatch.pl and ./scripts/get_maintainer.pl on the
patch. It should give you an idea of proper code style and what
people/lists to send it to. I am sure, google will help you with
how to prepare proper kernel patches, too.
> but after rootfs is up, this msg appears in dmesg:
>
> platform sound: Driver asoc-simple-card requests probe deferral
This is, as always, a missing dependency of simple-card node.
Either you are missing drivers for Dove audio (kirkwood-i2s) or
one of the drivers involved in your simple-card fails.
Look for errors in your dmesg output - also please name the
kernel version you are working with. There have been some changes
in clk handling that break a bunch of drivers in 4.0-rc1.
Sebastian
prev parent reply other threads:[~2015-02-24 21:54 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-23 20:27 Ask for help. ARM:mvebu (CM-A510), add sound support using TLV320AIC23 audio-codec Gabriel Dobato
2015-02-24 6:42 ` Marcin Wojtas
2015-02-24 8:41 ` Sebastian Hesselbarth
2015-02-24 21:16 ` Gabriel Dobato
2015-02-24 21:54 ` Sebastian Hesselbarth [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54ECF317.10603@gmail.com \
--to=sebastian.hesselbarth@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).