* [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name
@ 2017-03-01 12:26 Liviu Dudau
2017-03-06 10:41 ` Mark Brown
0 siblings, 1 reply; 4+ messages in thread
From: Liviu Dudau @ 2017-03-01 12:26 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai
Cc: ALSA devel, LKML, Liviu Dudau
Without this I am getting the following messages at boot on my Trimslice:
tlv320aic23-codec 2-001a: Control not supported for path LLINEIN -> [NULL] -> Line Input
tlv320aic23-codec 2-001a: ASoC: no dapm match for LLINEIN --> NULL --> Line Input
tlv320aic23-codec 2-001a: ASoC: Failed to add route LLINEIN -> NULL -> Line Input
tlv320aic23-codec 2-001a: Control not supported for path RLINEIN -> [NULL] -> Line Input
tlv320aic23-codec 2-001a: ASoC: no dapm match for RLINEIN --> NULL --> Line Input
tlv320aic23-codec 2-001a: ASoC: Failed to add route RLINEIN -> NULL -> Line Input
tlv320aic23-codec 2-001a: Control not supported for path MICIN -> [NULL] -> Mic Input
tlv320aic23-codec 2-001a: ASoC: no dapm match for MICIN --> NULL --> Mic Input
tlv320aic23-codec 2-001a: ASoC: Failed to add route MICIN -> NULL -> Mic Input
tegra-snd-trimslice sound: tlv320aic23-hifi <-> 70002800.i2s mapping ok
Signed-off-by: Liviu Dudau <liviu@dudau.co.uk>
---
sound/soc/codecs/tlv320aic23.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic23.c b/sound/soc/codecs/tlv320aic23.c
index 410cae0f2060..628a8eeaab68 100644
--- a/sound/soc/codecs/tlv320aic23.c
+++ b/sound/soc/codecs/tlv320aic23.c
@@ -174,10 +174,9 @@ static const struct snd_soc_dapm_route tlv320aic23_intercon[] = {
{"ROUT", NULL, "Output Mixer"},
/* Inputs */
- {"Line Input", "NULL", "LLINEIN"},
- {"Line Input", "NULL", "RLINEIN"},
-
- {"Mic Input", "NULL", "MICIN"},
+ {"Line Input", NULL, "LLINEIN"},
+ {"Line Input", NULL, "RLINEIN"},
+ {"Mic Input", NULL, "MICIN"},
/* input mux */
{"Capture Source", "Line", "Line Input"},
--
2.11.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name
2017-03-01 12:26 [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name Liviu Dudau
@ 2017-03-06 10:41 ` Mark Brown
2017-03-07 0:54 ` Liviu Dudau
0 siblings, 1 reply; 4+ messages in thread
From: Mark Brown @ 2017-03-06 10:41 UTC (permalink / raw)
To: Liviu Dudau
Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, ALSA devel, LKML
[-- Attachment #1: Type: text/plain, Size: 358 bytes --]
On Wed, Mar 01, 2017 at 12:26:28PM +0000, Liviu Dudau wrote:
> Without this I am getting the following messages at boot on my Trimslice:
> tlv320aic23-codec 2-001a: Control not supported for path LLINEIN -> [NULL] -> Line Input
Please use subject lines matching the style for the subsystem. This
makes it easier for people to identify relevant patches.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name
2017-03-06 10:41 ` Mark Brown
@ 2017-03-07 0:54 ` Liviu Dudau
2017-03-07 12:06 ` Mark Brown
0 siblings, 1 reply; 4+ messages in thread
From: Liviu Dudau @ 2017-03-07 0:54 UTC (permalink / raw)
To: Mark Brown; +Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, ALSA devel, LKML
Hi Mark,
On Mon, Mar 06, 2017 at 11:41:18AM +0100, Mark Brown wrote:
> On Wed, Mar 01, 2017 at 12:26:28PM +0000, Liviu Dudau wrote:
> > Without this I am getting the following messages at boot on my Trimslice:
> > tlv320aic23-codec 2-001a: Control not supported for path LLINEIN -> [NULL] -> Line Input
>
> Please use subject lines matching the style for the subsystem. This
> makes it easier for people to identify relevant patches.
Other than the additional ALSA: prefix I have matched the format used
by previous commits touching the file. I'm not clear where I have made the
mistake and what would have been the better subject line.
Best regards,
Liviu
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name
2017-03-07 0:54 ` Liviu Dudau
@ 2017-03-07 12:06 ` Mark Brown
0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2017-03-07 12:06 UTC (permalink / raw)
To: Liviu Dudau
Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, ALSA devel, LKML
[-- Attachment #1: Type: text/plain, Size: 739 bytes --]
On Tue, Mar 07, 2017 at 12:54:08AM +0000, Liviu Dudau wrote:
> On Mon, Mar 06, 2017 at 11:41:18AM +0100, Mark Brown wrote:
> > Please use subject lines matching the style for the subsystem. This
> > makes it easier for people to identify relevant patches.
> Other than the additional ALSA: prefix I have matched the format used
> by previous commits touching the file. I'm not clear where I have made the
> mistake and what would have been the better subject line.
It's the extra ALSA prefix, the start of the subject line is the most
important bit to get right since it's that that people are most likely
to pattern match on. You also used caps for the name of the CODEC,
normally they use lower case, though that is less important.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-07 13:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-01 12:26 [PATCH] ALSA: ASoC: TLV320AIC23: Unquote NULL from control name Liviu Dudau
2017-03-06 10:41 ` Mark Brown
2017-03-07 0:54 ` Liviu Dudau
2017-03-07 12:06 ` Mark Brown
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).