scsi: qla4xxx: Move 'qla4_82xx_reg_tbl' to the only place its used
authorLee Jones <lee.jones@linaro.org>
Tue, 21 Jul 2020 16:41:27 +0000 (17:41 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Sat, 25 Jul 2020 02:10:00 +0000 (22:10 -0400)
Fixes the following W=1 kernel build warning(s):

 In file included from drivers/scsi/qla4xxx/ql4_def.h:43,
 from drivers/scsi/qla4xxx/ql4_mbx.c:9:
 drivers/scsi/qla4xxx/ql4_nx.h:602:23: warning: ‘qla4_82xx_reg_tbl’ defined but not used [-Wunused-const-variable=]
 602 | static const uint32_t qla4_82xx_reg_tbl[] = {
 | ^~~~~~~~~~~~~~~~~
 [...]
 NB: Lots of these

Link: https://lore.kernel.org/r/20200721164148.2617584-20-lee.jones@linaro.org
Cc: QLogic-Storage-Upstream@qlogic.com
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla4xxx/ql4_nx.h
drivers/scsi/qla4xxx/ql4_os.c

index 98fe78613eb7b6934974e76ef7591d3753ca9ec6..b7a6e7f169ca9b77ab0bf5e7097aae43763fdbf3 100644 (file)
@@ -599,23 +599,6 @@ enum qla_regs {
        QLA8XXX_CRB_TEMP_STATE,
 };
 
-static const uint32_t qla4_82xx_reg_tbl[] = {
-       QLA82XX_PEG_HALT_STATUS1,
-       QLA82XX_PEG_HALT_STATUS2,
-       QLA82XX_PEG_ALIVE_COUNTER,
-       QLA82XX_CRB_DRV_ACTIVE,
-       QLA82XX_CRB_DEV_STATE,
-       QLA82XX_CRB_DRV_STATE,
-       QLA82XX_CRB_DRV_SCRATCH,
-       QLA82XX_CRB_DEV_PART_INFO,
-       QLA82XX_CRB_DRV_IDC_VERSION,
-       QLA82XX_FW_VERSION_MAJOR,
-       QLA82XX_FW_VERSION_MINOR,
-       QLA82XX_FW_VERSION_SUB,
-       CRB_CMDPEG_STATE,
-       CRB_TEMP_STATE,
-};
-
 /* Every driver should use these Device State */
 #define QLA8XXX_DEV_COLD               1
 #define QLA8XXX_DEV_INITIALIZING       2
index 27064c602dc704b2fe6a35f922facb7cda826caa..2572f7aef8f886e86deef3fc613399faa481e828 100644 (file)
@@ -188,6 +188,23 @@ static int qla4xxx_sysfs_ddb_logout_sid(struct iscsi_cls_session *cls_sess);
 static struct qla4_8xxx_legacy_intr_set legacy_intr[] =
     QLA82XX_LEGACY_INTR_CONFIG;
 
+static const uint32_t qla4_82xx_reg_tbl[] = {
+       QLA82XX_PEG_HALT_STATUS1,
+       QLA82XX_PEG_HALT_STATUS2,
+       QLA82XX_PEG_ALIVE_COUNTER,
+       QLA82XX_CRB_DRV_ACTIVE,
+       QLA82XX_CRB_DEV_STATE,
+       QLA82XX_CRB_DRV_STATE,
+       QLA82XX_CRB_DRV_SCRATCH,
+       QLA82XX_CRB_DEV_PART_INFO,
+       QLA82XX_CRB_DRV_IDC_VERSION,
+       QLA82XX_FW_VERSION_MAJOR,
+       QLA82XX_FW_VERSION_MINOR,
+       QLA82XX_FW_VERSION_SUB,
+       CRB_CMDPEG_STATE,
+       CRB_TEMP_STATE,
+};
+
 static const uint32_t qla4_83xx_reg_tbl[] = {
        QLA83XX_PEG_HALT_STATUS1,
        QLA83XX_PEG_HALT_STATUS2,