diff options
author | Clemens Ladisch <clemens@ladisch.de> | 2012-03-18 14:01:39 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2012-03-18 17:15:38 -0400 |
commit | 32c507f7b73d74c00caae6bbbd539f368be5d108 (patch) | |
tree | ca6994989dd5735833dbe1b9133e2e6133b45790 /drivers/firewire | |
parent | a7611e84bd06c70c8b0cd88cd1e474c3c42fee3e (diff) |
firewire: ohci: copy_iso_headers(): make comment match the code
The comment incorrectly talked about one little-endian quadlet, while
there are actually two. Furthermore, the endianness of the remaining
headers depends on whatever protocol is used, so don't mention them.
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/firewire')
-rw-r--r-- | drivers/firewire/ohci.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/firewire/ohci.c b/drivers/firewire/ohci.c index e52615a14382..74ff1a8f4fcb 100644 --- a/drivers/firewire/ohci.c +++ b/drivers/firewire/ohci.c | |||
@@ -2685,10 +2685,9 @@ static void copy_iso_headers(struct iso_context *ctx, void *p) | |||
2685 | return; | 2685 | return; |
2686 | 2686 | ||
2687 | /* | 2687 | /* |
2688 | * The iso header is byteswapped to little endian by | 2688 | * The two iso header quadlets are byteswapped to little |
2689 | * the controller, but the remaining header quadlets | 2689 | * endian by the controller, but we want to present them |
2690 | * are big endian. We want to present all the headers | 2690 | * as big endian for consistency with the bus endianness. |
2691 | * as big endian, so we have to swap the first quadlet. | ||
2692 | */ | 2691 | */ |
2693 | if (ctx->base.header_size > 0) | 2692 | if (ctx->base.header_size > 0) |
2694 | *(u32 *) (ctx->header + i) = __swab32(*(u32 *) (p + 4)); | 2693 | *(u32 *) (ctx->header + i) = __swab32(*(u32 *) (p + 4)); |