diff options
author | Takashi Iwai <tiwai@suse.de> | 2015-01-19 10:08:02 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-01-20 02:18:27 -0500 |
commit | b2a3b02392836b8cc0aa4246d7496028911f1539 (patch) | |
tree | 125bf1beb4e1d05af7420d60871acae86939210e /sound | |
parent | 10e3a023c97e67975812027c1a96db874be2e78d (diff) |
ALSA: line6: Remove CHECK_RETURN macro
Such a macro doesn't improve readability.
Tested-by: Chris Rorvick <chris@rorvick.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/usb/line6/driver.h | 7 | ||||
-rw-r--r-- | sound/usb/line6/pod.c | 12 | ||||
-rw-r--r-- | sound/usb/line6/toneport.c | 10 |
3 files changed, 15 insertions, 14 deletions
diff --git a/sound/usb/line6/driver.h b/sound/usb/line6/driver.h index 8e4d946a130c..f484aa09840b 100644 --- a/sound/usb/line6/driver.h +++ b/sound/usb/line6/driver.h | |||
@@ -48,13 +48,6 @@ | |||
48 | 48 | ||
49 | #define LINE6_CHANNEL_MASK 0x0f | 49 | #define LINE6_CHANNEL_MASK 0x0f |
50 | 50 | ||
51 | #define CHECK_RETURN(x) \ | ||
52 | do { \ | ||
53 | err = x; \ | ||
54 | if (err < 0) \ | ||
55 | return err; \ | ||
56 | } while (0) | ||
57 | |||
58 | #define CHECK_STARTUP_PROGRESS(x, n) \ | 51 | #define CHECK_STARTUP_PROGRESS(x, n) \ |
59 | do { \ | 52 | do { \ |
60 | if ((x) >= (n)) \ | 53 | if ((x) >= (n)) \ |
diff --git a/sound/usb/line6/pod.c b/sound/usb/line6/pod.c index 6b30deb6b157..6f4b09982589 100644 --- a/sound/usb/line6/pod.c +++ b/sound/usb/line6/pod.c | |||
@@ -429,9 +429,15 @@ static int pod_create_files2(struct device *dev) | |||
429 | { | 429 | { |
430 | int err; | 430 | int err; |
431 | 431 | ||
432 | CHECK_RETURN(device_create_file(dev, &dev_attr_device_id)); | 432 | err = device_create_file(dev, &dev_attr_device_id); |
433 | CHECK_RETURN(device_create_file(dev, &dev_attr_firmware_version)); | 433 | if (err < 0) |
434 | CHECK_RETURN(device_create_file(dev, &dev_attr_serial_number)); | 434 | return err; |
435 | err = device_create_file(dev, &dev_attr_firmware_version); | ||
436 | if (err < 0) | ||
437 | return err; | ||
438 | err = device_create_file(dev, &dev_attr_serial_number); | ||
439 | if (err < 0) | ||
440 | return err; | ||
435 | return 0; | 441 | return 0; |
436 | } | 442 | } |
437 | 443 | ||
diff --git a/sound/usb/line6/toneport.c b/sound/usb/line6/toneport.c index 3097a75a9bec..24544b7ab476 100644 --- a/sound/usb/line6/toneport.c +++ b/sound/usb/line6/toneport.c | |||
@@ -431,10 +431,12 @@ static int toneport_init(struct usb_interface *interface, | |||
431 | line6_read_data(line6, 0x80c2, &toneport->firmware_version, 1); | 431 | line6_read_data(line6, 0x80c2, &toneport->firmware_version, 1); |
432 | 432 | ||
433 | if (toneport_has_led(toneport->type)) { | 433 | if (toneport_has_led(toneport->type)) { |
434 | CHECK_RETURN(device_create_file | 434 | err = device_create_file(&interface->dev, &dev_attr_led_red); |
435 | (&interface->dev, &dev_attr_led_red)); | 435 | if (err < 0) |
436 | CHECK_RETURN(device_create_file | 436 | return err; |
437 | (&interface->dev, &dev_attr_led_green)); | 437 | err = device_create_file(&interface->dev, &dev_attr_led_green); |
438 | if (err < 0) | ||
439 | return err; | ||
438 | } | 440 | } |
439 | 441 | ||
440 | toneport_setup(toneport); | 442 | toneport_setup(toneport); |