* [PATCH 2/3] scsi: 3w-xxxx: Fix a function name in comments
2021-09-25 12:53 [PATCH 1/3] scsi: qla1280: Fix a function name in comments Cai Huoqing
@ 2021-09-25 12:53 ` Cai Huoqing
2021-09-29 3:13 ` Martin K. Petersen
2021-09-25 12:53 ` [PATCH 3/3] scsi: lpfc: " Cai Huoqing
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Cai Huoqing @ 2021-09-25 12:53 UTC (permalink / raw)
To: caihuoqing
Cc: Adam Radford, James E.J. Bottomley, Martin K. Petersen,
James Smart, Dick Kennedy, Michael Reed, linux-scsi,
linux-kernel
Use dma_map_single() instead of pci_map_single(),
because only dma_map_single() is called here.
Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
---
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
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] scsi: 3w-xxxx: Fix a function name in comments
2021-09-25 12:53 ` [PATCH 2/3] scsi: 3w-xxxx: " Cai Huoqing
@ 2021-09-29 3:13 ` Martin K. Petersen
0 siblings, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2021-09-29 3:13 UTC (permalink / raw)
To: Cai Huoqing
Cc: Adam Radford, James E.J. Bottomley, Martin K. Petersen,
James Smart, Dick Kennedy, Michael Reed, linux-scsi,
linux-kernel
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] scsi: lpfc: Fix a function name in comments
2021-09-25 12:53 [PATCH 1/3] scsi: qla1280: Fix a function name in comments Cai Huoqing
2021-09-25 12:53 ` [PATCH 2/3] scsi: 3w-xxxx: " Cai Huoqing
@ 2021-09-25 12:53 ` Cai Huoqing
2021-09-29 3:15 ` Martin K. Petersen
2021-09-29 3:12 ` [PATCH 1/3] scsi: qla1280: " Martin K. Petersen
2021-10-05 4:34 ` Martin K. Petersen
3 siblings, 1 reply; 7+ messages in thread
From: Cai Huoqing @ 2021-09-25 12:53 UTC (permalink / raw)
To: caihuoqing
Cc: Adam Radford, James E.J. Bottomley, Martin K. Petersen,
James Smart, Dick Kennedy, Michael Reed, linux-scsi,
linux-kernel
Use dma_map_sg() instead of pci_map_sg(),
because only dma_map_sg() is called here.
Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
---
drivers/scsi/lpfc/lpfc_scsi.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/lpfc/lpfc_scsi.c b/drivers/scsi/lpfc/lpfc_scsi.c
index 27ea8f552ede..83ad03d22a49 100644
--- a/drivers/scsi/lpfc/lpfc_scsi.c
+++ b/drivers/scsi/lpfc/lpfc_scsi.c
@@ -894,7 +894,7 @@ lpfc_scsi_prep_dma_buf_s3(struct lpfc_hba *phba, struct lpfc_io_buf *lpfc_cmd)
bpl += 2;
if (scsi_sg_count(scsi_cmnd)) {
/*
- * The driver stores the segment count returned from pci_map_sg
+ * The driver stores the segment count returned from dma_map_sg
* because this a count of dma-mappings used to map the use_sg
* pages. They are not guaranteed to be the same for those
* architectures that implement an IOMMU.
@@ -2589,7 +2589,7 @@ lpfc_bg_scsi_prep_dma_buf_s3(struct lpfc_hba *phba,
bpl += 2;
if (scsi_sg_count(scsi_cmnd)) {
/*
- * The driver stores the segment count returned from pci_map_sg
+ * The driver stores the segment count returned from dma_map_sg
* because this a count of dma-mappings used to map the use_sg
* pages. They are not guaranteed to be the same for those
* architectures that implement an IOMMU.
@@ -3249,7 +3249,7 @@ lpfc_scsi_prep_dma_buf_s4(struct lpfc_hba *phba, struct lpfc_io_buf *lpfc_cmd)
*/
if (scsi_sg_count(scsi_cmnd)) {
/*
- * The driver stores the segment count returned from pci_map_sg
+ * The driver stores the segment count returned from dma_map_sg
* because this a count of dma-mappings used to map the use_sg
* pages. They are not guaranteed to be the same for those
* architectures that implement an IOMMU.
@@ -3488,7 +3488,7 @@ lpfc_bg_scsi_prep_dma_buf_s4(struct lpfc_hba *phba,
*/
if (scsi_sg_count(scsi_cmnd)) {
/*
- * The driver stores the segment count returned from pci_map_sg
+ * The driver stores the segment count returned from dma_map_sg
* because this a count of dma-mappings used to map the use_sg
* pages. They are not guaranteed to be the same for those
* architectures that implement an IOMMU.
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] scsi: lpfc: Fix a function name in comments
2021-09-25 12:53 ` [PATCH 3/3] scsi: lpfc: " Cai Huoqing
@ 2021-09-29 3:15 ` Martin K. Petersen
0 siblings, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2021-09-29 3:15 UTC (permalink / raw)
To: Cai Huoqing
Cc: Adam Radford, James E.J. Bottomley, Martin K. Petersen,
James Smart, Dick Kennedy, Michael Reed, linux-scsi,
linux-kernel
Cai,
> Use dma_map_sg() instead of pci_map_sg(),
Applied to 5.16/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] scsi: qla1280: Fix a function name in comments
2021-09-25 12:53 [PATCH 1/3] scsi: qla1280: Fix a function name in comments Cai Huoqing
2021-09-25 12:53 ` [PATCH 2/3] scsi: 3w-xxxx: " Cai Huoqing
2021-09-25 12:53 ` [PATCH 3/3] scsi: lpfc: " Cai Huoqing
@ 2021-09-29 3:12 ` Martin K. Petersen
2021-10-05 4:34 ` Martin K. Petersen
3 siblings, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2021-09-29 3:12 UTC (permalink / raw)
To: Cai Huoqing
Cc: Adam Radford, James E.J. Bottomley, Martin K. Petersen,
James Smart, Dick Kennedy, Michael Reed, linux-scsi,
linux-kernel
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] scsi: qla1280: Fix a function name in comments
2021-09-25 12:53 [PATCH 1/3] scsi: qla1280: Fix a function name in comments Cai Huoqing
` (2 preceding siblings ...)
2021-09-29 3:12 ` [PATCH 1/3] scsi: qla1280: " Martin K. Petersen
@ 2021-10-05 4:34 ` Martin K. Petersen
3 siblings, 0 replies; 7+ messages in thread
From: Martin K. Petersen @ 2021-10-05 4:34 UTC (permalink / raw)
To: Cai Huoqing
Cc: Martin K . Petersen, James Smart, linux-scsi, Michael Reed,
Adam Radford, Dick Kennedy, James E.J. Bottomley, linux-kernel
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
^ permalink raw reply [flat|nested] 7+ messages in thread