diff options
author | Takashi Iwai <tiwai@suse.de> | 2015-03-10 10:42:14 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-03-10 10:42:14 -0400 |
commit | ec0e9937aaa8b0a4b0633711c4d70d622acd9a7f (patch) | |
tree | 7c19729d7bbd4d97784b45b2e2ec1b13dd005dd6 /sound/core/oss | |
parent | 24db8bbaa3fcfaf0c2faccbff5864b58088ac1f6 (diff) |
ALSA: core: Drop superfluous error/debug messages after malloc failures
The kernel memory allocators already report the errors when the
requested allocation fails, thus we don't need to warn it again in
each caller side.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core/oss')
-rw-r--r-- | sound/core/oss/mixer_oss.c | 4 | ||||
-rw-r--r-- | sound/core/oss/pcm_oss.c | 1 |
2 files changed, 1 insertions, 4 deletions
diff --git a/sound/core/oss/mixer_oss.c b/sound/core/oss/mixer_oss.c index 5e6349f00ecd..056f8e274851 100644 --- a/sound/core/oss/mixer_oss.c +++ b/sound/core/oss/mixer_oss.c | |||
@@ -1212,10 +1212,8 @@ static void snd_mixer_oss_proc_write(struct snd_info_entry *entry, | |||
1212 | /* not changed */ | 1212 | /* not changed */ |
1213 | goto __unlock; | 1213 | goto __unlock; |
1214 | tbl = kmalloc(sizeof(*tbl), GFP_KERNEL); | 1214 | tbl = kmalloc(sizeof(*tbl), GFP_KERNEL); |
1215 | if (! tbl) { | 1215 | if (!tbl) |
1216 | pr_err("ALSA: mixer_oss: no memory\n"); | ||
1217 | goto __unlock; | 1216 | goto __unlock; |
1218 | } | ||
1219 | tbl->oss_id = ch; | 1217 | tbl->oss_id = ch; |
1220 | tbl->name = kstrdup(str, GFP_KERNEL); | 1218 | tbl->name = kstrdup(str, GFP_KERNEL); |
1221 | if (! tbl->name) { | 1219 | if (! tbl->name) { |
diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index 80423a4ccab6..58550cc93f28 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core/oss/pcm_oss.c | |||
@@ -854,7 +854,6 @@ static int snd_pcm_oss_change_params(struct snd_pcm_substream *substream) | |||
854 | params = kmalloc(sizeof(*params), GFP_KERNEL); | 854 | params = kmalloc(sizeof(*params), GFP_KERNEL); |
855 | sparams = kmalloc(sizeof(*sparams), GFP_KERNEL); | 855 | sparams = kmalloc(sizeof(*sparams), GFP_KERNEL); |
856 | if (!sw_params || !params || !sparams) { | 856 | if (!sw_params || !params || !sparams) { |
857 | pcm_dbg(substream->pcm, "No memory\n"); | ||
858 | err = -ENOMEM; | 857 | err = -ENOMEM; |
859 | goto failure; | 858 | goto failure; |
860 | } | 859 | } |