diff options
| author | Darrick J. Wong <djwong@us.ibm.com> | 2007-01-29 23:48:22 -0800 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2007-02-03 08:16:19 -0600 |
| commit | 058e2c474897dc53c88ac9162f9a9b16a879b8cd (patch) | |
| tree | fbfe6f7d6be148f129389d62e255ff96225c81c1 /drivers | |
| parent | a9344e68ac0a656475006737dbc258d69fe4f7b0 (diff) | |
| download | kernel_samsung_tuna-058e2c474897dc53c88ac9162f9a9b16a879b8cd.zip kernel_samsung_tuna-058e2c474897dc53c88ac9162f9a9b16a879b8cd.tar.gz kernel_samsung_tuna-058e2c474897dc53c88ac9162f9a9b16a879b8cd.tar.bz2 | |
[SCSI] aic94xx: Remove TMF result code munging
In asd_initiate_ssp_tmf, the TMF result code is replaced with
TMF_RESP_FUNC_FAILED except when the TMF returns a result code immediately.
However, TMFs can return result codes via an ESCB... yet these codes are
also replaced with "FAILED". The only values that can fall into that case
are TMF_* codes anyway, so get rid of this code where COMPLETE and SUCCESS
are turned into FAILED. This also lets us propagate those TMF_* codes up
to the caller.
Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/scsi/aic94xx/aic94xx_tmf.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/scsi/aic94xx/aic94xx_tmf.c b/drivers/scsi/aic94xx/aic94xx_tmf.c index fd5269e..9a14a6d 100644 --- a/drivers/scsi/aic94xx/aic94xx_tmf.c +++ b/drivers/scsi/aic94xx/aic94xx_tmf.c @@ -566,14 +566,7 @@ static int asd_initiate_ssp_tmf(struct domain_device *dev, u8 *lun, res = TMF_RESP_FUNC_ESUPP; break; default: - if (tmf == TMF_QUERY_TASK) { - ASD_DPRINTK("%s: QUERY_SSP_TASK response: 0x%x\n", - __FUNCTION__, res); - break; - } - ASD_DPRINTK("%s: converting result 0x%x to TMF_RESP_FUNC_FAILED\n", - __FUNCTION__, res); - res = TMF_RESP_FUNC_FAILED; + /* Allow TMF response codes to propagate upwards */ break; } out_err: |