diff options
author | Takashi Iwai <tiwai@suse.de> | 2006-02-20 05:57:34 -0500 |
---|---|---|
committer | Jaroslav Kysela <perex@suse.cz> | 2006-03-22 04:30:43 -0500 |
commit | 8278ca8feb2748cf02d756ac6c5b9ab2e047c84a (patch) | |
tree | a58ddfa8d7ac897c02e4ba3bda9b77e278348fbf /sound/drivers | |
parent | 1a56f8d662ec7fc86f2c408d289fa07cdb781746 (diff) |
[ALSA] Fix check of enable module option
Fix the check of enable module option in probe of platform_device drivers.
It shouldn't break the loop but just ignore if enable[i] is false.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/drivers')
-rw-r--r-- | sound/drivers/dummy.c | 4 | ||||
-rw-r--r-- | sound/drivers/mpu401/mpu401.c | 4 | ||||
-rw-r--r-- | sound/drivers/serial-u16550.c | 4 | ||||
-rw-r--r-- | sound/drivers/virmidi.c | 4 |
4 files changed, 12 insertions, 4 deletions
diff --git a/sound/drivers/dummy.c b/sound/drivers/dummy.c index 14e1a671b5cf..e35fd5779a9d 100644 --- a/sound/drivers/dummy.c +++ b/sound/drivers/dummy.c | |||
@@ -669,8 +669,10 @@ static int __init alsa_card_dummy_init(void) | |||
669 | return err; | 669 | return err; |
670 | 670 | ||
671 | cards = 0; | 671 | cards = 0; |
672 | for (i = 0; i < SNDRV_CARDS && enable[i]; i++) { | 672 | for (i = 0; i < SNDRV_CARDS; i++) { |
673 | struct platform_device *device; | 673 | struct platform_device *device; |
674 | if (! enable[i]) | ||
675 | continue; | ||
674 | device = platform_device_register_simple(SND_DUMMY_DRIVER, | 676 | device = platform_device_register_simple(SND_DUMMY_DRIVER, |
675 | i, NULL, 0); | 677 | i, NULL, 0); |
676 | if (IS_ERR(device)) { | 678 | if (IS_ERR(device)) { |
diff --git a/sound/drivers/mpu401/mpu401.c b/sound/drivers/mpu401/mpu401.c index 915589a402ab..9d10d79e27af 100644 --- a/sound/drivers/mpu401/mpu401.c +++ b/sound/drivers/mpu401/mpu401.c | |||
@@ -240,8 +240,10 @@ static int __init alsa_card_mpu401_init(void) | |||
240 | return err; | 240 | return err; |
241 | 241 | ||
242 | devices = 0; | 242 | devices = 0; |
243 | for (i = 0; i < SNDRV_CARDS && enable[i]; i++) { | 243 | for (i = 0; i < SNDRV_CARDS; i++) { |
244 | struct platform_device *device; | 244 | struct platform_device *device; |
245 | if (! enable[i]) | ||
246 | continue; | ||
245 | #ifdef CONFIG_PNP | 247 | #ifdef CONFIG_PNP |
246 | if (pnp[i]) | 248 | if (pnp[i]) |
247 | continue; | 249 | continue; |
diff --git a/sound/drivers/serial-u16550.c b/sound/drivers/serial-u16550.c index 112ddf705402..13b46d12564a 100644 --- a/sound/drivers/serial-u16550.c +++ b/sound/drivers/serial-u16550.c | |||
@@ -989,8 +989,10 @@ static int __init alsa_card_serial_init(void) | |||
989 | return err; | 989 | return err; |
990 | 990 | ||
991 | cards = 0; | 991 | cards = 0; |
992 | for (i = 0; i < SNDRV_CARDS && enable[i]; i++) { | 992 | for (i = 0; i < SNDRV_CARDS; i++) { |
993 | struct platform_device *device; | 993 | struct platform_device *device; |
994 | if (! enable[i]) | ||
995 | continue; | ||
994 | device = platform_device_register_simple(SND_SERIAL_DRIVER, | 996 | device = platform_device_register_simple(SND_SERIAL_DRIVER, |
995 | i, NULL, 0); | 997 | i, NULL, 0); |
996 | if (IS_ERR(device)) { | 998 | if (IS_ERR(device)) { |
diff --git a/sound/drivers/virmidi.c b/sound/drivers/virmidi.c index 4258723de2ab..a3ee306239c9 100644 --- a/sound/drivers/virmidi.c +++ b/sound/drivers/virmidi.c | |||
@@ -163,8 +163,10 @@ static int __init alsa_card_virmidi_init(void) | |||
163 | return err; | 163 | return err; |
164 | 164 | ||
165 | cards = 0; | 165 | cards = 0; |
166 | for (i = 0; i < SNDRV_CARDS && enable[i]; i++) { | 166 | for (i = 0; i < SNDRV_CARDS; i++) { |
167 | struct platform_device *device; | 167 | struct platform_device *device; |
168 | if (! enable[i]) | ||
169 | continue; | ||
168 | device = platform_device_register_simple(SND_VIRMIDI_DRIVER, | 170 | device = platform_device_register_simple(SND_VIRMIDI_DRIVER, |
169 | i, NULL, 0); | 171 | i, NULL, 0); |
170 | if (IS_ERR(device)) { | 172 | if (IS_ERR(device)) { |