diff options
author | Tejun Heo <htejun@gmail.com> | 2006-01-22 23:09:36 -0500 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2006-01-26 22:33:49 -0500 |
commit | 77853bf2b48e34449e826a9ef4df5ea0dbe947f4 (patch) | |
tree | b46a186c141c61f05352b7a1199b2940fd9a2065 /include | |
parent | 4ba946e9d8e10fada7bbce527f6ea05842592e06 (diff) |
[PATCH] libata: make the owner of a qc responsible for freeing it
qc used to be freed automatically on command completion. However, as
a qc can carry information about its completion status, it can be
useful to its owner/issuer after command completion. This patch makes
freeing qc responsibility of its owner. This simplifies
ata_exec_internal() and makes command turn-around for atapi request
sensing less hackish.
This change was originally suggested by Jeff Garzik.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/libata.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/libata.h b/include/linux/libata.h index 46ccea215892..d58b659cf3f5 100644 --- a/include/linux/libata.h +++ b/include/linux/libata.h | |||
@@ -235,7 +235,7 @@ struct ata_port; | |||
235 | struct ata_queued_cmd; | 235 | struct ata_queued_cmd; |
236 | 236 | ||
237 | /* typedefs */ | 237 | /* typedefs */ |
238 | typedef int (*ata_qc_cb_t) (struct ata_queued_cmd *qc); | 238 | typedef void (*ata_qc_cb_t) (struct ata_queued_cmd *qc); |
239 | 239 | ||
240 | struct ata_ioports { | 240 | struct ata_ioports { |
241 | unsigned long cmd_addr; | 241 | unsigned long cmd_addr; |