diff options
author | Hannes Eder <hannes@hanneseder.net> | 2008-11-20 15:25:25 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2008-11-21 02:13:15 -0500 |
commit | a39c4ad1089be34d8dc66e926e93a52c44993a0a (patch) | |
tree | 9cd3373dbd7d481c80014c382bb48006beaa5d59 | |
parent | 9bf1a2445f3c569098b8de7097ca324e65abecc2 (diff) |
sound/sound_core: Fix sparse warnings
Fix the following sparse warnings:
sound/sound_core.c:460:2: warning: returning void-valued expression
sound/sound_core.c:477:2: warning: returning void-valued expression
sound/sound_core.c:510:5: warning: symbol 'soundcore_open' was not
declared. Should it be static?
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/sound_core.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sound/sound_core.c b/sound/sound_core.c index a75b289a5d78..10ba4218161b 100644 --- a/sound/sound_core.c +++ b/sound/sound_core.c | |||
@@ -457,7 +457,7 @@ EXPORT_SYMBOL(unregister_sound_mixer); | |||
457 | 457 | ||
458 | void unregister_sound_midi(int unit) | 458 | void unregister_sound_midi(int unit) |
459 | { | 459 | { |
460 | return sound_remove_unit(&chains[2], unit); | 460 | sound_remove_unit(&chains[2], unit); |
461 | } | 461 | } |
462 | 462 | ||
463 | EXPORT_SYMBOL(unregister_sound_midi); | 463 | EXPORT_SYMBOL(unregister_sound_midi); |
@@ -474,7 +474,7 @@ EXPORT_SYMBOL(unregister_sound_midi); | |||
474 | 474 | ||
475 | void unregister_sound_dsp(int unit) | 475 | void unregister_sound_dsp(int unit) |
476 | { | 476 | { |
477 | return sound_remove_unit(&chains[3], unit); | 477 | sound_remove_unit(&chains[3], unit); |
478 | } | 478 | } |
479 | 479 | ||
480 | 480 | ||
@@ -507,7 +507,7 @@ static struct sound_unit *__look_for_unit(int chain, int unit) | |||
507 | return NULL; | 507 | return NULL; |
508 | } | 508 | } |
509 | 509 | ||
510 | int soundcore_open(struct inode *inode, struct file *file) | 510 | static int soundcore_open(struct inode *inode, struct file *file) |
511 | { | 511 | { |
512 | int chain; | 512 | int chain; |
513 | int unit = iminor(inode); | 513 | int unit = iminor(inode); |