diff options
| author | Dan Williams <dan.j.williams@intel.com> | 2011-06-13 00:51:30 -0700 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2011-07-03 04:04:51 -0700 |
| commit | 0d0cf14c9bd2943ed5afd15df459f564d85eacde (patch) | |
| tree | b996109708782750f7f3d58fea957b5cf19f0fa6 /drivers/scsi/isci/request.h | |
| parent | 994a9303d33f8238d57f58c26067b6d4ac9af222 (diff) | |
| download | blackbird-op-linux-0d0cf14c9bd2943ed5afd15df459f564d85eacde.tar.gz blackbird-op-linux-0d0cf14c9bd2943ed5afd15df459f564d85eacde.zip | |
isci: cleanup request allocation
Rather than return an error code and update a pointer that was passed by
reference just return the request object directly (or null if allocation
failed).
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/scsi/isci/request.h')
| -rw-r--r-- | drivers/scsi/isci/request.h | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/drivers/scsi/isci/request.h b/drivers/scsi/isci/request.h index ac9368c5a6b5..8de2542f081f 100644 --- a/drivers/scsi/isci/request.h +++ b/drivers/scsi/isci/request.h @@ -679,16 +679,13 @@ static inline void isci_request_free(struct isci_host *isci_host, #define isci_request_access_tmf(req) ((req)->ttype_ptr.tmf_task_ptr) -int isci_request_alloc_tmf(struct isci_host *isci_host, - struct isci_tmf *isci_tmf, - struct isci_request **isci_request, - struct isci_remote_device *isci_device, - gfp_t gfp_flags); - +struct isci_request *isci_request_alloc_tmf(struct isci_host *ihost, + struct isci_tmf *isci_tmf, + struct isci_remote_device *idev, + gfp_t gfp_flags); int isci_request_execute(struct isci_host *isci_host, struct sas_task *task, - struct isci_request **request, gfp_t gfp_flags); /** |
