staging: asus_oled: Remove superfluous loop
authorPeter Huewe <peterhuewe@gmx.de>
Sat, 25 Feb 2012 00:19:41 +0000 (01:19 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 29 Feb 2012 23:43:21 +0000 (15:43 -0800)
This patch removes a superfluous loop in asus_oled.c
The code is equivalent to do{...} while (0) and thus executes the code
exactly once -> so we can simply remove the loop.

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/asus_oled/asus_oled.c

index 1df9586..83549d9 100644 (file)
@@ -159,7 +159,6 @@ static void setup_packet_header(struct asus_oled_packet *packet, char flags,
 
 static void enable_oled(struct asus_oled_dev *odev, uint8_t enabl)
 {
-       int a;
        int retval;
        int act_len;
        struct asus_oled_packet *packet;
@@ -178,17 +177,15 @@ static void enable_oled(struct asus_oled_dev *odev, uint8_t enabl)
        else
                packet->bitmap[0] = 0xae;
 
-       for (a = 0; a < 1; a++) {
-               retval = usb_bulk_msg(odev->udev,
-                       usb_sndbulkpipe(odev->udev, 2),
-                       packet,
-                       sizeof(struct asus_oled_header) + 1,
-                       &act_len,
-                       -1);
+       retval = usb_bulk_msg(odev->udev,
+               usb_sndbulkpipe(odev->udev, 2),
+               packet,
+               sizeof(struct asus_oled_header) + 1,
+               &act_len,
+               -1);
 
-               if (retval)
-                       dev_dbg(&odev->udev->dev, "retval = %d\n", retval);
-       }
+       if (retval)
+               dev_dbg(&odev->udev->dev, "retval = %d\n", retval);
 
        odev->enabled = enabl;