Use the scatterlist iterators and remove direct indexing of the
scatterlist array.

This way allows us to pre-allocate one small scatterlist, which can be
chained with one runtime allocated scatterlist if the pre-allocated one
isn't enough for the whole request.

Signed-off-by: Ming Lei <ming....@redhat.com>
---
 drivers/scsi/wd33c93.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/wd33c93.c b/drivers/scsi/wd33c93.c
index 74be04f2357c..ae5935c0a149 100644
--- a/drivers/scsi/wd33c93.c
+++ b/drivers/scsi/wd33c93.c
@@ -744,7 +744,7 @@ transfer_bytes(const wd33c93_regs regs, struct scsi_cmnd 
*cmd,
  * source or destination for THIS transfer.
  */
        if (!cmd->SCp.this_residual && cmd->SCp.buffers_residual) {
-               ++cmd->SCp.buffer;
+               cmd->SCp.buffer = sg_next(cmd->SCp.buffer);
                --cmd->SCp.buffers_residual;
                cmd->SCp.this_residual = cmd->SCp.buffer->length;
                cmd->SCp.ptr = sg_virt(cmd->SCp.buffer);
-- 
2.20.1

Reply via email to