Date: Thu, 29 Jul 2010 09:31:56 +0200 (CEST) From: Stefan Richter Subject: firewire: core: small clarifications in core-cdev Make a note on the seemingly unused linux/sched.h. Rename an irritatingly named variable. Signed-off-by: Stefan Richter --- drivers/firewire/core-cdev.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) Index: b/drivers/firewire/core-cdev.c =================================================================== --- a/drivers/firewire/core-cdev.c +++ b/drivers/firewire/core-cdev.c @@ -34,7 +34,7 @@ #include #include #include -#include +#include /* required for linux/wait.h */ #include #include #include @@ -994,7 +994,7 @@ static int ioctl_queue_iso(struct client struct fw_cdev_queue_iso *a = &arg->queue_iso; struct fw_cdev_iso_packet __user *p, *end, *next; struct fw_iso_context *ctx = client->iso_context; - unsigned long payload, buffer_end, header_length; + unsigned long payload, buffer_end, transmit_header_bytes; u32 control; int count; struct { @@ -1043,7 +1043,7 @@ static int ioctl_queue_iso(struct client if (ctx->type == FW_ISO_CONTEXT_TRANSMIT) { if (u.packet.header_length % 4 != 0) return -EINVAL; - header_length = u.packet.header_length; + transmit_header_bytes = u.packet.header_length; } else { /* * We require that header_length is a multiple of @@ -1052,15 +1052,15 @@ static int ioctl_queue_iso(struct client if (u.packet.header_length == 0 || u.packet.header_length % ctx->header_size != 0) return -EINVAL; - header_length = 0; + transmit_header_bytes = 0; } next = (struct fw_cdev_iso_packet __user *) - &p->header[header_length / 4]; + &p->header[transmit_header_bytes / 4]; if (next > end) return -EINVAL; if (__copy_from_user - (u.packet.header, p->header, header_length)) + (u.packet.header, p->header, transmit_header_bytes)) return -EFAULT; if (u.packet.skip && ctx->type == FW_ISO_CONTEXT_TRANSMIT && u.packet.header_length + u.packet.payload_length > 0)