]> Pileus Git - ~andy/linux/commitdiff
sound: Remove unnecessary semicolon
authorPeter Senna Tschudin <peter.senna@gmail.com>
Fri, 28 Sep 2012 09:24:57 +0000 (11:24 +0200)
committerTakashi Iwai <tiwai@suse.de>
Sat, 6 Oct 2012 14:47:53 +0000 (16:47 +0200)
A simplified version of the semantic patch that finds this problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@r1@
statement S;
position p,p1;
@@
S@p1;@p

@script:python r2@
p << r1.p;
p1 << r1.p1;
@@
if p[0].line != p1[0].line_end:
        cocci.include_match(False)
@@
position r1.p;
@@
-;@p
// </smpl>

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
20 files changed:
sound/drivers/opl3/opl3_midi.c
sound/drivers/vx/vx_pcm.c
sound/isa/sb/emu8000_callback.c
sound/isa/wavefront/wavefront_synth.c
sound/oss/audio.c
sound/oss/opl3.c
sound/oss/pss.c
sound/oss/sb_ess.c
sound/oss/sb_mixer.c
sound/oss/sys_timer.c
sound/oss/uart6850.c
sound/pci/asihpi/asihpi.c
sound/pci/au88x0/au88x0_game.c
sound/pci/au88x0/au88x0_pcm.c
sound/pci/emu10k1/emu10k1_callback.c
sound/pci/emu10k1/emupcm.c
sound/pci/intel8x0.c
sound/pci/mixart/mixart_hwdep.c
sound/sparc/amd7930.c
sound/sparc/dbri.c

index 2bfe4bcb7a7dd8b4b667895e9a96012197207230..0c796bcbc0a36947453b8a3632daae48d5b24abc 100644 (file)
@@ -163,7 +163,7 @@ static int opl3_get_voice(struct snd_opl3 *opl3, int instr_4op,
        struct best *bp;
 
        for (i = 0; i < END; i++) {
-               best[i].time = (unsigned int)(-1); /* XXX MAX_?INT really */;
+               best[i].time = (unsigned int)(-1); /* XXX MAX_?INT really */
                best[i].voice = -1;
        }
 
index 5e897b236cec4f3b5c5371f41bb9e925bfab94f7..deed5efff33c9c57c2d462f7d05de89dc31bf2d7 100644 (file)
@@ -184,7 +184,7 @@ static int vx_set_format(struct vx_core *chip, struct vx_pipe *pipe,
        default : 
                snd_BUG();
                return -EINVAL;
-        };
+       }
 
        return vx_set_stream_format(chip, pipe, header);
 }
index 344b4355be1cd79b898227e5a39661129227f3e9..72a9ac5efb4061360d539fb1ed13300033f13d95 100644 (file)
@@ -175,7 +175,7 @@ get_voice(struct snd_emux *emu, struct snd_emux_port *port)
        hw = emu->hw;
 
        for (i = 0; i < END; i++) {
-               best[i].time = (unsigned int)(-1); /* XXX MAX_?INT really */;
+               best[i].time = (unsigned int)(-1); /* XXX MAX_?INT really */
                best[i].voice = -1;
        }
 
index 405f8b6a58b53db482376efd456c6e2ddaf6e2a3..b1bf8d4e6494f4561ea2e11a407f4663f5882f98 100644 (file)
@@ -538,7 +538,7 @@ munge_int32 (unsigned int src,
                                    /* Note: we leave the upper bits in place */ 
 
                dst++;
-       };
+       }
        return dst;
 };
 
index 4b958b1c497c86a75b3984af6d43b553b639c401..09c932f899b8158c8fde31eb9f8dbd5eac440e72 100644 (file)
@@ -354,7 +354,7 @@ int audio_read(int dev, struct file *file, char __user *buf, int count)
 
                        if(copy_to_user(&(buf)[p], fixit, l))
                                return -EFAULT;
-               };
+               }
 
                DMAbuf_rmchars(dev, buf_no, l);
 
index 407cd677950bccb68059fdcd1abec6da914b7a38..c5c24409ceb065b49d8f7bb05d010085e4588d47 100644 (file)
@@ -1190,7 +1190,7 @@ static int opl3_init(int ioaddr, struct module *owner)
 
                for (i = 0; i < 18; i++)
                        pv_map[i].ioaddr = devc->left_io;
-       };
+       }
        conf_printf2(devc->fm_info.name, ioaddr, 0, -1, -1);
 
        for (i = 0; i < SBFM_MAXINSTR; i++)
index 0f32a561f15f6a6575d75888abc733b3ceebd377..145e36b2cfd0db9ff84232047fa89e2a3e929928 100644 (file)
@@ -359,7 +359,7 @@ static int pss_download_boot(pss_confdata * devc, unsigned char *block, int size
                {
                        /*_____ Send the next byte */
                        outw (*block++, REG (PSS_DATA));
-               };
+               }
                count++;
        }
 
index 5c773dff5ac5dd702d2f463c348373e999f0ef65..c0be085e4a200e4878097138b9294fe0d4a0855c 100644 (file)
@@ -1104,15 +1104,15 @@ int ess_init(sb_devc * devc, struct address_info *hw_config)
                default:
                        printk (KERN_ERR "Invalid esstype=%d specified\n", devc->sbmo.esstype);
                        return 0;
-               };
+               }
                if (submodel != -1) {
                        devc->submodel = submodel;
                        sprintf (modelname, "ES%d", devc->sbmo.esstype);
                        chip = modelname;
-               };
+               }
                if (chip == NULL && (ess_minor & 0x0f) < 8) {
                        chip = "ES688";
-               };
+               }
 #ifdef FKS_TEST
 FKS_test (devc);
 #endif
@@ -1122,7 +1122,7 @@ FKS_test (devc);
                 */
                if (chip == NULL && devc->sbmo.esstype == ESSTYPE_LIKE20) {
                        chip = "ES1688";
-               };
+               }
 
                if (chip == NULL) {
                        int type;
@@ -1150,8 +1150,8 @@ FKS_test (devc);
                                if ((type & 0x00ff) != ((type >> 8) & 0x00ff)) {
                                        printk ("ess_init: Unrecognized %04x\n", type);
                                }
-                       };
-               };
+                       }
+               }
 #if 0
                /*
                 * this one failed:
@@ -1182,10 +1182,10 @@ FKS_test (devc);
                                chip = "ES1788";
                                devc->submodel = SUBMDL_ES1788;
                        }
-               };
+               }
                if (chip == NULL) {
                        chip = "ES1688";
-               };
+               }
 
            printk ( KERN_INFO "ESS chip %s %s%s\n"
                , chip
@@ -1293,7 +1293,7 @@ printk(KERN_INFO "ess_set_dma_hw: dma8=%d,dma16=%d,dup=%d\n"
                        default:
                                printk(KERN_ERR "ESS1887: Invalid DMA16 %d\n", dma);
                                return 0;
-                       };
+                       }
                        ess_chgmixer (devc, 0x78, 0x20, dma16_bits);
                        ess_chgmixer (devc, 0x7d, 0x07, dma_bits);
                }
@@ -1584,7 +1584,7 @@ printk(KERN_INFO "FKS: write mixer %x: %x\n", port, value);
                udelay(20);
                outb(((unsigned char) (value & 0xff)), MIXER_DATA);
                udelay(20);
-       };
+       }
        spin_unlock_irqrestore(&devc->lock, flags);
 }
 
@@ -1761,7 +1761,7 @@ int ess_mixer_reset (sb_devc * devc)
                        ess_chgmixer(devc, 0x7a, 0x18, 0x08);
                        ess_chgmixer(devc, 0x1c, 0x07, 0x07);
                        break;
-               };
+               }
                /*
                 * Call set_recmask for proper initialization
                 */
index f8f3b7a66b7377fcc25fc84b76c8b9b63af44181..acf7586aeb47de2ba6184c208e69777d45cb4c4a 100644 (file)
@@ -410,7 +410,7 @@ static int set_recmask(sb_devc * devc, int mask)
                case MDL_SMW:
                        if (devc->model == MDL_ESS && ess_set_recmask (devc, &devmask)) {
                                break;
-                       };
+                       }
                        if (devmask != SOUND_MASK_MIC &&
                                devmask != SOUND_MASK_LINE &&
                                devmask != SOUND_MASK_CD)
@@ -666,7 +666,7 @@ static void sb_mixer_reset(sb_devc * devc)
 
        if (devc->model != MDL_ESS || !ess_mixer_reset (devc)) {
                set_recmask(devc, SOUND_MASK_MIC);
-       };
+       }
 }
 
 int sb_mixer_init(sb_devc * devc, struct module *owner)
index 8db6aefe15e4043224a788c2d9bc314e1c2055a9..9f039831114c2f75e11789e061dd505933f87beb 100644 (file)
@@ -57,7 +57,7 @@ poll_def_tmr(unsigned long dummy)
                  {
                          def_tmr.expires = (1) + jiffies;
                          add_timer(&def_tmr);
-                 };
+                 }
 
                  if (tmr_running)
                    {
@@ -103,7 +103,7 @@ def_tmr_open(int dev, int mode)
        {
                def_tmr.expires = (1) + jiffies;
                add_timer(&def_tmr);
-       };
+       }
 
        return 0;
 }
index f3f914aa92ee3aaf25a81fa80f02bde9b27b3077..1079133dd6ab17fc856677a328ea54563028c555 100644 (file)
@@ -146,7 +146,7 @@ static int uart6850_open(int dev, int mode,
        {
 /*               printk("Midi6850: Midi busy\n");*/
                  return -EBUSY;
-       };
+       }
 
        uart6850_cmd(UART_RESET);
        uart6850_input_loop();
index a51e3ce3c800b2f8d97c82337a1598aa918e4c7a..eedc017c1cd8f7f65f537ddca7c445c6e3b14111 100644 (file)
@@ -2658,7 +2658,7 @@ static int __devinit snd_card_asihpi_mixer_new(struct snd_card_asihpi *asihpi)
                                        hpi_ctl.dst_node_type,
                                        hpi_ctl.dst_node_index);
                        continue;
-               };
+               }
                if (err < 0)
                        return err;
        }
index c07c792bde8d27a8ad8c6627ac0c8de784c2608e..30a456700d899bb88cbc643d3d5ea782a2f3d684 100644 (file)
@@ -100,7 +100,7 @@ static int __devinit vortex_gameport_register(vortex_t * vortex)
        if (!gp) {
                printk(KERN_ERR "vortex: cannot allocate memory for gameport\n");
                return -ENOMEM;
-       };
+       }
 
        gameport_set_name(gp, "AU88x0 Gameport");
        gameport_set_phys(gp, "pci%s/gameport0", pci_name(vortex->pci_dev));
index e59f120742a48e4140a54d80c63f6592c325c220..b2405020284c6476814b8039a27cb416d64bf668 100644 (file)
@@ -585,7 +585,7 @@ static int snd_vortex_pcm_vol_put(struct snd_kcontrol *kcontrol,
                                case 4:
                                        mixin = p->mixin[i];
                                        break;
-                               };
+                               }
                                vol = p->vol[i];
                                vortex_mix_setinputvolumebyte(vortex,
                                        vortex->mixplayb[i], mixin, vol);
index a0afa5057488895b9e5710ccc57017db6d21e6d4..cae36597aa718c3fc13abe95221a2308627c2c8c 100644 (file)
@@ -228,7 +228,7 @@ lookup_voices(struct snd_emux *emu, struct snd_emu10k1 *hw,
        int  i;
 
        for (i = 0; i < V_END; i++) {
-               best[i].time = (unsigned int)-1; /* XXX MAX_?INT really */;
+               best[i].time = (unsigned int)-1; /* XXX MAX_?INT really */
                best[i].voice = -1;
        }
 
index e22b8e2bbd884099339b807045176ceb33ee37d9..0e6664fa6cd997fe7b64137bd420523beef0444e 100644 (file)
@@ -1310,7 +1310,7 @@ static int snd_emu10k1_capture_efx_open(struct snd_pcm_substream *substream)
                        runtime->hw.channels_min =
                                runtime->hw.channels_max = 16;
                        break;
-               };
+               }
 #endif
 #if 0
                /* For 96kHz */
index 848102e5d864d6d1ac5007af668737ca087aced3..ea4b706c8d635925c248bfb497a021361d990a27 100644 (file)
@@ -2226,7 +2226,7 @@ static int __devinit snd_intel8x0_mixer(struct intel8x0 *chip, int ac97_clock,
                case DEVICE_INTEL_ICH4:
                        chip->spdif_idx = ICHD_SPBAR;
                        break;
-               };
+               }
        }
 
        chip->in_ac97_init = 1;
index bfbdc91e4cb30cc89ad028a8627a6204d2b12919..e0f4d87555a04ace24920b66fe96bf8994d6e111 100644 (file)
@@ -538,7 +538,7 @@ static int mixart_dsp_load(struct mixart_mgr* mgr, int index, const struct firmw
 
                if ((err = snd_card_register(chip->card)) < 0)
                        return err;
-       };
+       }
 
        snd_printdd("miXart firmware downloaded and successfully set up\n");
 
index b63b3a86d3f40f45eeef765a960b705268dd8d77..5701787c0e6bce406603ebb8590668d853900fd6 100644 (file)
@@ -813,7 +813,7 @@ static int snd_amd7930_get_volume(struct snd_kcontrol *kctl, struct snd_ctl_elem
        default:
                swval = &amd->pgain;
                break;
-       };
+       }
 
        ucontrol->value.integer.value[0] = *swval;
 
@@ -838,7 +838,7 @@ static int snd_amd7930_put_volume(struct snd_kcontrol *kctl, struct snd_ctl_elem
        default:
                swval = &amd->pgain;
                break;
-       };
+       }
 
        spin_lock_irqsave(&amd->lock, flags);
 
index a6b0deb77746c11646e42dd9f5f6353311e94656..ae35f5342e105dc3700e5589da6d23a4f4a6ab63 100644 (file)
@@ -592,7 +592,7 @@ static __u32 reverse_bytes(__u32 b, int len)
                break;
        default:
                printk(KERN_ERR "DBRI reverse_bytes: unsupported length\n");
-       };
+       }
 
        return b;
 }