diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2007-07-16 20:13:48 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2007-07-18 17:53:27 -0400 |
commit | 5d59a6f1ba90f15132d55d9d4d5a632f15a43d84 (patch) | |
tree | 41482875bd19d9568f2f3af7be8d912f05e5e2fd /drivers | |
parent | b980f5a224f3df6c884dbf5ae48797ce352ba139 (diff) |
firewire: remove bogus check in fw_core_handle_request
This check is bogus:
- Maximum asynchronous payload size for S800...S3200 is 4096.
- The p->payload_length is totally uninteresting. Only the
request->length of the subsequently allocated and initialized
struct fw_request is of significance.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Kristian Høgsberg <krh@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/firewire/fw-transaction.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/firewire/fw-transaction.c b/drivers/firewire/fw-transaction.c index 80d0121463d0..c4b3417a5dac 100644 --- a/drivers/firewire/fw-transaction.c +++ b/drivers/firewire/fw-transaction.c | |||
@@ -628,11 +628,6 @@ fw_core_handle_request(struct fw_card *card, struct fw_packet *p) | |||
628 | unsigned long flags; | 628 | unsigned long flags; |
629 | int tcode, destination, source; | 629 | int tcode, destination, source; |
630 | 630 | ||
631 | if (p->payload_length > 2048) { | ||
632 | /* FIXME: send error response. */ | ||
633 | return; | ||
634 | } | ||
635 | |||
636 | if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE) | 631 | if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE) |
637 | return; | 632 | return; |
638 | 633 | ||