Staging: line6: minor coding style cleanups
authorGreg Kroah-Hartman <gregkh@suse.de>
Tue, 21 Sep 2010 23:58:00 +0000 (16:58 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 21 Sep 2010 23:58:00 +0000 (16:58 -0700)
This fixes up all of the remaining coding style issues that
make any sense to make in the line6 driver.

Cc: Markus Grabner <grabner@icg.tugraz.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
14 files changed:
drivers/staging/line6/audio.c
drivers/staging/line6/capture.c
drivers/staging/line6/config.h
drivers/staging/line6/driver.c
drivers/staging/line6/driver.h
drivers/staging/line6/midi.c
drivers/staging/line6/pcm.c
drivers/staging/line6/pcm.h
drivers/staging/line6/playback.c
drivers/staging/line6/playback.h
drivers/staging/line6/pod.c
drivers/staging/line6/toneport.c
drivers/staging/line6/usbdefs.h
drivers/staging/line6/variax.c

index b1488d321c7b73d3f777d839d7eca6bb128c8204..61db1f99b0c8ffe5c62eae7b8ad505da141de5df 100644 (file)
@@ -37,7 +37,9 @@ int line6_init_audio(struct usb_line6 *line6)
        strcpy(card->id, line6->properties->id);
        strcpy(card->driver, DRIVER_NAME);
        strcpy(card->shortname, line6->properties->name);
-       sprintf(card->longname, "Line6 %s at USB %s", line6->properties->name, dev_name(line6->ifcdev));        /* 80 chars - see asound.h */
+       /* longname is 80 chars - see asound.h */
+       sprintf(card->longname, "Line6 %s at USB %s", line6->properties->name,
+               dev_name(line6->ifcdev));
        return 0;
 }
 
index 06a6db889fbd79f4d6686dd2fa2e095345e06170..1e3bb14063319620ea05b7b0a815521c0d09b2d9 100644 (file)
@@ -164,15 +164,19 @@ void line6_capture_copy(struct snd_line6_pcm *line6pcm, char *fbuf, int fsize)
                               len * bytes_per_frame);
                        memcpy(runtime->dma_area, fbuf + len * bytes_per_frame,
                               (frames - len) * bytes_per_frame);
-               } else
-                       dev_err(line6pcm->line6->ifcdev, "driver bug: len = %d\n", len);        /* this is somewhat paranoid */
+               } else {
+                       /* this is somewhat paranoid */
+                       dev_err(line6pcm->line6->ifcdev,
+                               "driver bug: len = %d\n", len);
+               }
        } else {
                /* copy single chunk */
                memcpy(runtime->dma_area +
                       line6pcm->pos_in_done * bytes_per_frame, fbuf, fsize);
        }
 
-       if ((line6pcm->pos_in_done += frames) >= runtime->buffer_size)
+       line6pcm->pos_in_done += frames;
+       if (line6pcm->pos_in_done >= runtime->buffer_size)
                line6pcm->pos_in_done -= runtime->buffer_size;
 }
 
@@ -181,15 +185,16 @@ void line6_capture_check_period(struct snd_line6_pcm *line6pcm, int length)
        struct snd_pcm_substream *substream =
            get_substream(line6pcm, SNDRV_PCM_STREAM_CAPTURE);
 
-       if ((line6pcm->bytes_in += length) >= line6pcm->period_in) {
+       line6pcm->bytes_in += length;
+       if (line6pcm->bytes_in >= line6pcm->period_in) {
                line6pcm->bytes_in %= line6pcm->period_in;
                snd_pcm_period_elapsed(substream);
        }
 }
 
 /*
-  Callback for completed capture URB.
-*/
* Callback for completed capture URB.
+ */
 static void audio_in_callback(struct urb *urb)
 {
        int i, index, length = 0, shutdown = 0;
index adad130c5dc2d0c15aa3e0d1b5f8b1d9ccf94380..f8a5149e3dadabc28d480d61709ce83f72701192 100644 (file)
@@ -18,9 +18,9 @@
 #endif
 
 
-/**
  Development tools.
-*/
+/*
* Development tools.
+ */
 #define DO_DEBUG_MESSAGES    0
 #define DO_DUMP_URB_SEND     DO_DEBUG_MESSAGES
 #define DO_DUMP_URB_RECEIVE  DO_DEBUG_MESSAGES
index c671427350ef9ad12e447502066b1aa527d5b7d4..ea9209d9ceb268ee67663c19ba455725af3133da 100644 (file)
@@ -634,8 +634,7 @@ int line6_write_data(struct usb_line6 *line6, int address, void *data,
                                "receiving status failed (error %d)\n", ret);
                        return ret;
                }
-       }
-       while (status == 0xff);
+       } while (status == 0xff);
 
        if (status != 0) {
                dev_err(line6->ifcdev, "write failed (error %d)\n", ret);
@@ -667,7 +666,7 @@ ssize_t line6_nop_read(struct device *dev, struct device_attribute *attr,
 /*
        No operation (i.e., unsupported).
 */
-ssize_t line6_nop_write(struct device * dev, struct device_attribute * attr,
+ssize_t line6_nop_write(struct device *dev, struct device_attribute *attr,
                        const char *buf, size_t count)
 {
        return count;
@@ -677,7 +676,7 @@ ssize_t line6_nop_write(struct device * dev, struct device_attribute * attr,
        "write" request on "raw" special file.
 */
 #ifdef CONFIG_LINE6_USB_RAW
-ssize_t line6_set_raw(struct device * dev, struct device_attribute * attr,
+ssize_t line6_set_raw(struct device *dev, struct device_attribute *attr,
                      const char *buf, size_t count)
 {
        struct usb_interface *interface = to_usb_interface(dev);
index 01fa27e757745e678fc573d6a5f31afa989eb6e2..553192f49317dff756dc82adc8e7b9211a0eda57 100644 (file)
@@ -70,10 +70,12 @@ do {                                \
                return err;     \
 } while (0)
 
-#define CHECK_STARTUP_PROGRESS(x, n)      \
-       if((x) >= (n))                     \
-               return;                    \
-       x = (n);
+#define CHECK_STARTUP_PROGRESS(x, n)   \
+do {                                   \
+       if ((x) >= (n))                 \
+               return;                 \
+       x = (n);                        \
+} while (0)
 
 extern const unsigned char line6_midi_id[3];
 extern struct usb_line6 *line6_devices[LINE6_MAX_DEVICES];
index fffd80503776d884dcec0174d5137c1997fd35e0..4304dfe6c16608debc95f34b7d2cf1dd3a49d681 100644 (file)
@@ -380,8 +380,10 @@ int line6_init_midi(struct usb_line6 *line6)
        int err;
        struct snd_line6_midi *line6midi;
 
-       if (!(line6->properties->capabilities & LINE6_BIT_CONTROL))
-               return 0;       /* skip MIDI initialization and report success */
+       if (!(line6->properties->capabilities & LINE6_BIT_CONTROL)) {
+               /* skip MIDI initialization and report success */
+               return 0;
+       }
 
        line6midi = kzalloc(sizeof(struct snd_line6_midi), GFP_KERNEL);
 
index 8fb5596efe9e86457d1d6cec18d804bdd184c1ad..e54770e34d2e79440c62e08460c401e5c484ef23 100644 (file)
@@ -361,9 +361,8 @@ static int snd_line6_pcm_free(struct snd_device *device)
 */
 static void pcm_disconnect_substream(struct snd_pcm_substream *substream)
 {
-       if (substream->runtime && snd_pcm_running(substream)) {
+       if (substream->runtime && snd_pcm_running(substream))
                snd_pcm_stop(substream, SNDRV_PCM_STATE_DISCONNECTED);
-       }
 }
 
 /*
index 3a50083cb30c5a2d7cde0cc7d33dbfc5f3876c2f..77055b3724ad1dac63379aca7b60cc891f305ac2 100644 (file)
@@ -45,7 +45,8 @@
 /*
        Get substream from Line6 PCM data structure
 */
-#define get_substream(line6pcm, stream) (line6pcm->pcm->streams[stream].substream)
+#define get_substream(line6pcm, stream)        \
+               (line6pcm->pcm->streams[stream].substream)
 
 /*
        PCM mode bits and masks.
@@ -312,13 +313,13 @@ extern void line6_pcm_disconnect(struct snd_line6_pcm *line6pcm);
 extern int line6_pcm_start(struct snd_line6_pcm *line6pcm, int channels);
 extern int line6_pcm_stop(struct snd_line6_pcm *line6pcm, int channels);
 
-#define PRINT_FRAME_DIFF(op) {                                                                                                                                         \
-       static int diff_prev = 1000;                                                                                                                                            \
+#define PRINT_FRAME_DIFF(op) {                                         \
+       static int diff_prev = 1000;                                    \
        int diff = line6pcm->last_frame_out - line6pcm->last_frame_in;  \
-       if((diff != diff_prev) && (abs(diff) < 100)) {                                                                  \
-               printk("%s frame diff = %d\n", op, diff);                                                                                       \
-               diff_prev = diff;                                                                                                                                                                                       \
-       }                                                                                                                                                                                                                                                               \
-       }
+       if ((diff != diff_prev) && (abs(diff) < 100)) {                 \
+               printk(KERN_INFO "%s frame diff = %d\n", op, diff);     \
+               diff_prev = diff;                                       \
+       }                                                               \
+}
 
 #endif
index 39218d686aa24a7b1ebde30c2758a62ea9e70698..29940fd1671b5108d32906ed3d755fad2cddc207 100644 (file)
@@ -165,9 +165,8 @@ static int submit_audio_out_urb(struct snd_line6_pcm *line6pcm)
                struct usb_iso_packet_descriptor *fout =
                    &urb_out->iso_frame_desc[i];
 
-               if (line6pcm->flags & MASK_CAPTURE) {
+               if (line6pcm->flags & MASK_CAPTURE)
                        fsize = line6pcm->prev_fsize;
-               }
 
                if (fsize == 0) {
                        int n;
@@ -237,7 +236,8 @@ static int submit_audio_out_urb(struct snd_line6_pcm *line6pcm)
 #endif
                }
 
-               if ((line6pcm->pos_out += urb_frames) >= runtime->buffer_size)
+               line6pcm->pos_out += urb_frames;
+               if (line6pcm->pos_out >= runtime->buffer_size)
                        line6pcm->pos_out -= runtime->buffer_size;
        } else {
                memset(urb_out->transfer_buffer, 0,
@@ -418,8 +418,8 @@ static void audio_out_callback(struct urb *urb)
                submit_audio_out_urb(line6pcm);
 
                if (test_bit(BIT_PCM_ALSA_PLAYBACK, &line6pcm->flags)) {
-                       if ((line6pcm->bytes_out +=
-                            length) >= line6pcm->period_out) {
+                       line6pcm->bytes_out += length;
+                       if (line6pcm->bytes_out >= line6pcm->period_out) {
                                line6pcm->bytes_out %= line6pcm->period_out;
                                snd_pcm_period_elapsed(substream);
                        }
index c6f2a172d6f9e4728fab0185e8e3352f6aeebfc2..f2fc8c0526e313d5d1aede620f2d49bc8e166a55 100644 (file)
 #include "driver.h"
 
 /*
-  When the TonePort is used with jack in full duplex mode and the outputs are
-  not connected, the software monitor produces an ugly noise since everything
-  written to the output buffer (i.e., the input signal) will be repeated in the
-  next period (sounds like a delay effect). As a workaround, the output buffer
-  is cleared after the data have been read, but there must be a better
-  solution. Until one is found, this workaround can be used to fix the problem.
-*/
+ * When the TonePort is used with jack in full duplex mode and the outputs are
+ * not connected, the software monitor produces an ugly noise since everything
+ * written to the output buffer (i.e., the input signal) will be repeated in
+ * the next period (sounds like a delay effect). As a workaround, the output
+ * buffer is cleared after the data have been read, but there must be a better
+ * solution. Until one is found, this workaround can be used to fix the
+ * problem.
+ */
 #define USE_CLEAR_BUFFER_WORKAROUND 1
 
 extern struct snd_pcm_ops snd_line6_playback_ops;
index e6a1c79cd8519e3a431980a0e1622fcc1c19087f..22e2cedcacf79f027c6b0505c99d66641ec03223 100644 (file)
@@ -1242,35 +1242,29 @@ static int pod_try_init(struct usb_interface *interface,
 
        /* create sysfs entries: */
        err = pod_create_files2(&interface->dev);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initialize audio system: */
        err = line6_init_audio(line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initialize MIDI subsystem: */
        err = line6_init_midi(line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initialize PCM subsystem: */
        err = line6_init_pcm(line6, &pod_pcm_properties);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* register monitor control: */
-       err =
-           snd_ctl_add(line6->card,
-                       snd_ctl_new1(&pod_control_monitor, line6->line6pcm));
-       if (err < 0) {
+       err = snd_ctl_add(line6->card,
+                         snd_ctl_new1(&pod_control_monitor, line6->line6pcm));
+       if (err < 0)
                return err;
-       }
 
        /*
           When the sound card is registered at this point, the PODxt Live
@@ -1295,9 +1289,8 @@ int line6_pod_init(struct usb_interface *interface, struct usb_line6_pod *pod)
 {
        int err = pod_try_init(interface, pod);
 
-       if (err < 0) {
+       if (err < 0)
                pod_destruct(interface);
-       }
 
        return err;
 }
@@ -1317,9 +1310,8 @@ void line6_pod_disconnect(struct usb_interface *interface)
                struct snd_line6_pcm *line6pcm = pod->line6.line6pcm;
                struct device *dev = &interface->dev;
 
-               if (line6pcm != NULL) {
+               if (line6pcm != NULL)
                        line6_pcm_disconnect(line6pcm);
-               }
 
                if (dev != NULL) {
                        /* remove sysfs entries: */
index d4ac734d744f1aa9d6c56340736b70f78676ff15..6a10b0f9749a4cee25e54a717072b734a67e5baf 100644 (file)
@@ -349,24 +349,20 @@ static int toneport_try_init(struct usb_interface *interface,
 
        /* initialize audio system: */
        err = line6_init_audio(line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initialize PCM subsystem: */
        err = line6_init_pcm(line6, &toneport_pcm_properties);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* register monitor control: */
-       err =
-           snd_ctl_add(line6->card,
-                       snd_ctl_new1(&toneport_control_monitor,
-                                    line6->line6pcm));
-       if (err < 0) {
+       err = snd_ctl_add(line6->card,
+                         snd_ctl_new1(&toneport_control_monitor,
+                                      line6->line6pcm));
+       if (err < 0)
                return err;
-       }
 
        /* register source select control: */
        switch (usbdev->descriptor.idProduct) {
@@ -376,16 +372,14 @@ static int toneport_try_init(struct usb_interface *interface,
                    snd_ctl_add(line6->card,
                                snd_ctl_new1(&toneport_control_source,
                                             line6->line6pcm));
-               if (err < 0) {
+               if (err < 0)
                        return err;
-               }
        }
 
        /* register audio system: */
        err = line6_register_audio(line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        line6_read_serial_number(line6, &toneport->serial_number);
        line6_read_data(line6, 0x80c2, &toneport->firmware_version, 1);
@@ -416,9 +410,8 @@ int line6_toneport_init(struct usb_interface *interface,
 {
        int err = toneport_try_init(interface, toneport);
 
-       if (err < 0) {
+       if (err < 0)
                toneport_destruct(interface);
-       }
 
        return err;
 }
index a8a43a28a9722f1ef06d8f4524b5c0123af28500..c6dffe6bc1a532285086b778ea882387d698a727 100644 (file)
@@ -74,7 +74,9 @@
 /* device support hardware monitoring */
 #define LINE6_BIT_HWMON                (1 << 2)
 
-#define LINE6_BIT_CONTROL_PCM_HWMON (LINE6_BIT_CONTROL | LINE6_BIT_PCM | LINE6_BIT_HWMON)
+#define LINE6_BIT_CONTROL_PCM_HWMON    (LINE6_BIT_CONTROL |    \
+                                        LINE6_BIT_PCM |        \
+                                        LINE6_BIT_HWMON)
 
 #define LINE6_FALLBACK_INTERVAL                10
 #define LINE6_FALLBACK_MAXPACKETSIZE   16
index 3a46104a14a18ab0a76cf6bd9b551a6c6f8739f1..894eee7f2317b62c392dc426be44dee4479fb89c 100644 (file)
@@ -664,15 +664,13 @@ static int variax_try_init(struct usb_interface *interface,
 
        /* initialize audio system: */
        err = line6_init_audio(&variax->line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initialize MIDI subsystem: */
        err = line6_init_midi(&variax->line6);
-       if (err < 0) {
+       if (err < 0)
                return err;
-       }
 
        /* initiate startup procedure: */
        variax_startup1(variax);
@@ -687,9 +685,8 @@ int line6_variax_init(struct usb_interface *interface,
 {
        int err = variax_try_init(interface, variax);
 
-       if (err < 0) {
+       if (err < 0)
                variax_destruct(interface);
-       }
 
        return err;
 }