Commit 22c1a660 authored by FUJITA Tomonori's avatar FUJITA Tomonori Committed by James Bottomley

[SCSI] aic7xxx_old: convert to use the data buffer accessors

- remove the unnecessary map_single path.

- convert to use the new accessors for the sg lists and the
parameters.
Signed-off-by: default avatarFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: default avatarDoug Ledford <dledford@redhat.com>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
parent 85289f2e
...@@ -2690,17 +2690,8 @@ aic7xxx_done(struct aic7xxx_host *p, struct aic7xxx_scb *scb) ...@@ -2690,17 +2690,8 @@ aic7xxx_done(struct aic7xxx_host *p, struct aic7xxx_scb *scb)
struct aic7xxx_scb *scbp; struct aic7xxx_scb *scbp;
unsigned char queue_depth; unsigned char queue_depth;
if (cmd->use_sg > 1) scsi_dma_unmap(cmd);
{
struct scatterlist *sg;
sg = (struct scatterlist *)cmd->request_buffer;
pci_unmap_sg(p->pdev, sg, cmd->use_sg, cmd->sc_data_direction);
}
else if (cmd->request_bufflen)
pci_unmap_single(p->pdev, aic7xxx_mapping(cmd),
cmd->request_bufflen,
cmd->sc_data_direction);
if (scb->flags & SCB_SENSE) if (scb->flags & SCB_SENSE)
{ {
pci_unmap_single(p->pdev, pci_unmap_single(p->pdev,
...@@ -3869,7 +3860,7 @@ aic7xxx_calculate_residual (struct aic7xxx_host *p, struct aic7xxx_scb *scb) ...@@ -3869,7 +3860,7 @@ aic7xxx_calculate_residual (struct aic7xxx_host *p, struct aic7xxx_scb *scb)
* the mid layer didn't check residual data counts to see if the * the mid layer didn't check residual data counts to see if the
* command needs retried. * command needs retried.
*/ */
cmd->resid = scb->sg_length - actual; scsi_set_resid(cmd, scb->sg_length - actual);
aic7xxx_status(cmd) = hscb->target_status; aic7xxx_status(cmd) = hscb->target_status;
} }
} }
...@@ -10137,6 +10128,7 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10137,6 +10128,7 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
struct scsi_device *sdptr = cmd->device; struct scsi_device *sdptr = cmd->device;
unsigned char tindex = TARGET_INDEX(cmd); unsigned char tindex = TARGET_INDEX(cmd);
struct request *req = cmd->request; struct request *req = cmd->request;
int use_sg;
mask = (0x01 << tindex); mask = (0x01 << tindex);
hscb = scb->hscb; hscb = scb->hscb;
...@@ -10209,8 +10201,10 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10209,8 +10201,10 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
memcpy(scb->cmnd, cmd->cmnd, cmd->cmd_len); memcpy(scb->cmnd, cmd->cmnd, cmd->cmd_len);
hscb->SCSI_cmd_pointer = cpu_to_le32(SCB_DMA_ADDR(scb, scb->cmnd)); hscb->SCSI_cmd_pointer = cpu_to_le32(SCB_DMA_ADDR(scb, scb->cmnd));
if (cmd->use_sg) use_sg = scsi_dma_map(cmd);
{ BUG_ON(use_sg < 0);
if (use_sg) {
struct scatterlist *sg; /* Must be mid-level SCSI code scatterlist */ struct scatterlist *sg; /* Must be mid-level SCSI code scatterlist */
/* /*
...@@ -10219,11 +10213,11 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10219,11 +10213,11 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
* differences and the kernel SG list uses virtual addresses where * differences and the kernel SG list uses virtual addresses where
* we need physical addresses. * we need physical addresses.
*/ */
int i, use_sg; int i;
sg = (struct scatterlist *)cmd->request_buffer;
scb->sg_length = 0; scb->sg_length = 0;
use_sg = pci_map_sg(p->pdev, sg, cmd->use_sg, cmd->sc_data_direction);
/* /*
* Copy the segments into the SG array. NOTE!!! - We used to * Copy the segments into the SG array. NOTE!!! - We used to
* have the first entry both in the data_pointer area and the first * have the first entry both in the data_pointer area and the first
...@@ -10231,10 +10225,9 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10231,10 +10225,9 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
* entry in both places, but now we download the address of * entry in both places, but now we download the address of
* scb->sg_list[1] instead of 0 to the sg pointer in the hscb. * scb->sg_list[1] instead of 0 to the sg pointer in the hscb.
*/ */
for (i = 0; i < use_sg; i++) scsi_for_each_sg(cmd, sg, use_sg, i) {
{ unsigned int len = sg_dma_len(sg);
unsigned int len = sg_dma_len(sg+i); scb->sg_list[i].address = cpu_to_le32(sg_dma_address(sg));
scb->sg_list[i].address = cpu_to_le32(sg_dma_address(sg+i));
scb->sg_list[i].length = cpu_to_le32(len); scb->sg_list[i].length = cpu_to_le32(len);
scb->sg_length += len; scb->sg_length += len;
} }
...@@ -10244,26 +10237,7 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10244,26 +10237,7 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
scb->sg_count = i; scb->sg_count = i;
hscb->SG_segment_count = i; hscb->SG_segment_count = i;
hscb->SG_list_pointer = cpu_to_le32(SCB_DMA_ADDR(scb, &scb->sg_list[1])); hscb->SG_list_pointer = cpu_to_le32(SCB_DMA_ADDR(scb, &scb->sg_list[1]));
} } else {
else
{
if (cmd->request_bufflen)
{
unsigned int address = pci_map_single(p->pdev, cmd->request_buffer,
cmd->request_bufflen,
cmd->sc_data_direction);
aic7xxx_mapping(cmd) = address;
scb->sg_list[0].address = cpu_to_le32(address);
scb->sg_list[0].length = cpu_to_le32(cmd->request_bufflen);
scb->sg_count = 1;
scb->sg_length = cmd->request_bufflen;
hscb->SG_segment_count = 1;
hscb->SG_list_pointer = cpu_to_le32(SCB_DMA_ADDR(scb, &scb->sg_list[0]));
hscb->data_count = scb->sg_list[0].length;
hscb->data_pointer = scb->sg_list[0].address;
}
else
{
scb->sg_count = 0; scb->sg_count = 0;
scb->sg_length = 0; scb->sg_length = 0;
hscb->SG_segment_count = 0; hscb->SG_segment_count = 0;
...@@ -10271,7 +10245,6 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd, ...@@ -10271,7 +10245,6 @@ static void aic7xxx_buildscb(struct aic7xxx_host *p, struct scsi_cmnd *cmd,
hscb->data_count = 0; hscb->data_count = 0;
hscb->data_pointer = 0; hscb->data_pointer = 0;
} }
}
} }
/*+F************************************************************************* /*+F*************************************************************************
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment