unsigned long flags;
        struct event *event;
        size_t size, total;
-       int i, retval;
+       int i, ret;
 
-       retval = wait_event_interruptible(client->wait,
-                                         !list_empty(&client->event_list) ||
-                                         fw_device_is_shutdown(client->device));
-       if (retval < 0)
-               return retval;
+       ret = wait_event_interruptible(client->wait,
+                       !list_empty(&client->event_list) ||
+                       fw_device_is_shutdown(client->device));
+       if (ret < 0)
+               return ret;
 
        if (list_empty(&client->event_list) &&
                       fw_device_is_shutdown(client->device))
        for (i = 0; i < ARRAY_SIZE(event->v) && total < count; i++) {
                size = min(event->v[i].size, count - total);
                if (copy_to_user(buffer + total, event->v[i].data, size)) {
-                       retval = -EFAULT;
+                       ret = -EFAULT;
                        goto out;
                }
                total += size;
        }
-       retval = total;
+       ret = total;
 
  out:
        kfree(event);
 
-       return retval;
+       return ret;
 }
 
 static ssize_t
 dispatch_ioctl(struct client *client, unsigned int cmd, void __user *arg)
 {
        char buffer[256];
-       int retval;
+       int ret;
 
        if (_IOC_TYPE(cmd) != '#' ||
            _IOC_NR(cmd) >= ARRAY_SIZE(ioctl_handlers))
                        return -EFAULT;
        }
 
-       retval = ioctl_handlers[_IOC_NR(cmd)](client, buffer);
-       if (retval < 0)
-               return retval;
+       ret = ioctl_handlers[_IOC_NR(cmd)](client, buffer);
+       if (ret < 0)
+               return ret;
 
        if (_IOC_DIR(cmd) & _IOC_READ) {
                if (_IOC_SIZE(cmd) > sizeof(buffer) ||
                        return -EFAULT;
        }
 
-       return retval;
+       return ret;
 }
 
 static long
        struct client *client = file->private_data;
        enum dma_data_direction direction;
        unsigned long size;
-       int page_count, retval;
+       int page_count, ret;
 
        if (fw_device_is_shutdown(client->device))
                return -ENODEV;
        else
                direction = DMA_FROM_DEVICE;
 
-       retval = fw_iso_buffer_init(&client->buffer, client->device->card,
-                                   page_count, direction);
-       if (retval < 0)
-               return retval;
+       ret = fw_iso_buffer_init(&client->buffer, client->device->card,
+                                page_count, direction);
+       if (ret < 0)
+               return ret;
 
-       retval = fw_iso_buffer_map(&client->buffer, vma);
-       if (retval < 0)
+       ret = fw_iso_buffer_map(&client->buffer, vma);
+       if (ret < 0)
                fw_iso_buffer_destroy(&client->buffer, client->device->card);
 
-       return retval;
+       return ret;
 }
 
 static int shutdown_resource(int id, void *p, void *data)
 
 at_context_transmit(struct context *ctx, struct fw_packet *packet)
 {
        unsigned long flags;
-       int retval;
+       int ret;
 
        spin_lock_irqsave(&ctx->ohci->lock, flags);
 
                return;
        }
 
-       retval = at_context_queue_packet(ctx, packet);
+       ret = at_context_queue_packet(ctx, packet);
        spin_unlock_irqrestore(&ctx->ohci->lock, flags);
 
-       if (retval < 0)
+       if (ret < 0)
                packet->callback(packet, &ctx->ohci->card, packet->ack);
 
 }
 {
        struct fw_ohci *ohci;
        unsigned long flags;
-       int retval = -EBUSY;
+       int ret = -EBUSY;
        __be32 *next_config_rom;
        dma_addr_t uninitialized_var(next_config_rom_bus);
 
 
                reg_write(ohci, OHCI1394_ConfigROMmap,
                          ohci->next_config_rom_bus);
-               retval = 0;
+               ret = 0;
        }
 
        spin_unlock_irqrestore(&ohci->lock, flags);
         * controller could need to access it before the bus reset
         * takes effect.
         */
-       if (retval == 0)
+       if (ret == 0)
                fw_core_initiate_bus_reset(&ohci->card, 1);
        else
                dma_free_coherent(ohci->card.device, CONFIG_ROM_SIZE,
                                  next_config_rom, next_config_rom_bus);
 
-       return retval;
+       return ret;
 }
 
 static void ohci_send_request(struct fw_card *card, struct fw_packet *packet)
        struct fw_ohci *ohci = fw_ohci(card);
        struct context *ctx = &ohci->at_request_ctx;
        struct driver_data *driver_data = packet->driver_data;
-       int retval = -ENOENT;
+       int ret = -ENOENT;
 
        tasklet_disable(&ctx->tasklet);
 
        driver_data->packet = NULL;
        packet->ack = RCODE_CANCELLED;
        packet->callback(packet, &ohci->card, packet->ack);
-       retval = 0;
-
+       ret = 0;
  out:
        tasklet_enable(&ctx->tasklet);
 
-       return retval;
+       return ret;
 }
 
 static int
 #else
        struct fw_ohci *ohci = fw_ohci(card);
        unsigned long flags;
-       int n, retval = 0;
+       int n, ret = 0;
 
        /*
         * FIXME:  Make sure this bitmask is cleared when we clear the busReset
        spin_lock_irqsave(&ohci->lock, flags);
 
        if (ohci->generation != generation) {
-               retval = -ESTALE;
+               ret = -ESTALE;
                goto out;
        }
 
        flush_writes(ohci);
  out:
        spin_unlock_irqrestore(&ohci->lock, flags);
-       return retval;
+
+       return ret;
 #endif /* CONFIG_FIREWIRE_OHCI_REMOTE_DMA */
 }
 
        descriptor_callback_t callback;
        u32 *mask, regs;
        unsigned long flags;
-       int index, retval = -ENOMEM;
+       int index, ret = -ENOMEM;
 
        if (type == FW_ISO_CONTEXT_TRANSMIT) {
                mask = &ohci->it_context_mask;
        if (ctx->header == NULL)
                goto out;
 
-       retval = context_init(&ctx->context, ohci, regs, callback);
-       if (retval < 0)
+       ret = context_init(&ctx->context, ohci, regs, callback);
+       if (ret < 0)
                goto out_with_header;
 
        return &ctx->base;
        *mask |= 1 << index;
        spin_unlock_irqrestore(&ohci->lock, flags);
 
-       return ERR_PTR(retval);
+       return ERR_PTR(ret);
 }
 
 static int ohci_start_iso(struct fw_iso_context *base,
 {
        struct iso_context *ctx = container_of(base, struct iso_context, base);
        unsigned long flags;
-       int retval;
+       int ret;
 
        spin_lock_irqsave(&ctx->context.ohci->lock, flags);
        if (base->type == FW_ISO_CONTEXT_TRANSMIT)
-               retval = ohci_queue_iso_transmit(base, packet, buffer, payload);
+               ret = ohci_queue_iso_transmit(base, packet, buffer, payload);
        else if (ctx->context.ohci->use_dualbuffer)
-               retval = ohci_queue_iso_receive_dualbuffer(base, packet,
-                                                        buffer, payload);
+               ret = ohci_queue_iso_receive_dualbuffer(base, packet,
+                                                       buffer, payload);
        else
-               retval = ohci_queue_iso_receive_packet_per_buffer(base, packet,
-                                                               buffer,
-                                                               payload);
+               ret = ohci_queue_iso_receive_packet_per_buffer(base, packet,
+                                                       buffer, payload);
        spin_unlock_irqrestore(&ctx->context.ohci->lock, flags);
 
-       return retval;
+       return ret;
 }
 
 static const struct fw_card_driver ohci_driver = {