scsi: qla2xxx: Change qla_trim_buf,__qla_adjust_buf functions to static

Message ID 20230114013724.3943580-1-trix@redhat.com
State New
Headers
Series scsi: qla2xxx: Change qla_trim_buf,__qla_adjust_buf functions to static |

Commit Message

Tom Rix Jan. 14, 2023, 1:37 a.m. UTC
  Smatch reports
drivers/scsi/qla2xxx/qla_mid.c:1189:6: warning: symbol 'qla_trim_buf' was not declared. Should it be static?
drivers/scsi/qla2xxx/qla_mid.c:1221:6: warning: symbol '__qla_adjust_buf' was not declared. Should it be static?

These functions are only used in qla_mid.c, so they should be static.

Fixes: 1f8f9c34127e ("scsi: qla2xxx: edif: Reduce memory usage during low I/O")
Signed-off-by: Tom Rix <trix@redhat.com>
---
 drivers/scsi/qla2xxx/qla_mid.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Martin K. Petersen Jan. 18, 2023, 11:47 p.m. UTC | #1
Tom,

> Smatch reports
> drivers/scsi/qla2xxx/qla_mid.c:1189:6: warning: symbol 'qla_trim_buf' was not declared. Should it be static?
> drivers/scsi/qla2xxx/qla_mid.c:1221:6: warning: symbol '__qla_adjust_buf' was not declared. Should it be static?
>
> These functions are only used in qla_mid.c, so they should be static.

Applied to 6.3/scsi-staging, thanks!
  
Martin K. Petersen Jan. 27, 2023, 3:23 a.m. UTC | #2
On Fri, 13 Jan 2023 20:37:24 -0500, Tom Rix wrote:

> Smatch reports
> drivers/scsi/qla2xxx/qla_mid.c:1189:6: warning: symbol 'qla_trim_buf' was not declared. Should it be static?
> drivers/scsi/qla2xxx/qla_mid.c:1221:6: warning: symbol '__qla_adjust_buf' was not declared. Should it be static?
> 
> These functions are only used in qla_mid.c, so they should be static.
> 
> 
> [...]

Applied to 6.3/scsi-queue, thanks!

[1/1] scsi: qla2xxx: Change qla_trim_buf,__qla_adjust_buf functions to static
      https://git.kernel.org/mkp/scsi/c/54c51253b3d5
  

Patch

diff --git a/drivers/scsi/qla2xxx/qla_mid.c b/drivers/scsi/qla2xxx/qla_mid.c
index c6ca39b8e23d..bd6bc1a968b2 100644
--- a/drivers/scsi/qla2xxx/qla_mid.c
+++ b/drivers/scsi/qla2xxx/qla_mid.c
@@ -1186,7 +1186,7 @@  int qla_get_buf(struct scsi_qla_host *vha, struct qla_qpair *qp, struct qla_buf_
 	return 0;
 }
 
-void qla_trim_buf(struct qla_qpair *qp, u16 trim)
+static void qla_trim_buf(struct qla_qpair *qp, u16 trim)
 {
 	int i, j;
 	struct qla_hw_data *ha = qp->vha->hw;
@@ -1218,7 +1218,7 @@  void qla_trim_buf(struct qla_qpair *qp, u16 trim)
 	       qp->id, trim, qp->buf_pool.num_alloc);
 }
 
-void __qla_adjust_buf(struct qla_qpair *qp)
+static void __qla_adjust_buf(struct qla_qpair *qp)
 {
 	u32 trim;