* [PATCH] ASoC: dapm: Fix return value in snd_soc_dapm_new_control()
@ 2012-08-23 14:12 Dimitris Papastamos
2012-08-23 14:43 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: Dimitris Papastamos @ 2012-08-23 14:12 UTC (permalink / raw)
To: Mark Brown; +Cc: alsa-devel, patches
Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
---
sound/soc/soc-dapm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c
index 99aac65..90b8fb9 100644
--- a/sound/soc/soc-dapm.c
+++ b/sound/soc/soc-dapm.c
@@ -2884,7 +2884,7 @@ snd_soc_dapm_new_control(struct snd_soc_dapm_context *dapm,
ret = PTR_ERR(w->priv);
dev_err(dapm->dev, "Failed to request %s: %d\n",
w->name, ret);
- return ret;
+ return NULL;
}
break;
default:
--
1.7.12
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ASoC: dapm: Fix return value in snd_soc_dapm_new_control()
2012-08-23 14:12 [PATCH] ASoC: dapm: Fix return value in snd_soc_dapm_new_control() Dimitris Papastamos
@ 2012-08-23 14:43 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2012-08-23 14:43 UTC (permalink / raw)
To: Dimitris Papastamos; +Cc: alsa-devel, patches
[-- Attachment #1.1: Type: text/plain, Size: 251 bytes --]
On Thu, Aug 23, 2012 at 03:12:43PM +0100, Dimitris Papastamos wrote:
> Signed-off-by: Dimitris Papastamos <dp@opensource.wolfsonmicro.com>
Which kernel is this against? Both for-3.6 and for-3.7 already have
the code like this (according to git am).
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-08-23 14:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-23 14:12 [PATCH] ASoC: dapm: Fix return value in snd_soc_dapm_new_control() Dimitris Papastamos
2012-08-23 14:43 ` 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).