On Thu, Jun 4, 2009 at 3:33 AM, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > dma_cache_sync() is supposed to be used only with > dma_alloc_noncoherent(). > > We don't need to call any dma_sync_ functions here too. > > Signed-off-by: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> Reviewed-by: Grant Grundler <grundler@xxxxxxxxxx> (jejb should probably also add his here too) thanks, grant > --- > drivers/scsi/advansys.c | 2 -- > 1 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c > index b756041..6b72e2d 100644 > --- a/drivers/scsi/advansys.c > +++ b/drivers/scsi/advansys.c > @@ -8349,8 +8349,6 @@ static __le32 advansys_get_sense_buffer_dma(struct scsi_cmnd *scp) > struct asc_board *board = shost_priv(scp->device->host); > scp->SCp.dma_handle = dma_map_single(board->dev, scp->sense_buffer, > SCSI_SENSE_BUFFERSIZE, DMA_FROM_DEVICE); > - dma_cache_sync(board->dev, scp->sense_buffer, > - SCSI_SENSE_BUFFERSIZE, DMA_FROM_DEVICE); > return cpu_to_le32(scp->SCp.dma_handle); > } > > -- > 1.6.0.6 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html