[SCSI] qla2xxx: Use #defines instead of hardcoded values for intr status.
authorArun Easi <arun.easi@qlogic.com>
Wed, 22 Aug 2012 18:21:16 +0000 (14:21 -0400)
committerJames Bottomley <JBottomley@Parallels.com>
Mon, 24 Sep 2012 08:10:50 +0000 (12:10 +0400)
Signed-off-by: Arun Easi <arun.easi@qlogic.com>
Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/qla2xxx/qla_def.h
drivers/scsi/qla2xxx/qla_isr.c

index ff8919293033914b922fd8fa92ea910b043695f0..c4e77d7bb08294129bd789761f36491685613bf8 100644 (file)
@@ -675,6 +675,17 @@ typedef struct {
 /* 83XX FCoE specific */
 #define MBA_IDC_AEN            0x8200  /* FCoE: NIC Core state change AEN */
 
+/* Interrupt type codes */
+#define INTR_ROM_MB_SUCCESS            0x1
+#define INTR_ROM_MB_FAILED             0x2
+#define INTR_MB_SUCCESS                        0x10
+#define INTR_MB_FAILED                 0x11
+#define INTR_ASYNC_EVENT               0x12
+#define INTR_RSP_QUE_UPDATE            0x13
+#define INTR_RSP_QUE_UPDATE_83XX       0x14
+#define INTR_ATIO_QUE_UPDATE           0x1C
+#define INTR_ATIO_RSP_QUE_UPDATE       0x1D
+
 /* ISP mailbox loopback echo diagnostic error code */
 #define MBS_LB_RESET   0x17
 /*
index 686099267dd57f7c6f5841570304ce9a8c04d933..a551827d3edce827b632d671e4bd15250319257d 100644 (file)
@@ -2522,29 +2522,29 @@ qla24xx_intr_handler(int irq, void *dev_id)
                        break;
 
                switch (stat & 0xff) {
-               case 0x1:
-               case 0x2:
-               case 0x10:
-               case 0x11:
+               case INTR_ROM_MB_SUCCESS:
+               case INTR_ROM_MB_FAILED:
+               case INTR_MB_SUCCESS:
+               case INTR_MB_FAILED:
                        qla24xx_mbx_completion(vha, MSW(stat));
                        status |= MBX_INTERRUPT;
 
                        break;
-               case 0x12:
+               case INTR_ASYNC_EVENT:
                        mb[0] = MSW(stat);
                        mb[1] = RD_REG_WORD(&reg->mailbox1);
                        mb[2] = RD_REG_WORD(&reg->mailbox2);
                        mb[3] = RD_REG_WORD(&reg->mailbox3);
                        qla2x00_async_event(vha, rsp, mb);
                        break;
-               case 0x13:
-               case 0x14:
+               case INTR_RSP_QUE_UPDATE:
+               case INTR_RSP_QUE_UPDATE_83XX:
                        qla24xx_process_response_queue(vha, rsp);
                        break;
-               case 0x1C: /* ATIO queue updated */
+               case INTR_ATIO_QUE_UPDATE:
                        qlt_24xx_process_atio_queue(vha);
                        break;
-               case 0x1D: /* ATIO and response queues updated */
+               case INTR_ATIO_RSP_QUE_UPDATE:
                        qlt_24xx_process_atio_queue(vha);
                        qla24xx_process_response_queue(vha, rsp);
                        break;
@@ -2673,29 +2673,29 @@ qla24xx_msix_default(int irq, void *dev_id)
                        break;
 
                switch (stat & 0xff) {
-               case 0x1:
-               case 0x2:
-               case 0x10:
-               case 0x11:
+               case INTR_ROM_MB_SUCCESS:
+               case INTR_ROM_MB_FAILED:
+               case INTR_MB_SUCCESS:
+               case INTR_MB_FAILED:
                        qla24xx_mbx_completion(vha, MSW(stat));
                        status |= MBX_INTERRUPT;
 
                        break;
-               case 0x12:
+               case INTR_ASYNC_EVENT:
                        mb[0] = MSW(stat);
                        mb[1] = RD_REG_WORD(&reg->mailbox1);
                        mb[2] = RD_REG_WORD(&reg->mailbox2);
                        mb[3] = RD_REG_WORD(&reg->mailbox3);
                        qla2x00_async_event(vha, rsp, mb);
                        break;
-               case 0x13:
-               case 0x14:
+               case INTR_RSP_QUE_UPDATE:
+               case INTR_RSP_QUE_UPDATE_83XX:
                        qla24xx_process_response_queue(vha, rsp);
                        break;
-               case 0x1C: /* ATIO queue updated */
+               case INTR_ATIO_QUE_UPDATE:
                        qlt_24xx_process_atio_queue(vha);
                        break;
-               case 0x1D: /* ATIO and response queues updated */
+               case INTR_ATIO_RSP_QUE_UPDATE:
                        qlt_24xx_process_atio_queue(vha);
                        qla24xx_process_response_queue(vha, rsp);
                        break;