Use dma_map_single() instead of pci_map_single(),
because only dma_map_single() is called here.
Signed-off-by: Cai Huoqing <[email protected]>
---
drivers/scsi/qla1280.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c
index d0b4e063bfe1..20b1709e8288 100644
--- a/drivers/scsi/qla1280.c
+++ b/drivers/scsi/qla1280.c
@@ -19,7 +19,7 @@
Rev 3.26, January 16, 2006 Jes Sorensen
- Ditch all < 2.6 support
Rev 3.25.1, February 10, 2005 Christoph Hellwig
- - use pci_map_single to map non-S/G requests
+ - use dma_map_single to map non-S/G requests
- remove qla1280_proc_info
Rev 3.25, September 28, 2004, Christoph Hellwig
- add support for ISP1020/1040
--
2.25.1
Use dma_map_single() instead of pci_map_single(),
because only dma_map_single() is called here.
Signed-off-by: Cai Huoqing <[email protected]>
---
drivers/scsi/3w-xxxx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/3w-xxxx.c b/drivers/scsi/3w-xxxx.c
index 4ee485ab2714..afe35220ed29 100644
--- a/drivers/scsi/3w-xxxx.c
+++ b/drivers/scsi/3w-xxxx.c
@@ -156,7 +156,7 @@
same card number.
Fix bug where cards were being shut down more than once.
1.02.00.029 - Add missing pci_free_consistent() in tw_allocate_memory().
- Replace pci_map_single() with pci_map_page() for highmem.
+ Replace dma_map_single() with dma_map_page() for highmem.
Check for tw_setfeature() failure.
1.02.00.030 - Make driver 64-bit clean.
1.02.00.031 - Cleanup polling timeouts/routines in several places.
--
2.25.1
Cai,
> Use dma_map_single() instead of pci_map_single(),
> because only dma_map_single() is called here.
pci_map_single() was the appropriate interface when the change log entry
was written. And the original entry accurately describes the change that
was made to the code back then. Whereas your proposed fix does not.
There is little benefit to having change logs in drivers in a modern
context. However, these entries predate git history and should therefore
be preserved.
--
Martin K. Petersen Oracle Linux Engineering
Cai,
> Use dma_map_single() instead of pci_map_single(),
> because only dma_map_single() is called here.
Same comment as the previous patch.
--
Martin K. Petersen Oracle Linux Engineering
On Sat, 25 Sep 2021 20:53:21 +0800, Cai Huoqing wrote:
> Use dma_map_single() instead of pci_map_single(),
> because only dma_map_single() is called here.
>
>
Applied to 5.16/scsi-queue, thanks!
[3/3] scsi: lpfc: Fix a function name in comments
https://git.kernel.org/mkp/scsi/c/8d807a068090
--
Martin K. Petersen Oracle Linux Engineering