diff options
author | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-02-05 00:06:27 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2008-02-06 06:59:32 -0500 |
commit | 37198e3051b63d3184886e9bb8235e7578e82628 (patch) | |
tree | 3ecd4e8b6b7c710b808909adbd2bf643a10e85b0 | |
parent | 837f5f8fb98d4357d49e9631c9ee2815f3c328ca (diff) |
libata: kill now unused n_iter and fix sata_fsl
qc->n_iter was used for libata's own sg walking before sg chaining
replaced it. During conversion, the field and its usage in sata_fsl
were left behind. Kill the filed and update sata_fsl.
tj: This was part of James's libata-use-block-layer-padding patch.
Separated out by me.
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: Tejun Heo <htejun@gmail.com>
Cc: Li Yang <leoli@freescale.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/ata/sata_fsl.c | 4 | ||||
-rw-r--r-- | include/linux/libata.h | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 922d7b2efba8..efcb66b6ccef 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c | |||
@@ -355,8 +355,8 @@ static unsigned int sata_fsl_fill_sg(struct ata_queued_cmd *qc, void *cmd_desc, | |||
355 | ata_port_printk(qc->ap, KERN_ERR, | 355 | ata_port_printk(qc->ap, KERN_ERR, |
356 | "s/g len unaligned : 0x%x\n", sg_len); | 356 | "s/g len unaligned : 0x%x\n", sg_len); |
357 | 357 | ||
358 | if ((num_prde == (SATA_FSL_MAX_PRD_DIRECT - 1)) && | 358 | if (num_prde == (SATA_FSL_MAX_PRD_DIRECT - 1) && |
359 | (qc->n_iter + 1 != qc->n_elem)) { | 359 | sg_next(sg) != NULL) { |
360 | VPRINTK("setting indirect prde\n"); | 360 | VPRINTK("setting indirect prde\n"); |
361 | prd_ptr_to_indirect_ext = prd; | 361 | prd_ptr_to_indirect_ext = prd; |
362 | prd->dba = cpu_to_le32(indirect_ext_segment_paddr); | 362 | prd->dba = cpu_to_le32(indirect_ext_segment_paddr); |
diff --git a/include/linux/libata.h b/include/linux/libata.h index 4374c4277780..bc5a8d0c7090 100644 --- a/include/linux/libata.h +++ b/include/linux/libata.h | |||
@@ -457,7 +457,6 @@ struct ata_queued_cmd { | |||
457 | unsigned long flags; /* ATA_QCFLAG_xxx */ | 457 | unsigned long flags; /* ATA_QCFLAG_xxx */ |
458 | unsigned int tag; | 458 | unsigned int tag; |
459 | unsigned int n_elem; | 459 | unsigned int n_elem; |
460 | unsigned int n_iter; | ||
461 | unsigned int mapped_n_elem; | 460 | unsigned int mapped_n_elem; |
462 | 461 | ||
463 | int dma_dir; | 462 | int dma_dir; |
@@ -1367,7 +1366,6 @@ static inline void ata_qc_reinit(struct ata_queued_cmd *qc) | |||
1367 | qc->nbytes = qc->raw_nbytes = qc->curbytes = 0; | 1366 | qc->nbytes = qc->raw_nbytes = qc->curbytes = 0; |
1368 | qc->n_elem = 0; | 1367 | qc->n_elem = 0; |
1369 | qc->mapped_n_elem = 0; | 1368 | qc->mapped_n_elem = 0; |
1370 | qc->n_iter = 0; | ||
1371 | qc->err_mask = 0; | 1369 | qc->err_mask = 0; |
1372 | qc->pad_len = 0; | 1370 | qc->pad_len = 0; |
1373 | qc->last_sg = NULL; | 1371 | qc->last_sg = NULL; |