diff options
| -rw-r--r-- | drivers/firewire/core-card.c | 4 | ||||
| -rw-r--r-- | drivers/firewire/core-transaction.c | 5 | ||||
| -rw-r--r-- | include/linux/firewire.h | 8 |
3 files changed, 10 insertions, 7 deletions
diff --git a/drivers/firewire/core-card.c b/drivers/firewire/core-card.c index 2bb5c036e806..0c312c4bb4bd 100644 --- a/drivers/firewire/core-card.c +++ b/drivers/firewire/core-card.c | |||
| @@ -570,12 +570,12 @@ static int dummy_set_config_rom(struct fw_card *card, | |||
| 570 | 570 | ||
| 571 | static void dummy_send_request(struct fw_card *card, struct fw_packet *packet) | 571 | static void dummy_send_request(struct fw_card *card, struct fw_packet *packet) |
| 572 | { | 572 | { |
| 573 | packet->callback(packet, card, -ENODEV); | 573 | packet->callback(packet, card, RCODE_CANCELLED); |
| 574 | } | 574 | } |
| 575 | 575 | ||
| 576 | static void dummy_send_response(struct fw_card *card, struct fw_packet *packet) | 576 | static void dummy_send_response(struct fw_card *card, struct fw_packet *packet) |
| 577 | { | 577 | { |
| 578 | packet->callback(packet, card, -ENODEV); | 578 | packet->callback(packet, card, RCODE_CANCELLED); |
| 579 | } | 579 | } |
| 580 | 580 | ||
| 581 | static int dummy_cancel_packet(struct fw_card *card, struct fw_packet *packet) | 581 | static int dummy_cancel_packet(struct fw_card *card, struct fw_packet *packet) |
diff --git a/drivers/firewire/core-transaction.c b/drivers/firewire/core-transaction.c index 5f5a7852f7ac..e2e4dc624fb6 100644 --- a/drivers/firewire/core-transaction.c +++ b/drivers/firewire/core-transaction.c | |||
| @@ -310,7 +310,10 @@ static int allocate_tlabel(struct fw_card *card) | |||
| 310 | * After the transaction is completed successfully or unsuccessfully, the | 310 | * After the transaction is completed successfully or unsuccessfully, the |
| 311 | * @callback will be called. Among its parameters is the response code which | 311 | * @callback will be called. Among its parameters is the response code which |
| 312 | * is either one of the rcodes per IEEE 1394 or, in case of internal errors, | 312 | * is either one of the rcodes per IEEE 1394 or, in case of internal errors, |
| 313 | * the firewire-core specific %RCODE_SEND_ERROR. | 313 | * the firewire-core specific %RCODE_SEND_ERROR. The other firewire-core |
| 314 | * specific rcodes (%RCODE_CANCELLED, %RCODE_BUSY, %RCODE_GENERATION, | ||
| 315 | * %RCODE_NO_ACK) denote transaction timeout, busy responder, stale request | ||
| 316 | * generation, or missing ACK respectively. | ||
| 314 | * | 317 | * |
| 315 | * Note some timing corner cases: fw_send_request() may complete much earlier | 318 | * Note some timing corner cases: fw_send_request() may complete much earlier |
| 316 | * than when the request packet actually hits the wire. On the other hand, | 319 | * than when the request packet actually hits the wire. On the other hand, |
diff --git a/include/linux/firewire.h b/include/linux/firewire.h index adc5b55e6e5f..0c38b8e97722 100644 --- a/include/linux/firewire.h +++ b/include/linux/firewire.h | |||
| @@ -285,10 +285,10 @@ struct fw_packet { | |||
| 285 | u32 timestamp; | 285 | u32 timestamp; |
| 286 | 286 | ||
| 287 | /* | 287 | /* |
| 288 | * This callback is called when the packet transmission has | 288 | * This callback is called when the packet transmission has completed. |
| 289 | * completed; for successful transmission, the status code is | 289 | * For successful transmission, the status code is the ack received |
| 290 | * the ack received from the destination, otherwise it's a | 290 | * from the destination. Otherwise it is one of the juju-specific |
| 291 | * negative errno: ENOMEM, ESTALE, ETIMEDOUT, ENODEV, EIO. | 291 | * rcodes: RCODE_SEND_ERROR, _CANCELLED, _BUSY, _GENERATION, _NO_ACK. |
| 292 | * The callback can be called from tasklet context and thus | 292 | * The callback can be called from tasklet context and thus |
| 293 | * must never block. | 293 | * must never block. |
| 294 | */ | 294 | */ |
