Commit 5d59a6f1 authored Jul 17, 2007 by Stefan Richter

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>

parent b980f5a2

drivers/firewire/fw-transaction.c

+0 −5

Original line number	Diff line number	Diff line
		@@ -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;