firewire: remove bogus check in fw_core_handle_request
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Tue, 17 Jul 2007 00:13:48 +0000 (02:13 +0200)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Wed, 18 Jul 2007 21:53:27 +0000 (23:53 +0200)
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>
drivers/firewire/fw-transaction.c

index 80d0121463d0d7fee8b0a49b6e56a3aab53b20f3..c4b3417a5dac47ec43944262b2518c1ef636a638 100644 (file)
@@ -628,11 +628,6 @@ fw_core_handle_request(struct fw_card *card, struct fw_packet *p)
        unsigned long flags;
        int tcode, destination, source;
 
-       if (p->payload_length > 2048) {
-               /* FIXME: send error response. */
-               return;
-       }
-
        if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE)
                return;